Reacties op het script SQL Classe (pgsql + mysql)
|
|
 Gepost op: 14 augustus 2007 - 20:51 |
|
|
|
Beheerder
|
Ziet er goed uit maar ik mis de documentatie.
Kan je je script van documentatie voorzien? Eventueel met phpDocumentor. (Het liefst wel  ) |
|
|
|
|
| Gepost op: 14 augustus 2007 - 21:44 |
|
|
|
Crew algemeen
|
| die phpdocumentator krijg ik niet onmiddelijk aan de praat, iets met docblocks. Anyway, ik zal handmatig wel wat documentatie maken. |
|
|
|
|
| Gepost op: 16 augustus 2007 - 15:19 |
|
|
|
PHP expert
|
je kunt de functies ook algemener maken he  maak van je interface een abstractie, want zo kun je ook functies defineren.
De functies select/delete zien er ongeveer hetzelfde uit qua QUERY opbouw. Dus als je die dan in je abstractie zet, kun je die makkelijker gebruiken.
bijv.
function constructSelect( $params.... )
{
// hier maak je dan de query aan en return je een string
}
function constructSelect( $params.... ) { // hier maak je dan de query aan en return je een string }
en die kun je dan weer gebruiken als iemand de functie select gebruikt. Minder code en efficienter. Ik weet dat er verschillen zijn tussen MySQL en postgreSQL, maar bij zulke simpele queries kan het makkelijker he |
|
|
|
|
| Gepost op: 29 augustus 2007 - 16:52 |
|
|
|
PHP expert
|
1) veel code komt meerdere keren voor zoals DELETE, SELECT, INSERT, UPDATE. Het enige wat verschilt zijn de functie namen  Je interface is goed maar ik zou er een abstract class bijmaken en dan kan iedere subclass (mysql, pgsql,...) die extenden.
2) de error handling kan beter, maak gebruik van throw new Exception(...);. De eindgebruiker moet deze maar in een mooi jasje stoppen.
Voor de rest ziet de class er in orde uit, al heb ik altijd problemen met scripters die een array|object fetchen. Maak één fetch functie en een optionele parameter waar de eindgebruiker kan kiezen welke fetch methode ze willen gebruiken. |
|
|
|
|
| Gepost op: 30 augustus 2007 - 13:02 |
|
|
|
Crew algemeen
|
stijn1989 schreef: 1) veel code komt meerdere keren voor zoals DELETE, SELECT, INSERT, UPDATE. Het enige wat verschilt zijn de functie namen Je interface is goed maar ik zou er een abstract class bijmaken en dan kan iedere subclass (mysql, pgsql,...) die extenden.
Ik gebruik -vooral bij kleinere scripts- gewoon dit, en maak gewoon een volledig nieuwe instantie van de classe aan.... zonder parent!
Citaat: 2) de error handling kan beter, maak gebruik van throw new Exception(...);. De eindgebruiker moet deze maar in een mooi jasje stoppen.
Een script maak ik voor mezelf, niet voor anderen. Ik zet mijn scripts op internet en anderen kunnen hier van leren of ze vrij gebruiken/aanpassen.
Citaat: Voor de rest ziet de class er in orde uit, al heb ik altijd problemen met scripters die een array|object fetchen. Maak één fetch functie en een optionele parameter waar de eindgebruiker kan kiezen welke fetch methode ze willen gebruiken.
zelfde als hierboven |
|
|
|
|
| Gepost op: 02 september 2007 - 10:32 |
|
|
|
Crew algemeen
|
Ik vind het niet echt een goede class, het is meer een wrapper voor de mysql/pgsql functies dan echt een database class 
Ik bedoel, je verwerkt niet eens zelf de resultaten, je kunt gewoon de result identifier van mysql doorgeven, en that's it, ik zou een result-class maken die dit allemaal zelf afhandelt, dit is niks bijzonders. |
|
|
| Enkel aanvullende informatie, vragen en antwoorden op vragen zijn welkom. |
|
|
|
Wat is jouw favoriete javascript framework? (S: 18, R: 2)
