correcte coding

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

8.610 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>';	
	}
?>

Martijn B

28-09-2010 22:49 gewijzigd op 28-09-2010 22:51

^^^^ ja dan kun je gelijk doen:
<?php
foreach( $artiest->fotos as $sleutel => $bestandsnaam )
{
...
}
?>

[edit]

Je had een fotos tabel moeten maken kun je meer dan 5 foto's gebruiken.

Onbekende gebruiker

28-09-2010 22:52

Radio Dancemania op 28/09/2010 22:48:29

[quote="Pim de Haan op 28/09/2010 22:47:05"]
Waarom niet $artiest->fotos[1] ?

omdat in mijn db de tabellen foto1 , foto2, foto3, foto4, foto5 zijn?
[/quote]

Karl Karl op 28/09/2010 22:49:07

(...) Ik heb het idee dat het datamodel niet klopt.

Bron quote.
Ha, ik heb gelijk.

Radio Dancemania

28-09-2010 22:53

oke complete code zoals ik hem nu dus heb(werkend met dank aan martijn)


<?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($artiest->foto1,$artiest->foto2,$artiest->foto3.$artiest->foto4,$artiest->foto5);
		foreach($dbfoto as $key => $foto)
		{
 			if ( $key && $foto )
 			{
  				echo '<a href="/images/artiesten/'.$foto.'.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>';	
	}
?>

iemand nog een tip?:)

Onbekende gebruiker

28-09-2010 22:56

Ga normaliseren.

Radio Dancemania

28-09-2010 22:57

Karl Karl op 28/09/2010 22:56:19

Ga normaliseren.

geef misschien antwoorden waar iemand wat aan heeft of zeg gewoon niks;)

- SanThe -

28-09-2010 22:58 gewijzigd op 28-09-2010 23:04

<?php
$dbfoto = array($artiest->foto1,$artiest->foto2,$artiest->foto3.$artiest->foto4,$artiest->foto5);
foreach($dbfoto as $key => $foto)
{
if ( $key && $foto )
{
echo '<a href="/images/artiesten/'.$foto.'.jpg" rel="lightbox['.$artiest->naam.']" title="'.$artiest->naam.'">';
}
}
?>
Waarom moeilijk als het ook zonder if() kan.
<?php
$dbfoto = array($artiest->foto2,$artiest->foto3,$artiest->foto4,$artiest->foto5);
foreach($dbfoto as $foto)
{
echo '<a href="/images/artiesten/'.$foto.'.jpg" rel="lightbox['.$artiest->naam.']" title="'.$artiest->naam.'">';
}
?>

Martijn B

28-09-2010 22:58 gewijzigd op 28-09-2010 23:01

De eerste foto uit de array laten heb je $key ook niet meer nodig:
<?php
$dbfoto = array($artiest->foto2,$artiest->foto3,$artiest->foto4,$artiest->foto5);
foreach($dbfoto as $foto)
{
if ( $foto )
{

}
}
?>

Als je alleen fotogroep 1 en 0 hebt dan:
<?php
if( $artiest->fotogroep )
{
...
}
else
{
...
}
?>

@SanThe Nvt:

Omdat misschien foto5 niet gevuld kan zijn.

- SanThe -

28-09-2010 23:00

Radio Dancemania op 28/09/2010 22:57:31

[quote="Karl Karl op 28/09/2010 22:56:19"]
Ga normaliseren.

geef misschien antwoorden waar iemand wat aan heeft of zeg gewoon niks;)

[/quote]

Tja, zo kan je het ook opvatten.

Echter, hij heeft wel gelijk.

Onbekende gebruiker

28-09-2010 23:01

Radio Dancemania op 28/09/2010 22:57:31

[quote="Karl Karl op 28/09/2010 22:56:19"]
Ga normaliseren.

geef misschien antwoorden waar iemand wat aan heeft of zeg gewoon niks;)

[/quote]
Je datadase model klopt niet.
Je hebt nu vijf tabellen zeg je.
Met daarin foto's
Tabel foto1, tabel foto2, enzovoort.
Dat klopt niet. Dat zorgt voor fouten. Dat is niet een genormaliseerd model.
Wat nu als je wilt dat er meerdere foto's kunnen toegevoegd worden?
En als je iets niet weet, dan moet je maar eens gaan googelen.

Radio Dancemania

28-09-2010 23:01

SanThe Nvt op 28/09/2010 22:58:32

<?php
$dbfoto = array($artiest->foto1,$artiest->foto2,$artiest->foto3.$artiest->foto4,$artiest->foto5);
foreach($dbfoto as $key => $foto)
{
if ( $key && $foto )
{
echo '<a href="/images/artiesten/'.$foto.'.jpg" rel="lightbox['.$artiest->naam.']" title="'.$artiest->naam.'">';
}
}
?>
Waarom moeilijk als het ook zonder if() kan.
<?php
$dbfoto = array(p$artiest->foto2,$artiest->foto3,$artiest->foto4,$artiest->foto5);
foreach($dbfoto as $foto)
{
echo '<a href="/images/artiesten/'.$foto.'.jpg" rel="lightbox['.$artiest->naam.']" title="'.$artiest->naam.'">';
}
?>

als je hem zonder if doet pakt ie dus alle 5 de fotos in het geval van martijn dus 4 foto's terwijl ik check of die foto's wel ingevoerd zijn?

[size=xsmall]Toevoeging op 28/09/2010 23:02:30:[/size]

Karl Karl op 28/09/2010 23:01:01

[quote="Radio Dancemania op 28/09/2010 22:57:31"]
[quote="Karl Karl op 28/09/2010 22:56:19"]
Ga normaliseren.

Reageren

Inloggen om te reageren