Origineel gepost door Ivo van B.

Ben ik niet met je eens, geef eens 1 voordeel van het plaatsen van een querystring in een aparte variabele alvorens deze uit te voeren.

Querystring?
Wat hebben die ermee te maken?

Origineel gepost door John Timmer

En dan heb je het plan van aanpak klaar, overzicht van functies, nette codeerstandaarden, documenteren, testen, alle fouten afvangen via het boekje... blijkt dat de klant alleen maar budget heeft voor het eerste punt, hehehe :)

[off]Aan dat soort klanten heb je weinig.... Goede code kost geld, budget hebben of opzoek naar een goedkopere scripter..[/off]

Origineel gepost door Vincent

Querystring?
Wat hebben die ermee te maken?

Querystring daarmee bedoel ik de SQL die je gaat uitvoeren, ik bedoel niet de $_SERVER['QUERY_STRING'], misschien kies ik die naam wat onhandig...

Origineel gepost door Ivo van B.

Ben ik niet met je eens, geef eens 1 voordeel van het plaatsen van een querystring in een aparte variabele alvorens deze uit te voeren.

welk gedeelte ben je het met mee eens dan?

zowel

Code:

$boe = "mysql error ga ik krijgen";
mysql_query($boe);

als

Code:

mysql_query("mysql error ga ik krijgen");

zijn netjes. om het voor de derde keer te herhalen. Wat is netjes? Niks is netjes en alles is netjes. Als jij afspraken maakt met jezelf, je klant of je baas en je houd je daarin. codeer je netjes. klaar.

voor de 2e keer het volgende:
er is geen php standaard. die is nooit geschreven en is ook nooit gedocumenteerd. dus alweer, alles is netjes.

Ik ben het met je eens dat code ook vies kan zijn, maar dat is ook een kwestie van persoonlijkheid. Ik heb wel eens code gezien waarvan ik werkelijk groen uitsloeg, totdat mn collega het uitlegde en toen had ik zoiets van.. Mja, best ok zo :)

Origineel gepost door Thijs Damen

welk gedeelte ben je het met mee eens dan?

zowel

Code:

$boe = "mysql error ga ik krijgen";
mysql_query($boe);

als

Code:

mysql_query("mysql error ga ik krijgen");

zijn netjes. om het voor de derde keer te herhalen. Wat is netjes? Niks is netjes en alles is netjes. Als jij afspraken maakt met jezelf, je klant of je baas en je houd je daarin. codeer je netjes. klaar.

voor de 2e keer het volgende:
er is geen php standaard. die is nooit geschreven en is ook nooit gedocumenteerd. dus alweer, alles is netjes.

Ik ben het met je eens dat code ook vies kan zijn, maar dat is ook een kwestie van persoonlijkheid. Ik heb wel eens code gezien waarvan ik werkelijk groen uitsloeg, totdat mn collega het uitlegde en toen had ik zoiets van.. Mja, best ok zo :)

Ik geef het op.. maar ik blijf het toch onnodig geheugengebruik vinden...

Origineel gepost door Ivo van B.

Ik geef het op.. maar ik blijf het toch onnodig geheugengebruik vinden...

1. we hebben het niet over optimalisatie
2. Dan gooi je het in een functie. word die op de stack gegooit. is die zo leeg.

Origineel gepost door Thijs Damen

1. we hebben het niet over optimalisatie
2. Dan gooi je het in een functie. word die op de stack gegooit. is die zo leeg.

Netheid en optimalisatie horen bij elkaar, in een functie proppen is ook zonde..

@ hieronder

Ik zei ook ik geef het op.. jij reageert daarna nog;)

ik ga hetn iet voor een 4e keer herhalen.

Origineel gepost door Ivo van B.

Ik geef het op.. maar ik blijf het toch onnodig geheugengebruik vinden...

Geef eens een voorbeeld van hoe jij het zou doen ;)

Origineel gepost door Vincent

Geef eens een voorbeeld van hoe jij het zou doen ;)

https://www.sitedeals.nl/scripting-ph...cripten-2.html

Eerste reactie daar, daar zie je code van mij, en bij Tutorials.
https://www.sitedeals.nl/tutorials/51...-notendop.html

Vind je het niet een beetje hypocriet om te zeggen dat het onnodig geheugengebruik is om een query netjes in een variable te zetten maar wel ondertussen wel een asterisk gebruiken in de query zelf wat vele malen meer geheugen zou gebruiken door het aanmaken van vars die wellicht niet eens gebruikt worden.

