Witam

Mam pytanie czy oplacalne jest tworzenie funkcji ktora zrzucalaby w petli wynik danego zapytania do tablicy dwuwymiarowej (wiecej niz jeden rekord), a nastepnia zwracalaby ta tablice poprzez return ?

Za:
- bardziej przejrzysta i poukladana struktura kodu
- po wywolaniu funkcji operacja na samej tablicy

Przeciw:
- niepotrzebne powtarzanie dwoch petli - jedna to odczyt z bazy i wrzucanie do tablicy, druga to odczyt z tablicy

Czy w tym przypadku oplaca sie tworzyc dodatkowa funkcje ?
Oczywiscie moge sobie napisac oba sposoby i porownac czas wykonania ale niestety same wyniki czasowe za wiele mi nie mowia.

Na takich operacjach opierają się wszystkie systemy szablonów. Musisz iść na kompromisy między optymalizacją a przejrzystością i wygodą pracy.

Ok. W takim razie postanowilem zrobic to sposobem opisanym w pierwszym poscie.
Niestety mam pewne problemy :/

Jezeli funkcja zwraca mi tablice jednowymiarowa to robie tak:

[PHP] pobierz, plaintext 
<?php
list($id, $from, $to) = message_show();
?>
[PHP] pobierz, plaintext 

wtedy dane pola zawieraja sie w zmiennych "id, from, to".


Sprawa sie troche komplikuje jak mam tablice dwuwymiarowa.
Proboje to robic tak ...

Miejsce w funkcji gdzie wrzucam wynik zapytania do tablicy dwuwymiarowej.

[PHP] pobierz, plaintext 
<?php
 
	$i = 0;
 
	  while ($result = mysql_fetch_array($query))
	{
 
	  $msg_list[$i]['id'] = $result['id'];
	  $msg_list[$i]['msg_from'] = $result['msg_from'];
	  $msg_list[$i]['msg_to'] = $result['msg_to'];
	  $msg_list[$i]['subject'] = $result['subject'];
	  $msg_list[$i]['text'] = $result['text'];
	  $msg_list[$i]['date'] = $result['date'];
	  $msg_list[$i]['status'] = $result['status'];
 
	  $i++;
 
	}
 
	return $msg_list;
 
?>
[PHP] pobierz, plaintext 

Pozniej juz poza funkcja proboje ja pobrac nastepujaco ...

[PHP] pobierz, plaintext 
<?php
 
list($id, $msg_from, $msg_to, $subject, $text, $date, $status) = message_show_list();
 
$i = '0';
 
while (!empty($id[$i]))
{
echo $id[$i]; // przykladowo chce tutaj wyswietlic wszystkie pola id ktore zostaly pobrane z baz
 
$i++;
}
 
?>
[PHP] pobierz, plaintext 

Niestety to nie dziala.
Wyswietla tylko:



Myslalem aby funkcje list() wywolywac w petli i po jednym wierszu pobierac dane ale wydajniej bedzie zrobic to za jednym razem.

Bede wdzieczny za jakies wskazowki jak to rozwiazac.

[PHP] pobierz, plaintext 
<?php
$i = 0;
?>
[PHP] pobierz, plaintext 

jak dodajesz apostrofy to jest wartość traktowana jako string, to tak na szybko

Łukasz

Już sprawdzalem zarowno z apostrofami jak i bez ale problem tkwi w czym innym :/
Jakies pomysly ?

Sypią się błędy, bo twoja zmienna $id zawiera:

$msg_from zawiera:

itd.


Zamień miejscami [$i] i ['id'] (analogicznie dla wszystkich elementów tablicy $msg_list), to ruszy.



[EDIT]
Jednak nie ruszy:


Chyba że ominiesz to w ten sposób:

[PHP] pobierz, plaintext 
<?php
 
	list( $a, $b, $c ) = array_values( twojaFunkcja() );
 
?>
[PHP] pobierz, plaintext 

Bardzo dziekuje za pomoc !

Co do tych indeksow to zmienilem na liczbowe, gdyz wlasciwie nie ma to zadnego znaczenia. Trzeba unikac rzeczy zbednych
I tak w funkcji list() definiuje zmienne ktore beda uzywane poza funkcja.

Jeszcze raz dzieki.
Pozdrawiam
J.