GET-VARIABLE in SQL-where-Klausel

carmelita · 24.11.2008, 11:51

Hallo
Ich möchte in einem Affenformular (eingabe1.php) das Abfragekriterium abfragen und dann ausgeben. Irgendwas mit der Zeichensetzung krieg ich nicht hin (Parse error: syntax error, unexpected T_VARIABLE in C:\xampp\htdocs\eingabe1.php on line 26).

Danke für Hilfe

PHP-Code:

  <?php

if ( empty ($_GET['sprache']) == TRUE )

 
{

echo '

 
      <form action="eingabe1.php" method="get" name="formular1" id="formular1">

 
      <p> Sprache:

      <input name="sprache" value="' . $_GET['sprache'] . '"/></p>

 
      <p> Teil:

      <input name="teil" value="' . $_GET['teil'] . '"/></p>

 
      <p>

      <input type="submit" name="sendebutton" value="absenden"/>  </p>

 
      </form>';

}

 
else

{

 
                   $db = mysql_connect (localhost,root,Kennwort1);

                   $SQL_Befehl = "SELECT Lernziel FROM tb_lernziele ";

                          $sqlab .= " WHERE tb_lernziele.teil = "$_GET['$teil']"";

                          $sqlab .= " WHERE tb_lernziele.sprache = "$_GET['sprache']"";

                   $resultat = mysql_db_query("aaa", $SQL_Befehl);

 
                   while ($row = mysql_fetch_array($resultat))

                   {

                   echo ("<option value=\"$row[Lernziel]\">$row[Lernziel]</option>");

                   }

 
 
 
}

 
?>

cycap · 24.11.2008, 11:55

Dann guck doch mal wie du es oben bei den Input-Feldern gemacht hast und vergleich es mit dem SQL-String, zu mal 2 mal WHERE wird der SQL-Server sowieso nicht annehmen.

carmelita · 24.11.2008, 12:02

Hallo du superprofi
steht in diesem Forum PHP Einsteiger??
Wie wärs, wenn du mir helfen würdest?
Hab Dank

nikosch · 24.11.2008, 12:16

cycap hat aber Recht. Beides sind Stringkombinationen und damit vergleichbar. Und oben ist es ja richtig gemacht. Mehrere Bedingungen in SQL erreichst Du über AND- oder OR-Verknüpfungen, nicht über meherer WHERE clauses.

HPR1974 · 24.11.2008, 12:40

btw. benutzt Du einmal die Variable $SQL_Befehl und dann $sqlab.

WHERE tb_lernziele.teil = "$_GET['$teil']""; hier fehlen zwei Dinge: einfache ' um im sql die werte zu umschliessen und der sogenannte dot operator um die Strings zu verbinden

und dann noch was: $_GET['$teil'] hier hast du ein $ zuviel es sei denn Du hast eine Variable $teil dann wären aber zwei ' zuviel

HPR1974 · 24.11.2008, 12:41

achja zu mysql_db_query() aus dem manual:
Anmerkung: Diese Funktion ist seit PHP 4.0.6 veraltet. Verwenden Sie diese Funktion nicht. Stattdessen sollten Sie die Funktionen mysql_select_db() und mysql_query() verwenden.

HPR1974 · 24.11.2008, 12:43

$db = mysql_connect (localhost,root,Kennwort1);
Hast Du Deine Verbindungsdaten in Konstanten untergebracht? Ansonsten sind die parameter der Funktion in Anführungszeichen zu setzen.

nikosch · 24.11.2008, 12:52

@HPR1974
Auch Du darfst Dich an die Forenregeln halten:

- Code in [php] Tags, Zitate in [quote] Tags
- Beiträge editieren, statt ständig neue Postings aufzumachen

Und dann würde ich Dir zum einfachen Editormodus raten. Dann hast Du auch kein Farbproblem mehr.

cycap · 24.11.2008, 12:59

Zitat:

Zitat von carmelita

Hallo du superprofi
steht in diesem Forum PHP Einsteiger??
Wie wärs, wenn du mir helfen würdest?
Hab Dank

