PHP, dodawanie informacji do bazy danych

Question

Hej, pomoże ktoś, jestem nowy w PHP, i nie mogę naprawić tego błędu.

Chodzi o mysql_query("INSERT INTO `uzytkownicy` (`email`, `password`, `password2`, `name`, ``lastname) VALUES ('".$email."','".$password1."','".$password2."','".$name."','".$lastName."');");

wyskauje mi taki błąd 
Fatal error: Uncaught Error: Call to undefined function mysql_query() in C:\xampp\htdocs\ajax\connect.php:27 Stack trace: #0 C:\xampp\htdocs\ajax\register.php(4): require_once() #1 {main} thrown in C:\xampp\htdocs\ajax\connect.php on line 27

else {
    
    if (isset($_POST['register']))
    {
        $email = $_POST['email'];
        $password1 = $_POST['password1'];
        $password2 = $_POST['password2'];
        $name = $_POST['name'];
        $lastName = $_POST['lastName'];
        // sprawdzamy czy login nie jest już w bazie

        if($password1 == $password2)
        {
            mysql_query("INSERT INTO `uzytkownicy` (`email`, `password`, `password2`, `name`, ``lastname) VALUES ('".$email."','".$password1."','".$password2."','".$name."','".$lastName."');");
            echo "Konto zostało utworzone";
        }
        else
        {
            echo "Hasła nie są takie same!";
        }
        
        mysqli_close($connect);
    }
}

 

Comments

http://php.net/manual/en/function.mysql-connect.php

Answer 1

Po pierwsze, tą część możesz usunąć, nie jest wymagana a tylko zwiększa długość kwerendy w kodzie, wystarczy że wpiszesz zmienne w takiej kolejności jak kolumny w tabeli: ("INSERT INTO `uzytkownicy` (`email`, `password`, `password2`, `name`, ``lastname) VALUES ('".$email."','".$password1."','".$password2."','".$name."','".$lastName."');");

Po drugie - tak się nie robi: "'.$email.'" , wystarczy dać zmienną w samych apostrofach np. '$email', '$password' itd.

Po trzecie zapisz sobie połączenie do bazy w zmiennej np. $connect

$database_servername = "localhost";
$database_username = "root";
$database_password = "";
$database_name = "nazwabazy";

$connect = new mysqli($database_servername, $database_username, $database_password, $database_name);

a potem możesz zrobić tak: $connect->query("tutaj polecenie do bazy");

Na przyszłość jak nie wiesz dlaczego ci polecenie nie działa chociaż wydaje ci sie że wpisujesz dobrze to polecam ci spróbować wykonać to polecenie w konsoli phpmyadmin, tam ładnie pokazuje wszystkie błędy

 

Comments

Doszlifuj php, bo masz lekkie braki. Mam nadzieje, że używasz edytora z podświetlaniem skladni. Jeśli masz problemy z jakimś zapytaniem SQL, to po prostu wklej sobie do osobnego pliku i wybierz SQL z listy skladni, lub zmien rozszerzenie pliku na sql.

Masz poprawnie zrobiony twoj przyklad:

function connect(): mysqli
{
    static $mysqli;
    if (null === $mysqli) {
        $mysqli = mysqli_connect(); // new mysqli();
        mysqli_report(MYSQLI_REPORT_STRICT);
    }

    return $mysqli;
}

function validateUser(string $email, string $password1, string $password2, string $name, string $lastname): bool
{
    if ($password1 != $password2) {
        return false;
    }

    if (existUser($email)) {
        return false;
    }

    return true;
}

function existUser(string $email): bool
{
    $stmt = connect()->prepare("SELECT COUNT(*) as `count` FROM `uzytkownicy` WHERE email = ?;");
    $stmt->bind_param('s', ...func_get_args());
    $stmt->execute();

    return $stmt->get_result()->fetch_assoc()['count'] > 0;
}

function addUser(string $email, string $password1, string $password2, string $name, string $lastname)
{
    $stmt = connect()->prepare("INSERT INTO `uzytkownicy` (`email`, `password`, `password2`, `name`, `lastname`) VALUES (?,?,?,?,?);");
    $stmt->bind_param('sssss', ...func_get_args());
    $stmt->execute();
}

Pożniej dajesz tylko: 

if(validUser($email, $password1, $password2, $name, $lastname)) {
    addUser($email, $password1, $password2, $name, $lastname);
}

Nie wiem, po co wrzucasz password2 do bazy.

Nie wiem, czemu nie szyfrujesz password1 w bazie

Zamiast wysylac kazdy parametr z osobna do funkcji mozna wyslac cala tablice, co jest lepsze.

Wypadaloby dodac lepszy walidator. Sprawdzić dlugość imienia, nazwiska, czy znaki są poprawne, czy email to email. etc

Nie używamy mysql_*, bo już nie jest supportowany przez php.

---

dzięki wielkie! A czy możesz mi powiedzieć gdzie jest błąd w moim przykadzie?

---

wyskauje mi taki błąd  
Fatal error: Uncaught Error: Call to undefined function mysql_query() in C:\xampp\htdocs\ajax\connect.php:27 Stack trace: #0 C:\xampp\htdocs\ajax\register.php(4): require_once() #1 {main} thrown in C:\xampp\htdocs\ajax\connect.php on line 27

 Nie używamy mysql_*, bo już nie jest supportowany przez php.

W twojej wersji php nie masz tej funkcji. Zmień wszystkie mysql na mysqli 

---

no jak to, mam mysqli wszdzie

 

<?php

$host = "localhost";
$user = "root";
$pass = "";
$base = "users";

$connect = new mysqli($host,$user,$pass,$base);

    //rejestracja
    if (isset($_POST['register']))
    {
        $email = $_POST['email'];
        $password1 = $_POST['password1'];
        $password2 = $_POST['password2'];
        $name = $_POST['name'];
        $lastName = $_POST['lastName'];
        // sprawdzamy czy login nie jest już w bazie

        if($password1 == $password2)
        {
            $connect->query("INSERT INTO `uzytkownicy` (`id`, `email`, `password`, `password2`, `name`, `lastname`) VALUES ('NULL', '$email','$password1','$password2','$name','$lastName')");
        }
        else
        {
            echo "Hasła nie są takie same!";
        }
    }
    
    //logowanie
    if(isset($_POST['login']))
    {
        $lemail = $_POST['lemail'];
        $lpass = $_POST['lpassword'];
        if ($connect->query("SELECT email, password FROM uzytkownicy WHERE email = '$lemail' AND password = '$lpass'"))
            {
               echo "Zalogowano";
            }
        else
            {
                echo "Wpisano złe dane";
            }
    }
?>

 

---

Z tego błędu wynika, że w pliku  connect.php na lini lini 27 uzywasz mysql_query.

Answer 2

Chyba pomyślałeś nazwę funkcji.

mysqli_query, zamiast

mysql_query.

Comments

nie pomogło