Variable in Mysql Query übergeben

[c4]

PHP-Code:

$result=mysql_query("SELECT feld FROM tabelle WHERE hersteller='$firma'"); 


Es wurde bereits gesagt, dass Du Texte sinnvoll 'klammern' sollst, dass Du LIKE meiden solltest, wenn keine Platzhalter genutzt werden und die ` nur bei reservierten Bezeichnern notwendig sind...

[FreAKAzoid]

PHP-Code:

$result = mysql_query
    (
    "SELECT a.`Hersteller`, a.`Bezeichnung` 
    FROM `warenbestand`.`wareneingang` a
    WHERE a.`Hersteller`=`$firma`
    ")
    or die (mysql_error())
    ; 


so sieht die Query (dank der Tipps und Hilfe von euch) jetzt aus.
und nach dem was C4 sagt sollte das auch laufen???? Tuts aber nicht.

ich selber bin Systemintegrator und kein Anwendungsentwickler... aber wenn ich in meiner dunklen Vergangenheit etwas grabe fällt mir ein, daß ich bei c++ vorne einige Includes angebe... muss ich so etwas auch bei PHP machen??? oder woran kann es sonst liegen???

[c4]

Du musst mysql_error() auch ausgeben, nicht nur aufrufen!

PHP-Code:

$result = mysql_query("SELECT Hersteller, Bezeichnung
    FROM wareneingang
    WHERE Hersteller='$firma'") or die (echo mysql_error()); 


Wieso weigerst Du Dich, $firma in ' zu setzen??

Zitat von FreAKAzoid

und nach dem was C4 sagt sollte das auch laufen???? Tuts aber nicht.

Man muss unterscheiden zwischen dem, was ich geschrieben habe und was Du daraus gemacht hast...

Zitat von FreAKAzoid

wenn ich in meiner dunklen Vergangenheit etwas grabe fällt mir ein, daß ich bei c++ vorne einige Includes angebe... muss ich so etwas auch bei PHP machen??? oder woran kann es sonst liegen???

Woran liegt was?
include()s kannst Du gerne machen (DB-Verbindung herstellen, Konfigurationen laden, Layout, usw.) musst Du aber alles nicht.

[xabbuh]

Zitat von c4

Du musst mysql_error() auch ausgeben, nicht nur aufrufen!

Da brauchst du kein echo/print verwenden.

[c4]

Argh! Ja, stimmt. Krass voll viel fett übersehen, dass das ja im die() steht!

[dachris]

Womit wir ja auch schon wieder beim sinnvollen Quellcodeformat sind :-)(Lesbarkeit und so....)

[c4]

mysql_error() hat auch nur zu Zeiten der Entwicklung im Code zu stehen, dann nicht mehr. Also ist es egal.

Aber Du hast auch wieder Recht, hätte man es so geschrieben, dann hätte ich das mit dem echo sicher nicht geschrieben: or die("".mysql_error()."") ;)

[dachris]

Eigentlich meinte ich eher so....

PHP-Code:

 $result = mysql_query("SELECT Hersteller, Bezeichnung
    FROM wareneingang
    WHERE Hersteller='$firma'") 

or 

die (echo mysql_error()); 


Es geht ja nicht darum an Kleinigkeiten rumzumotzen oder ob man " kommentiert oder nicht.....

aber ich habe schon soviele Fehler (meistens selbst geabut) gesehen welche aufgrund von aneinandergefrickelten unkommentierten und unstrukturierten Code entstanden sind...

Die meisten Fehler lassen sich einfach vermeiden wenn man ein paar Programmierstandards einhält und dazu gehört nun mal korrekte sinnvolle Syntax, keine Kurzschreibweisen, Struktur und Kommentare sowie OOP sowie eine sinnvolle Nomenklatur....
Das unterscheidet nunmmal einen professionellen Code welcher skalierbar und weiterverwendbar ist vom typischen dirtyCoded Schnellschuss...

Es geht ja meist nicht nur darum das man selber Fhelr besser erkennt sondern das andere den Code evtl. weiterwarten müssen....

Aber ich bin dafür das wir für die Diskussion einen iegenen Thread aufmachen, das hat nun wirklich nix mehr mit dem Ursprungsthema zu tun....

[FreAKAzoid]

Zitat von c4

Wieso weigerst Du Dich, $firma in ' zu setzen??

Asche auf mein Haupt, Salz in meine Wunden und PHP in mein Hirn!

ich danke euch für eure Unterstützung bei diesem Problem... wenn ich denn irgendwann auf Anhieb erkennen kann ob es sich um ein ` oder ein ' dreht denke ich habe ich PHP verstanden.
Der Query mit dem es klappt sieht so aus:

