Frage: Eintrag in die Datenbank per PHP

[TW1920]

Hallo,

ich habe ein problem mit php. ich weiß nicht wie man daten von einer variable in eine datenbank speichert. ich weiß wie man eine db-verbindung herstellt und wie man normal daten speichert. allerdings muss ich über einen variablenwert eine gewisse zeile aktualisieren.

Code: Alles auswählen

		$sql = "UPDATE *entfernt*_topics
			SET feed = $feed,
			WHERE topic_id = $topic";
		$db->sql_query($sql);

habe das mal so probiert, aberes funktioniert nicht. Ich hoffe mir kann einer helfen

Danke im vorraus

[Pyramide]

aberes funktioniert nicht.

Windows startet neu? Der Monitor explodiert? Eine Fehlermeldung wird angezeigt?

KB:programmierfragen

[P7BB]

guck mal hinter "$feed"... da ist ein komma und ich denke, dort sollte keins sein

[TW1920]

Nein , leider hat das nix geholfen.

Code: Alles auswählen

		$db->sql_transaction('begin');

		$sql = "UPDATE twc_de_topics
			SET star = $star
			WHERE topic_id = " $topic;
		$db->sql_query($sql);

		$sql = "UPDATE twc_de_topics
			SET feed = $feed
			WHERE topic_id = " $topic;
		$db->sql_query($sql);


		$db->sql_transaction('commit');

In derZeile vom oberen Topic ist Zeile 62:

Code: Alles auswählen

Parse error: syntax error, unexpected T_VARIABLE in /var/www/vhosts/twcmail.de/httpdocs/de/star.php on line 62

Ich weiß nicht warum der fehlerda ist.

wenn ich

Code: Alles auswählen

		$db->sql_transaction('begin');

		$sql = "UPDATE twc_de_topics
			SET star = $star
			WHERE topic_id =  $topic ";
		$db->sql_query($sql);

		$sql = "UPDATE twc_de_topics
			SET feed = $feed
			WHERE topic_id =  $topic ";
		$db->sql_query($sql);


		$db->sql_transaction('commit');

Dann kommt nur:

SQL ERROR [ mysql4 ]

Unknown column '$topic' in 'where clause' [1054]

An sql error occurred while fetching this page. Please contact an administrator if this problem persists.

Ich hoffe jemand kann damit was anfangen,

danke im vorraus...

[gn#36]

In der oberen Variante fehlt ein Punkt zwischen " und $topic; Die zweite sieht so aus als würde es eine Variable namens $topic nicht geben...

[P7BB]

Beim 2. muss es nicht ... = $topic sein, sondern ... = '$topic'
angenommen $topic hätte den wert "bla". Dann würde dort bei der jetzigen Variante stehen:
WHERE ... = bla
Es muss aber dort = 'bla' stehen
Zumindest hatte ich einmal einen ähnlichen Fehler und da lag es tatsächlich an den Anführungsstrichen

[TW1920]

danke, es funktioniert nun.

[gn#36]

Naja wenn er mit einem Feld vergleicht in dem eine Zahl steht braucht er die Anführungszeichen eigentlich nicht, denn ob der Wert eine Zahl ist sollte man in jedem Fall vorher prüfen wenn man sowas in ein Query einbaut.

[P7BB]

ok, man muss es nicht machen, aber ich mach es normalerweise trotzdem immer, weil es eben dann eindeutig ist

[gn#36]

Eigentlich nicht. Denn die Anführungszeichen sind für Strings, nicht für Zahlen. Du solltest vorher prüfen ob du eine Zahl einfügst oder nicht, vor allem wenn das Feld eigentlich nur Zahlen akzeptiert. Um Fehler in deinen Programmen zu finden ist es meist besser wenn sie Probleme nicht durch die Verwendung von falschen Datentypen umgehen. Das macht das Verhalten im Fehlerfall dann nämlich abhängig von weiteren Komponenten auf die du möglicherweise keinen Einfluss hast. Vielleicht wird ein String, sollte er in der Variablen stehen, dann in eine 0 umgewandelt? Vielleicht gibt's einen SQL Fehler? Vielleicht gibts auch eine SQL Injection weil du nicht ganz aufgepasst hast um das zu verhindern? Wenn ein Programm falsche Daten bekommt sollte es den Dienst verweigern und nicht versuchen "irgendwas" mit den Daten zu machen.