[MySQL][PHP]Sprawdzanie czy rekord jest już w bazie Opcje

[StadiPL]

Witam po raz kolejny ;]
Postanowiłem dodać sobie możliwość sprawdzenia czy użykownik jaki chce się zarejestrować nie istnieje już w bazie oraz czy jego email nie został już użyty.

[PHP] pobierz, plaintext 
//Oczywiście łączę się z bazą danych ;p
$nickname_q = mysql_query("SELECT * FROM users WHERE nickname = '$nickname'") or die(mysql_error());
$nickname_check = mysql_num_rows($nickname_q);
$email_q = mysql_query("SELECT * FROM users WHERE email= '$email'") or die(mysql_error());
$email_check = mysql_num_rows($email_q);
 
//Nieistotna część kodu
else if ($nickname_check=1) {echo $existing_nickname;}
else if ($email_check=1) {echo $extisting_email;}
{
echo 'Wykonanie skryptu ';
 
//na razie wartość id zmieniam ręcznie, ale dodam wkrótce dodam żeby automatycznie sczytywał największą wartość i dodawał o 1 wyższą
$id="1";
 
$ins = @mysql_query("INSERT INTO users SET id='$id', nickname='$nickname', password='$password', email='$email' ");
if ($ins) echo "OK"; else echo "LIPA";
[PHP] pobierz, plaintext 

Czemu nie działa poprawnie?

Ten post edytował StadiPL 12.05.2013, 04:28:25

[Sobak]

[PHP] pobierz, plaintext 
$nickname_check=1
[PHP] pobierz, plaintext 

= to operator przypisania. Ty chcesz porównać wartości - czyli użyj ==. Już nie wgłębiając się w szczegóły, w tym wypadku użycie = zwróci Ci zawsze true więc efekt jest niezgodny z oczekiwanym.

Jeśli chodzi o samo sprawdzenie czy rekord się powtarza, to pamiętaj, że w SQL masz funkcję COUNT() a najlepiej po prostu nadać polu UNIQUE (IMG:style_emoticons/default/smile.gif)

na razie wartość id zmieniam ręcznie, ale dodam wkrótce dodam żeby automatycznie sczytywał największą wartość i dodawał o 1 wyższą

AUTO_INCREMENT na pole z ID w bazie jest tym czego szukasz.

[StadiPL]

No więc teraz ta część mojego kodu wygląda tak:

[PHP] pobierz, plaintext 
$nickname_check=count($nickname);
$email_check=count($email);
 
//Sprawdzenie warunków czy można dokonać rejestracji
if(empty($nickname)){$no_nickname;}
else if(empty($password) || empty($password_r)){echo $no_password;}
else if($password != $password_r){echo $different_password;}
else if(empty($email) || empty($email_r)){echo $no_email;}
else if($email != $email_r){echo $different_email;}
else if(strlen($nickname) < 3 || !preg_match('/^[A-Za-z0-9]+$/', $nickname)) {echo $bad_nickname;}
else if(strlen($password) < 3 || !preg_match('/^[A-Za-z0-9]+$/', $password)) {echo $bad_password;}
else if(!filter_var($email,FILTER_VALIDATE_EMAIL)){echo $bad_email;}
else if($nickname_check==1) {echo $existing_nickname;}
else if($email_check==1) {echo $extisting_email;}
else {
echo 'Wykonanie skryptu ';
 
$ins = @mysql_query("INSERT INTO users SET nickname='$nickname', password='$password', email='$email' ");
if ($ins) echo "OK"; else echo "LIPA";
 
mysql_close($connection);
}
[PHP] pobierz, plaintext 

I zawsze zwróci mi wartość $existing_nickname przeglądałem milion razy i nigdzie nie widze błędu. ktos mnie oswieci? ;]

[session]

[PHP] pobierz, plaintext 
("SELECT COUNT(*) FROM `users` WHERE `nickname` = '".$nickname."'")
[PHP] pobierz, plaintext 

Spróbuj tak i już potem nie licz żadnym count(), ani mysql_num_rows(), takie zapytanie samo zwróci Ci ilość.

[PHP] pobierz, plaintext 
$nickname_check=mysql_fetch_row($nickname_q);
if($nickname_check[0]!=0){echo $existing_nickname;}
[PHP] pobierz, plaintext 

-- EDIT --
@Sobak przeoczyłem, że pisałeś o COUNT()

Ten post edytował session 12.05.2013, 10:49:49

[StadiPL]

Próbowałem już tym sposobem. Problem musi leżeć w tej części kodu:

[PHP] pobierz, plaintext 
if($nickname_check==1) {echo $existing_nickname;}
else if($email_check==1) {echo $extisting_email;}
else {
...
}
[PHP] pobierz, plaintext 

[session]

Tak problem jest w tym, że funkcją count() zliczasz ilość elementów tablicy, gdzie zawsze jest conajmniej 1, a nie wartość tych elementów. Spróbuj tak jak pisałem.

Ten post edytował session 12.05.2013, 10:48:59

[StadiPL]

Parse error: syntax error, unexpected 'SELECT' (T_STRING) in D:\_XAMPP\serwer\htdocs\log\register_function.php on line 30

