PHP: Error in your SQL syntax

so richtig kenne ich mich damit nicht aus aber ich würde auf das ";" in 
der abfrage tippen.

Flo schrieb:
> Also kommt der Fehler nur wenn der Mysql keine Tabellen zurückliefert.

dann liefere doch einfach einen wert zurück:

$browser_sql  = "SELECT Count(*) as Anz FROM `BrowserOS` WHERE browser = 
'$user_agent'";

und dann solltest du dich mir Prepared SQL beschäftigen, denn so hast du 
schon die erste Sicherheitslücke in deinem Programm.

Flo schrieb:
> $insert_sql = "INSERT INTO BrowserOS (browser) VALUES ($user_agent);";

versuche es mal damit:

$insert_sql = "INSERT INTO BrowserOS (browser) VALUES ('$user_agent');";

eingast schrieb:
> versuche es mal damit:
>
> $insert_sql = "INSERT INTO BrowserOS (browser) VALUES ('$user_agent');";

Der Insert ist nicht das Problem, wenn ich die if Abfrage auskommentiere 
dann trägt er den Wert richtig ein, das Problem liegt nur in der Abfrage 
wenn es noch keinen Eintrag gibt..

Das mit Count ist eine gute Idee, das werde ich ausprobieren

Rufus Τ. Firefly schrieb:
> Warum hier "Backticks" um BrowserOS und Hochkommata um $user_agent,
> $insert_sql = "INSERT INTO BrowserOS (browser) VALUES ($user_agent);";
>
> ... wenn sie hier nicht nötig sind?

ja sieht merkwürdig aus, ist aber ok.

das war dann ein Copy&Paste Fehler von mir... habe mir teilweise die 
Querys aus PHPMyadmin kopiert.

ich habe es jetzt mal mit count versucht, an das Ergebnis (im moment 0 
weil die Tabelle leer ist) komme ich noch nicht ganz:

1

$browser_result = mysql_query($browser_sql) OR die(mysql_error());

2

$count = mysql_result($browser_result);

3

4

if($count == 0){

5

  $insert_sql = "INSERT INTO BrowserOS (browser) VALUES $user_agent);";

6

  $insert_result = mysql_query($insert_sql) OR die(mysql_error());

7

  echo "Wird hinzugefuegt";    

8

    };

Ich glaube mysql_result($browser_result); ist hier nicht das richtige 
oder?
Es kommt weiterhin:You have an error in your SQL syntax; check the 
manual that corresponds to your MySQL server version for the right 
syntax to use near '(Windows NT 6.1; WOW64)...'

Karl Heinz schrieb:
> Vor allen Dingen fehlt hier ein ; im SQL Statement

nein, auch das ist nicht der Fehler. Das Braucht man um im Client das 
Kommando an den Server zu übergeben. Wenn man die Datenbank über PHP 
anspricht ist es nicht notwendig.

vergessen:  $browser_sql  = "SELECT count(*) FROM `BrowserOS` WHERE 
browser = '$user_agent'";

Flo schrieb:
> Der Insert ist nicht das Problem, wenn ich die if Abfrage auskommentiere
> dann trägt er den Wert richtig ein, das Problem liegt nur in der Abfrage
> wenn es noch keinen Eintrag gibt..

Würde ich jetzt mal so nicht behaupten, denn die Fehlermeldung deutet 
auf das ";" in der Browserbezeichnng hin, welches als SQL-Kommandoende 
interpretiert wird. Deshalb versuche es erstmal mit den "'" und dann 
kann man weiter diskutieren!

eingast schrieb:
> Würde ich jetzt mal so nicht behaupten, denn die Fehlermeldung deutet
> auf das ";" in der Browserbezeichnng hin, welches als SQL-Kommandoende
> interpretiert wird. Deshalb versuche es erstmal mit den "'" und dann
> kann man weiter diskutieren!

SUPER, danke, der Fehler kam vom insert...

Jetzt habe ich es so:

1

2

$browser_sql  = "SELECT count(*) FROM `BrowserOS` WHERE browser = $user_agent'";

3

4

$browser_result = mysql_query($browser_sql) OR die(mysql_error());

5

$count = mysql_result($browser_result);

6

7