Daarbij is PDO sowieso altijd meer serverintensief als direct in de eigen taal/dialect querien (logisch, want pdo moet simpel gezegd geconverteerd worden naar het eigen dialect).


Dus je argument gaat niet echt op ;)

Origineel gepost door Vincent

Vind je het niet een beetje hypocriet om te zeggen dat het onnodig geheugengebruik is om een query netjes in een variable te zetten maar wel ondertussen wel een asterisk gebruiken in de query zelf wat vele malen meer geheugen zou gebruiken door het aanmaken van vars die wellicht niet eens gebruikt worden.

Daarbij is PDO sowieso altijd meer serverintensief als direct in de eigen taal/dialect querien (logisch, want pdo moet simpel gezegd geconverteerd worden naar het eigen dialect).


Dus je argument gaat niet echt op ;)

Origineel gepost door Vincent

* gebruiken is natuurlijk geen optie als je het hebt over besparen, overigens moet je dan ook geen fetch_array gebruiken maar fetch_object aangezien de array versie 2x zoveel vars bevat.

Die asterisk is nodig op dat stuk omdat ik ALLE velden uit die tabel wil hebben, die asterisk dus daar dus prima op z'n plaats, normaliter gebruik ik dat zo niet. Op andere plekken gebruik ik, zoals je ziet geen asterisk.
Overal plaats ik, waar nodig een LIMIT zodat MySQL niet onnodig verder gaat zoeken als het resultaat bereikt is, dat komt ook ten goede aan de snelheid.

Over fetch_object, dat is juist niet slim om te gebruiken, beter is om assoc of array te gebruiken, de array fetch kun je zo instellen dat je key's met namen krijgt of key's met getallen, of standaard, beide.

PDO is intensiever, eens, maar PDO levert zoveel voordeel op dat het nadeel snelheid niet meer opweegt. De SQL in een string plaatsen alvorens die uit te voeren biedt geen voordelen en wel nadelen.

Origineel gepost door Vincent

Maar hoeveel queries maak jij die zo lang zijn

Code:

SELECT * FROM tabel WHERE id = $x

(en als je het wel doet vraag ik me toch enigzins af hoe optmaal je code is als het bestaat uit veel kleine queries)

Een minimale querie die ik zou schrijven is toch wel als voorbeeld

Code:

$query = mysql_query("SELECT celnaam, celnaam, celnaam, celnaam, celnaam, celnaam FROM table WHERE table_id = " . $db->escape($var) . " AND status = 1");

Hoe vind je dat dan nog overzichtelijk, zeker voor toekomstige wijzigingen.


* gebruiken is natuurlijk geen optie als je het hebt over besparen, overigens moet je dan ook geen fetch_array gebruiken maar fetch_object aangezien de array versie 2x zoveel vars bevat.

Ik doelde met 'alles op 1 regel' dat je de querystring en queryfunctie (al dan niet PDO) niet apart moet doen. Dit wordt gelukkig later in dit topic nog bevestigd door Ivo. Daarnaast sluit ik me ook aan bij Ivo's vorige post: PDO is intensiever maar levert bij goed gebruik zoveel meer mogelijkheden dat ik het iedereen aanraad.

Betreft de query's: doe dit netjes zoals Vincent. Je zou nog een functie voor errorafhandeling kunnen maken, of gewoon een simpel berichtje neerzetten:

----

$query = mysql_query("<jouw query>") or die("Foutmelding:<br />".Mysql_error());

----
Je kunt nog beter zelfs dit doen:

----

$query['select']['profielgegevens'] = mysql_query("<jouw query>") or die("Foutmelding:<br />".Mysql_error());

----

Origineel gepost door Joris W

Betreft de query's: doe dit netjes zoals Vincent. Je zou nog een functie voor errorafhandeling kunnen maken, of gewoon een simpel berichtje neerzetten:

----

$query = mysql_query("<jouw query>") or die("Foutmelding:<br />".Mysql_error());

----
Je kunt nog beter zelfs dit doen:

----

$query['select']['profielgegevens'] = mysql_query("<jouw query>") or die("Foutmelding:<br />".Mysql_error());

----

Waarom zou je or die gebruiken? Zulke vieze en slechte error-handling waaraan je zo goed als niets hebt..
En waarom je queries allemaal opslaan in een array? Wat is daarvan het voordeel?

Het gaat hier om een voorbeeldje van netjes scripten, in mijn scripts gebruik ik mijn eigen functies om de fouten af te handelen. Het voordeel van de queries in een array, is dat het overzichtelijker is. Je ziet namelijk meteen wat de query doet.