correcte coding

Door Radio Dancemania op 28-09-2010 22:19

8.607 views

Beste PHPhulpers,

Ik heb in mijn vorige topic een probleem beschreven..(waarbij ik niet voldoende hulp heb mogen ontvangen ,helaas).. Nu heb ik het probleem ondertussen zelf opgelost en vroeg ik me af of deze code correct is. Hij werkt goed alleen vraag ik me af is hij veilig e.d. en wat zouden jullie anders doen?
Hier komt ie dan:


<?php
if ($artiest->fotogroep == '1')
	{
		echo'
		<a href="/images/artiesten/'.$artiest->foto1.'.jpg" rel="lightbox['.$artiest->naam.']" title="'.$artiest->naam.'">
		<img src="/images/artiesten/'.$artiest->foto1.'.jpg" alt="'.$artiest->foto1.'" width="200" height="200" />';
		
		$dbfoto = array(false,$artiest->foto1,$artiest->foto2,$artiest->foto3.$artiest->foto4,$artiest->foto5);
		$fotos = range(1,5);
		foreach($fotos as $value) 
		{
			if (!empty($dbfoto[$value]) && $value != '1')
			{ 
				echo '<a href="/images/artiesten/'.$dbfoto[$value].'.jpg" rel="lightbox['.$artiest->naam.']"  title="'.$artiest->naam.'">';
			}
		}
		echo'</a>';
	}
	elseif($artiest->fotogroep == '0')
	{
		echo'
		<a href="/images/artiesten/'.$artiest->foto1.'.jpg" rel="lightbox" title="'.$artiest->naam.'">
		<img src="/images/artiesten/'.$artiest->foto1.'.jpg" alt="'.$artiest->foto1.'" width="200" height="200" /></a>';	
	}
?>

- SanThe -

28-09-2010 23:15

Ik ben het met Karl eens: Ga normaliseren.
Deze manier van scripten is waardeloos.
Ik stop mijn support aan dit topic.

Hipska BE

28-09-2010 23:15

En wat anderen al de hele tijd proberen te zeggen is dat je die foto3, foto4 en foto5 helemaal niet moet controlleren moest je een goed datamodel hebben.

Bij zo'n goed datamodel zou je meteen een array van foto's van een artiest kunnen krijgen uit de DB. En dan hoefde je je hoofd niet meer te breken over dit soort brakke en onlogische code met dubbel werk.

Onbekende gebruiker

28-09-2010 23:16

SanThe Nvt op 28/09/2010 23:15:21

Ik ben het met Karl eens: Ga normaliseren.
Deze manier van scripten is waardeloos.
Ik stop mijn support aan dit topic.

Hipska BE op 28/09/2010 23:15:42

En wat anderen al de hele tijd proberen te zeggen is dat je die foto3, foto4 en foto5 helemaal niet moet controlleren moest je een goed datamodel hebben.

Bij zo'n goed datamodel zou je meteen een array van foto's van een artiest kunnen krijgen uit de DB. En dan hoefde je je hoofd niet meer te breken over dit soort brakke en onlogische code met dubbel werk.

AMEN!

Radio Dancemania

28-09-2010 23:17 gewijzigd op 28-09-2010 23:18

en hoe zien jullie zo'n extra DBveld ??

[size=xsmall]Toevoeging op 28/09/2010 23:18:11:[/size]

SanThe Nvt op 28/09/2010 23:15:21

Ik ben het met Karl eens: Ga normaliseren.
Deze manier van scripten is waardeloos.
Ik stop mijn support aan dit topic.

je hebt ook nog geen nuttig antwoord gegeven dus maakt verder weinig uit.. ik kom hier alleen om hulp maar ik merk al dat hier 2 uit de hoogte geblazen figuren zitten die het altijd beter weten en maar al te goed over willen komen.

erg jammer niet iedereen is zo goed als jullie;)

Martijn B

28-09-2010 23:25

Dit zou een betere database structuur zijn.

Tabellen:

- artiesten (id, naam, omschrijving, ...)

- artiest_fotos (id, artiest_id, bestandsnaam, ...)

- artiest_liedjes (id, artiest_id, naam, ...)

1 artiest meerdere foto's
1 artiest meerdere liedjes

Radio Dancemania

28-09-2010 23:26

Martijn B op 28/09/2010 23:25:08

Dit zou een betere database structuur zijn.

Tabellen:

- artiesten (id, naam, omschrijving, ...)

- artiest_fotos (id, artiest_id, bestandsnaam, ...)

- artiest_liedjes (id, artiest_id, naam, ...)

1 artiest meerdere foto's
1 artiest meerdere liedjes

en dit dus controleren door artiest_id te controleren met id van artiesten ?

Martijn B

28-09-2010 23:28 gewijzigd op 28-09-2010 23:29

Ja, de kolom artiest_id wordt ook wel vreemde sleutel genoemd. (foreign key)

Deze verwijst naar een id in een andere tabel.

[edit]

Meestal zet je ook een index op een vreemde sleutel. Hierdoor voorkom je dat MySQL de hele tabel moet doorzoeken naar een bepaalde artiest.

Synaps Framework

28-09-2010 23:28

Nee. Hij geeft je juist nuttige antwoorden. Jij hebt alleen een blok voor je kop en negeert het goede antwoord. Jij zit te wachten totdat iemand je verteld hoe je dit het beste kunt oplossen met een verkeerde database structuur.

Zoals Hipska al aangeeft. Stop je energie nou gewoon in het normaliseren van je database structuur. Is je probleem meteen verholpen.

Radio Dancemania

28-09-2010 23:29

Synaps Framework op 28/09/2010 23:28:25

Nee. Hij geeft je juist nuttige antwoorden. Jij hebt alleen een blok voor je kop en negeert het goede antwoord. Jij zit te wachten totdat iemand je verteld hoe je dit het beste kunt oplossen met een verkeerde database structuur.

Zoals Hipska al aangeeft. Stop je energie nou gewoon in het normaliseren van je database structuur. Is je probleem meteen verholpen.

hier wil ik ook aan beginnen maar zoals ik aangeef.. niet iedereen is zo slim als de rest hier.. ik zit hier met een reden om hulp te vragen. En als alle "pro" scripters hier dan met dingen aankomen die niet duidelijk zijn dan is de naam PHPhulp toch echt verkeerd genomen.

- SanThe -

28-09-2010 23:35

Omdat het jou blijkbaar te veel moeite kost om het zelf te vinden. Normaliseren.

Reageren

Inloggen om te reageren