PHP-Code:

    $result = mysql_query
    (
    "SELECT w.`Hersteller`, w.`Bezeichnung` 
    FROM `warenbestand`.`wareneingang` w
    WHERE w.`Hersteller` like '%$firma%'
    ")
    or die (mysql_error()) 


Dachris, C4 und allen anderen ein herzliches Dankeschön (werde jetzt ein wenig an der Struktur arbeiten damit es lesefreundlicher wird)

[R. Woitaschik]

Hi dachris,

noch zu info:

PHP-Code:

die (echo mysql_error()); 


ist quatsch. Wenn wir schon bei den Programmierstandards sind, die Funktion "mysql_error()" liefert als ein Ergebnis einen String mit der Fehlermeldung, die Funktion "die()" erwartet einen String, den es vor dem Beenden ausgibt. Ein "echo" macht hier wohl wenig Sin.

Außerdem zum Thema "" . $variable . "" oder "$variable": Die Lesbarkeit eines Quelltextes hängt stark vom Syntax-Highlighting deines Editors ab. Wenn du also einen Editor nimmst, der in doppelten Anführungszeichen trotzdem alle Variablen andersfarbig darstellt (z.B. jEdit), dann ist die zweite Variante besser lesbar.

Außerdem ist es gerade die Stärke von PHP Daten und Ausgabe zu vermengen. Die Programmiersprache wird interpretiert nicht compiliert. Eine Trennung von Daten und Visualisierung führt hier fast immer zu einer Performanceeinbuße. Ich gebe zu, eine Anweisung der Form:

PHP-Code:

printf('<a href="%s">%s</a>', $url, $caption); 


ist besser Lesbar (falls man mit printf umgehen kann), aber diese Version ist schneller:

PHP-Code:

<a href"<?=$url?>"><?=$caption?></a>

Klar, bei kleinen Ausgaben macht es keinen Unterschied aber multipliziere mal die Anweisungen mit 100.
Ich bevorzuge bei der Programmierung mit PHP eine "parserfreundliche" Version, d.h. wenn möglichk, die Ausgabe direkt im Quelltext. Wie gesagt, mit einem guten Editor ist ein so geschriebener Text auch entsprechend gut lesbar.

Gruß
Richard

Zitat von dachris

Eigentlich meinte ich eher so....

PHP-Code:

 $result = mysql_query("SELECT Hersteller, Bezeichnung
    FROM wareneingang
    WHERE Hersteller='$firma'") 

or 

die (echo mysql_error()); 


Es geht ja nicht darum an Kleinigkeiten rumzumotzen oder ob man " kommentiert oder nicht.....

aber ich habe schon soviele Fehler (meistens selbst geabut) gesehen welche aufgrund von aneinandergefrickelten unkommentierten und unstrukturierten Code entstanden sind...

Die meisten Fehler lassen sich einfach vermeiden wenn man ein paar Programmierstandards einhält und dazu gehört nun mal korrekte sinnvolle Syntax, keine Kurzschreibweisen, Struktur und Kommentare sowie OOP sowie eine sinnvolle Nomenklatur....
Das unterscheidet nunmmal einen professionellen Code welcher skalierbar und weiterverwendbar ist vom typischen dirtyCoded Schnellschuss...

Es geht ja meist nicht nur darum das man selber Fhelr besser erkennt sondern das andere den Code evtl. weiterwarten müssen....

Aber ich bin dafür das wir für die Diskussion einen iegenen Thread aufmachen, das hat nun wirklich nix mehr mit dem Ursprungsthema zu tun....