[PHP][MySQL]Co nie tak w tym kodzie? Opcje

[misterious]

To znów ja....
Co jest tutaj nie tak:

[PHP] pobierz, plaintext 
$host = "host";
$user = "user";
$pass = "haslo";
$baza = "baza";
$polaczenie = mysql_connect($host, $user, $pass) or die("Błąd serwera bazy danych: " . mysql_error());
$baza = mysql_select_db($baza)or die("Błąd bazy danych: " . mysql_error());
$ip = $_SERVER['REMOTE_ADDR'];
$data = time();
$przegladarka = $_SERVER["HTTP_USER_AGENT"];
$query = "INSERT INTO `ip_phpbb` (`ip`, `czas`, `przegladarka`) VALUES ('$ip', '$data', '$przegladarka')";
mysql_close(); 
[PHP] pobierz, plaintext 

?

Ten post edytował misterious 7.11.2012, 13:26:26

[viking]

Zapewne to:

Kod

MYSQL. Use of this extension is discouraged. Instead, the MySQLi or PDO_MySQL extension should be used. See also MySQL: choosing an API guide and related FAQ for more information. Alternatives to this function include:

    mysqli_connect()
    PDO::__construct()

[misterious]

nie rozumiem...

[b4rt3kk]

Brakuje wykonania zapytania.

[PHP] pobierz, plaintext 
$query = "INSERT INTO `ip_phpbb` (`ip`, `czas`, `przegladarka`) VALUES ('$ip', '$data', '$przegladarka')";
mysql_query($query);
[PHP] pobierz, plaintext 

a viking chciał powiedzieć, że odradzane jest używanie mysql_, wkrótce zostanie zdeprecjonowane.

[redeemer]

Dodatkowo Twój kod jest podatny na SQL injection.

[misterious]

Już działa... Przeoczyłem

[PHP] pobierz, plaintext 
mysql_query($query);
[PHP] pobierz, plaintext 

. Co do SQL injection - wiem o tym To ma tylko funkcjonować