[MySQL][PHP] Własna funkcja w warunku while Opcje

[o2w5n778]

Witam!
Chciałbym umieścić w warunku funkcji while stworzoną wcześniej funkcję, zawiera ona zapytania i zwraca mysql_fetch_array. Niestety nie działa mi to.

[PHP] pobierz, plaintext 
<form method="post" action="#">
<input type="text" name="ala" /><input type="submit" />
</form>
<?php
	mysql_connect('localhost', 'root', '');
	mysql_select_db('project1');
 
	function loop(){
		$query = mysql_query("SELECT * FROM pr_pages WHERE page_id=".$_POST['ala']."");
		$sas = mysql_fetch_array($query);
		return $sas;
	}
 
	while(loop()){
		$sas['page_id'];
	}
 
 
?>
[PHP] pobierz, plaintext 

W czym problem?

[nospor]

Przecież loop zawsze zwraca pierwszy rekord wiec ci się petla while zapętla. A teraz zadanie dla ciebie: zastanów się dlaczego loop() zawsze zwraca pierwszy rekord. To nie jest pytanie podchwytliwe.

[o2w5n778]

Chcę stworzyć funkcję, którą będzie się podawało w pętli do wyświetlania artykułów (w szablonie), tylko w szablonie nie będę pisał zapytań itd. więc potrzebuję funkcji.

Co do pytania to odpowiem wprost, niestety, nie wiem.
Liczę na jakieś wskazówki.

Ten post edytował o2w5n778 26.04.2012, 06:20:20

[viking]

Po pierwsze sql injection (mysql_real_escape_string dla danych POST). Po drugie, poczytaj w dokumentacji (return dla funkcji, jest przykład z opisem) o zasięgu zmiennych bo możesz zapomnieć że $sas będzie dostępne poza funkcją. A po trzecie w dokumentacji mysql_fetch_array masz przykłady jak to zrobić. Pierwsze zdanie tej samej strony dokumentacji opisuje Ci to o czym nospor pisał.

Ten post edytował viking 26.04.2012, 06:47:54

[o2w5n778]

Nie ma przykładu jak to zrobić w manualu funkcji mysql_fetch_array.

[markonix]

POD i fetchAll albo musisz w ciele funkcji utworzyć tymczasową tablicę i ją zwrócić.
Funkcja będzie zwracać tablicę (asocjacyjną) i sobie ładnie będziesz ładnie wyświetlał listy za pomocą foreach.

[ethann]

@markonix
Tak, jest to dobre rozwiązanie ale dwukrotnie użyjesz pętli, przez co czas wykonywania kodu się minimalnie zwiększy.

[PHP] pobierz, plaintext 
<form method="post" action="#">
<input type="text" name="ala" /><input type="submit" />
</form>
<?php
	mysql_connect('localhost', 'root', '');
	mysql_select_db('project1');
 
	$query = mysql_query("SELECT * FROM pr_pages WHERE page_id=".intval($_POST['ala']));
	while($sas = mysql_fetch_array($query)){
		echo $sas['page_id']."<br />";
	}
?>
[PHP] pobierz, plaintext 

Jak pisał viking - bezpieczeństwo jest ważne, chyba że projekt polega na ukazaniu słabości. Stąd umieszczenie funkcji intval() - skoro page_id to pomyślałem że to liczba .
I wygląda to niemal jak przykład z php.net.

Jeśli koniecznie chcesz użyć własnej funkcji to wykonaj wcześniej mysql_query() i przekaż jego wynik jako argument Twojej funkcji.

Ten post edytował ethann 26.04.2012, 12:19:26

[o2w5n778]

Chyba troszkę źle się wyraziłem lub mnie nie zrozumieliście
Chcę zrobić w moim systemie tak, abym mógł w szablonie za pomocą pętli wyświetlać dane. Niestety niezbyt będzie to optymalne jeżeli w pliku szablonu będę tworzył zapytania itd.
Mój problem polega na tym że nie mogę stworzyć funkcji, którą będę mógł umieścić we warunku pętli while/foreach (obojętnie) i ma ona zawierać zapytanie itd. A w szablonie za pomocą tablicy będę wyświetlał dane.

Prościej mówiąc rozwiązanie wordpress'owe

Ten post edytował o2w5n778 26.04.2012, 12:27:39

[ethann]

To tak jak pisał markonix. Tylko Jego pomysłem było użycie do tego PDO.

[PHP] pobierz, plaintext 
<?php
[... connect ...]
 
function foo() {
 $q = mysql_query("SELECT * FROM pr_pages WHERE page_id=".intval($_POST['ala']));
 $ret = array();
 while($r = mysql_fetch_assoc($q)) $ret[] = $r;
 
 return $ret;
}
 
foreach(foo() as $value) {
  echo $value['pr_pages'];
}
 
?>
[PHP] pobierz, plaintext 

Ten post edytował ethann 26.04.2012, 12:35:42

[markonix]

Funkcja może zwracać resurce mysql a potem w "widoku" można zrobić pętle mysql_fetch_assoc czy inną.
Wtedy jednak brzydko to wygląda (funkcja mysql wmieszana w HTML) dlatego tymczasowa tablica to chyba jedyna opcja.
PDO i fetchAll pewnie też na tej zasadzie działa tylko szybciej.