Dlaczego ta pętla (for) nie działa?, Po prostu nic nie robi... Opcje

[Ripper]

Witam,

Tworzę skrypt i mam takie pytanko, otóż mam taką pętlę for:

[PHP] pobierz, plaintext 
<?php
$wolne_id = 1;
for ($sprawdzane_id = 1; $wolne_id = 0; ++$sprawdzane_id) {
	$sprawdz_w_bazie = dbquery("SELECT 1 FROM ".$db_prefix."users WHERE id='".$sprawdzane_id."'");
	$wolne_id = dbrows($sprawdz_w_bazie);
}
?>
[PHP] pobierz, plaintext 

Kod ma sprawdzać ile jest wyników w bazie danych (tabeli ...users) o danym id i zapisywać je do zmiennej $wolne_id, jeżeli nie będzie żadnego wyniku to $wolne_id = 0 i skrypt powinien się zatrzymać, a ja powinienem otrzymać i zmienną $wolne_id (która mi się nie przyda) i zmienną $sprawdzane_id kiedy $wolne_id wynosi 0 (będzie to id nowego usera rejestrującego się na stronie). Mógłbym równie dobrze zrobić, żeby liczyło userów i dodawalo usera o id wyższym niż jest, ale wtedy dublowaly by się id po usunięciu kogoś. funkcja dbquery i dbrows jest dobra gdyż używałem jej wiele razy wcześniej i działała (ma na celu to co w nazwie ;P), baza danych też jest odpowiednio przedstawiona... Co może być źle, aby znajdywało mi id, którego nie ma jeszcze w tabeli coś_users?

[nithajasz]

Chłopie konstrukcja pętli for się kłania.

[PHP] pobierz, plaintext 
<?php
for ( incjacja licznika; warunek dla licznika; operacje na liczniku){
	//kod
}
?>
[PHP] pobierz, plaintext 

Tak swoją drogą co Ty kombinujesz? Bo nie bardzo rozumiem..

Ten post edytował nithajasz 16.06.2008, 17:10:26

[nevt]

[PHP] pobierz, plaintext 
<?php
//zamiast
$wolne_id = 0
//daj
$wolne_id == 0
?>
[PHP] pobierz, plaintext 

przecież to ma być porównanie a nie podstawienie...

[Ripper]

