pętla for i mysql Opcje

[olszam]

mam skrypt w pętli for z bazą wyświetla się tak jak powinno ale czy można coś zrobić by to trochę szybciej się wyświetlało?

[PHP] pobierz, plaintext 
<?php
require 'polacz.php';
$poczatek = 0; $koniec = 3; $koniec2 = 1; 
$ile2=mysql_query("SELECT * 
FROM zdjecia
INNER JOIN zawodnik ON zdjecia.idzawodnik = zawodnik.idzawodnik
INNER JOIN danezdjecia ON zdjecia.iddanezdjecia = danezdjecia.iddanezdjecia
INNER JOIN danezawody ON danezdjecia.iddanezawody = danezawody.iddanezawody
WHERE danezdjecia.nazwapliku LIKE 'podium/%'");
$ile=mysql_num_rows($ile2);
$ile=ceil($ile/$koniec);
 
for($koniec2;$koniec2<=$ile;$koniec2++){
$poczateka = 2; $koniecb = 1; $koniec2c = 5; 
$ile2d=mysql_query("SELECT * 
FROM zdjecia
INNER JOIN zawodnik ON zdjecia.idzawodnik = zawodnik.idzawodnik
INNER JOIN danezdjecia ON zdjecia.iddanezdjecia = danezdjecia.iddanezdjecia
INNER JOIN danezawody ON danezdjecia.iddanezawody = danezawody.iddanezawody
WHERE danezdjecia.nazwapliku LIKE 'podium/%'");
$ilee=mysql_num_rows($ile2d);
$ilee=ceil($ilee/$koniecb);
for($koniec2c;$koniec2c=$ilee;$koniec2c++){
 
	$rezultatg = mysql_query("SELECT * 
FROM zdjecia
INNER JOIN zawodnik ON zdjecia.idzawodnik = zawodnik.idzawodnik
INNER JOIN danezdjecia ON zdjecia.iddanezdjecia = danezdjecia.iddanezdjecia
INNER JOIN danezawody ON danezdjecia.iddanezawody = danezawody.iddanezawody
WHERE danezdjecia.nazwapliku LIKE 'podium/%' LIMIT ".$poczateka.", ".$koniecb);
while($wynikf = mysql_fetch_array($rezultatg)){
		echo $wynikf['nazwapliku'].'<br/>';
	}
	$rezultat = mysql_query("SELECT * 
FROM zdjecia
INNER JOIN zawodnik ON zdjecia.idzawodnik = zawodnik.idzawodnik
INNER JOIN danezdjecia ON zdjecia.iddanezdjecia = danezdjecia.iddanezdjecia
INNER JOIN danezawody ON danezdjecia.iddanezawody = danezawody.iddanezawody
WHERE danezdjecia.nazwapliku LIKE 'podium/%' LIMIT ".$poczatek.", ".$koniec);
	while($wynik = mysql_fetch_array($rezultat)){
		echo $wynik['imiezawodnik'].' '.$wynik['nazwiskozawodnik'].'<br/>';
 
	}
 
	echo '<br/><br/>';
	$poczatek = $koniec;
	$koniec += 3;
	$koniecb = $koniecb + 3;
	$poczateka = $poczateka + 3;
}
}
?>
[PHP] pobierz, plaintext 

po włączeniu skrytpu tak to wygląda

podium/Ragna_the_Bloodedge_(Emblem,_Crest).png
Mike Powell
Larry Myricks
Ivan Pedroso


podium/Nu-13_(Emblem,_Crest).png
Ivan Pedroso
Mike Powell
Robert Emmijan

Ten post edytował olszam 20.03.2012, 19:03:47

[230005]

Nie obraź się, ale dokładnie w taki sposób nie powinno się pisać kodu (w żadnym języku, nie tylko php).
A żeby działało szybciej to może nie rób selectów w pętli (i do tego zagnieżdzonej)? Z tego co widzę, to chyba da się to załatwić tylko jednym zapytaniem poza pętlą. No i jeśli coś liczysz, to nie rób select * from tabela i potem na tym mysql_num_rows tylko np select COUNT(id) from tabela. Poleciłbym też żebyś zapoznał się z rozszerzeniem PDO, ale tak w oderwaniu od powyższego i jak powyższe zacznie działać .

EDIT:
Tak po namyśle stwierdzam, że naprawienie tego kodu byłoby całkiem dobrym zadaniem na test z programowania...

Ten post edytował 230005 20.03.2012, 20:02:09

[r4xz]

EDIT:
Tak po namyśle stwierdzam, że naprawienie tego kodu byłoby całkiem dobrym zadaniem na test z programowania...

tylko pojawia się jeden problem... "co autor miał na myśli?"

[olszam]

kod by i może lepiej wyglądał gdyby nie baza którą mam zbudowaną bo profesor będzie się czepiał na zmiany w bazie, jemu trudno coś powiedzieć że ktoś może mieć racje a on się myli np jak mu pokazywałem że jak się wpisze tam daną odległość skoku i wybierze się operator porównania np. mniejsze. to sprawdza czy skok1>$wartosc or skok2>$wartosc or skok3>$wartosc i jeśli jedno z nich będzie prawdziwe to wyświetli dane które chce a on mi wysuwa z jakąś teorią że sprawdza to po najdłuższym skoku i weź tu się z kim takim dogadaj.

sorka że tak na forum się wyżalam ale już mnie korci by mu przywalić i zaczął słuchać co inni mówią a nie bujać we własnym świecie

[nospor]

kod by i może lepiej wyglądał gdyby nie baza którą mam zbudowaną bo profesor będzie się czepiał na zmiany w bazie, jemu trudno coś powiedzieć że ktoś może mieć racje

Ty przestań narzekać na profesora a przyjrzyj się sobie.... Nikt tu ci nie karze zmieniać bazy. Tu ci mówią o kodzie PHP, który jest totalnie beznadziejny. Za takie coś pała się należy z miejsca. I jeszcze te narzekania bezsensowne....

Jako przykład sam początek:

[PHP] pobierz, plaintext 
$ile2=mysql_query("zapytanie....");
$ile=mysql_num_rows($ile2);
$ile=ceil($ile/$koniec);
 
for($koniec2;$koniec2<=$ile;$koniec2++){
$poczateka = 2; $koniecb = 1; $koniec2c = 5; 
$ile2d=mysql_query("zapytanie.....");
[PHP] pobierz, plaintext 

Najpierw zliczasz liczbę rekordów z zapytania.
Potem to samo zapytanie wkładasz do pętli i w petli za każadym razem wyliczasz to, co już wyliczyłeś przed pętlą. Totalna bzdura.

Na dodatek te nieszczęsne mysql_num_rows o którym już pisano.
Brakuje ci elementarnej wiedzy więc się trochę doszkól a nie zwalasz winę na profesora.

[cudny]

A jak Pan Profesor Ci zaliczy coś takiego to sami damy mu pałę taki dżołk sytuacyjny