Töltsd fel valahová, kérlek, bebizonyítom, hogy Lorennek nincs igaza mutatok valamit.
Komolyra fordítva a szót, néhány komolyabb hiba /van más is, de most nincs kedvem az összeset felsorolni/:
- az általad használt mysql függvények elavultak, javasolt a mysqli, vagy PDO használata
- minden egy fájlban van /átláthatatlan, stb./
- teljes mértékben statikus, a dinamikusság jeleit még csak nem is mutatja
- nincs semmiféle autentikáció
- $row[iD] -> $row[\'ID\']
- print helyett inkább echo
- htmlspecialchars
- injection ellen mysql_real_escape_string(), ami szintén elavult
- én nem írnám ki a mysql errort csak úgy bárkinek
- én nem használnék ilyen buttonokat
- feleslegesen csinálsz egy sor helyett öt sort
- echo \"asd = $valami\" => echo \'asd = \'.$valami;
$query = mysql_query(\"SELECT * FROM messages WHERE ISTOPIC = 1\");
while($row = mysql_fetch_array($query)) {
$res = mysql_query(\"SELECT * FROM messages WHERE ISTOPIC = 0 AND TOPICID = $row[iD]\");
$valaszok = mysql_num_rows($res);
print(\"<tr><td><a href=\'?option=show&id=$row[iD]\'>$row[TITLE]</a></td><td><i>$row[AUTHOR]</i></td><td>$valaszok válasz</td></tr>\");
}
print(\"</table>\");
helyett valami ilyesmi is jó (és az elavult mysql függvényekkel például nem foglalkoztam):
$query = mysql_query(\"SELECT * FROM messages WHERE ISTOPIC = 1\");
while($row = mysql_fetch_array($query)) {
echo \'<tr><td><a href=\"?option=show&id=\'.$row[\"ID\"].\'\">\'.htmlspecialchars($row[\"TITLE\"].\'</a></td><td><i>\'.htmlspecialchars($row[\"AUTHOR\"]).\'</i></td><td>\'.mysql_num_rows(mysql_query(\"SELECT * FROM messages WHERE ISTOPIC = 0 AND TOPICID = \".$row[\'ID\'])).\' válasz</td></tr>\';
}
echo \"</table>\";
Üzenetek megjelenítése
