Forum PHP.pl > [PHP] Usuwanie użytkowników pętlą po uwzględnieniu warunku

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [PHP] Usuwanie użytkowników pętlą po uwzględnieniu warunku

Lethys

10.08.2015, 11:44:06

Witam,

Chcę zrobić prosty skrypt, który mi wypiszę użytkowników którzy dodali artykuł ze słaba punktacją (-5 bądź mniej punktów). W tabeli artykuly, każdy artykuł ma kolumnę id_dodajacy na podstawie której chcę wypisać użytkowników ktorzy dodali słaby artykuł.

[PHP] pobierz, plaintext 
 
	  $artykul_dostep = mysql_fetch_array(mysql_query("select * from artykuly where punkty<=-5")) or mysql_error();
 
 
		  foreach($artykul_dostep as $userr)
	  {
		$dany_user = mysql_fetch_array(mysql_query("select * from uzytkownicy where id=$userr[id_dodajacego]")) or mysql_error();
 
		echo '<br>'.$dany_user[login].'</br>';	
	  }
 
 
[PHP] pobierz, plaintext

Co źle robię? Chcę to zrobić pętlą foreach.

kapslokk

10.08.2015, 11:47:28

Tak na pierwszy rzut oka to:

[PHP] pobierz, plaintext 
$dany_user[login]
[PHP] pobierz, plaintext

a powinno być

[PHP] pobierz, plaintext 
$dany_user['login']
[PHP] pobierz, plaintext

i w zapytaniu też nie masz apostrofów

Lethys

10.08.2015, 12:07:00

Cytat(kapslokk @ 10.08.2015, 11:47:28 )

Tak na pierwszy rzut oka to:

[PHP] pobierz, plaintext 
$dany_user[login]
[PHP] pobierz, plaintext

a powinno być

[PHP] pobierz, plaintext 
$dany_user['login']
[PHP] pobierz, plaintext

i w zapytaniu też nie masz apostrofów

Niby skrypt ruszył ale nie działa jak powinien. Wyświetla mi nick pierwszego z brzegu usera kilka razy (nie zgadadza się z liczbą z warunku), pomijając innych userów oraz co jakiś czas zostawiając puste linijki

Kishin

10.08.2015, 12:13:36

Po co robisz dodatkowe zapytanie w pętli?

Nie lepiej?

[SQL] pobierz, plaintext 
SELECT * FROM artykuly a
LEFT JOIN uzytkownicy u ON u.id_dodajacego=a.id
WHERE punkty<=-5
[SQL] pobierz, plaintext

i potem foreacha zrobić i wyciągasz już sobie co tylko chcesz

com

10.08.2015, 12:13:52

pokaż poprawiony kod

Lethys

10.08.2015, 12:25:12

Cytat(Kishin @ 10.08.2015, 12:13:36 )

Po co robisz dodatkowe zapytanie w pętli?

Nie lepiej?

[SQL] pobierz, plaintext 
SELECT * FROM artykuly a
LEFT JOIN uzytkownicy u ON u.id_dodajacego=a.id
WHERE punkty<=-5
[SQL] pobierz, plaintext

i potem foreacha zrobić i wyciągasz już sobie co tylko chcesz

Nie działa takie zapytanie

[PHP] pobierz, plaintext 
	   $podobienstwa_dostep = mysql_fetch_array(mysql_query("select * from artykuly a left join uzytkownicy u on u.id_dodajacego=a.id where punkty<=-5")) or mysql_error();
 
	   	  foreach($podobienstwa_dostep as $userr)
	  {
 
		echo ''.$userr['login'].'</br>';	
	  }
[PHP] pobierz, plaintext

Kishin

10.08.2015, 12:33:55

ehh mój błąd

[SQL] pobierz, plaintext 
 
SELECT * FROM artykuly a
 
LEFT JOIN uzytkownicy u ON u.id=a.id_dodajacego
 
WHERE punkty<=-5
 
 
 
[SQL] pobierz, plaintext

Lethys

10.08.2015, 12:47:05

Cytat(Kishin @ 10.08.2015, 12:33:55 )

ehh mój błąd

[SQL] pobierz, plaintext 
 
SELECT * FROM artykuly a
 
LEFT JOIN uzytkownicy u ON u.id=a.id_dodajacego
 
WHERE punkty<=-5
 
 
 
[SQL] pobierz, plaintext

Nadal nie działą, zrobie to standardowo ale skrypt mi wyświetla losowe kilka nicków z bazy zamiast moje nicki według warunku.