@nithajasz - wiem z czego składa się pętla for, ale i tak dzięki za przypomnienie...
@nevt - wiem, zauważyłem to po napisaniu posta ale coś mi po tym komp zawiesił, reset i zapomnialem poprawić (oprócz tego, że mam "==" poróbowałem też != 1 (bo oprócz 0 może być tylko 1) i nic ...

pętla ma jak już mówiłem wybierać najmniejsze nieużyte id, czyli:
- Pobierać ilośc wyników z id równym X
- Liczyć ilość wyników
- Jeżeli ilość wyników nie równa się 0, zwiększyć X o jeden i powtórzyć pętlę (już ze zwiększonym X - czyli ID).

pętla ma po prostu zdobywać najmniejszy wolny id, który nie został użyty w bazi danych, dla nowego usera, który będzie się rejestrował. Poprawiłem w/w błędy i dalej nie może nic znaleźć, mógłby mi ktoś napisać od nowa taką pętlę? Jeśli chodzi o bazy danych to pole z id mam tak:
Baza: baza_test
tabela: bt_users
pole: id

najlepiej gdybyście zamiast całych regułek podstawili do kodu od razu z funkcjami dbquery i dbrows (użycie:
$zapytanie = dbquery("_ZAPYTANIE_"); np. $zapytanie = dbquery("SELECT nazwisko FROM personaila WHERE imie='Jan');
$licz = dbrows($zmienna_z_zapytania); np. $licz = dbrows($zapytanie);
coś takiego)

Z góry dziękuję i mam nadzieję, że pomożecie (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)

PS. Dobrze by było, gdyby zmienną odpowiadającą za id w pętli przy którym zwraca wynik $wolne_id zero, było zapisane do jakieś zmiennej tak, abym mógł go potem użyć w dalszej części pliku/lub po zaincludowaniu

Ten post edytował Ripper 16.06.2008, 18:22:41

[pyro]

[PHP] pobierz, plaintext 
<?php
for ($sprawdzane_id = 1; $wolne_id = 0; ++$sprawdzane_id) 
?>
[PHP] pobierz, plaintext 

Spójrz na to... Po pierwsze dałeś znak przypisania a nie porównania, powinno być ==...

Pętla for działa tak, że dopóki ine natknie się na wartość false to się wykonuje (czyli jak jest wartośc true), u ciebie to wygląda tak, że od razu masz wartość false bo $wolne id == 1, a ty dajesz warunek że musi mieć wartość 0, dlatewgo ma wartość false i nic nie robi, kumasz czacze?

[Mlodycompany]

petla powinna wygladac tak

[PHP] pobierz, plaintext 
<?php
for($i = 0; $i < 100; $i++;){
	//instrukcje
}
?>
[PHP] pobierz, plaintext 

Nie rozumiem twojej petli i jej zmiennych

[Ripper]

@pyro o == i = już wyjaśnione - w skrypcie mam == a nawet miałęm != więc też powinno działać (no i <= też miałem raz)
Czego nie wiedziałem, to że działa, aż nie napotka wartości false, a myślałem, że true :/ Spróbuje i odp.

@Mlodycomapny dzięki za ogólny zarys pętli for (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) (IMG:http://forum.php.pl/style_emoticons/default/haha.gif)

[tomeksobczak]

do Twojego zastosowania to chyba bardziej pasuje while a nie for

[nevt]

a nie powinno jeszcze być:

[PHP] pobierz, plaintext 
<?php
// zamiast
$wolne_id = dbrows($sprawdz_w_bazie);
// coś w stylu
$wolne_id = count(dbrows($sprawdz_w_bazie));
//albo
$wolne_id = dbrows($sprawdz_w_bazie)[0][0];
?>
[PHP] pobierz, plaintext 

(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

[Ripper]

Nie ;P Jak już pisałem te funkcje dbrows i dbquery (jak i inne) robią po prostu wszystko za mnie (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) podają gotową liczbę (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Kod już mi działa i znajduje (dodałem echo i widać co i jak) - teraz podam przykład:
Zarejestrowani są 2 userzy o id 1 i 2, po echo widać:
Id: 1 - Tak
Id: 2 - Tak
Id 3 - Nie

(Nie/Tak przez if zrobiłem (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) ) i teraz powinno być $sprawdzane_id jako 3 a zapisuje się tak jakby po następnym dopiero locie for i dodaje o jedno (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) (IMG:http://forum.php.pl/style_emoticons/default/haha.gif) poradziłem sobie odejmując zaraz za pętlą te id o 1 i wszystko śmiga. Temat do zamknięcia - bardzo dziękuję za pomoc - szczególnie @pyro - Leci "Pomógł" ;D Innym, którzy pomogli także ;D

[nithajasz]

pętla ma po prostu zdobywać najmniejszy wolny id, który nie został użyty w bazi danych, dla nowego usera, który będzie się rejestrował.

Moim zdaniem to jest złe rozwiązanie. Chyba, że w bazie przetrzymujesz jedynie userów i nic z nimi nie wiążesz, np jakiś komentarzy czy postów (lub przez nick co też mi się nigdy nie podobało). Ale to tylko taka moja mała uwaga.

[Ripper]

Po prostu robie sobie dla testów "Jak uda mi sie" własnego prostego cms'a - na razie zaczynam od rejestracji i logowania gdyż to będzie potrzebne mi potem - logowanie mam a przy rejestracji pętla ma wykrywać najmniejsze wolne ID - wolne czyli przypisane do żadnego usera. Dlaczego tak zrobiłem, a nie policzyć ile mam userów i dodać 1 do nich? Bo gdybym miał 20 userów, usunął tego z id powiedzmy 10, ktoś by się zarejestrował to były by 2 osoby z id 20 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) rozumiesz? (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif)

Pozdro, temat można zamknąć

[Shili]

A nie lepiej zrobić AUTO_INCREMENT? I nic nie musisz sam liczyć.

[tomeksobczak]

a słyszałeś o auto_increment albo jeżeli już musisz to robić ręcznie to max() ?