[MySQL][PHP] chyba problem z funkcami warunkowymi, to ma być w przyszłości skrypt logowania Opcje

[Mr.Niceguy]

ok przyznam, że w php jestem noobem, jednak do tygodnia staram się jak
mogę ;]

nich ktoś mi powie dlaczego ten super skrypt nie działa
z bazą danych się łączy

[PHP] pobierz, plaintext 
<?php
 
include ('funkcje/db_konekt.php');
 
$pass="haslo";
$user="nazwa@o2.pl";
 
function checkPass(){
 
        db_konekt();
        $zap1="SELECT * FROM users WHERE email='".$user."'";
        $result= mysql_query($zap1);
        $row = mysql_fetch_assoc($result);
        if(!$row){
                echo ("Probem z bazą danych. Odrzucone zapytanie.");
 
}
 
if($row["password"] != $pass){
        echo("Podałes nieprawidłowe dane");
        return false;
        }
 
if($row["password"] === $pass){
        echo("Podałes prawidlowe dane");
        return true;
        }
 
}
 
?> 
[PHP] pobierz, plaintext 

[PanGuzol]

Linie 8 do 17 to definicja funkcji checkPass której nigdzie nie wywołujesz. Czyli kod między 8 a 17 nigdy się nie wykonał.
Jeśli hasło będzie różne to zastosuj else a nie stosuj drugiego ifa dla przypadku jak będzie prawidłowe.

[darko]

Domyślam się, że jednak wywołujesz funkcję checkPass() gdzieś dalej. Błąd może tkwić w tym, że zmienna $pass jest niewidoczna wewnątrz funkcji checkPass(). Poczytaj o zasięgu zmiennych, to są podstawy. Inne możliwości, to:
- jednak nie ma połączenia z bazą
- błędne użycie funkcji mysql_fetch_assoc (zastosuj pętlę while lub skorzystaj z funkcji mysql_fetch_row)

[Mr.Niceguy]

ok, więc zastosowałem pętle while i zastąpiłem drugiego ifa funkcją else
no i dalej jest problem. Skrypt działa jeśli zmienna $user jest taka jak w bazie czyli nazwa@o2.pl , jeśli ja zmienię (w skrypcie nie w bazie) to zamiast komunikatu "Dane NIEprawidłowe" mam błąd połączenia

[PHP] pobierz, plaintext 
<?php
include ('funkcje/db_konekt.php');
 
$user="nazwa@o2.pl"; //jak tu zmieniam wartość zmiennej tak żeby była inna niż w bazie to wyświetla błąd 
 
db_konekt();
$result = mysql_query("SELECT * FROM users WHERE email='".$user."'");
while (!$row = mysql_fetch_assoc($result)) {
	echo ("Problem!!");
	mysql_close();
	}
 
   // echo $row["name"];
		//echo '<br>';
	//echo $row["password"];
		//echo '<br>';
 
if ($row["email"]!=$user){
				echo ("Dane NIEprawidłowe") ;
				//mysql_close();
			}					
			else{
			echo ("Dane prawidłowe");
			//mysql_close();
			}
 
?>
[PHP] pobierz, plaintext 

[darko]

Trochę namieszałeś spróbuj tak:

[PHP] pobierz, plaintext 
$result = mysql_query("SELECT email FROM users WHERE email='" . $user . "'") or die(mysql_error());
$row = mysql_fetch_row($result);
 
if ($row[0] != $user)
{
  echo 'dane nieprawidłowe';
}
else
{
  echo 'ok';
}
[PHP] pobierz, plaintext 

Jeśli dalej będziesz miał błąd połączenia, to przekaż do funkcji mysql_query zasób połączenia (zwrócony przez funkcją mysql_connect). Teraz przynajmniej będziesz miał komunikat błędu.

[Mr.Niceguy]

dzięki... jednak zostanę przy funkcji mysql_fetch_assoc, jest dla mnie bardziej czytelna


witam, ten skrypt ma mi sprawdzać po emailu, czy ktoś może się dodać do bazy
powinien zwracać "Użytkownik o takim emailu juz istnije w naszje bazie"
zamiast tego mam Duplicate entry 'ja@com.pl' for key 'PRIMARY'

[PHP] pobierz, plaintext 
<?php 
include ('funkcje/db_konekt.php');
 
db_konekt();
 
//w bazie istnieje taki wiersz 
$pass="lala";
$email="ja@com.pl";
$name="stefan";
 
$zap0= sprintf("SELECT 'email' FROM users WHERE email='$email'");
$res=mysql_query($zap0) or die (mysql_error());
 
$row2 = mysql_fetch_assoc($res);
 
		if ($row2['email'] == $email)
			{
			echo ("Użytkownik o takim emailu juz istnije w naszje bazie<br>") ;
			die;
			}					
			else{
			$zap1 = sprintf("INSERT INTO users (`email` , `password` , `name`) VALUES ('$email' , '$pass' , '$name') ");
 
			$result = mysql_query($zap1)  or die (mysql_error());
			exit;
			}
 