Aktualny skrypt:

[PHP] pobierz, plaintext 
 
	  $podobienstwa_dostep = mysql_fetch_array(mysql_query("select * from artykuly where punkty<=-5")) or mysql_error();
 
 
		  foreach($podobienstwa_dostep as $userr)
	  {
		$dany_user = mysql_fetch_array(mysql_query("select * from uzytkownicy where id='$userr[id_dodajacego]'")) or mysql_error();
 
		echo ''.$dany_user['login'].'</br>';	
	  }
 
 
[PHP] pobierz, plaintext

kapslokk

10.08.2015, 12:51:21

[PHP] pobierz, plaintext 
		$dany_user = mysql_fetch_array(mysql_query("select * from uzytkownicy where id='$userr[id_dodajacego]'")) or mysql_error();
[PHP] pobierz, plaintext

[PHP] pobierz, plaintext 
		$dany_user = mysql_fetch_array(mysql_query("select * from uzytkownicy where id=$userr['id_dodajacego']")) or mysql_error();
[PHP] pobierz, plaintext

Left join to tak czy inaczej lepsze rozwiązanie - Twój sposób zamiast zrobić jedno zapytanie zrobi ich tyle ile będzie tych wpisów w bazie.

Lethys

10.08.2015, 12:55:02

Cytat(kapslokk @ 10.08.2015, 12:51:21 )

[PHP] pobierz, plaintext 
		$dany_user = mysql_fetch_array(mysql_query("select * from uzytkownicy where id='$userr[id_dodajacego]'")) or mysql_error();
[PHP] pobierz, plaintext

[PHP] pobierz, plaintext 
		$dany_user = mysql_fetch_array(mysql_query("select * from uzytkownicy where id=$userr['id_dodajacego']")) or mysql_error();
[PHP] pobierz, plaintext

Left join to tak czy inaczej lepsze rozwiązanie - Twój sposób zamiast zrobić jedno zapytanie zrobi ich tyle ile będzie tych wpisów w bazie.

zamiana id='$userr[id_dodajacego] na $userr['id_dodajacego'] sprawia, że skrypt teraz nic nie wyświetla nawet błędnych loginów..

kapslokk

10.08.2015, 12:58:29

Legendy głoszą, że jak dasz:

[PHP] pobierz, plaintext 
or die(mysql_error());
[PHP] pobierz, plaintext

to wyświetli się np błąd z mysql'a.

Lethys

10.08.2015, 13:07:52

Cytat(kapslokk @ 10.08.2015, 12:58:29 )

Legendy głoszą, że jak dasz:

[PHP] pobierz, plaintext 
or die(mysql_error());
[PHP] pobierz, plaintext

to wyświetli się np błąd z mysql'a.

Pojawia się:

[PHP] pobierz, plaintext 
Warning: Illegal string offset 'id_dodajacego'
[PHP] pobierz, plaintext

odnośnie tej linijki:

[PHP] pobierz, plaintext 
$dany_user = mysql_fetch_array(mysql_query("select * from uzytkownicy where id='$userr[id_dodajacego]'")) or die(mysql_error());
[PHP] pobierz, plaintext

Tylko, że jeżeli dam id=$user['id_dodajacego'] to nic mi skrypt nie wyrzuca, więc też nie działa ..

kapslokk

10.08.2015, 13:15:45

Warning Ci wywala, bo nie masz apostrofów. Znajdź sobie jakiś kurs php i tablice asocjacyjne. Popraw to zapytanie tak aby apostrofy były, potem sobie zrób

[PHP] pobierz, plaintext 
echo("select * from uzytkownicy where id=$userr['id_dodajacego']");
[PHP] pobierz, plaintext

i wrzuć zapytanie do bazy np przez phpMyAdmin'a. Może nie znajduje użytkowników którzy by pasowali do zapytania ?

Kishin

10.08.2015, 13:27:47

Cytat(kapslokk @ 10.08.2015, 14:15:45 )

Warning Ci wywala, bo nie masz apostrofów. Znajdź sobie jakiś kurs php i tablice asocjacyjne. Popraw to zapytanie tak aby apostrofy były, potem sobie zrób

[PHP] pobierz, plaintext 
echo("select * from uzytkownicy where id=$userr['id_dodajacego']");
[PHP] pobierz, plaintext

i wrzuć zapytanie do bazy np przez phpMyAdmin'a. Może nie znajduje użytkowników którzy by pasowali do zapytania ?

