[PHP][MYSQL][Smarty] niewyświetla Opcje

[-k1ilo-]

Witam mam problem mam klasę

[PHP] pobierz, plaintext 
<?php
class news
{
	private $Baza;
	public function __construct() {
		$this->Baza = mysql_connect("localhost", "user", "pass")
		or die('connect_error'.mysql_error());
		mysql_select_db('dbname')or die('database_error'.mysql_error());
	}
 
	public function wyswietl($ile) {
		$query = 'SELECT id, tytul, tresc, data FROM news ORDER BY id DESC LIMIT 0, "'.$ile.'"';
		mysql_query($query);
		$aNews = array();
			while($r = mysql_fetch_array($query)) {
				$aNews[] = $r;
			}
		return $aNews;
	}
}
?>
[PHP] pobierz, plaintext 

i teraz nie potrafię wyświetlić wyników

[PHP] pobierz, plaintext 
<?php
require './news.php';
 
$smarty = new blog;
 
$a = new news;
$a -> wyswietl(1);
 
$smarty -> assign('news', $a -> wyswietl());
$smarty -> display('news.tpl');
?>
[PHP] pobierz, plaintext 

Pewnie coś źle robię tylko nie mam już pomysłu co.

[Shili]

Klasa blog dziedziczy po klasie smarty? Zmieniłeś nazwę domyślnej klasie smarty?
Wyświetlają się może jakieś błędy?

Jeśli nie, to uważaj jaki obiekt inicjujesz i również napisz czy wyświetlają się błędy. Zamiast zainicjować obiekt Smarty zanicjowałeś obiekt blog i za jego pomocą próbujesz coś wyświetlić na ekranie. Prawdopodobnie powinno wyglądać to tak:

[PHP] pobierz, plaintext 
<?php
$smarty = new Smarty();
?>
[PHP] pobierz, plaintext 

Ten post edytował Shili 14.08.2008, 09:04:25

[-k1ilo-]

klasa blog dziedziczy po smarty i w niej ustawiam debug itd...
kiedy robie

[PHP] pobierz, plaintext 
<?php
$a = new news;
$smarty -> assign('wNews', $a -> wyswietl(1));
?>
[PHP] pobierz, plaintext 

otrzymuje błąd
Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in news.php on line 32

a z okna debug smarty wiem iż na pewno tablica $wNews nic nie zawiera.

[nospor]

nie:

[PHP] pobierz, plaintext 
<?php
$query = 'SELECT id, tytul, tresc, data FROM news ORDER BY id DESC LIMIT 0, "'.$ile.'"';
?>
[PHP] pobierz, plaintext 

a:

[PHP] pobierz, plaintext 
<?php
$query = 'SELECT id, tytul, tresc, data FROM news ORDER BY id DESC LIMIT 0, '.$ile.'';
?>
[PHP] pobierz, plaintext 

Dodatkowo na przyszlosc sprawdzaj jaki masz blad zapytania:
http://nospor.pl/php-faq-n29.html#faq-6

[-k1ilo-]

[PHP] pobierz, plaintext 
<?php
public function wyswietl($ile) {
		$query = 'SELECT id, tytul, tresc, data FROM news ORDER BY id DESC LIMIT 0, '.$ile.'';
		$this->Baza = mysql_query($query);
		$aNews = array();
*		while($r = mysql_fetch_array($query)) {
			$aNews[] = $r;
		}
		print_r ($aNews);
		return $aNews;
	}
?>
[PHP] pobierz, plaintext 

nadal Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource news.php

linia z * wskazana jest przez parser
chciałem sprawdzić czy ta tablica naprawdę nic nie zawiera wiec zrobiłem
print_r ($aNews);
i dostaje tylko Array();

[nospor]

ale czemu robisz: $this->Baza = mysql_query($query);
ma byc: $query = mysql_query($query); tak jak do tej pory (hehe, w zasadzie do tej pory tez tak nie miales. Wiec ma tak byc jak teraz ci napisalem)

[Shili]

Skorzystałeś z podpowiedzi nospora?

Poza tym nie

