Je reçois cette erreur PHP, qu'est-ce que cela signifie ?
Notice: Undefined offset: 0 in
C:\xampp\htdocs\mywebsite\reddit_vote_tut\src\votes.php on line 41
A partir de ce code :
<?php
include("config.php");
function getAllVotes($id)
{
$votes = array();
$q = "SELECT * FROM entries WHERE id = $id";
$r = mysql_query($q);
if(mysql_num_rows($r)==1)//id found in the table
{
$row = mysql_fetch_assoc($r);
$votes[0] = $row['votes_up'];
$votes[1] = $row['votes_down'];
}
return $votes;
}
function getEffectiveVotes($id)
{
$votes = getAllVotes($id);
$effectiveVote = $votes[0] - $votes[1]; //ERROR THROWN HERE
return $effectiveVote;
}
$id = $_POST['id'];
$action = $_POST['action'];
//get the current votes
$cur_votes = getAllVotes($id);
//ok, now update the votes
if($action=='vote_up') //voting up
{
$votes_up = $cur_votes[0]+1; //AND ERROR THROWN HERE
$q = "UPDATE threads SET votes_up = $votes_up WHERE id = $id";
}
elseif($action=='vote_down')
{
$votes_down = $cur_votes[1]+1;
$q = "UPDATE threads SET votes_down = $votes_down WHERE id = $id";
}
$r = mysql_query($q);
if($r)
{
$effectiveVote = getEffectiveVotes($id);
echo $effectiveVote." votes";
}
elseif(!$r) //voting failed
{
echo "Failed!";
}
?>
21 votes
Votre code est dangereux ! il peut être utilisé pour des injections sql !
0 votes
@Bernd Ott Comment faire pour régler ce problème ?
2 votes
Utiliser une couche de base de données qui autorise les paramètres de requête et/ou utiliser mysql_real_escape_string voir php.net/manual/de/function.mysql-real-escape-string.php il y a aussi de beaux échantillons.
0 votes
J'aimerais souligner que mysqli_real_escape_string est toujours vulnérable aux injections SQL et devrait être déconseillé au moment où j'écris ces lignes (2020). Les instructions préparées sont la voie à suivre maintenant.