Coś chyba mylisz. Taki zapis jak tu masz jest nieprawidłowy

powinno być:

[PHP] pobierz, plaintext 
echo("select * from uzytkownicy where id='$userr[id_dodajacego]'");
[PHP] pobierz, plaintext

Lethys rozumiem ze id na pewno równa się id_dodajacego w Twoich tabelkach?

kapslokk

10.08.2015, 13:36:28

Cytat(Kishin @ 10.08.2015, 14:27:47 )

Coś chyba mylisz. Taki zapis jak tu masz jest nieprawidłowy

powinno być:

[PHP] pobierz, plaintext 
echo("select * from uzytkownicy where id='$userr[id_dodajacego]'");
[PHP] pobierz, plaintext

Lethys rozumiem ze id na pewno równa się id_dodajacego w Twoich tabelkach?

To Ty też sobie poczytaj o tablicach asocjacyjnych. Albo wyjaśnij skąd warning? Można wziąć dodatkowo

[PHP] pobierz, plaintext 
$userr[id_dodajacego]
[PHP] pobierz, plaintext

w apostrofy, ale jeżeli jest liczbą to nie ma takiej potrzeby.

Lethys

10.08.2015, 13:36:56

Cytat(Kishin @ 10.08.2015, 13:27:47 )

Coś chyba mylisz. Taki zapis jak tu masz jest nieprawidłowy

powinno być:

[PHP] pobierz, plaintext 
echo("select * from uzytkownicy where id='$userr[id_dodajacego]'");
[PHP] pobierz, plaintext

Lethys rozumiem ze id na pewno równa się id_dodajacego w Twoich tabelkach?

Teraz odkryłem, że skrypt zawsze zwraca ID:1 (nie wpadłem na to bo testowałem swoim kontem z id 1)

[PHP] pobierz, plaintext 
 $podobienstwa_dostep = mysql_fetch_array(mysql_query("select * from artykuly where punkty<=-5 and id_dodajacego IS NOT NULL")) or die(mysql_error());
 
 
		  foreach($podobienstwa_dostep as $userr)
	  {
		$dany_user = mysql_fetch_array(mysql_query("select * from `uzytkownicy` where id='$userr[id_dodajacego]'")) or die(mysql_error());
 
//$userr['id_dodajacego'] WYNOSI ZAWSZE 1 MIMO, IŻ W BAZIE UŻYTKOWNIK, KTÓRY PISAŁ DANY ARTYKUŁ MA INNY ID
 
		echo ''.$userr['id_dodajacego'].'</br>';
		echo ''.$dany_user['login'].'</br>';	
	  }
[PHP] pobierz, plaintext

Kishin

10.08.2015, 14:03:42

Cytat(kapslokk @ 10.08.2015, 14:36:28 )

To Ty też sobie poczytaj o tablicach asocjacyjnych. Albo wyjaśnij skąd warning? Można wziąć dodatkowo

[PHP] pobierz, plaintext 
$userr[id_dodajacego]
[PHP] pobierz, plaintext

w apostrofy, ale jeżeli jest liczbą to nie ma takiej potrzeby.

Wiesz że warning jest przy zaproponowanym przez Ciebie zapytaniu?
Proponuje napisać takie zapytanie i przekonać się na własnej skórze.

Cytat(Lethys @ 10.08.2015, 14:36:56 )

Teraz odkryłem, że skrypt zawsze zwraca ID:1 (nie wpadłem na to bo testowałem swoim kontem z id 1)

[PHP] pobierz, plaintext 
 $podobienstwa_dostep = mysql_fetch_array(mysql_query("select * from artykuly where punkty<=-5 and id_dodajacego IS NOT NULL")) or die(mysql_error());
 
 
		  foreach($podobienstwa_dostep as $userr)
	  {
		$dany_user = mysql_fetch_array(mysql_query("select * from `uzytkownicy` where id='$userr[id_dodajacego]'")) or die(mysql_error());
 
//$userr['id_dodajacego'] WYNOSI ZAWSZE 1 MIMO, IŻ W BAZIE UŻYTKOWNIK, KTÓRY PISAŁ DANY ARTYKUŁ MA INNY ID
 
		echo ''.$userr['id_dodajacego'].'</br>';
		echo ''.$dany_user['login'].'</br>';	
	  }
[PHP] pobierz, plaintext

Możesz wrzucić jakieś screeny tych tabelek?