[PHP] pobierz, plaintext 
<?php
$this->Baza = mysql_query($query);
		$aNews = array();
		while($r = mysql_fetch_array($query)) {
			$aNews[] = $r;
		}
?>
[PHP] pobierz, plaintext 

Do fetch array nie wciskasz zapytania, tylko wynik zapytania. Wynik, który masz w $this->Baza.

[nospor]

@Shili to podajmy juz caly kod bo po naszych odpowiedziach sie zamota (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[PHP] pobierz, plaintext 
<?php
public function wyswietl($ile) {
		$query = 'SELECT id, tytul, tresc, data FROM news ORDER BY id DESC LIMIT '.$ile;
		$res = mysql_query($query)  or die('Zapytanie: '.$query.' --- błąd: '.mysql_error());
		$aNews = array();
		while($r = mysql_fetch_array($res)) {
			$aNews[] = $r;
		}
		print_r ($aNews);
		return $aNews;
	}
?>
[PHP] pobierz, plaintext 

[-k1ilo-]

dobra udało się rzeczywiście cały czas plułem do fetch'a zapytanie
dzięki za pomoc.

[Shili]

Oczywiście, ja w takim razie jeszcze pozwolę sobie na drobną optymalizację (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

[PHP] pobierz, plaintext 
<?php
public function wyswietl($ile) {
		$query = 'SELECT id, tytul, tresc, data FROM news ORDER BY id DESC LIMIT '.$ile;
		$res = mysql_query($query)  or die('Zapytanie: '.$query.' --- błąd: '.mysql_error());
		$aNews = array();
		while($aNews[] = mysql_fetch_array($res));
		print_r ($aNews);
		return $aNews;
	}
?>
[PHP] pobierz, plaintext 

Ten post edytował Shili 14.08.2008, 10:45:12

[nospor]

ooo, Shili za ten jęzor to ci dowale:

Twoja optymalizacja wrzuca jako ostatni element tablicy false. Srednio mily wynik. Pozatym mam pewne obawy czy wlasnie nie zrobilas never ending loop (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

edit: tak, juz mam pewnosc, wlasnie zrobilas never ending loop
I jeszcze jeden jęzorek na koniec (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

[Shili]

Neverending raczej nie. Po tym false już się powinno zatrzymać.
Teoretycznie nie powinno być tego jednego (z doświadczenia z innymi językami z tego co pamiętam), jednakże jeśli jest, to przepraszam za wprowadzanie w błąd. Zapamiętam sobie przynajmniej (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Zawsze można dodać array_pop($aNews), ale nie wiem czy jest sens przy tak drobnych danych ^^

@edit
Zmieniłam kawałek swojej klasy dla testu

[PHP] pobierz, plaintext 
<?php
public function select($query)
	{
		$result = $this->query($query);
		$row = array();
		while($row[] = mysql_fetch_assoc($result));
		array_pop($row);
		return $row;
	}
?>
[PHP] pobierz, plaintext 

Pętla się kończy, wywala mi wszystkie wyniki.
Błędy: E_ALL | E_STRICT, PHP 5coś tam.

@down
Faktycznie, jakby nie patrzeć powinieneś mieć rację.
Ciekawe czemu u mnie to cudo działa jak powinno o_0

Ten post edytował Shili 14.08.2008, 10:58:10

[nospor]

Neverending na pewno nie

Jak juz pisalem: napewno tak (do czasu skonczenia sie czasu na skrypt )
Przeciez w while sprawdzasz calą tablice a nie jej ostatni element. Sprawdź sobie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

polecam prosty test:

[PHP] pobierz, plaintext 
<?php
$ar = array('aa');
while ($ar[] = false){
	echo 'uppps, petelka';
}
echo 'koniec';
?>
[PHP] pobierz, plaintext 

edit: hmmm.... i mi zaczelo dzialac po Twojemu :/ co jest... chyba mi sie apacha przepalil (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
w takim razie zwracam honor (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

edit2:

@down
Faktycznie, jakby nie patrzeć powinieneś mieć rację.
Ciekawe czemu u mnie to cudo działa jak powinno o_0

Nie, nie. Jednak ty masz racje jak napisalem w edicie wyżej.
$ar[] = false
zwraca ostatni element a nie całą tablicę.