[MySQL][PHP]Sprawdzenie czy 2 wartości istnieją w bazie Opcje

[Turson]

Piszę skrypt rejestracji i ma sprawdzić, czy w bazie danych istnieje już login jaki podał użytkownik oraz czy jest już tam taki e-mail.

[PHP] pobierz, plaintext 
if(mysql_query("SELECT login FROM users WHERE login=$login") OR mysql_query("SELECT mail FROM users WHERE mail=$mail"))
{
echo 'Konto z takim loginem lub adresem e-mail już istnieje.';
}
else
{
/* OK */
}
[PHP] pobierz, plaintext 

Ale działa to źle. Podam:
istniejący już login + nowy mail = info o błędzie
nowy login + istniejący mail = bez błędu

[Greg0]

A co z kombinacjami
istniejący już login + istniejący mail
nowy login + nowy mail
Bo na dobrą sprawę na razie wystarczy ci sprawdzić tylko login

[widmo_91]

[PHP] pobierz, plaintext 
$wp = mysql_query("SELECT login FROM users WHERE login=$login OR mail=$mail");
if(mysql_num_rows($wp))
{
echo 'Konto z takim loginem lub adresem e-mail już istnieje.';
}
else
{
/* OK */
}
[PHP] pobierz, plaintext 

Ten post edytował widmo_91 21.07.2013, 23:38:30

[Turson]

@widmo_91
Istniejący login + istniejący mail PRZECHODZI

[widmo_91]

Jeżeli OK jest jak: nie ma loginu i nie ma email albo jest login i jest email to:

[PHP] pobierz, plaintext 
mysql_query("SELECT login FROM users WHERE login=$login XOR mail=$mail");
if(mysql_num_rows())
{
echo 'Konto z takim loginem lub adresem e-mail już istnieje.';
}
else
{
/* OK */
}
[PHP] pobierz, plaintext 

Ten post edytował widmo_91 21.07.2013, 22:55:40

[Turson]

Nie, nie o to chodzi. Ma być:
Istniejący login + nowy mail = FALSE
Nowy login + istniejący mail = FALSE
Istniejący login + istniejący mail = FALSE

Ten post edytował TursoN 21.07.2013, 22:57:08

[Greg0]

1.

istniejący już login + nowy mail = info o błędzie
nowy login + istniejący mail = bez błędu

2.

Istniejący login + nowy mail = FALSE
Nowy login + istniejący mail = FALSE

Zdecyduj się i wypisz wszystkie kombinacje + dla których ma być błąd, a dla których nie

Ten post edytował Greg0 21.07.2013, 23:03:21

[Turson]

Pomyliłem się.

Ma przejść tylko w jednym przypadku - gdy żadna z wartości nie widnieje jeszcze w bazie. Inne opcje mają dać błąd.

[widmo_91]

No to masz odpowiedź w pierwszym moim poście w tym temacie.

[Turson]

Nie do końca. Jeśli sprawdzę login i nie ma takiego w bazie, to skrypt założy nowe konto, a tymczasem jest jeszcze druga wartość do sprawdzenia, która nie może w tejże bazie być.

istniejący login+nowy mail=FALSE
istniejący login+istniejący mail=FALSE
nowy login + istniejący mail = FALSE

nowy login+nowy mail=TRUE

Czyli jeśli przynajmniej jedna z wartości login lub mail istnieje w bazie skrypt ma nie przepuścić tego

[widmo_91]

Rezygnuję z dalszych wyjaśnień, zagadnienie mnie przerosło

[Turson]

Wymyśliłem dziwną alternatywę...

[PHP] pobierz, plaintext 
$valid=0;
 
if(mysql_query("SELECT login FROM users WHERE login=$login"))
{}
else
{
$valid++;
}
 
if(mysql_query("SELECT mail FROM users WHERE mail=$mail"))
{}
else
{
$valid++;
}
if($valid)==2)
{
/* zakłada konto */
}
else
{
echo "Login lub haslo istnieje";
}
[PHP] pobierz, plaintext 

Ma to racje bytu?

Ten post edytował TursoN 21.07.2013, 23:25:09

[widmo_91]

