jezeli to jest koncowy kod to:
1) jezeli $haslo itd. to nazwy zmiennych prosto pobieranych z formularza to nie rob tak, tylko korzystaj z $_POST -> generalnie poczytaj o register globals
2) po co te dwa zapytania?
$zapytanie = "SELECT * FROM uzytkownicy"; $wykonaj = mysql_query($zapytanie);
$zapytanie = "SELECT login FROM uzytkownicy WHERE login='$login'";
wystarczy, ze wykonasz drugie zapytanie i sprawdzisz czy jest taki uzytkownik
3) mozesz nalozyc w projekcie bazy na pole login artybut UNIQUE przez co bedziesz mial zapewniona integralnosc danych w bd (przy zapytaniu, probie dodania uzytkownika o istniejacym loginie system uniemozliwi taka czynnosc)
4) nie musisz tworzyc zmiennej $date wytarczy ze zapiszesz:
<?php
, '\".date('Y-m-d G:i:s').\"')\";
?>
5) nie sprawdzasz czy zapytanie zostalo faktycznie wykonane, tylko z gory zakladasz, ze jest ok...
<?php
?>
ps. to kilka uwag na poczatek... (jeszcze mozna by napisac o braku walidacji danych... i innych)
---edit---
Cytat
Brakuje średnika zamień na:
Kod php:
1
2
3
4
5
<?php
$q ="INSERT INTO uzytkownicy (`login`, `haslo`, `imie`, 'nazwisko', 'mail', 'typ', 'data') VALUES ('$login' , '$haslo' , '$imie, '$nazwisko', '$mail', 'testowani', '$data');";
?>
Pozdrawiam
nie brakuje... w zapytaniach wysylanych z poziomu skryptu nie stosuje sie tego, jezeli natomiast pracujesz na konsoli - to tak...