Zapytanie do MySQL, Problem z obiektowym zapytaniem do MySQL Opcje

[jml]

Mam taką bazę danych:

[SQL] pobierz, plaintext 
CREATE TABLE user (
username varchar(20) NOT NULL PRIMARY KEY,
password char(40) NOT NULL,
email varchar(30) NOT NULL,
number tinyint(7) UNSIGNED NOT NULL
);
[SQL] pobierz, plaintext 

Taki skrypt php:

[PHP] pobierz, plaintext 
<?php
function reg_register($user, $pass, $email, $number)
{
	$connect=connect_db();
	$result=$connect->query("select * from user where username='$user'");
	if(!$result)
	{
		throw new exception('Username already exist.');
	}
	// zapisanie wyniku do bazy danych
	$result=$connect->query("insert into user values('$user',sha1('$pass'),'$email','$number')");
	if(!$result)
	{
		throw new exception('MySQL server is offline, please try again later.');
	}
}
}
?>
[PHP] pobierz, plaintext 

Nie działa mi kod w tym miejscu:

[PHP] pobierz, plaintext 
<?php
$result=$connect->query("insert into user values('$user',sha1('$pass'),'$email','$number')");
?>
[PHP] pobierz, plaintext 

W tym miejscu wywala mi błąd.

Co muszę poprawić? Siedzę już nad tym sporo czasu i w necie nic nie znalazłem. Chcę to zrobić obiektowo.

Ten post edytował jml 10.02.2007, 00:17:22

[bne]

Kod

<?php
$result = $connect -> query("insert into user(username, pass, email, number) values('".$user."',sha1('".$pass."'),'".$email."',".$number.")");
?>

Ten post edytował bne 10.02.2007, 00:46:37

[jml]

Ciągle nie działa.

[bne]

A poprawiłeś pass na password? Czy przepisałeś to co podałem nawet nie sprawdzając co tam pisze?

[jml]

Sorki, nie zauważyłem... Poprawione i też nie działa.

[siemakuba]

Chcę to zrobić obiektowo.

Zacznij od nieobiektowego mysql_error zaraz po zapytaniu i pokaż/przeanalizuj zwrócony błąd. To, że wywala ci błąd niestety mówi niewiele ;)

pozdr.

[jml]

Mam w bloku catch coś takiego:

[PHP] pobierz, plaintext 
catch(exception $e)
{
	make_head1(error);
	?>	<!-- w tym bloku metatagi i style, wszystko co w head oprócz title -->	
<style type="text/css">@import url("style.css");</style>
	<?php
	make_head2();
 
 
	echo $e->getmessage();
 
	make_side(1);
	make_foot();
}
[PHP] pobierz, plaintext 

Niestety 'echo $e->getmessage();' nie pokazuje mi żadnego komunikatu o błędzie - czy coś tu jest źle tzn w żądaniu komunikatu błędu?

Próbowałem uzyskać błąd za pomocą mysql_error() ale po pierwsze nie wiem jak to zastosować dokładnie (tzn stosowałem, bez rezultatu), po drugie musiałbym sporo przerobić żeby wywołać komunikat o o błędzie tą funkcją.

'echo $e->getmessage();' powinno chyba pokazać komunikat błędu, prawda?

[Cienki1980]

Skoro nie wiesz jak posłużyć się mysql_error() to wyświelt zapytanie które tworzysz w 11 lini tego kodu co pokazałeś.
Dla pewności zrób np tak:

[PHP] pobierz, plaintext 
<?php
echo "insert into users .....";
exit;
?>
[PHP] pobierz, plaintext 

Zobacz co Ci powie phpMyAdmin bądź też jakieś inne narzędzie do MySQL'a z którego korzystasz podczas próby wykonania tego zapytania "z ręki".

[jml]

Ok, wszystko jest w porządku. Błąd znalazłem wpisując zapytanie do bazy z wiersza poleceń.

Cały skrypt jest w porządku, błąd był w pliku sql tworzącym bazę, wystarczyło zmienić tinyint na int w kolumnie 'number' (za dużych liczb używałem jak na tinyint). Ehhh, 2 dni szukałem tego błędu... No ale nic, dziękuję wszystkim za pomoc

Jeszcze jedno pytanie, odnośnie tego fragmentu powyższego kodu:

[PHP] pobierz, plaintext 
<?php
$result=$connect->query("select * from user where username='$user'");
	if(!$result)
	{
		throw new exception('Query to database not successed.');
	}
if(!$connect->num_rows>0)
	{
		throw new exception('Username already exist.');
	}
?>
[PHP] pobierz, plaintext 

Sprawdza on, czy podany user instnieje już w bazie danych. Jeśli jednak w bazie jest user o nazwie 'ktos', a chcemy zarejestrować nowego usera 'ktos1', to wywali nam, że taki już istnieje. Jak zrobić, żeby nie można było zarejestrować tylko identycznego usera, a nie też takiego, który ma podobny człon jak podawany w formularzu?

Ten post edytował jml 10.02.2007, 13:50:11

[siemakuba]

[PHP] pobierz, plaintext 
<?php
if(!$connect->num_rows>0)
?>
[PHP] pobierz, plaintext 

No, nie wiem co tam się dzieje w twoim obiekcie $connect, ale ten warunek chyba trochę przekombinowałeś. Sprawdzasz czy są już rekordy z pasującym loginem ($connect->num_rows>0) ale na początek odwracasz logiczną wartość tego sprawdzenia przez !. Czyli tak naprawdę, warunek się sprawdzi jeżeli nie będzie żadnych rekordów zwróconych przez zapytanie. IMHO usunięcie wykrzyknika powinno wystarczyć.

pozdr.

[jml]

[PHP] pobierz, plaintext 
<?php
if(!$connect->num_rows>0)
?>
[PHP] pobierz, plaintext 

No, nie wiem co tam się dzieje w twoim obiekcie $connect, ale ten warunek chyba trochę przekombinowałeś. Sprawdzasz czy są już rekordy z pasującym loginem ($connect->num_rows>0) ale na początek odwracasz logiczną wartość tego sprawdzenia przez !. Czyli tak naprawdę, warunek się sprawdzi jeżeli nie będzie żadnych rekordów zwróconych przez zapytanie. IMHO usunięcie wykrzyknika powinno wystarczyć.

pozdr.

Dzięki, poza tym powinienem wywoływać to na rzecz obiektu 'result' Teraz działa.

Ten post edytował jml 10.02.2007, 19:53:32