Also sorry, ich gehe nicht hin und schreibe deinen Code um. Ich habe dich schon mit der Nase darauf gestoßen, das muss doch wohl reichen. Du willst 2 mal das gleiche machen (Strings miteinander verbinden) und bei ersten mal machst du es richtig und beim SQL-String nicht...

HPR1974 · 24.11.2008, 13:02

@nikosch ich schreibe hier so wie es für mich am bequemsten ist.

- für wenig code die php tags zu benutzen ist müll, ebenso der editor hier ansich mit seinen farbkodierungen.
- zitieren aus anderen posts ja wenn ich aus dem manual zitiere nein
- ständiges editieren machen hier einige gerne, das verändert aber die posts manchmal so sehr dass der inhalt gar verfälscht wird

und so am Rande: Locker die Hälfte der Moderatoren Posts hier handeln von Forumsregeln, und php Tags. Also entweder sind die Regeln mit dem gegebenen Tool nicht gut einzuhalten, sie sind Schrott oder Ihr seid einfach unterbeschäftigt und meint ständig alles und jeden moderieren zu müssen.

Und bevor Du das hier als Diskussion in die Länge ziehst und ein thread mal wiedet OT wird schreib mir ne PN oder ne mail.

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

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Variable per Session übergeben? Spricht etwas dagegen?	Lapje	PHP Tipps 2008	6	20.10.2008 21:49
Undefined variable Undefined property: block	xxJJxx	PHP Tipps 2008	16	05.08.2008 15:30
erweiterte abfrage - where	DarkManX	Datenbanken	2	02.02.2007 17:14
Parsen von Delimitern	Basti	Tutorials	0	30.09.2006 21:24
Kleine Frage zu WHERE Klausel in SQL String	Plague	Datenbanken	1	09.08.2006 11:23
Variable in SQL Anweisung verwenden	Beatbox	PHP Tipps 2006	2	08.08.2006 16:32
Probleme mit Where Klausel		Datenbanken	4	07.02.2006 14:54
Variable durch 2. Variable bestimmen	Quagga	PHP Tipps 2005-2	11	26.10.2005 17:34
Befehlsoptimierung	faux	Datenbanken	4	31.05.2005 19:11
[Erledigt] Variable in DB		Datenbanken	1	17.04.2005 15:01
where string von einer funktion zu anderen transportieren		PHP Tipps 2005	5	03.03.2005 16:29
Variable in einer Variable	Beatbox	PHP Tipps 2005	7	15.01.2005 15:55
Warum wird Variable nicht übergeben???	Anuschka	PHP Tipps 2005	2	06.01.2005 13:22
2 Schleifen durchlaufen		PHP Tipps 2004	1	14.09.2004 14:33

24.11.2008, 11:51
carmelita Neuer Benutzer Registriert seit: 20.11.2008 Beiträge: 24	GET-VARIABLE in SQL-where-Klausel Hallo Ich möchte in einem Affenformular (eingabe1.php) das Abfragekriterium abfragen und dann ausgeben. Irgendwas mit der Zeichensetzung krieg ich nicht hin (Parse error: syntax error, unexpected T_VARIABLE in C:\xampp\htdocs\eingabe1.php on line 26). Danke für Hilfe PHP-Code: <?php if ( empty ($_GET['sprache']) == TRUE ) { echo ' <form action="eingabe1.php" method="get" name="formular1" id="formular1"> <p> Sprache: <input name="sprache" value="' . $_GET['sprache'] . '"/></p> <p> Teil: <input name="teil" value="' . $_GET['teil'] . '"/></p> <p> <input type="submit" name="sendebutton" value="absenden"/> </p> </form>'; } else { $db = mysql_connect (localhost,root,Kennwort1); $SQL_Befehl = "SELECT Lernziel FROM tb_lernziele "; $sqlab .= " WHERE tb_lernziele.teil = "$_GET['$teil']""; $sqlab .= " WHERE tb_lernziele.sprache = "$_GET['sprache']""; $resultat = mysql_db_query("aaa", $SQL_Befehl); while ($row = mysql_fetch_array($resultat)) { echo ("<option value=\"$row[Lernziel]\">$row[Lernziel]</option>"); } } ?>


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