?>
[PHP] pobierz, plaintext 

[darko]

pokaż:

[PHP] pobierz, plaintext 
print_r($row2['email']);
[PHP] pobierz, plaintext 

Dlaczego nie po prostu:

[PHP] pobierz, plaintext 
$res=mysql_query("SELECT email FROM users WHERE email='" . $email . "'") or die (mysql_error());
[PHP] pobierz, plaintext 

?

[Mr.Niceguy]

Dlaczego nie po prostu:

[PHP] pobierz, plaintext 
$res=mysql_query("SELECT email FROM users WHERE email='" . $email . "'") or die (mysql_error());
[PHP] pobierz, plaintext 

?

bo nie lubię długich wierszy w kodzie

następny tyci problemik
chodzi o to co ma się dziać jeśli nie wypełni się pola tekstowego w formie, a powinno się wypełnić

[PHP] pobierz, plaintext 
<?php
 
$z= $_POST['email'];
 
if ($z = false ){               <=== to 'false' to ściema. po prostu kombinowałem ;]  
echo ("podaj email");
}
else{
print ("$z");
}
?>
[PHP] pobierz, plaintext 

[darko]

[PHP] pobierz, plaintext 
if(isset($_POST['email']))
{
//...
[PHP] pobierz, plaintext 

// edit
poczytaj chociaż jakiś poradnik do php a najlepiej jakąś polskojęzyczną książkę, bo zapis:
if($z = false)
nie oznacza sprawdzenia, sprawdzamy tak:
if($z == false)
to są kompletne podstawy


Ten post edytował darko 21.12.2010, 23:35:56

[modern-web]

= to przyporządkowanie, natomiast == to 'równa się'
Zapamiętaj to sobie. Najczęściej popelniany błąd wsród początkujących programistów

Podobnie poczytaj o instrukcjach warunkowych - wykorzystuje się je praktycznie wszędzie, a co za tym idzie jest to podstawa...
Polecam Ci lekturke na noc: manual

Pozdrawiam i dobrej nocy życzę

[Mr.Niceguy]

Witam!
czuję się zobowiązany przed Panem darko pozkazać jak mi idzie nauka php
tyle udało mi się zrobić do tej pory hello world, hello php ;]
oraz dziękuję darko oraz modern-web za konstruktywny ochrzan ;]

ps. czasami mam wrażenie, ze css jest trudniejszy od php

ps2. pewnie zahakujecie to w 5sec :], 100% mój kod

Ten post edytował Mr.Niceguy 4.01.2011, 12:51:12

[darko]

Miło mi, proszę bardzo, ale... ekhm...
popraw:

TO JEST STRONA ZABESPIECZONA

Z
Jeszcze gdzieś był jakiś byk, ale nie pamiętam już gdzie. Po za tym sprawdzaj czy faktycznie udało się zarejestrować użytkownika, bo podejrzewam, że tego nie sprawdzasz - zarejestrowałem jakiegoś fikcyjnego koleżkę, a nie mogę się na niego zalogować, czyli o co chodzi? Został zarejestrowany czy nie ?

// edit
Whooa! Nie zauważyłem wcześniej. Spójrz w kod strony. Dołączasz dwukrotnie doctype, rozpoczynasz dwukrotnie <html> i sekcję <head> Popracuj nad tym, bo coś źle inkludujesz. Nie mam czasu sprawdzać zabezpieczeń jeżeli jakieś są w ogóle.

Ten post edytował darko 4.01.2011, 13:14:58

[Mr.Niceguy]

Miło mi, proszę bardzo, ale... ekhm...
popraw:

Z
Jeszcze gdzieś był jakiś byk, ale nie pamiętam już gdzie. Po za tym sprawdzaj czy faktycznie udało się zarejestrować użytkownika, bo podejrzewam, że tego nie sprawdzasz - zarejestrowałem jakiegoś fikcyjnego koleżkę, a nie mogę się na niego zalogować, czyli o co chodzi? Został zarejestrowany czy nie ?

// edit
Whooa! Nie zauważyłem wcześniej. Spójrz w kod strony. Dołączasz dwukrotnie doctype, rozpoczynasz dwukrotnie <html> i sekcję <head> Popracuj nad tym, bo coś źle inkludujesz. Nie mam czasu sprawdzać zabezpieczeń jeżeli jakieś są w ogóle.

Sorry za byka;}
został zarejestrowany, po prostu konta były nieaktywne,
konta są już aktywne

poprawiłem wyświetlającą się informacyjne na
Niewłaściwy email lub hasło <br>Lub<br>
Twoje konto nie jest aktywowane!

Linie 8 do 17 to definicja funkcji checkPass której nigdzie nie wywołujesz. Czyli kod między 8 a 17 nigdy się nie wykonał.
Jeśli hasło będzie różne to zastosuj else a nie stosuj drugiego ifa dla przypadku jak będzie prawidłowe.

Mówisz o kodzie, który wklejałem na forum? Ten kod uległ radykalnym modyfikacjom ;]

Ten post edytował Mr.Niceguy 4.01.2011, 14:02:55