System logowania wyrzuca błędy., mysql_fetch_assoc(): supplied argument is not a valid MySQL result res Opcje

[Put3r]

Witam.
Na początku powinienem przeprosić za temat. Jest nieprecyzyjny, ale nawet nie wiem w czym leży problem.
Gdy tworzyłem strone, a dokładnie system logowania, w trakcie testowania skryptów coś się stało i teraz cały czas wyskakuje mi błąd:
Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in C:\...\inc.php on line 12
Nie wiem czy problem leży po stronie bazy danych, czy coś z sesją się popsuło.

Kod do pliku inc.php:

[PHP] pobierz, plaintext 
<?php
ob_start();
session_start();
mysql_connect('localhost','root','') or die(mysql_error());
mysql_select_db('gra2') or die(mysql_error());
function vtxt($var)
{
	return trim(mysql_real_escape_string(strip_tags($var)));
}
function row($sql)
{
	return mysql_fetch_assoc(mysql_query($sql));
}
 
function getUser($id)
{
	return row("SELECT * FROM user WHERE id=$id");
}
$user = getUser($_SESSION['id']);
?>
[PHP] pobierz, plaintext 

Jeżeli natomiast usunę linijkę 19:

[PHP] pobierz, plaintext 
$user = getUser($_SESSION['id']);
[PHP] pobierz, plaintext 

to wszystko działa dobrze. Szukałem rozwiązania w google ale nie znalazłem sensownej pomocy w tym problemi. Czytałem gdzies w necie, że z czasem powinno samo zacząć działać.

Czy jest jakiś sposób na szybkie rozwiązanie tego problemu?

[Turson]

Kod sam w sobie jest prawidłowy. Sprawdź co siedzi w $_SESSION['id']

[Kshyhoo]

[PHP] pobierz, plaintext 
function getUser($id)
{
  return row("SELECT * FROM user WHERE id=$id");
}
[PHP] pobierz, plaintext 

I subtelna różnica:

[PHP] pobierz, plaintext 
function getUser($id)
{
  return row("SELECT * FROM user WHERE id='$id'");
}
[PHP] pobierz, plaintext 

[Turson]

Ksyhoo, tak czy inaczej powinno działać bo sam sprawdzałem

[Put3r]

Udało się. Dodałem @ i działa:

[PHP] pobierz, plaintext 
function row($sql)
{
	return @mysql_fetch_assoc(mysql_query($sql));
}
[PHP] pobierz, plaintext 

a to '$id' i działa bez tego tez.

Tak czy inaczej @ chyba usówa błędy więc, chyba to nie załatwiło prawdziwego problemu, tylko go ukryło. Tak mi się wydaje.

[Kshyhoo]

Tak czy inaczej @ chyba usówa błędy więc, chyba to nie załatwiło prawdziwego problemu, tylko go ukryło. Tak mi się wydaje.

Nie usuwa, tylko maskuje. Błąd jest nadal.

[Turson]

Zamiast dwóch funkcji zrób jedną, która pobierze wyniki, bo chyba to jest przyczyną problemów

Ten post edytował Turson 26.12.2013, 15:10:54

[Put3r]

Zamiast których dwóch funkcji? Bo każda u mnie służy co czego innego i nie wiem które mógł bym scalić.

[Turson]

Usuń @ przy zapytaniu a dodaj to na końcu gdzieś i zobacz czy wyrzuci błąd

[PHP] pobierz, plaintext 
if(mysql_error()==true){
	echo mysql_error();
}
[PHP] pobierz, plaintext 

[Put3r]

Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in C:\...\inc.php on line 18

[Turson]

[PHP] pobierz, plaintext 
echo $_SESSION['id'];
[PHP] pobierz, plaintext 

na pewno daje integer?

[Put3r]

Raczej tak - zwraca id użytkownika.

Na wypadek dodałem jeszcze if by to sprawdzić:
echo $_SESSION['id'];
if(!ctype_digit($_SESSION['id'])) echo 'to nie liczba<br>';

Zauważyłem, że błąd wyskakuje tylko jak jestem niezalogowany.

[com]

no bo pewnie wtedy $_SESSION['id'] jest null

a odnośnie
http://forum.php.pl/index.php?s=&showt...t&p=1082181
To on ma w "" wiec '' niczego tam nie zmienia bo mysql jest na tyle idioto odporne że dla inta dla niego to bez znaczenia (IMG:style_emoticons/default/wink.gif)

musisz to przypisywać dopiero po zalogowaniu, bo dopiero wtedy nadajesz id jakoś wartość a tak ona nie istnieje (IMG:style_emoticons/default/wink.gif)

Ten post edytował com 27.12.2013, 20:03:06

[Put3r]

Jeżeli dobrze rozumiem to powinno być coś takiego?

[PHP] pobierz, plaintext 
function row($sql)
{
	if(!isset($_SESSION['id'])) 
	{
		return @mysql_fetch_assoc(mysql_query($sql));
	}
}
[PHP] pobierz, plaintext 

[com]

nie to co zrobiłeś nie ma sensu bo nic nie wnosi...

[PHP] pobierz, plaintext 
$user = getUser($_SESSION['id']);
[PHP] pobierz, plaintext 

daj dopiero po zalogowaniu

i nie wygłuszał tych błędów @ (IMG:style_emoticons/default/exclamation.gif)

[killzone]

Taka mała uwaga:

[PHP] pobierz, plaintext 
ob_start();
session_start();
mysql_connect('localhost','root','') or die(mysql_error());
mysql_select_db('gra2') or die(mysql_error());
[PHP] pobierz, plaintext 

http://www.php.net/manual/en/class.mysqli.php

Tak na przyszłość to lepiej używać klasy mysqli do zarządzania bazą.
A twoje komendy stracą wspracie.

[Put3r]

Dopiero zaczynam ogarniać podstawy php, klasy może sobie na razie odpuszczę (IMG:style_emoticons/default/smile.gif) Kupie jakąś książkę do php, zeby ogarnąć podstawy i potem się za to zabiorę. Bo ciężko tak z manuala słabo ogarniająć język, nauczyć się obsługi baz.