Неустранимая ошибка: допустимый объем памяти 134217728 байт исчерпан (попытка выделить 64 байта) в строке 305
Я только что добавил новый query на мою веб-страницу, которая использует while loop производить array результатов. Однако, когда я пытаюсь запустить страницу, я получаю сообщение об ошибке: "Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 64 bytes) on line 305", Я не могу понять, почему это произошло внезапно, или что может быть утечка памяти. Я не хочу увеличивать размер PHP memory_limit,
У меня другой вопрос … Я прошу совета о том, что может быть причиной утечки памяти, а не о том, что это за фатальная ошибка или что она означает.
Запрос в вопросе:
<?php
$result1 = $con->query("SELECT SkillID FROM userskills WHERE UserID = '$User'") or die(mysqli_error($con));
$current_skills = array();
while (($skillrow = mysqli_fetch_array($result1, MYSQLI_NUM)) !== false){
$current_skills[] = $skillrow;
}
?>
Полная страница:
<?php
error_reporting(E_ALL); ini_set('display_errors', 1);
require 'Assets/Connections/Connections.php';
session_start();
if(isset($_SESSION["UserID"]))
{
}
else
{
header('Location: LogIn.php');
die();
}
$User = (int)$_SESSION["UserID"];
$result = $con->query("SELECT * FROM user WHERE UserID ='$User'") or die(mysqli_error($con));
$row = $result->fetch_array(MYSQLI_BOTH);
$_SESSION["FirstName"] = $row['Fname'];
$_SESSION["LastName"] = $row['Lname'];
$_SESSION["Email"] = $row['Email'];
$_SESSION["Role"] = $row['JobRole'];
$skillresult = $con->query("SELECT userskills.SkillID, Description, Experience FROM User INNER JOIN userskills ON User.UserId = userskills.UserId JOIN Skills ON userskills.SkillID = Skills.SkillID WHERE user.UserID ='$User'")
or die(mysqli_error($con));
$skills_array = array();
while($r=mysqli_fetch_array($skillresult)){
if (!isset($skills_array[$r['SkillID']])){
$skills_array[$r['SkillID']] = array();
}
$skills_array[$r['SkillID']][] = $r['Description'];
}
if(isset($_POST['Update']))
{
$UpdateFName = $_SESSION["FirstName"];
if ($_POST['FirstName'] != '' ) { $UpdateFName = $_POST['FirstName'];}
$UpdateLName = $_SESSION["LastName"];
if ($_POST['LastName'] != '' ) { $UpdateLName = $_POST['LastName'];}
$UpdateEmail = $_SESSION["Email"];
if ($_POST['Email'] != '' ) { $UpdateEmail = $_POST['Email'];}
$UpdateRole = $_SESSION["Role"];
if ($_POST['JobRole'] != '' ) { $UpdateRole = $_POST['JobRole'];}
$PasswordCheck = $_POST['Password'];
if(password_verify($PasswordCheck, $row['Password']))
{
$sql = $con->query("UPDATE user SET
Fname = '{$UpdateFName}',
Lname = '{$UpdateLName}',
Email = '{$UpdateEmail}',
JobRole = '{$UpdateRole}'
WHERE UserID = $User") or die(mysqli_error($con));
if(!empty($_FILES['file']['name']))
{
$file = basename($_FILES['file']['name']);
move_uploaded_file($_FILES['file']['tmp_name'], 'Assets/Images/'.$file);
}
if(isset($file))
{
$sql = $con->query("UPDATE user SET ProfileImage = '".$_FILES['file']['name']."' WHERE UserID = $User") or die(mysqli_error($con));
}
$default = 0;
foreach($skills_array AS $skills_id=>$skills_name)
{
if (isset($_POST[$skills_name]))
{
if (empty($_POST[$skills_name.'exp']))
{
$exp = $default;
}
else
{
$exp = $_POST[$skills_name.'exp'];
}
$sql = $con->query("SELECT count(UserID) as total FROM userskills WHERE UserID = $User AND SkillID = ".$skills_id) or die(mysqli_error($con));
if ($row = mysqli_fetch_assoc($sql))
{
$sql = $con->query("INSERT INTO userskills ( UserID, SkillID, Experience) VALUES ($User, $skills_id, $exp)");
//If the checkbox is not checked it will check to see if skill is already a skill assigned to the user. If they are it will delete it. If not it will ignore.
}
else
{
$sql = $con->query("UPDATE userskills SET Experience = $exp WHERE UserID = $User AND SkillID = ".$skills_id);
}
}
else
{
$sql = $con->query("DELETE FROM userskills WHERE UserID = $User AND SkillID = ".$skills_id);
}
}
header('Location: Account.php');
die();
}
else
{
echo 'Incorrect password please try again.';
}
}
?>
<!doctype html>
<html>
<head>
<link href="Assets/CSS/Master.css" rel="stylesheet" type="text/css" />
<link href="Assets/CSS/Menu.css" rel="stylesheet" type="text/css" />
<meta charset="utf-8">
<title>Update Account</title>
</head>
<body>
<div class="Container">
<div class="Header"></div>
<div class="Menu">
<div id="Menu">
<nav>
<ul class="cssmenu">
<li><a href="Home.php">Home</a></li>
<li><a href="Account.php">Account</a></li>
<li><a href="Projects.php">Projects</a></li>
<li><a href="Users.php">Users</a></li>
<li><a href="LogOut.php">LogOut</a></li>
</ul>
</nav>
</div>
</div>
<div class="LeftBody">
<form id="form1" name="form1" method="post" enctype="multipart/form-data">
<div class="FormElement">
<input name="FirstName" type="text" class="TField" id="FirstName" placeholder="First Name" value="<?php echo $_SESSION["FirstName"]; ?>">
</div>
<div class="FormElement">
<input name="LastName" type="text" class="TField" id="LastName" placeholder="Last Name" value="<?php echo $_SESSION["LastName"]; ?>">
</div>
<div class="FormElement">
<input name="Email" type="email" class="TField" id="Email" placeholder="Email Address" value="<?php echo $_SESSION["Email"]; ?>">
</div>
<div class="FormElement">
<input name="JobRole" type="text" class="TField" id="JobRole" placeholder="Job Role" value="<?php echo $_SESSION["Role"]; ?>">
</div>
<div class="FormElement">
<input name="Password" type="password" class="TField" id="Password" placeholder="Password" required="requried">
</div>
<div class="FormElement">
<input type="file" name="file">
<br>
<br>
</div>
<div class="FormElement">
<input name="Update" type="submit" class="button" id="Update" value="Submit Changes">
</div>
</form>
</div>
<div class="RightBody">
<form id="form2" name="form2" method="post" enctype="multipart/form-data">
<p><h3>Skills:</h3>
<?php
//advice given from stackoverflow. Suggests looping around the results of this to output
$result1 = $con->query("SELECT skills.SkillID, skills.Description, COUNT(userskills.SkillID) AS SkillUserHas, MAX(Experience) AS Experience
FROM
(
SELECT 1 AS SkillID, 'Java' AS Description
UNION
SELECT 7 AS SkillID, 'iOS' AS Description
UNION
SELECT 9 AS SkillID, 'PHP' AS Description
UNION
SELECT 3 AS SkillID, 'SQL' AS Description
UNION
SELECT 4 AS SkillID, 'Windows' AS Description
UNION
SELECT 5 AS SkillID, 'Linux' AS Description
UNION
SELECT 6 AS SkillID, 'Unix' AS Description
UNION
SELECT 8 AS SkillID, 'Requirements Elicitation' AS Description
) skills
LEFT OUTER JOIN userskills
ON skills.SkillID = userskills.SkillID AND userskills.UserID = '$User'
GROUP BY skills.SkillID, skills.Description
ORDER BY FIELD(skills.SkillID, 1, 7, 9, 3, 4, 5, 6, 8)")
or die(mysqli_error($con));while ($skillrow = $result1->fetch_assoc())
{
?>
<div class="CheckboxText">
<?php
echo '<label>';
echo '<input type="checkbox" name="'.$skillrow['Description'].'" id="CheckboxGroup1_'.$skillrow['SkillID'].'" class="skillselect" value="yes" '.(($skillrow['SkillUserHas'] > 0) ? 'checked' : '').'>';
echo $skillrow['Description'].'</label>';
echo '<input type="number" name="'.$skillrow['Description'].'exp" class="expnumber" placeholder="Enter Experience in years." value="'.$skillrow['Experience'].'">';
echo '<br />';
echo '<br />';
}
?>
</div>
</p>
</form>
</div>
<div class="Footer">
<footer class="footer-basic-centered">
<p class="footer-company-motto">We Always Believe</p>
<p class="footer-links"> <a href="Home.php">Home</a> · <a href="Account.php">Account</a> · <a href="Projects.php">Projects</a> · <a href="Users.php">Users</a> · <a href="LogOut.php">LogOut</a> </p>
<p class="footer-company-name">Project Mainframe © 2016</p>
</footer>
</div>
</div>
</body>
</html>
Решение
Обратите внимание, что согласно документам mysqli_fetch_array возвращает NULL, если в результирующем наборе больше нет строк. Вы проверяете на ложность специально, а не на то, что равно ложному. Таким образом, у вас есть бесконечный цикл. Измените его на следующее в качестве краткосрочного исправления, пока не исправите другие проблемы:
while (($skillrow = mysqli_fetch_array($result1, MYSQLI_NUM)) != false)
{
$current_skills[] = $skillrow;
}
Выполнение остальной части этого в качестве ответа, хотя это скорее комментарий, но это слишком долго.
Ваш код содержит огромное количество повторяющегося кода.
Например, вы делаете в значительной степени тот же код, что и ниже, для каждого навыка: —
//If the Unix checkbox is checked it will check to see if Unix is already a skill assigned to the user. If so it will ignore, if not it will add.
if (isset($_POST['unix'])){
if (empty($_POST['unixexp'])){
$unixexp = $default;
}else{
$unixexp = $_POST['unixexp'];}
$sql = $con->query("SELECT count(UserID) as total FROM userskills WHERE UserID = $User AND SkillID = 6")
or die(mysqli_error($con));
$row = mysqli_fetch_assoc($sql);
if ($row ['total'] == "0"){
$sql = $con->query("INSERT INTO userskills ( UserID, SkillID, Experience) VALUES ($User, 6, $unixexp)");
//If the Unix checkbox is not checked it will check to see if Unix is already a skill assigned to the user. If they are it will delete it. If not it will ignore.
}} else{
$sql = $con->query("SELECT count(UserID) as total FROM userskills WHERE UserID = $User AND SkillID = 6")
or die(mysqli_error($con));
$row = mysqli_fetch_assoc($sql);
if ($row ['total'] == "1"){
$sql = $con->query("DELETE FROM userskills
WHERE UserID = $User AND SkillID = 6");
}}
Это можно легко сделать, зацикливая массив навыков, или лучше зацикливая результаты запроса таблицы, в которой хранятся эти навыки. Это сделает код короче, проще и гораздо проще в обслуживании (так как в будущем вы можете просто добавить новый навык в массив сверху или лучше в таблицу навыков и не менять сам скрипт).
Кроме того, если у вас есть таблица навыков, запрос пользователей навыков может быть выполнен как: —
SELECT a.SkillID, a.SkillName, COUNT(b.SkillID) AS SkillUserHas
FROM all_skills a
LEFT OUTER JOIN auserskills b
ON a.SkillID = b.SkillID
AND b.UserID = '$User'
GROUP BY a.SkillID, a.SkillName
Затем вы можете просто зациклить результаты этого, чтобы вывести список флажков для пользователя, чтобы пометить / снять отметку с навыков (это будет возвращать 1 строку на навык, независимо от того, имел он пользователь или нет, и столбец, который будет равен 0, если у них нет этого навыка и> = 1, если этот навык записан для них).
Обратите внимание, что вы также, кажется, сделали нулевую очистку переменных. Вам нужно использовать mysqli_real_escape_string или эквивалентный, иначе результаты могут быть неприятными, так как пользователи могут потворствовать внедрению SQL.
Другие решения
Быстрая попытка очистки кода. Это просто использование закодированного списка навыков, а не хранение их на столе, но, надеюсь, даст вам некоторые идеи.
Можно было бы сделать лучше, если бы таблица навыков имела идентификатор пользователя / идентификатор навыка в качестве уникального индекса, тогда вы могли бы просто выполнить вставку / обновление дубликата ключа, а не пытаться прочитать значения, чтобы решить, вставлять или обновлять запись
<?php
error_reporting(E_ALL); ini_set('display_errors', 1);
require 'Assets/Connections/Connections.php';
session_start();
if(isset($_SESSION["UserID"]))
{
}
else
{
header('Location: LogIn.php');
die();
}
$User = (int)$_SESSION["UserID"];
$result = $con->query("SELECT * FROM user WHERE UserID ='$User'") or die(mysqli_error($con));
$row = $result->fetch_array(MYSQLI_BOTH);
$_SESSION["FirstName"] = $row['Fname'];
$_SESSION["LastName"] = $row['Lname'];
$_SESSION["Email"] = $row['Email'];
$_SESSION["Role"] = $row['JobRole'];
$skills_array = array(1=>'Java',
7=>'iOS',
9=>'PHP',
3=>'SQL',
4=>'Windows',
5=>'Linux',
6=>'Unix',
8=>'Requirements Elicitation');
if(isset($_POST['Update']))
{
$UpdateFName = $_SESSION["FirstName"];
if ($_POST['FirstName'] != '' ) { $UpdateFName = $_POST['FirstName'];}
$UpdateLName = $_SESSION["LastName"];
if ($_POST['LastName'] != '' ) { $UpdateLName = $_POST['LastName'];}
$UpdateEmail = $_SESSION["Email"];
if ($_POST['Email'] != '' ) { $UpdateEmail = $_POST['Email'];}
$UpdateRole = $_SESSION["Role"];
if ($_POST['JobRole'] != '' ) { $UpdateRole = $_POST['JobRole'];}
$PasswordCheck = $_POST['Password'];
if(password_verify($PasswordCheck, $row['Password']))
{
$sql = $con->query("UPDATE user SET
Fname = '{$UpdateFName}',
Lname = '{$UpdateLName}',
Email = '{$UpdateEmail}',
JobRole = '{$UpdateRole}'
WHERE UserID = $User") or die(mysqli_error($con));
if(!empty($_FILES['file']['name']))
{
$file = basename($_FILES['file']['name']);
move_uploaded_file($_FILES['file']['tmp_name'], 'Assets/Images/'.$file);
}
if(isset($file))
{
$sql = $con->query("UPDATE user SET ProfileImage = '".$_FILES['file']['name']."' WHERE UserID = $User") or die(mysqli_error($con));
}
$default = 0;
foreach($skills_array AS $skills_id=>$skills_name)
{
if (isset($_POST[$skills_name]))
{
if (empty($_POST[$skills_name.'exp']))
{
$exp = $default;
}
else
{
$exp = $_POST[$skills_name.'exp'];
}
$sql = $con->query("SELECT count(UserID) as total FROM userskills WHERE UserID = $User AND SkillID = ".$skills_id) or die(mysqli_error($con));
if ($row = mysqli_fetch_assoc($sql))
{
$sql = $con->query("INSERT INTO userskills ( UserID, SkillID, Experience) VALUES ($User, $skills_id, $exp)");
//If the checkbox is not checked it will check to see if skill is already a skill assigned to the user. If they are it will delete it. If not it will ignore.
}
else
{
$sql = $con->query("UPDATE userskills SET Experience = $exp WHERE UserID = $User AND SkillID $skills_id");
}
}
else
{
$sql = $con->query("DELETE FROM userskills WHERE UserID = $User AND SkillID = ".$skills_id);
}
}
header('Location: Account.php');
die();
}
else
{
echo 'Incorrect password please try again.';
}
}
?>
<!doctype html>
<html>
<head>
<link href="Assets/CSS/Master.css" rel="stylesheet" type="text/css" />
<link href="Assets/CSS/Menu.css" rel="stylesheet" type="text/css" />
<meta charset="utf-8">
<title>Update Account</title>
</head>
<body>
<div class="Container">
<div class="Header">
</div>
<div class="Menu">
<div id="Menu">
<nav>
<ul class="cssmenu">
<li><a href="Home.php">Home</a></li>
<li><a href="Account.php">Account</a></li>
<li><a href="Projects.php">Projects</a></li>
<li><a href="Users.php">Users</a></li>
<li><a href="LogOut.php">LogOut</a></li>
</ul>
</nav>
</div>
</div>
<div class="LeftBody"></div>
<div class="RightBody">
<form id="form1" name="form1" method="post" enctype="multipart/form-data">
<div class="FormElement">
<input name="FirstName" type="text" class="TField" id="FirstName" placeholder="First Name" value="<?php echo $_SESSION["FirstName"]; ?>">
</div>
<div class="FormElement">
<input name="LastName" type="text" class="TField" id="LastName" placeholder="Last Name" value="<?php echo $_SESSION["LastName"]; ?>">
</div>
<div class="FormElement">
<input name="Email" type="email" class="TField" id="Email" placeholder="Email Address" value="<?php echo $_SESSION["Email"]; ?>">
</div>
<div class="FormElement">
<input name="JobRole" type="text" class="TField" id="JobRole" placeholder="Job Role" value="<?php echo $_SESSION["Role"]; ?>">
</div>
<div class="FormElement">
<input name="Password" type="password" class="TField" id="Password" placeholder="Password" required="requried">
</div>
<div class="FormElement">
<input type="file" name="file">
<br />
<br />
</div>
<p>
<?php
$result1 = $con->query("SELECT all_skills.SkillID, all_skills.SkillName, COUNT(userskills.SkillID) AS SkillKnown, MAX(Experience) AS Experience
FROM
(
SELECT 1 AS SkillID, 'Java' AS SkillName
UNION
SELECT 7 AS SkillID, 'iOS' AS SkillName
UNION
SELECT 9 AS SkillID, 'PHP' AS SkillName
UNION
SELECT 3 AS SkillID, 'SQL' AS SkillName
UNION
SELECT 4 AS SkillID, 'Windows' AS SkillName
UNION
SELECT 5 AS SkillID, 'Linux' AS SkillName
UNION
SELECT 6 AS SkillID, 'Unix' AS SkillName
UNION
SELECT 8 AS SkillID, 'Requirements Elicitation'
) all_skills
LEFT OUTER JOIN userskills
ON all_skills.SkillID = userskills.SkillID AND userskills.UserID = '$User'
GROUP BY all_skills.SkillID, all_skills.SkillName
ORDER BY FIELD(all_skills.SkillID, 1, 7, 9, 3, 4, 5, 6, 8") or die(mysqli_error($con));
while ($skillrow = mysqli_fetch_array($result1, MYSQLI_ASSOC))
{
echo '<label>';
echo '<input type="checkbox" name="'.$skillrow['SkillName'].'" id="CheckboxGroup1_'.$skillrow['SkillID'].'" class="skillselect" value="yes" '.(($skillrow['SkillKnown'] > 0) ? 'checked' : '').'>';
echo $skillrow['SkillName'].'</label>';
echo '<input type="number" name="'.$skillrow['SkillName'].'exp" class="expnumber" placeholder="Enter Experience in years." value="'.$skillrow['Experience'].'">';
echo '<br />';
echo '<br />';
}
?>
</p>
<div class="FormElement">
<input name="Update" type="submit" class="button" id="Update" value="Submit Changes">
</div>
</form>
</div>
<div class="Footer">
<footer class="footer-basic-centered">
<p class="footer-company-motto">We Always Believe</p>
<p class="footer-links"> <a href="Home.php">Home</a> · <a href="Account.php">Account</a> · <a href="Projects.php">Projects</a> · <a href="Users.php">Users</a> · <a href="LogOut.php">LogOut</a> </p>
<p class="footer-company-name">Project Mainframe © 2016</p>
</footer>
</div>
</div>
</body>
</html>