if($count == 0){

8

  $insert_sql = "INSERT INTO BrowserOS (browser) VALUES    ('$user_agent')";

9

  $insert_result = mysql_query($insert_sql) OR die(mysql_error());

10

  echo "Wird hinzugefuegt";    

11

    };

leider Ignoriert er jetzt das if($count == 0) und macht den Insert bei 
jedem aufruf.

Flo schrieb:
> leider Ignoriert er jetzt das if($count == 0) und macht den Insert bei
> jedem aufruf.

das ist ein anderes Thema und dabei wird dir die Doku helfen...

Flo schrieb:
> $count = mysql_result($browser_result);
>
> if($count == 0){

ist ja auch Blödsinn.

woher soll er denn wissen welche Spalte du haben willst?

Nachtrag:

du musst doch mindestens erst mal eine Zeilen "fetchen" und dann kann 
man von der Zeile eine Spalte abfragen.

Das Problem lag nur am Insert, ich habe es jetzt wie folgt gelöst:

1

$browser_sql  = "SELECT * FROM `BrowserOS` WHERE browser = '$user_agent'";

2

$browser_result = mysql_query($browser_sql) OR die(mysql_error());

3

4

    if(mysql_num_rows($browser_result)<1){  

5

    $insert_sql = "INSERT INTO BrowserOS (browser) VALUES  

6

                ('$user_agent')";

7

    $insert_result = mysql_query($insert_sql) OR 

8

                die(mysql_error());

9

    echo "Wird hinzugefuegt";    

10

    };

Εrnst B✶ schrieb:
> Und: mach das unbedingt noch sicher!
>
> Wie oben schon geschrieben: So ist das eine Sicherheitslücke.
>
> Der "User Agent" ist vom Webseiten-Besucher frei vorgebbar,

Wäre das eine mögliche Lösung?
$user_agent = str_replace("'","Einbruchsversuch",$user_agent);

Super, danke für den Tip! Ich habe mir mal eben ein Beispiel gesucht und 
es auf meine Anwendung angepasst, das mit dem schließen ist mir noch 
etwas unklar...

1

<?php 

2

$handler = new mysqli("dbhost", "dbuser", "dbpass", "db");

3

4

$getuseragent = $handler->prepare("SELECT * FROM people WHERE (user_agent = ?)");

5

$getuseragent->bind_param('s', $user_agent);

6

$user_agent = $_SERVER['HTTP_USER_AGENT'];

7

$getuseragent->execute();

8

9

if (!$getuseragent->get_result()) {

10

    echo("Browser nicht vorhanden, wird hinzugefuegt...");

11

    $insertbrowser = $handler->prepare("INSERT INTO BrowserOS (browser) VALUES (?)");

12

  $insertbrowser->bind_param("s", $browsername);

13

  $name = $user_agent;

14

    $insertbrowser->execute();

15

  //nicht sicher ob das hier stimmt

16

  $insertbrowser->close();

17

18

}; 

19

  //nicht sicher ob das hier stimmt

20

$getuseragent->close();

21

 ?>

Flo schrieb:
> if (!$getuseragent->get_result()) {
>     echo("Browser nicht vorhanden, wird hinzugefuegt...");

das sieht mir komisch aus. Auch ein leeres Result ist auch ein Result. 
Es kommt aus einen Select immer eine Result zurück, eventuell hat es 
aber 0 Zeilen.

vermutlich ist es dann so besser:
$getuseragent->execute();
$getuseragent->store_result();

if ($stmt->num_rows<1) {
....

Falls du noch keine Lösung hast:

http://php.net/manual/de/function.mysql-num-rows.php


<?php

$link = mysql_connect("localhost", "mysql_user", "mysql_password");
mysql_select_db("database", $link);

$result = mysql_query("SELECT * FROM table1", $link);
$num_rows = mysql_num_rows($result);

echo "$num_rows Zeilen\n";

?>

Auf ersten Augenblick schein mir so zu sein (ich hab nicht ausprobiert):
Du hast den Connection ($link) für mysql_query nicht übergeben. Damit 
ist es ein Aufrufproblem (kein SQL Problem an der Stelle), $result war 
undefiniert, und dann bei mysql_num_rows ist es rausgestiegen.

Das ist nur ein Tipp ;)