hej

zastanawiam sie czy sa jakies przeciwskazania lub inne lepsze rozwiazanie

Opis:

Strony wyswietlaja <tr>,<td>,<p>,<span> itp. w ten sam sposob pobierajac tresc z bazy danych. W zaleznosci jaka strona ma byc wyswietlana maja byc rozne klasy styli definiujace wyglad zawartosci.

chce zrobic case i za pomoca echo wyswietlac odpowiedni styl

tak jest zle (normalnie tam gdzie echo bylby case),blad parse error dla styli :

[PHP] pobierz, plaintext 
<style type="text/css">
<!--
 
.article_first {
	font-family: Arial, Helvetica, sans-serif;
	font-size: 12px;
	font-weight: bold;
<?php
		echo'
 
	background-color: #355CAA;
	color: #FFFFFF;
';?>
	margin: 10px;
	padding: 10px;	
}
-->	
</style> 
[PHP] pobierz, plaintext 

ale tak wydaje sie byc dobrze:

[PHP] pobierz, plaintext 
<?php
		echo'
<style type="text/css">
<!--
 
.article_first {
	font-family: Arial, Helvetica, sans-serif;
	font-size: 12px;
	font-weight: bold;
	background-color: #355CAA;
	color: #FFFFFF;
 
	margin: 10px;
	padding: 10px;	
}
-->	
</style> ';?>
[PHP] pobierz, plaintext 

Ponownie tam gdzie echo bylby case

prosze o opinie/krytyke tego rozwiazania

A po co dajesz style do środka?

Nie możesz tak:

[PHP] pobierz, plaintext 
<?php
 
$strAction = ( empty( $_GET[ 'style' ] ) ) ? 'main' : $_GET[ 'style' ];
 
$arrStyleFiles = array(
						 'main' => 'style_main.css',
						 'links' => 'style_links.css',
						 'download' => 'style_download.css'
);
 
?>
<style type="text/css" src="<?=  $arrStyleFiles[ $_GET[ 'style' ] ]; ?>">
[PHP] pobierz, plaintext 

Coś a tym stylu.
I tylko pliki podmieniasz.

W zewnetrznym stylu mam zdefiniowane okolo 30 klas a tylko ta jedna ma byc rozna, pomyslalem zeby tylko ta jedna zrobic w srodku, w takiej sytuacji troche bez sensu wydaje mi sie zapisywanie dokladnie 18 plikow style1-18.css gdzie roznica jest tylko jednej klasy a kazdy plik ma 29 zdefiniowanych identycznych klas.

doszedlem do wniosku ze wlasnie najlepszy bylby case (i dla mnie najlatwiejszy do wykonania), ciekaw jestem czy w tej sytuacji proponujesz tez tablice?

Nowy przykład

[PHP] pobierz, plaintext 
<?php
 
$strAction = ( empty( $_GET[ 'style' ] ) ) ? 'main' : $_GET[ 'style' ];
 
$arrStyleFiles = array(
						 'some' => 'style_some.css',
						 'links' => 'style_links.css',
						 'download' => 'style_download.css'
);
 
?>
<style type="text/css" src="global.css">
<!-- wszystkie globalne klasy -->
 
<style type="text/css" src="<?=  $arrStyleFiles[ $_GET[ 'style' ] ]; ?>">
<!-- czarna owca ;-) -->
[PHP] pobierz, plaintext 

hehe

no dobra ale teraz jeszcze potrzebuje wyjasnienia na co mi jest potrzebne 18 plikow css z jedna klasa przeciez za 2 tygodnie pierdykne jak bede mial sprawdzac w ktorym pliku jest to czy tamto


i na czy mpolega wyzszosc tablicy z zewnetrznymi plikami nad czyms takim

[PHP] pobierz, plaintext 
<?php
$change_first_style=$_GET['op'];
					  switch($change_first_style)
						   {
								case '600':
									echo'
											<style type="text/css">
											<!--
 
											.article_first {
												font-family: Arial, Helvetica, sans-serif;
												font-size: 12px;
												font-weight: bold;
												background-color: #355CAA;
												color: #FFFFFF;
 
												margin: 10px;
												padding: 10px;	
											}
											-->	
											</style> 
										';
									break;
 
								case '601':
									echo'
											<style type="text/css">
											<!--
 
											.article_first {
												font-family: Arial, Helvetica, sans-serif;
												font-size: 12px;
												font-weight: bold;
												background-color: #355CAA;
												color: #FFFFFF;
 
												margin: 10px;
												padding: 10px;	
											}
											-->	
											</style> 
										';
									break;
 
									case '602':
									break;
									case '603':
									break;
									...
 
						   }
?>
[PHP] pobierz, plaintext 

Wszystko w jednym miejscu i w ogole
pozdr

Myślisz bardzo mało przyszłościowo.
Gwarantuję Ci że dzięki Twojemy rozwiązaniu pierdyknisz jeszcze bardziej.

Bo będziesz miał switch'a z kilkunastoma sekcjami case, w każdej kod php wymieszany z CSS.
Dopiero wtedy problemem bedzie odnalezienie czegoś konkretnego.

A jak sobie pomyślisz że jeszcze jedna klasa ma być wyjątkowa?
To co, będziesz się grzebał wycinając coś z .css w wklejając to do .php. Paranoja.

Dzięki oddzieleniu logki od wizualizacji będziesz miał same przyjemne rzeczy.

zeby nie bylo

zgadzam sie w 100% z tym zeby oddzielac logike od wizualizacji

uwazam rowniez ze trzeba znalezc kompromis cena/jakosc
oraz jakosc/czas w przypadku gdy skrypt jest robiony za marne grosze nie ma sensu sie wczuwac tylko oszczedzic czas dla projektu ktory jest lepiej platny i czeka w kolejce...


i tak rozwazania o rozwiazanaich technicznych poszly w filozofie...