[PHP] pobierz, plaintext 
$q1 = "("SELECT COUNT(*) FROM `users` WHERE `nickname` = '".$nickname."'")";
$r1 = mysql_query($q1);
$q2 = "("SELECT COUNT(*) FROM `users` WHERE `email` = '".$email."'")";
$r2 = mysql_query($q2);
 
//Sprawdzenie warunków czy można dokonać rejestracji
if(empty($nickname)) {echo $no_nickname;}
else if(empty($password) || empty($password_r)){echo $no_password;}
else if($password != $password_r){echo $different_password;}
else if(empty($email) || empty($email_r)){echo $no_email;}
else if($email != $email_r){echo $different_email;}
else if(strlen($nickname) < 3 || !preg_match('/^[A-Za-z0-9]+$/', $nickname)) {echo $bad_nickname;}
else if(strlen($password) < 3 || !preg_match('/^[A-Za-z0-9]+$/', $password)) {echo $bad_password;}
else if(!filter_var($email,FILTER_VALIDATE_EMAIL)){echo $bad_email;}
else if($r1==1) {echo $existing_nickname;}
else if($r2==1) {echo $existing_email;}
else
[PHP] pobierz, plaintext 

[session]

[PHP] pobierz, plaintext 
$nickname_q = mysql_query("SELECT COUNT(*) FROM `users` WHERE `nickname` = '".$nickname."'") or die(mysql_error());
$nickname_check = mysql_fetch_row($nickname_q);
$email_q = mysql_query("SELECT COUNT(*) FROM `users` WHERE `email`= '".$email."'") or die(mysql_error());
$email_check = mysql_fetch_row($email_q);
 
