warum geht das nicht? - Seite 2

robo47 · 20.05.2005, 17:20

also nur mal ne grobe kritik an deinem script

1) du überprüfst keinerlei werte die du via post bekommst,

http://php-faq.de/q/q-sicherheit-parameter.html

http://php-faq.de/q/q-security-variablen.html
2)

PHP-Code:

  <?php

$_POST['name'] == "$row->name"

?>

die " sind unnötig

3)

bei step 2 gibt er nur was aus wenn deine daten stimmen und das $row->name in " könnte da beim vergleich ein problem darstellen.

4)
select * from
Warum soll ich nicht SELECT * schreiben?

http://www.php-faq.de/q/q-sql-select.html

mysql error

man sollte in seinen Scripten immer mysql_error() verwenden, in etwa so:

PHP-Code:

  $sql = "SELECT feld1,feld2 FROM table WHERE id=5";

$result = mysql_query($sql) or die(mysql_error().'
Query: '.$sql);

und der standard-tip:

error-reporting

Zum testen von Scripts setzt man normalerweise am anfang des scripts (oder direkt in einer config-datei die immer Includet wird)

PHP-Code:

  error_reporting(E_ALL);

damit werden dann alle Fehler ausgegeben.
Mögliche Fehler die dann auftreten und wie man sie beheben kann:
Notice: Use of undefined constant ...
Notice: Undefined variable ...

mfg
robo47

notyyy · 20.05.2005, 17:29

ich denke ich hab den fehler, eine schleife war falsch zugemacht, also hätte erst später geschlossen werden dürfen, thx besonders an robo47

20.05.2005, 17:53

Ähm, kannst du mir mal verraten, warum ich mir die Mühe gemacht habe, deinen Schrott mal ordentlich einzurücken? Damit du das ignorierst und beim nächsten mal wieder denselben Käse hier reinhaust?

Kopfschüttel

Gruß
phpfan

notyyy · 20.05.2005, 18:07

sorry, tut mir leid, ich hab einfach nicht gerallt was du mir sagen wolltest ^^
guckmal, anhand deiner beiträge und posts kann man erkennen, dass du gut php kannst, ich hingegen bin 14 hab wenig zeit und versuche es zu lernen ...
...irgendwann reg ich mich auch mal über die noobs auf, die immernochnicht wissen das ne if mit } anfängt und mit { aufhört

20.05.2005, 18:11

Aber du kannst die Augen aufmachen und Copy/Paste kannst doch bestimmt auch.
Dir hätte auch als 14-jähriger auffallen können, dass nach dem Einrücken das Script besser zu lesen ist, oder? Und glaubst du, ich schreibe sowas nur aus Langeweile. Man sollte sich die Tipps von Leuten, die schon ein paar Tage länger Programmieren, auch beachten.

Gruß
phpfan

notyyy · 20.05.2005, 21:59

ja hast ja recht, tut mir wirklich leid

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

20.05.2005, 17:20
robo47 Moderator Registriert seit: 03.09.2004 Beiträge: 11.792 PHP-Kenntnisse: Fortgeschritten	also nur mal ne grobe kritik an deinem script 1) du überprüfst keinerlei werte die du via post bekommst, http://php-faq.de/q/q-sicherheit-parameter.html http://php-faq.de/q/q-security-variablen.html 2) PHP-Code: `<?php $_POST['name'] == "$row->name" ?>` die " sind unnötig 3) bei step 2 gibt er nur was aus wenn deine daten stimmen und das $row->name in " könnte da beim vergleich ein problem darstellen. 4) *select from** Warum soll ich nicht SELECT * schreiben? http://www.php-faq.de/q/q-sql-select.html mysql error man sollte in seinen Scripten immer mysql_error() verwenden, in etwa so: PHP-Code: `$sql = "SELECT feld1,feld2 FROM table WHERE id=5"; $result = mysql_query($sql) or die(mysql_error().' Query: '.$sql);` und der standard-tip: error-reporting Zum testen von Scripts setzt man normalerweise am anfang des scripts (oder direkt in einer config-datei die immer Includet wird) PHP-Code: `error_reporting(E_ALL);` damit werden dann alle Fehler ausgegeben. Mögliche Fehler die dann auftreten und wie man sie beheben kann: Notice: Use of undefined constant ... Notice: Undefined variable ... mfg robo47 __________________ robo47.net - Blog, Codeschnipsel und mehr \| Caching-Klassen und Opcode Caches in php \| Robo47 Components - PHP Library extending Zend Framework


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

20.05.2005, 17:29
notyyy Erfahrener Benutzer Registriert seit: 12.05.2005 Beiträge: 1.038 PHP-Kenntnisse: Fortgeschritten	ich denke ich hab den fehler, eine schleife war falsch zugemacht, also hätte erst später geschlossen werden dürfen, thx besonders an robo47

20.05.2005, 17:53
Gast Beiträge: n/a	Ähm, kannst du mir mal verraten, warum ich mir die Mühe gemacht habe, deinen Schrott mal ordentlich einzurücken? Damit du das ignorierst und beim nächsten mal wieder denselben Käse hier reinhaust? Kopfschüttel Gruß phpfan

20.05.2005, 18:07
notyyy Erfahrener Benutzer Registriert seit: 12.05.2005 Beiträge: 1.038 PHP-Kenntnisse: Fortgeschritten	sorry, tut mir leid, ich hab einfach nicht gerallt was du mir sagen wolltest ^^ guckmal, anhand deiner beiträge und posts kann man erkennen, dass du gut php kannst, ich hingegen bin 14 hab wenig zeit und versuche es zu lernen ... ...irgendwann reg ich mich auch mal über die noobs auf, die immernochnicht wissen das ne if mit } anfängt und mit { aufhört

20.05.2005, 18:11
Gast Beiträge: n/a	Aber du kannst die Augen aufmachen und Copy/Paste kannst doch bestimmt auch. Dir hätte auch als 14-jähriger auffallen können, dass nach dem Einrücken das Script besser zu lesen ist, oder? Und glaubst du, ich schreibe sowas nur aus Langeweile. Man sollte sich die Tipps von Leuten, die schon ein paar Tage länger Programmieren, auch beachten. Gruß phpfan

20.05.2005, 21:59
notyyy Erfahrener Benutzer Registriert seit: 12.05.2005 Beiträge: 1.038 PHP-Kenntnisse: Fortgeschritten	ja hast ja recht, tut mir wirklich leid