Metoda zwracająca tematy forum Opcje

[acknex]

Witam, to mój pierwszy post na forum i od razu zaczynam problemem Ale do rzeczy. Posiadam metodę w klasie która jest odpowiedzialna za wyświetlenie for w moim skrypcie. Po przeanalizowaniu modelu MCV stwierdzam, że nie jest ona do końca z nim zgodna, planuję ją zmodyfikować i tutaj pojawia się problem. Metoda wygląda tak:

[PHP] pobierz, plaintext 
		public function listForum() {
			require_once("include/class.database.php");
 
			$connector = new DbConnector();
			$result = $connector->query("SELECT * FROM forum_question ORDER BY id DESC");
 
			$color = 1;
 
			while($rows = $connector->fetchArray($result)){
				echo '
							   <tr class="row'.$color.'">
			                  <td align="left"><img src="./images/folder_open.png" border="0" width="15" height="15" />&nbsp;<a href="view.topic.php?id='.$rows['id'].'">'.$rows['topic'].'</a><BR></td>
			                  <td align="center">'.$rows['view'].'</td>
			                  <td align="center">'.$rows['reply'].'</td>
			                  <td align="center">'.$rows['datetime'].' przez <a href="./profile.php?login='.$rows['name'].'">'.$rows['name'].'</a></td>
              </tr>
			  ';
			  $view = 'aaa';
			  if ($color == 1) { $color = 2;} else { $color = 1;}
			}
		}
[PHP] pobierz, plaintext 

Chcę przerobić tą metodę tak, żeby zwracała wszystkie tematy po kolei ale nie bardzo mam pomysł jak to rozwiązać. Będę wdzięczny za wskazówki.

Ten post edytował acknex 2.11.2011, 13:24:16

[phpion]

Nie możesz zwrócić po prostu $connector->fetchArray($result)? Jeśli nie zwraca to tablicy to przeleć po elementach zwróconych przez tą metodę, przypisz je do tablicy i zwróć. Wyświetlanie rekordów w postaci tabelki zostaw widokowi, który dostanie tablicę danych.

[acknex]

Rozwiązałem to tak:

Plik: include/class.forum.php

[PHP] pobierz, plaintext 
 
		public function listForum() {
			require_once("include/class.database.php");
 
			$connector = new DbConnector();
			$result = $connector->query("SELECT * FROM forum_question ORDER BY id DESC");
 
			while($rows = $connector->fetchArray($result)){
				$_data[] = $rows['id']."@".$rows['topic']."@".$rows['view']."@".$rows['reply']."@".$rows['datetime']."@".$rows['name'];
			}
 
			return $_data;
		}
[PHP] pobierz, plaintext 

Plik: forum.php

[PHP] pobierz, plaintext 
			<?php
				require_once("include/class.forum.php");
				require_once("include/class.users.php");
 
				$forum = new Forum();
				$rows = $forum->listForum();
 
				$color = 1;
 
				for ($i=0; $i < count($rows); $i++) {
					$data = explode("@", $rows[$i]);
 
					echo '<tr class="row'.$color.'">
					<td align="left"><img src="./images/folder_open.png" border="0" width="15" height="15" /> <a href="view.topic.php?id='.$data[0].'">'.$data[1].'</a><BR></td>
					<td align="center">'.$data[2].'</td>
					<td align="center">'.$data[3].'</td>
					<td align="center">'.$data[4].' przez <a href="./profile.php?login='.$data[5].'">'.$data[5].'</a></td>
					</tr>';			
 
					if ($color == 1) { $color = 2; } else { $color = 1;}
				}
?>
[PHP] pobierz, plaintext 

Mimo wszystko nie jestem do końca usatysfakcjonowany tym rozwiązaniem, więc jeśli ktoś ma jakieś sugestie to chętnie nad tym popracuję :)

Ten post edytował acknex 2.11.2011, 14:59:07

[marcio]

Po co ci ta malpa tam?
Zwroc normalnie tablice.