24.11.2008, 11:55
cycap Moderator Registriert seit: 13.02.2008 Beiträge: 6.816 PHP-Kenntnisse: Fortgeschritten	Dann guck doch mal wie du es oben bei den Input-Feldern gemacht hast und vergleich es mit dem SQL-String, zu mal 2 mal WHERE wird der SQL-Server sowieso nicht annehmen.

24.11.2008, 12:02
carmelita Neuer Benutzer Registriert seit: 20.11.2008 Beiträge: 24	Kindergärtner Hallo du superprofi steht in diesem Forum PHP Einsteiger?? Wie wärs, wenn du mir helfen würdest? Hab Dank

24.11.2008, 12:16
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.988 PHP-Kenntnisse: Fortgeschritten	cycap hat aber Recht. Beides sind Stringkombinationen und damit vergleichbar. Und oben ist es ja richtig gemacht. Mehrere Bedingungen in SQL erreichst Du über AND- oder OR-Verknüpfungen, nicht über meherer WHERE clauses. __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

24.11.2008, 12:40
HPR1974 Erfahrener Benutzer Registriert seit: 28.03.2008 Beiträge: 1.847	btw. benutzt Du einmal die Variable $SQL_Befehl und dann $sqlab. WHERE tb_lernziele.teil = "$_GET['$teil']""; hier fehlen zwei Dinge: einfache ' um im sql die werte zu umschliessen und der sogenannte dot operator um die Strings zu verbinden und dann noch was: $_GET['$teil'] hier hast du ein $ zuviel es sei denn Du hast eine Variable $teil dann wären aber zwei ' zuviel

24.11.2008, 12:41
HPR1974 Erfahrener Benutzer Registriert seit: 28.03.2008 Beiträge: 1.847	achja zu mysql_db_query() aus dem manual: Anmerkung: Diese Funktion ist seit PHP 4.0.6 veraltet. Verwenden Sie diese Funktion nicht. Stattdessen sollten Sie die Funktionen mysql_select_db() und mysql_query() verwenden.

24.11.2008, 12:43
HPR1974 Erfahrener Benutzer Registriert seit: 28.03.2008 Beiträge: 1.847	$db = mysql_connect (localhost,root,Kennwort1); Hast Du Deine Verbindungsdaten in Konstanten untergebracht? Ansonsten sind die parameter der Funktion in Anführungszeichen zu setzen.

24.11.2008, 12:52
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.988 PHP-Kenntnisse: Fortgeschritten	@HPR1974 Auch Du darfst Dich an die Forenregeln halten: - Code in [php] Tags, Zitate in [quote] Tags - Beiträge editieren, statt ständig neue Postings aufzumachen Und dann würde ich Dir zum einfachen Editormodus raten. Dann hast Du auch kein Farbproblem mehr. __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

24.11.2008, 13:02
HPR1974 Erfahrener Benutzer Registriert seit: 28.03.2008 Beiträge: 1.847	@nikosch ich schreibe hier so wie es für mich am bequemsten ist. - für wenig code die php tags zu benutzen ist müll, ebenso der editor hier ansich mit seinen farbkodierungen. - zitieren aus anderen posts ja wenn ich aus dem manual zitiere nein - ständiges editieren machen hier einige gerne, das verändert aber die posts manchmal so sehr dass der inhalt gar verfälscht wird und so am Rande: Locker die Hälfte der Moderatoren Posts hier handeln von Forumsregeln, und php Tags. Also entweder sind die Regeln mit dem gegebenen Tool nicht gut einzuhalten, sie sind Schrott oder Ihr seid einfach unterbeschäftigt und meint ständig alles und jeden moderieren zu müssen. Und bevor Du das hier als Diskussion in die Länge ziehst und ein thread mal wiedet OT wird schreib mir ne PN oder ne mail.