correcte coding
Pagina: « vorige 1 2 3 volgende »
Radio Dancemania op 28/09/2010 23:01:32:
(...)
nee ik heb geen tabel foto's ik heb een tabel met bijv artiesten:
naam, blabla,foto1,foto2,foto3,foto4 enz verder..
Karl Karl op 28/09/2010 23:01:01:
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 op 28/09/2010 22:57:31:
geef misschien antwoorden waar iemand wat aan heeft of zeg gewoon niks;)
Karl Karl op 28/09/2010 22:56:19:
Ga normaliseren.
geef misschien antwoorden waar iemand wat aan heeft of zeg gewoon niks;)
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.
nee ik heb geen tabel foto's ik heb een tabel met bijv artiesten:
naam, blabla,foto1,foto2,foto3,foto4 enz verder..
Dat klopt dus ook niet. Zelfde probleem als ik hierboven uitleg.
het probleem van meer foto's krijg ik niet op deze pagina omdat ik maximaal 5 foto's toelaat per artiest.. meer is niet nodig daar hebben ze hun eigen website voor.
Radio Dancemania op 28/09/2010 23:05:22:
het probleem van meer foto's krijg ik niet op deze pagina omdat ik maximaal 5 foto's toelaat per artiest.. meer is niet nodig daar hebben ze hun eigen website voor.
Dat is niet het hoofd probleem. Het hoofdprobleem is dat je die vage code van dat vorige en dit topic moest schrijven. Als je gewoon een standaard 1:n relatie had gedaan (of misschien zelfs wel een n:m relatie voor artiesten die samenwerken en dus samen op een foto staan) had je deze problemen niet gehad. Dan had je gewoon een net lijstje met foto's gehad.
En wat als je in de toekomst het wilt veranderen?
En wat als iemand maar één foto heeft?
Radio Dancemania op 28/09/2010 23:01:32:
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?
Beter kijken. Ik heb ook de eerste weggelaten. Om met een if() te kijken is totaal zinloos omdat de foreach() alle records uit het array() afloopt en niet meer. Dus die waarde bestaat altijd.
Karl Karl op 28/09/2010 23:07:29:
Dat is niet het hoofd probleem. Het hoofdprobleem is dat je die vage code van dat vorige en dit topic moest schrijven. Als je gewoon een standaard 1:n relatie had gedaan (of misschien zelfs wel een n:m relatie voor artiesten die samenwerken en dus samen op een foto staan) had je deze problemen niet gehad. Dan had je gewoon een net lijstje met foto's gehad.
En wat als je in de toekomst het wilt veranderen?
En wat als iemand maar één foto heeft?
Radio Dancemania op 28/09/2010 23:05:22:
het probleem van meer foto's krijg ik niet op deze pagina omdat ik maximaal 5 foto's toelaat per artiest.. meer is niet nodig daar hebben ze hun eigen website voor.
Dat is niet het hoofd probleem. Het hoofdprobleem is dat je die vage code van dat vorige en dit topic moest schrijven. Als je gewoon een standaard 1:n relatie had gedaan (of misschien zelfs wel een n:m relatie voor artiesten die samenwerken en dus samen op een foto staan) had je deze problemen niet gehad. Dan had je gewoon een net lijstje met foto's gehad.
En wat als je in de toekomst het wilt veranderen?
En wat als iemand maar één foto heeft?
Quote:
En wat als iemand maar één foto heeft?
check de code nogmaals;)
Radio Dancemania op 28/09/2010 23:09:46:
check de code nogmaals;)
Karl Karl op 28/09/2010 23:07:29:
Dat is niet het hoofd probleem. Het hoofdprobleem is dat je die vage code van dat vorige en dit topic moest schrijven. Als je gewoon een standaard 1:n relatie had gedaan (of misschien zelfs wel een n:m relatie voor artiesten die samenwerken en dus samen op een foto staan) had je deze problemen niet gehad. Dan had je gewoon een net lijstje met foto's gehad.
En wat als je in de toekomst het wilt veranderen?
En wat als iemand maar één foto heeft?
Radio Dancemania op 28/09/2010 23:05:22:
het probleem van meer foto's krijg ik niet op deze pagina omdat ik maximaal 5 foto's toelaat per artiest.. meer is niet nodig daar hebben ze hun eigen website voor.
Dat is niet het hoofd probleem. Het hoofdprobleem is dat je die vage code van dat vorige en dit topic moest schrijven. Als je gewoon een standaard 1:n relatie had gedaan (of misschien zelfs wel een n:m relatie voor artiesten die samenwerken en dus samen op een foto staan) had je deze problemen niet gehad. Dan had je gewoon een net lijstje met foto's gehad.
En wat als je in de toekomst het wilt veranderen?
En wat als iemand maar één foto heeft?
Quote:
En wat als iemand maar één foto heeft?
check de code nogmaals;)
Je gaat van de code uit. De code moet dus jou fouten in de database opvangen?
SanThe Nvt op 28/09/2010 23:08:44:
Beter kijken. Ik heb ook de eerste weggelaten. Om met een if() te kijken is totaal zinloos omdat de foreach() alle records uit het array() afloopt en niet meer. Dus die waarde bestaat altijd.
Radio Dancemania op 28/09/2010 23:01:32:
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?
Beter kijken. Ik heb ook de eerste weggelaten. Om met een if() te kijken is totaal zinloos omdat de foreach() alle records uit het array() afloopt en niet meer. Dus die waarde bestaat altijd.
Het gaat mij erom.. dat als bijv $artiest->foto3 en $artiest->foto4 e.d. leeg zijn, niet getoond worden en ik dus ook geen lightbox krijg met alleen het plaatje loading(omdat die foto er gewoonweg niet is)
Gewijzigd op 28/09/2010 23:12:09 door Radio Dancemania
Radio Dancemania op 28/09/2010 23:10:49:
Het gaat mij erom.. dat als bijv $artiest->foto3 en $artiest->foto4 e.d. leeg zijn, niet getoond worden en ik dus ook geen lightbox krijg met alleen het plaatje loading(omdat die foto er gewoonweg niet is)
SanThe Nvt op 28/09/2010 23:08:44:
Beter kijken. Ik heb ook de eerste weggelaten. Om met een if() te kijken is totaal zinloos omdat de foreach() alle records uit het array() afloopt en niet meer. Dus die waarde bestaat altijd.
Radio Dancemania op 28/09/2010 23:01:32:
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?
Beter kijken. Ik heb ook de eerste weggelaten. Om met een if() te kijken is totaal zinloos omdat de foreach() alle records uit het array() afloopt en niet meer. Dus die waarde bestaat altijd.
Het gaat mij erom.. dat als bijv $artiest->foto3 en $artiest->foto4 e.d. leeg zijn, niet getoond worden en ik dus ook geen lightbox krijg met alleen het plaatje loading(omdat die foto er gewoonweg niet is)
Zie je nou zelf niet dat deze database je allerlei problemen oplevert? Je zit zelf nu ook dus met dat probleem als er maar één foto is dat er niet meer moeten worden getoond.
Martijn B op 28/09/2010 23:12:11:
juist ja.. zoals ik hem ook nu heb staan.. als ik die if weglaat krijg ik dus 4 extra foto's die niet bestaan daarom dus die if;)
Deze manier van scripten is waardeloos.
Ik stop mijn support aan dit topic.
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.
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.
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.
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!
Toevoeging op 28/09/2010 23:18:11:
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.
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;)
Gewijzigd op 28/09/2010 23:18:31 door Radio Dancemania
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
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
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 ?
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.
Meestal zet je ook een index op een vreemde sleutel. Hierdoor voorkom je dat MySQL de hele tabel moet doorzoeken naar een bepaalde artiest.
Gewijzigd op 28/09/2010 23:29:56 door Martijn B
Zoals Hipska al aangeeft. Stop je energie nou gewoon in het normaliseren van je database structuur. Is je probleem meteen verholpen.
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.
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.