Przepisywanie kodu do MVC Opcje

[walus16]

Witam, napisałem kiedyś aplikację w PHP strukturalnym i aby ją unowocześnić i rozbudować postanowiłem zacząć przepisywać ją do MVC. Wszystko szło fajnie aż do momentu, w którym natrafiłem na pewien problem. W starym kodzie w pętli "while" zrobiony co drugi wiersz tabeli innym kolorem ($cell_color). Po przepisaniu tego do "foreach" osiągnąłem kolorowanie co drugiej komórki a nie wiersza. Proszę o pomoc.

Stary kod:

[PHP] pobierz, plaintext 
$result = mysql_query("SELECT * FROM ".DB_PREFIX."studios ORDER BY studio_name");
if (mysql_num_rows($result) != 0){
	$i = 0;
	echo "<td class='tbl-header'>Nazwa</td>\n";
	echo "<td class='tbl-header'>Opis</td>\n";
	echo "<td class='tbl-header'>Opcje</td>\n";
	echo "</tr>\n";
	while ($data = mysql_fetch_assoc($result)){
		$cell_color = ($i % 2 == 0 ? "tbl1" : "tbl2");
		echo "<tr>\n";
		echo "<td class='$cell_color'>".$data['studio_name']."\n</td>\n";
		echo "<td class='$cell_color'>".$data['studio_desc']."\n</td>\n";
		echo "<td class='$cell_color'>".$data['studio_id']."\n</td>\n";
		echo "</tr>\n";
		$i++;
	}
	echo "</table>\n";
} else {
	echo "<tr><td align='center' class='tbl1'>Brak studiów.</td></tr>\n</table>\n";
}
[PHP] pobierz, plaintext 

Nowy kod:

[PHP] pobierz, plaintext 
$result = mysql_query('SELECT * FROM mb_studios ORDER BY studio_name');
$posts = array();
while ($row = mysql_fetch_assoc($result)){
	$posts[] = $row;
}
 
?>
 
<?php if ($posts): ?>
<table width="100%">
<tr>
	<td class="tbl-header">Nazwa</td>
	<td class="tbl-header">Opis</td>
	<td class="tbl-header">Opcje</td>
</tr>
<?php foreach ($posts as $post): ?>
<tr>
	<td class=""><?php echo $post['studio_name'] ?></td>
	<td class=""><?php echo $post['studio_desc'] ?></td>
	<td class=""><?php echo $post['studio_id'] ?></td>
</tr>
<?php endforeach; ?>
</table>
<?php else: ?>
	Testowy komunikat
<?php endif ?>
[PHP] pobierz, plaintext 

Ten post edytował walus16 3.07.2014, 19:02:32

[SmokAnalog]

A nie lepiej kolorować w CSS? http://www.w3schools.com/cssref/sel_nth-child.asp

P.S. Plus dla Ciebie za używanie składni alternatywnej i nie wypluwanie HTML-a w echo!

[Turson]

Dlaczego nie użyjesz po prostu while i będzie identycznie jak wcześniej? While() też ma przyjazną formę

[PHP] pobierz, plaintext 
<?php while(): ?>
html
<?php endwhile; ?>
[PHP] pobierz, plaintext 

[walus16]

Dzięki, nie słyszałem o to takim rozwiązaniu muszę się temu bliżej przyjrzeć. Jest jeszcze jakieś rozwiązanie i idąc bardziej w stronę PHP.

A nie lepiej kolorować w CSS? http://www.w3schools.com/cssref/sel_nth-child.asp

P.S. Plus dla Ciebie za używanie składni alternatywnej i nie wypluwanie HTML-a w echo!

[SmokAnalog]

Jak coś da się zrobić w CSS, to lepiej w CSS. Ale zanim pojawiło się nth-child w CSS, to ja to robiłem tak:

[PHP] pobierz, plaintext 
for ($even = true; warunek; $even = !$even) {...}
[PHP] pobierz, plaintext 

I potem przypisywałem klasę na podstawie zmiennej $even (true to parzysty wiersz, a false to nieparzysty).

[walus16]

@Turson, z "while" to w taki sposób mi to nie działa.

Dlaczego nie użyjesz po prostu while i będzie identycznie jak wcześniej? While() też ma przyjazną formę

[PHP] pobierz, plaintext 
<?php while(): ?>
html
<?php endwhile; ?>
[PHP] pobierz, plaintext 

[Turson]

[PHP] pobierz, plaintext 
<?php while ($post = mysql_fetch_assoc($result)): ?>
<tr>
<td class=""><?php echo $post['studio_name'] ?></td>
<td class=""><?php echo $post['studio_desc'] ?></td>
<td class=""><?php echo $post['studio_id'] ?></td>
</tr>
<?php endwhile; ?>
[PHP] pobierz, plaintext 

[walus16]

Mój zamysł jest taki, że logikę aplikacji będę trzymał w katalogu model a widoki html w katalogu view więc takie rozwiązanie z while ($post = mysql_fetch_assoc($result)) odpada

Ten post edytował walus16 3.07.2014, 19:47:58

[SmokAnalog]

Ale w czym problem? To użyj foreach tak jak masz teraz i wywal te klasy, tak jak Ci podpowiedziałem. Kod będzie czystszy i nowocześniejszy.

[walus16]

Dokładnie tak zrobię. Pozostanę przy foreach i dodam rozwiązanie z CSS.

[Turson]

Mój zamysł jest taki, że logikę aplikacji będę trzymał w katalogu model a widoki html w katalogu view więc takie rozwiązanie z while ($post = mysql_fetch_assoc($result)) odpada

Więc co robi to w "nowym kodzie"

[PHP] pobierz, plaintext 
$result = mysql_query('SELECT * FROM mb_studios ORDER BY studio_name');
$posts = array();
while ($row = mysql_fetch_assoc($result)){
$posts[] = $row;
}
[PHP] pobierz, plaintext 

[walus16]

Ten kod jest dlatego żeby było widać różnicę między starym, a nowym. No i dlatego, że nie mam jeszcze napisanego kontrolera. Sorry za zamęt.

Więc co robi to w "nowym kodzie"

[PHP] pobierz, plaintext 
$result = mysql_query('SELECT * FROM mb_studios ORDER BY studio_name');
$posts = array();
while ($row = mysql_fetch_assoc($result)){
$posts[] = $row;
}
[PHP] pobierz, plaintext