Wcale nie ma to racji bytu.
Gratuluje udziwnienia zacznijmy od tego, że mysql_query nie zwraca liczby wyników!!! Zwraca zasób i jeżeli zapytanie jest poprawne to zawsze da true, polecam zainteresować się mysql_num_rows()

Ten post edytował widmo_91 21.07.2013, 23:39:21

[Turson]

Nie założyłem, że mysql_query zwraca liczbę wyników.

Jeśli login nie istnieje $valid++, jeśli mail nie istnieje $valid++
o to chodziło

[widmo_91]

wiem o co chodziło ale mysql_query dla selecta zawsze zwróci true jeżeli zapytanie jest poprawne a niekoniecznie zapytanie musi zwracać jakieś wyniki.
Napisałeś to samo co 1 poście tylko 3 razy dłużej i 9 razy bardziej skomplikowanie.
Poza tym zamiast:

[PHP] pobierz, plaintext 
if(mysql_query("SELECT login FROM users WHERE login=$login"))
{}
else
{
$valid++;
}
[PHP] pobierz, plaintext 

Polecam:

[PHP] pobierz, plaintext 
if(!mysql_query("SELECT login FROM users WHERE login=$login")) 
$valid++;
[PHP] pobierz, plaintext 

Ten post edytował widmo_91 22.07.2013, 00:03:24

[Turson]

Rozwiązanie było jednak banalne...

[PHP] pobierz, plaintext 
if(mysql_query("SELECT login FROM users WHERE login=$login")==FALSE)
{
if(mysql_query("SELECT mail FROM users WHERE mail=$mail")==FALSE)
{
echo 'Konto z takim adresem e-mail już istnieje.<br><a href="index.php?page=rejestracja">-Powrót-</a>';
}
else
{
mysql_query("INSERT INTO users VALUES (null, '$login', '$mail', '$password', 0, 0)") or die(mysql_error());
$lh = $login . $password;
$md5 = md5($lh);
mysql_query("INSERT INTO kod_aktywacyjny VALUES ('$login', '$md5')") or die(mysql_error());
echo "Konto zostało założone.";
}
}
[PHP] pobierz, plaintext 

Ten post edytował TursoN 22.07.2013, 00:03:35

[r4xz]

@TursoN: nadal nie rozumiesz...

Return Values

For SELECT, SHOW, DESCRIBE, EXPLAIN and other statements returning resultset, mysql_query() returns a resource on success, or FALSE on error.

For other type of SQL statements, INSERT, UPDATE, DELETE, DROP, etc, mysql_query() returns TRUE on success or FALSE on error.

The returned result resource should be passed to mysql_fetch_array(), and other functions for dealing with result tables, to access the returned data.

Use mysql_num_rows() to find out how many rows were returned for a SELECT statement or mysql_affected_rows() to find out how many rows were affected by a DELETE, INSERT, REPLACE, or UPDATE statement.

mysql_query() will also fail and return FALSE if the user does not have permission to access the table(s) referenced by the query.

mysql_query

[Turson]

[PHP] pobierz, plaintext 
if(mysql_query("SELECT login FROM users WHERE login=$login"))
{
//jeśli istnieje 
}
else
{
// jeśli nie istnieje
}
[PHP] pobierz, plaintext 

Więc nie działa to tak? Nie sprawdzam ile jest wartości, ale czy istnieje w ogole

[widmo_91]

Ty sprawdzasz czy zapytanie jest poprawne a nie czy istnieje taki login w bazie.
Ja nie wiem jak ty sprawdzasz poprawność swojego kodu :|
Powtórzę ostatni raz poprawne rozwiązanie masz w 3 poście w tym temacie.

[Turson]

Sprawdzałem wszystkie kombinacje i działa prawidłowo...

Oto cały kod:

[PHP] pobierz, plaintext 
<?PHP 
 
 
 
			if(isset($_POST['submit']))
			{
				$login = mysql_real_escape_string(htmlspecialchars($_POST["login"]));
				$mail = mysql_real_escape_string(htmlspecialchars($_POST["mail"]));
				$password = md5($_POST["password"]);
				$password2 = md5($_POST["password2"]);
				$kod = $_POST["kod"];
				$wylosowany = $_POST["wylosowany"];
 
				if(!empty($login) && !empty($mail) && !empty($password) && !empty($password2) && !empty($kod))
				{
					if(($password)==($password2))
					{
						if(strlen($login)>2 && strlen($password)>4)
						{
							if(($wylosowany)==($kod))
							{
								if(mysql_query("SELECT login FROM users WHERE login=$login")==FALSE)
									{
										if(mysql_query("SELECT mail FROM users WHERE mail=$mail")==FALSE)
										{
											echo 'Konto z takim adresem e-mail już istnieje.<br><a href="index.php?page=rejestracja">-Powrót-</a>';
										}
										else
										{
											mysql_query("INSERT INTO users VALUES (null, '$login', '$mail', '$password', 0, 0)") or die(mysql_error());
											$lh = $login . $password;
											$md5 = md5($lh);
											mysql_query("INSERT INTO kod_aktywacyjny VALUES ('$login', '$md5')") or die(mysql_error());
											echo "Konto zostało założone.";
										}
 
									}
									else{echo 'Konto z takim loginem już istnieje.<br><a href="index.php?page=rejestracja">-Powrót-</a>';}
							}
							else
							{
								echo 'Błędnie przepisany kod z obrazka<br><a href="index.php?page=rejestracja">-Powrót-</a>';
							}
						}
							else 
							{
								echo 'Login lub hasło są za krótkie<br><a href="index.php?page=rejestracja">-Powrót-</a>';
							}
						}
					else
					{
						echo 'Hasła nie są takie same<br><a href="index.php?page=rejestracja">-Powrót-</a>';
					}
				} 
				else 
				{
					echo 'Nie zostały uzupełnione wszystkie pola formularza.<br><a href="index.php?page=rejestracja">-Powrót-</a>';
				}
			}
			else
			{
			/* Kod z obrazka */
				$l1 = rand(1,9);
				$l2 = rand(1,9);
				$l3 = rand(1,9);
				$l4 = rand(1,9);
				$l5 = rand(1,9);
				$losuj_kod = $l1 . $l2 . $l3 . $l4 . $l5;
			echo "
			<form action='index.php?page=rejestracja' method='post'>
			<table style='width:400px;margin:0;padding:0;background:none;border-spacing:0px;'>
				<tr style='margin:0;padding:0;background:none;'><td class='rej_td'>Login</td><td><input type='text' name='login' class='logowanie_form' maxlength=20></td><td class='rej_info'>(min. 3 znaki)</td></tr>
				<tr style='margin:0;padding:0;background:none'><td class='rej_td'>Adres e-mail</td><td><input type='text' name='mail' class='logowanie_form' maxlength=40></td></tr>
				<tr style='margin:0;padding:0;background:none'><td class='rej_td'>Hasło</td><td><input type='password' name='password' class='logowanie_form' maxlength=20></td><td class='rej_info'>(min. 5 znaków)</td></tr>
				<tr style='margin:0;padding:0;background:none'><td class='rej_td'>Powtórz hasło</td><td><input type='password' name='password2' class='logowanie_form' maxlength=20></td></tr>
					<tr style='margin:0;padding:0;background:none'><td></td><td><div class='obrazek'>";
					$img=ImageCreateFromJPEG('images/kod_bg.jpg');
					$kolor_bialy = ImageColorAllocate($img, 234, 0, 11);
					$nazwa = "nazwa.jpg";
					ImageFill($img, 0, 0, 0);
					ImageString($img, 5, 60, 15, $losuj_kod, $kolor_bialy);
					imagepng($img, $nazwa);
					ImageDestroy($img);
					echo "<img src=\"$nazwa\" alt=\"Wygenerowany obrazek\" />";
					echo "
					</div></td></tr>
				<tr style='margin:0;padding:0;background:none'><td class='rej_td'>Kod z obrazka</td><td><input type='text' name='kod' class='logowanie_form' maxlength=5></td></tr>
			</table>
			<input type='hidden' value=$losuj_kod name='wylosowany'>
			<input type='submit' value='Załóż konto' class='logowanie_submit' name='submit'>";
			}
			?>
[PHP] pobierz, plaintext