Webwereld

[ph] Hoe zou deze query en code beter kunnen

Gestart doorAarclay13 reacties75 weergaven
Log in om mee te praten

14 berichten · pagina 1 van 1

Aarclay

Lid · Topicstarter

Berichten
10.421
Reputatie
4
Punten
0,00
Lid sinds
Laatst gezien
[COLOR= #0000BB]<?php
$result_cats[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$db[/COLOR][COLOR= #007700]->[/COLOR]query[COLOR= #007700]([/COLOR][COLOR= #DD0000]"SELECT id, title FROM news_categories"[/COLOR][COLOR= #007700]);[/COLOR]
    [URL="http://www.php.net/while"]while[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$newscat[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$db[/COLOR][COLOR= #007700]->[/COLOR]fetch_assoc[COLOR= #007700]([/COLOR][COLOR= #0000BB]$result_cats[/COLOR][COLOR= #007700])) {[/COLOR]
        [COLOR= #0000BB]$sql[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #DD0000]"SELECT id, title, date_posted, auteur FROM news WHERE catID ="[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$newscat[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]id[/COLOR][COLOR= #007700]].[/COLOR][COLOR= #DD0000]" AND active=1 ORDER BY date_posted DESC LIMIT 5"[/COLOR][COLOR= #007700];[/COLOR]
       
         [COLOR= #0000BB]$result_items[/COLOR] [COLOR= #007700]=[/COLOR][COLOR= #0000BB]$db[/COLOR][COLOR= #007700]->[/COLOR]query[COLOR= #007700]([/COLOR][COLOR= #0000BB]$sql[/COLOR][COLOR= #007700]);[/COLOR]
        [URL="http://www.php.net/while"]while[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$item[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$db[/COLOR][COLOR= #007700]->[/COLOR]fetch_assoc[COLOR= #007700]([/COLOR][COLOR= #0000BB]$result_items[/COLOR][COLOR= #007700])) {[/COLOR]
            [COLOR= #0000BB]$data[/COLOR][COLOR= #007700][[/COLOR][COLOR= #0000BB]$newscat[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]title[/COLOR][COLOR= #007700]]][] =[/COLOR] [COLOR= #0000BB]$item[/COLOR][COLOR= #007700];
        }
    }[/COLOR]
    [COLOR= #0000BB]$tpl[/COLOR][COLOR= #007700]->[/COLOR]assign[COLOR= #007700]([/COLOR][COLOR= #DD0000]"news_categories"[/COLOR][COLOR= #007700],[/COLOR][COLOR= #0000BB]$data[/COLOR][COLOR= #007700]);[/COLOR]
    [COLOR= #0000BB]$tpl[/COLOR][COLOR= #007700]->[/COLOR]display[COLOR= #007700]([/COLOR][COLOR= #DD0000]"news_archive_index.tpl"[/COLOR][COLOR= #007700]);[/COLOR]
?>


Dit geeft een toffe multi-dimensionale array met alle categorieën, en daaronder de bijbehorende items. Een geweldig stukje code van mij van anderhalfjaar geleden. yummie
Echter, om steeds per categorie een query uit te voeren kan wel zwaar worden als je steeds meer categorieën hebt. Iemand enig idee hoe ik dit stukje code van verbeteren qua performance? Naar joins heb ik al gekeken, maar dat lijkt de juiste oplossing niet tussen te zitten...

Iemand die even advies kan geven?
0
Daiman

Lid

Berichten
5.795
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Het is denk ik toch beter om met joins hier te gaan werken.
Want om voor elke categorie een apparte query te doen voor de berichten is veel zwaarder dan joins gebruiken.

[COLOR= #0000BB]<?php[/COLOR]
[URL="http://www.php.net/mysql_query"]mysql_query[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]"SELECT news_categories.id, news_categories.title, news.id, news.title, news.date_posted, news.auteur FROM news_categories INNER JOIN news ON( news.catID = news_categories.id )"[/COLOR][COLOR= #007700]);[/COLOR]
?>


iets in die trant
0
Aarclay

Lid · Topicstarter

Berichten
10.421
Reputatie
4
Punten
0,00
Lid sinds
Laatst gezien
Daar zat ik ook aan te denken, maar het resultaat heeft toen niet gewerkt. Ik zal straks nog eens testen..
0
Smiegles

Lid

Berichten
4.912
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Aarclay schreef:
Daar zat ik ook aan te denken, maar het resultaat heeft toen niet gewerkt. Ik zal straks nog eens testen..
als je er al aan zat te denken waarom heb je het dan niet eerst zelf getest en daarom dit topic aangemaakt samen met die query die niet werkt / wel werkt dus dit hele topic niet nodig was?
0
One2xs

Bot

Berichten
93.699
Reputatie
0
Punten
1,27
Lid sinds
Laatst gezien
Smiegles schreef:
Eerdere citaten tonen
Aarclay schreef:
Daar zat ik ook aan te denken, maar het resultaat heeft toen niet gewerkt. Ik zal straks nog eens testen..
als je er al aan zat te denken waarom heb je het dan niet eerst zelf getest en daarom dit topic aangemaakt samen met die query die niet werkt / wel werkt dus dit hele topic niet nodig was?
Omdat hij vraagt of er een betere manier is
0
Smiegles

Lid

Berichten
4.912
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Nielsie schreef:
Eerdere citaten tonen
Smiegles schreef:
Eerdere citaten tonen
Aarclay schreef:
Daar zat ik ook aan te denken, maar het resultaat heeft toen niet gewerkt. Ik zal straks nog eens testen..
als je er al aan zat te denken waarom heb je het dan niet eerst zelf getest en daarom dit topic aangemaakt samen met die query die niet werkt / wel werkt dus dit hele topic niet nodig was?
Omdat hij vraagt of er een betere manier is
maar meneer had zelf al een betere manier gevondenemo
0
Alemaalte

Lid

Berichten
5.957
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Smiegles schreef:
Eerdere citaten tonen
Nielsie schreef:
Eerdere citaten tonen
Smiegles schreef:
Eerdere citaten tonen
Aarclay schreef:
Daar zat ik ook aan te denken, maar het resultaat heeft toen niet gewerkt. Ik zal straks nog eens testen..
als je er al aan zat te denken waarom heb je het dan niet eerst zelf getest en daarom dit topic aangemaakt samen met die query die niet werkt / wel werkt dus dit hele topic niet nodig was?
Omdat hij vraagt of er een betere manier is
maar meneer had zelf al een betere manier gevondenemo
Maar meneer kan niet lezen? Waar zegt hij dat hij een betere manier heeft? Je vraagt ook waarom hij het niet test, terwijl hij zei: "maar het resultaat heeft toen niet gewerkt." wat toch wel indiceert dat hij het geprobeerd heeft. Gewoon je bullshit bijhouden.
0
Smiegles

Lid

Berichten
4.912
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Alemaalte schreef:
Eerdere citaten tonen
Smiegles schreef:
Eerdere citaten tonen
Nielsie schreef:
Eerdere citaten tonen
Smiegles schreef:
Eerdere citaten tonen
Aarclay schreef:
Daar zat ik ook aan te denken, maar het resultaat heeft toen niet gewerkt. Ik zal straks nog eens testen..
als je er al aan zat te denken waarom heb je het dan niet eerst zelf getest en daarom dit topic aangemaakt samen met die query die niet werkt / wel werkt dus dit hele topic niet nodig was?
Omdat hij vraagt of er een betere manier is
maar meneer had zelf al een betere manier gevondenemo
Maar meneer kan niet lezen? Waar zegt hij dat hij een betere manier heeft? Je vraagt ook waarom hij het niet test, terwijl hij zei: "maar het resultaat heeft toen niet gewerkt." wat toch wel indiceert dat hij het geprobeerd heeft. Gewoon je bullshit bijhouden.
Aarclay schreef:
Daar zat ik ook aan te denken, maar het resultaat heeft toen niet gewerkt. Ik zal straks nog eens testen..
emo
0
Alemaalte

Lid

Berichten
5.957
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Smiegles schreef:
Eerdere citaten tonen
Alemaalte schreef:
Eerdere citaten tonen
Smiegles schreef:
Eerdere citaten tonen
Nielsie schreef:
Eerdere citaten tonen
Smiegles schreef:
Eerdere citaten tonen
Aarclay schreef:
Daar zat ik ook aan te denken, maar het resultaat heeft toen niet gewerkt. Ik zal straks nog eens testen..
als je er al aan zat te denken waarom heb je het dan niet eerst zelf getest en daarom dit topic aangemaakt samen met die query die niet werkt / wel werkt dus dit hele topic niet nodig was?
Omdat hij vraagt of er een betere manier is
maar meneer had zelf al een betere manier gevondenemo
Maar meneer kan niet lezen? Waar zegt hij dat hij een betere manier heeft? Je vraagt ook waarom hij het niet test, terwijl hij zei: "maar het resultaat heeft toen niet gewerkt." wat toch wel indiceert dat hij het geprobeerd heeft. Gewoon je bullshit bijhouden.
Eerdere citaten tonen
Aarclay schreef:
Daar zat ik ook aan te denken, maar het resultaat heeft toen niet gewerkt. Ik zal straks nog eens testen..
emo
Dus? Hij zat aan zoiets te denken? Wil dat zeggen dat dat het beste is? Daarbij zei hij dat het niet werkte. Dus emo voor jou, want je gedraagt je nogal dom.
0
Aarclay

Lid · Topicstarter

Berichten
10.421
Reputatie
4
Punten
0,00
Lid sinds
Laatst gezien
Kunnen we nu normaal doen, i.p.v. armoedig geziek?
Thnx worshippy
0
Smiegles

Lid

Berichten
4.912
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Alemaalte schreef:
Eerdere citaten tonen
Smiegles schreef:
Eerdere citaten tonen
Alemaalte schreef:
Eerdere citaten tonen
Smiegles schreef:
Eerdere citaten tonen
Nielsie schreef:
Eerdere citaten tonen
Smiegles schreef:
Eerdere citaten tonen
Aarclay schreef:
Daar zat ik ook aan te denken, maar het resultaat heeft toen niet gewerkt. Ik zal straks nog eens testen..
als je er al aan zat te denken waarom heb je het dan niet eerst zelf getest en daarom dit topic aangemaakt samen met die query die niet werkt / wel werkt dus dit hele topic niet nodig was?
Omdat hij vraagt of er een betere manier is
maar meneer had zelf al een betere manier gevondenemo
Maar meneer kan niet lezen? Waar zegt hij dat hij een betere manier heeft? Je vraagt ook waarom hij het niet test, terwijl hij zei: "maar het resultaat heeft toen niet gewerkt." wat toch wel indiceert dat hij het geprobeerd heeft. Gewoon je bullshit bijhouden.
Eerdere citaten tonen
Aarclay schreef:
Daar zat ik ook aan te denken, maar het resultaat heeft toen niet gewerkt. Ik zal straks nog eens testen..
emo
Dus? Hij zat aan zoiets te denken? Wil dat zeggen dat dat het beste is? Daarbij zei hij dat het niet werkte. Dus emo voor jou, want je gedraagt je nogal dom.
Mijn punt :
Meneer post een topic met daar in een vraag hoe hij iets het beste kan doen.
Meneer krijgt daar een antwoord op en inplaats van : bedankt , maar het werkt nog niet helemaal is het antwoord : Dacht ik ook al aan.
post dit dan OOk in het topic als je daar al aan dacht inplaats van antwoorden dat je dat al in gedachten had.
meneer post namelijk half Werk en daar erger ik me aan.
EN ALS IETS NIET WERKT VRAAG DAT DAN KOE
0
Marvinn

Lid

Berichten
2.609
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
//Laatmaar
0
Aarclay

Lid · Topicstarter

Berichten
10.421
Reputatie
4
Punten
0,00
Lid sinds
Laatst gezien
Wat mekkeren we toch in dit topic zeg...............
Kap eens met die onzin en ga lekker via PM met elkaar bekvechten.

Heren zoek het maar lekker uit. De join van Daiman ga ik uittesten en verder zoek ik het wel zelf uit.

Slotje op verzoek van deze TS!
0
Sjoerd

Lid

Berichten
13.482
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Deze dubbelpost van Aarclay is achtergebleven na het verwijderen van nogal wat onnodige posts. Hier komt een slotje op.
0

Jouw reactie

Deel je idee, stel een vraag of help een ander verder.

Editor laden…

Praat mee met de One2xs-community.

Inloggen om te reageren