for() - optymalizacja skryptu Opcje

[Joachim Peters]

Witam,

Mam taki kawałek kodu w swojej klasie stronnicowania:

[PHP] pobierz, plaintext 
<?php
$count = ceil($this->totalRecords / $this->maxRecords);
		$j = (($this->recordStart / $this->maxRecords) > 1) ? $this->recordStart / $this->maxRecords : $count;
 
		for($i = 0; $i < $count; $i++) {
			$pageId = ($i * $this->maxRecords);
			$pager .= ($pageId != $this->recordStart) ? '<a href="'.$pageId.'">'.($i + 1).'</a>&nbsp;' : '<b>'.($i + 1).'</b>&nbsp;';
 
			if($i == 2) {
				if($count > 6) {
					if(($count - 3) >= $j) {
						$i = $j - 2;
						$pager .= '... ';
					} else {
						$i = $count - 4;
						$pager .= '... ';
					}
				}	
			}	
			if($i > $j && ($count - 3) >= $j) $d++;
			if($d == 1) break;
		}
?>
[PHP] pobierz, plaintext 

Walczę z nim już drugi dzień, ale ciągle zdarzają się błędy typu "Fatal error: Allowed memory size of". Macie sugestie co do tego kodu, jak go zoptymalizować?

[Cysiaczek]

Pokaż kod wyciągający dane z bazy - te, które stronicujesz. Głównie napisz jak pobierasz ilość rekordów z bazy dancch i czy potem przerzucasz je do tablic/obiektów

Pozdrawiam.

[Joachim Peters]

Testuje to jeszcze na "sucho", poprzez wprowadzenie do konstruktora ilości wszystkich rekordów i liczby wyświetlanych na jednej stronie.

[zimi]

Twój kod jest dla mnie zagadką
wstawmy sobie przykładowo dane
$this->recordStart = 90;
$this->maxRecords = 30;
$this->totalRecords = 317;
Stąd będziemy mieli:
$count = 11;
$j = 3;
dochodzimy w pętli do $i = 2 i wykonuję się warunek...
otrzymujemy: if(11-3>3) $i = 3 - 2 = 1;
i jedziemy znowu do for bo kolejne warunki się nie wykonują, znowu dostaniemy 2 znowu ten sam warunek i pętla nieskończona...

tak naprawdę nie mam pojęcia do czego służy zmienna $d i $j dość mało opisowe nazwy :P

co do wykropkowywania stosujesz całą masę magicznych liczb a więc w kod trzeba się wpatrzeć aby zrozumieć o co w nim chodzi,
ja do tego proponuję trochę inne rozwiązanie które kiedyś sobie wydumałem:

[PHP] pobierz, plaintext 
<?php
$stronyWidoczne = array( 1, 2, 3, $ileStron-2, $ileStron-1, $ileStron, $stronaAktualna-2, $stronaAktualna-1, $stronaAktualna, $stronaAktualna+1, $stronaAktualna+2);
//inicjalizacja najprostsza z możliwych... można ją trochę zautymatyzować... bądź 
zmodyfikować żeby była edytowalna...
sort($stronyWidoczne);
$poprzedniaStrona=0;
foreach($stronyWidoczne as $numerStrony){
	if($numerStrony>0&&$numerStrony<=$ileStron){
		if($poprzedniaStrona+1==$numerStrony)
		{
			if($stronaAktualna!=$numerStrony)
			{
				$pager.='<a href="'.$numerStrony.'">'.$numerStrony.'</a> ';
			}
			else
			{
				$pager.='['.$numerStrony.'] ';
			}
		}
		else
		{
			$pager.='... <a href="'.$numerStrony.'">'.$numerStrony.'</a> ';
		}
		$poprzedniaStrona=$numerStrony;
	}
}
?>
[PHP] pobierz, plaintext