kapslokk

10.08.2015, 14:06:08

Ok, spoko. Ale dajmy w takim razie:

[PHP] pobierz, plaintext 
    echo("select * from uzytkownicy where id={$userr['id_dodajacego']}");
[PHP] pobierz, plaintext

a nie

[PHP] pobierz, plaintext 
    echo("select * from uzytkownicy where id='$userr[id_dodajacego]'");
[PHP] pobierz, plaintext

nadal niewiele to zmieni, bo pewnie problem jest w zapytaniu 1 lub w samej bazie.

Lethys

10.08.2015, 14:12:06

Cytat(Kishin @ 10.08.2015, 14:03:42 )

Wiesz że warning jest przy zaproponowanym przez Ciebie zapytaniu?
Proponuje napisać takie zapytanie i przekonać się na własnej skórze.

Możesz wrzucić jakieś screeny tych tabelek?

Jasne

Jak widać istnieje użytkownik z ID 8, który ma artykuły z minimum -5 punktami, a skrypt mi pokazuje usera z ID:1 zamiast 3 echo z userem o ID:8

Uzytkownicy:

Artykuły:

Kishin

10.08.2015, 14:40:53

wrzuc jeszcze przed foreachem takie coś i napisz co wypluło

[PHP] pobierz, plaintext 
 
echo '<pre>';
print_r($podobienstwa_dostep );
echo '</pre>';
 
[PHP] pobierz, plaintext

Lethys

10.08.2015, 15:03:40

Cytat(Kishin @ 10.08.2015, 14:40:53 )

wrzuc jeszcze przed foreachem takie coś i napisz co wypluło

[PHP] pobierz, plaintext 
 
echo '<pre>';
print_r($podobienstwa_dostep );
echo '</pre>';
 
[PHP] pobierz, plaintext

W sumie to poprawnie mi wywaliło:

[PHP] pobierz, plaintext 
 
Array
(
    [0] => 144
    [id1] => 144
    [1] => 478
    [id2] => 478
    [2] => 0
    [klimat] => 0
    [3] => 0
    [akcja] => 0
    [4] => 0
    [tematyka] => 0
    [5] => 0
    [seria] => 0
    [6] => -5
    [punkty] => -5
    [7] => 2159
    [id_podobienstwa] => 2159
    [8] => 8
    [id_dodajacego] => 8
)
[PHP] pobierz, plaintext

id_dodajacego sie zgadza, punkty też (ale jeden rekord tylko wypluło)

kapslokk

10.08.2015, 15:56:48

Jeden rekord, bo mysql_fetch_array powinieneś używać w pętli while() żeby wypluło ich więcej.
http://php.net/manual/en/function.mysql-fetch-array.php
Example #2

Kishin

11.08.2015, 07:50:32

Zamiast

[PHP] pobierz, plaintext 
$dany_user = mysql_fetch_array(mysql_query("select * from `uzytkownicy` where id='$userr[id_dodajacego]'")) or die(mysql_error());
 
[PHP] pobierz, plaintext

Spróbuj tak

[PHP] pobierz, plaintext 
 
while ($dany_user=mysql_fetch_array(mysql_query("select * from `uzytkownicy` where id='$userr[id_dodajacego]'"))){
echo  $dany_user['login'];
}
 
 
 
[PHP] pobierz, plaintext

Lethys

11.08.2015, 09:20:27

Cytat(Kishin @ 11.08.2015, 06:50:32 )

Zamiast

[PHP] pobierz, plaintext 
$dany_user = mysql_fetch_array(mysql_query("select * from `uzytkownicy` where id='$userr[id_dodajacego]'")) or die(mysql_error());
 
[PHP] pobierz, plaintext

Spróbuj tak

[PHP] pobierz, plaintext 
 
while ($dany_user=mysql_fetch_array(mysql_query("select * from `uzytkownicy` where id='$userr[id_dodajacego]'"))){
echo  $dany_user['login'];
}
 
 
 
[PHP] pobierz, plaintext

W nieskończoność mi echuje mój nick czyli usera z ID:1

kapslokk

11.08.2015, 09:27:21

[PHP] pobierz, plaintext 
$res = mysql_query("select * from `uzytkownicy` where id='$userr[id_dodajacego]'");
while ($dany_user=mysql_fetch_array($res)){
echo  $dany_user['login'];
}
[PHP] pobierz, plaintext

Dałem Ci link do dokumentacji... Wystarczyło zerknąć.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.