PHP problem z komendą UPDATE SQL

Question

Hej ! Mam problem . Chciałbym zaktualizowac baze danych poprzez php lecz wyskakuje mi błąd :

Fatal error: Uncaught PDOException: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'Temat: serv, WHERE temat = serverlog.txt AND plik = Plik log' at line 5 in C:\xamppp\htdocs\save.php:153 Stack trace: #0 C:\xamppp\htdocs\save.php(153): PDO->prepare('UPDATE pliki ...') #1 {main} thrown in C:\xamppp\htdocs\save.php on line 153

Mój kod odpowiedzialny za to :

$sql = "UPDATE pliki   
                 SET akcja = $akcja[$t],
                 valid = $wynik[$t],
                 data = $daty[$t], 
                 WHERE temat = $temat AND plik = $pliki[$t] "   ;

                 $statement = $db->prepare($sql);
                 $statement ->execute();

Prosze Was o pomoc . Z góry dziękuje !

Answer 1

Wszystkie wartości, które w zapytaniu są stringami, wypadałoby zapisać w apostrofy lub cudzysłowy - nie wiem czym jest $akcja czy $wynik, ale już $temat to ewidentnie wygląda na stringa.

A jeszcze lepiej byłoby użyć bindowania, tzn. wszystkie wartości, które z zewnątrz idą do bazy bindować. Wtedy masz pewną obronę przed atakami sql injection.

Comments

Zapisałem w apostrofach lecz nic się nie zmieniło . W moim przypadku odpornoć na ataki nie jest potrzebna.

 

$sql = "UPDATE pliki   
                 SET akcja = '$akcja[$t]',
                 valid = '$wynik[$t]',
                 data = '$daty[$t]', 
                 WHERE temat = '$temat' AND plik = '$pliki[$t]' "   ;

 

---

W moim przypadku odpornoć na ataki nie jest potrzebna.

Raczej średnie założenie, polecam jednak obrać sobie nawyk i zawsze bindować. 

data = '$daty[$t]', 

Za ostatnią wartością przed WHERE usuń przecinek.

---

Serdecznie dziękuje !

---

Answer 2

Przykład łączenia z bazą MySQL:

$servername = "(nazwa hosta, lokalnie użyj 'localhost')";
$username = "(nazwa użytkownika, zwykle 'root')";
$password = "(hasło, zwykle puste)";
$dbname = "(nazwa bazy danych)";

// Create connection
$conn = new mysqli($servername, $username, $password, $dbname);
// Check connection
if ($conn->connect_error) {
    die("Connection failed: " . $conn->connect_error);
} 

$sql = "zapytanie";

if ($conn->query($sql) === TRUE) {
    // sukcess
} else {
    // blad
    echo "Bład: " . $conn->error;
}

$conn->close();

Jeśli chcesz mogę znaleźć błąd, tylko musisz mi dać twój kod PHP.

Comments

A dlaczego nie PDO?