Prezentacja danych i ich przygotowanie

[Largo]

Witam,

Mam pewien problem, ale rozwiązałem go, mimo to, zastanawia mnie pewna rzecz. W funkcji do prezentacji logów mam taki kod:

				switch ( $row['request_status'] )
			{
				case 0:
					$row['status']  = '<span style="color: blue;font-weight: bold;">'.$this->ipsclass->lang['waiting_for_medi'].'</span>';
					break;
				case 1:
					$row['status']  = '<span style="color: grey;font-weight: bold;">'.$this->ipsclass->lang['cancelled'].'</span>';
					break;
				case 2:
					$row['status']	= '<span style="color: red;font-weight: bold;">'.$this->ipsclass->lang['unapprove'].'</span>';
					break;
				case 3:
					$row['status']	= $row['mediator'];
					break;
			}

Widać, że dane są przygotowane do szablonu, ale nie to mnie zastanawia. Jadę switch po polu z bazy ( request_status ), a sam tworzę dynamiczny rekord status. W szablonie dodaję taki kod, ale czy opłaca się mieszać HTML do logiki? W szablonach nie ma switch, a foreach, żeby sprawdzał statusy? Wchodzi w to foreach i if, więc więcej warunków, a log może mieć jeden status. To samo mam z serwerami gry ( switch i przypisanie HTML ), czy jest to dobre wyjście?

Pozdrawiam,

Largo

[Batman]

rozumiem ze dopiero w szablonie html są if-y i pętle foreach tak jest to czasami stosowane choć niezbyt eleganckie , switch nie jest równoważny do foreach tylko do elseif (elseif uważa się za szybsze)

foreach wykonuje pętle na wszystkich elementach tablicy wyświetlając szablon do każdej pozycji z tabeli

czy jest to dobre działa tak samo to tylko miejsce umieszczenia kodu, można tu dyskutować .

[Largo]

rozumiem ze dopiero w szablonie html są if-y i pętle foreach tak jest to czasami stosowane choć niezbyt eleganckie , switch nie jest równoważny do foreach tylko do elseif (elseif uważa się za szybsze)

foreach wykonuje pętle na wszystkich elementach tablicy wyświetlając szablon do każdej pozycji z tabeli

czy jest to dobre działa tak samo to tylko miejsce umieszczenia kodu, można tu dyskutować .

Witaj Batman,

Dziękuje za pzypomnienie podstaw, które są mi znane od par lat :-) W szablonie jest <if> i <foreach>, czy masz tam inne instrukcje warunkowe? A foreach w szablonach jest stosowany dosyć często ( w większych projektach ), więc mi to nie robi różnicy, ja osobiście używam foreach w szablonie. A masz pomysł jak nadać te statusy w szablonie? W myśl tego powinno to wyglądać tak:

<if="is_array( $data ) AND count( $data )">

<foreach="$data AS $row">

<if="$row['request_status'] == 0">

Dla 0

</if>

<if=$row['request_status'] == 1">

Dla 1

Co daje 4 iteracje i if, zamiast jednego switch, który dokładnie jednemu rekordowi się dopasowuje.

Pozdrawiam,

Largo

[Batman]

W szablonach nie ma switch, a foreach,

wybacz pomyślałem ze pomyliłeś te dwie funkcje

wracając

if , else , elseif jest bardziej optymalnym rozwiązaniem i zaleca się zastępowanie nimi funkcji switch przy optymalizacji kodu

więc może to tak nie wyglądać ale lepiej jest stosować if-y

[DawPi]

Wg mnie nie powinno się mieszać nigdy wyglądu ( html ) z logiką, co wynika choćby z wzorca MVC. Najlepiej to wszelkie obliczenia, sprawdzenia zrobić na poziomie kodu, a potem wynik tylko pokazać za pomocą bitów.

Wiesz, dla nas może być łatwo zmienić coś w takim bicie, ale nie wszyscy mogą się znać.

[Batman]

a no nie powinno się , ale różnie to wychodzi

[DawPi]

Postaw się w sytuacji gdy ktoś chce zmienić sam wygląd, ale widząc jakiś 'dziwny' kod foreach czy jakieś ify nie wie o co chodzi. Wygląd to wygląd, sterowanie wyglądem to sterowanie.

[Batman]

DawPi ja to rozumiem mówię tylko ze nie zawsze tak jest (minerva seo, samo IPB itd)

[DawPi]

Ja też rozumiem, że wiesz, i że tak jest. Sam się czasem łapię na tym.

Mówimy tutaj o programowaniu idealnym, a to, że czasem nie wyjdzie.....

[Largo]

Witam,

Czyli podsumowując - mam przekazywać do szablonu informację w postaci cyfr, pętelka foreach i if wszędzie dla każdej możliwej sytuacji? Bo teoria teorią, a tutaj chodzi o żywy przykład, chyba rozumiecie, z samych nazw zmiennych?

Pozdrawiam,

Largo

[DawPi]

Teoria to teoria. To jak to zrobisz to Twój wybór. Jeśli uznasz, że wolisz dodawać w bitach warunki logiczne czy pętle to nikt Cię od tego nie odwiedzie. Ja tylko mówię, że powinno się maksymalnie oddzielać wygląd od wszelkiego kodu sterującego.