[PHP] Usuwanie użytkowników pętlą po uwzględnieniu warunku Opcje

[Lethys]

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]

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 (IMG:style_emoticons/default/smile.gif)

Ten post edytował kapslokk 10.08.2015, 11:48:07

[Lethys]

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 (IMG:style_emoticons/default/smile.gif)

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]

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]

pokaż poprawiony kod (IMG:style_emoticons/default/smile.gif)

[Lethys]

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 (IMG:style_emoticons/default/sad.gif)

[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]

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]

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]

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

na

[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.

Ten post edytował kapslokk 10.08.2015, 12:51:47

[Lethys]

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

na

[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]

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.

Ten post edytował kapslokk 10.08.2015, 12:58:47

[Lethys]

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]

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]

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? (IMG:style_emoticons/default/wink.gif)

[kapslokk]

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? (IMG:style_emoticons/default/wink.gif)

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]

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? (IMG:style_emoticons/default/wink.gif)

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 

Ten post edytował Lethys 10.08.2015, 13:37:20

[Kishin]

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.

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]

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.

Ten post edytował kapslokk 10.08.2015, 14:07:25

[Lethys]

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:

(IMG:http://s2.postimg.org/hz7psqkq1/uzytkownicy.png)

Artykuły:

(IMG:http://s22.postimg.org/wfd6c1vf5/podobienstwo.png)

[Kishin]

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