//Nieistotna część kodu
else if ($nickname_check[0]!=0) {echo $existing_nickname;}
else if ($email_check[0]!=0) {echo $extisting_email;}
{
echo 'Wykonanie skryptu ';
 
//na razie wartość id zmieniam ręcznie, ale dodam wkrótce dodam żeby automatycznie sczytywał największą wartość i dodawał o 1 wyższą
$id="1";
 
$ins = @mysql_query("INSERT INTO users SET id='$id', nickname='$nickname', password='$password', email='$email' ");
if ($ins) echo "OK"; else echo "LIPA";
[PHP] pobierz, plaintext 

INSERT i tak Ci nie zadziała prawidłowo, ponieważ podajesz błędne id, musisz ustawić w MySQL pole id na auto_increment a w zapytaniu INSERT przypisać do id wartość NULL: id=NULL.

Ten post edytował session 12.05.2013, 11:10:13

[StadiPL]

Może zapodam po prostu najnowszą wersję całości, bo przykładowo ten problem już rozwiązałem (ktoś na forum mi podpowiedział):

[PHP] pobierz, plaintext 
<?php
//zamiana zmiennych POST na zwykłe zmienne
$nickname=$_POST['nickname'];
$password=$_POST['password'];
$password_r=$_POST['password_r'];
$email=$_POST['email'];
$email_r=$_POST['email_r'];
 
//Wyświetlane teksty
$no_nickname="Podanie nazwy uzytkownika jest wymagane";
$no_password="Podanie hasla i jego powtorzenie jest wymagane";
$different_password="Podane hasla roznia sie'";
$no_email="Podanie e-maila i jego powtorzenie jest wymagane";
$different_email="Podane e-maile roznia sie";
$bad_nickname="Niedozwolona nazwa uzytkownika";
$bad_password="Niedozwolone haslo";
$bad_email="Nieprawidłowy adres email";
$existing_nickname="Istnieje już użytkownik '$nickname' w bazie. Wybierz inna nazwę uzytkownika!";
$existing_email="Adres email '$email' już istnieje w bazie";
 
//Dodać łączenie się z bazą danych. Najlepiej z osobnego zabezpieczonego pliku
$connection = @mysql_connect('127.0.0.1', 'root', '')
or die('Brak połączenia z serwerem MySQL.<br />Błąd: '.mysql_error());
$db = @mysql_select_db('jink', $connection)
or die('Nie mogę połączyć się z bazą danych<br />Błąd: '.mysql_error());
 
//Zapytania o występowanie podanych danych oraz zapisanie wyniku tej operacji w postaci 0 lub 1
/*$nickname_check=count($nickname);
$email_check=count($email);*/
$q1 = '("SELECT COUNT(*) FROM `users` WHERE `nickname` = '".$nickname."'")';
$r1 = mysql_query($q1);
$q2 = '("SELECT COUNT(*) FROM `users` WHERE `email` = '".$email."'")';
$r2 = mysql_query($q2);
 
//Sprawdzenie warunków czy można dokonać rejestracji
if(empty($nickname)) {echo $no_nickname;}
else if(empty($password) || empty($password_r)){echo $no_password;}
else if($password != $password_r){echo $different_password;}
else if(empty($email) || empty($email_r)){echo $no_email;}
else if($email != $email_r){echo $different_email;}
else if(strlen($nickname) < 3 || !preg_match('/^[A-Za-z0-9]+$/', $nickname)) {echo $bad_nickname;}
else if(strlen($password) < 3 || !preg_match('/^[A-Za-z0-9]+$/', $password)) {echo $bad_password;}
else if(!filter_var($email,FILTER_VALIDATE_EMAIL)){echo $bad_email;}
else if($r1==1) {echo $existing_nickname;}
else if($r2==1) {echo $existing_email;}
else
 
 
/*if($nickname_check==1) {echo $existing_nickname;}
else if($email_check==1) {echo $extisting_email;}
else*/ {
echo 'Zarejestrowano ';
 
//Generowanie tokenu
$token_znaki = '1234567890qwertyuiopasdfghjklzxcvbnmQWERTYUIOPASDFGHJKLZXCVBNM';
$token = '';
for ($i=0; $i<18; $i++) { $token .= $token_znaki[rand()%(strlen($token_znaki))]; } // 18 to długość ciągu
 
$ins = @mysql_query("INSERT INTO users SET nickname='$nickname', password='$password', email='$email', token='$token' ");
if ($ins) echo "pomyślnie"; else echo "niepomyślnie";
 
mysql_close($connection);
}
?>
[PHP] pobierz, plaintext 

[session]

Dlaczego nie zmienisz na to jak Ci napisałem ? (IMG:style_emoticons/default/tongue.gif) Twój kod nie ma prawa działać prawidłowo. Jest w nim dużo błędów zaczynając od nieprawidłowego używania ' , brak odczytu wyniku zapytania, problem z wyświetlaniem błędów użytkownikom (pisałem o tym w innym Twoim temacie (IMG:style_emoticons/default/smile.gif) )

[StadiPL]

To nie zmieniłem na to co mi napisałeś? ;D Jak to źle używam ' ? Użytkownikom wszystkie błędy wyświetla dobrze.. nie licząc tego nie działającego sprawdzania czy nick i email juz występuje, z którym się już nieźle męczę na wszystkie sposoby.

[session]

Zamień:

[PHP] pobierz, plaintext 
$q1 = '("SELECT COUNT(*) FROM `users` WHERE `nickname` = '".$nickname."'")';
$r1 = mysql_query($q1);
$q2 = '("SELECT COUNT(*) FROM `users` WHERE `email` = '".$email."'")';
$r2 = mysql_query($q2);
[PHP] pobierz, plaintext 

Na:

[PHP] pobierz, plaintext 
$nickname_q = mysql_query("SELECT COUNT(*) FROM `users` WHERE `nickname` = '".$nickname."'") or die(mysql_error());
$nickname_check = mysql_fetch_row($nickname_q);
$email_q = mysql_query("SELECT COUNT(*) FROM `users` WHERE `email`= '".$email."'") or die(mysql_error());
$email_check = mysql_fetch_row($email_q);
[PHP] pobierz, plaintext 

Oraz:

[PHP] pobierz, plaintext 
else if($r1==1) {echo $existing_nickname;}
else if($r2==1) {echo $existing_email;}
[PHP] pobierz, plaintext 

Na:

[PHP] pobierz, plaintext 
else if ($nickname_check[0]!=0) {echo $existing_nickname;}
else if ($email_check[0]!=0) {echo $extisting_email;}
[PHP] pobierz, plaintext 

Niepoprawnie używasz właśnie w tym co Ci nie działa (IMG:style_emoticons/default/wink.gif) Praktycznie w trakcie zamykasz ' a potem ciąg dalszy którego nie łączysz

[StadiPL]

A więc pozmieniałem tak jak mówisz.. Gdy podaje dane nieistniejącego użytkownika to działa. Gdy podaje dane z istniejącym loginem to prawidłowo wyskakuje błąd. Natomiast gdy podam dane z użytym już adresem email, to jest coś takiego: Istnieje już użytkownik 'kupa' w bazie. Wybierz inna nazwę uzytkownika!
On powinien to wyświetlić gdy uzytkownik juz istnieje. w przypadku emaila jest zmienna $extisting_email
EDIT: czyli w sumie wracam do punktu wyjscia ;d

Ten post edytował StadiPL 12.05.2013, 11:45:08

[session]

No nie bardzo jest to możliwe, albo istnieje już kupa i działa dobrze ten błąd, albo pozamieniałeś zmienne z błędami. Albo masz coś źle w formularzu, np. dwa razy nickname, innej możliwości nie widzę. Posprawdzaj dobrze (IMG:style_emoticons/default/wink.gif)

[StadiPL]

YEEEAHH! ;] To byla zwykla !@#!@#!@# literówka! A sie na nia wnerwilem ;]

[session]

"$extisting_email" ? (IMG:style_emoticons/default/tongue.gif)

[StadiPL]

A tak jeszcze wracając na chwilkę, to lepsze będzie te Twoje rozwiązanie czy count? ;]

[session]

W jakim znaczeniu lepsze ? COUNT(*) jako komenda MySQL zwraca liczbę wyników, a funkcja count() z PHP liczy ilość elementów w tablicy, czyli coś co kompletnie tutaj Cie nie interesuje. Tylko i wyłącznie COUNT(*) w zapytaniu MySQL.