Reacties op het script Database class
|
|
 Gepost op: 26 februari 2005 - 14:32 |
|
|
|
Crew algemeen
|
| Hij is anders wel goedgekeurd.... Ook op een andere site |
|
|
|
|
| Gepost op: 26 februari 2005 - 15:47 |
|
|
|
Onbekend
|
Citaat: [I][B]wimmarien[/B] schreef op 26 Februari 2005 - 14:32[/I]
Hij is anders wel goedgekeurd.... Ook op een andere site
Opmerkingen:
- je gebruikt $this->connection en $this->verbinding door elkaar.
Citaat: Database connectie (opnieuw) maken; m.a.w. meerdere hosts mogelijk
classes zijn ervoor om meerdere objecten van te maken. Als je met 2 hosts tegelijk wilt werken maak je dus 2 objecten.
Anders kun je er net zo goed functies van maken, zoals dit van Fenrir:
http://www.site...amp;id=545
Verder lijkt het me wel een goed script, ik zou alleen 1 fetch() functie maken met als 2e parameter het type, bijv. als constante, zoals in Fenrirs script.
En ipv de set_ en unset_ functies zou je ook 1 functie kunnen maken met een boolean als parameter. |
|
|
|
|
| Gepost op: 26 februari 2005 - 15:50 |
|
|
|
Crew algemeen
|
Citaat: [I][B]numlockrond[/B] schreef op 26 Februari 2005 - 15:47[/I]
[..quote..] Tegen wie heb je het??
Opmerkingen:
- je gebruikt $this->connection en $this->verbinding door elkaar.
Die reactie is ondertussen al verwijderd....
Citaat: [I][B]numlockrond[/B] schreef op 26 Februari 2005 - 15:47[/I]
[..quote..] waarom maak je er dan nog een class van??
classes zijn ervoor om meerdere objecten van te maken. Als je met 2 hosts tegelijk wilt werken maak je dus 2 objecten.
Anders kun je er net zo goed functies van maken, zoals dit van Fenrir:
http://www.site...amp;id=545
Verder lijkt het me wel een goed script, ik zou alleen 1 fetch() functie maken met als 2e parameter het type, bijv. als constante, zoals in Fenrirs script.
En ipv de set_ en unset_ functies zou je ook 1 functie kunnen maken met een boolean als parameter.
Dan is het mijn classe niet meer ;)
Bovendien had ik eerst $verbinding, maar vond $connection achteraf beter qua mijn gewentes van scripten....:s |
|
|
|
|
| Gepost op: 26 februari 2005 - 16:49 |
|
|
|
Onbekend
|
Citaat: Dan is het mijn classe niet meer ;)
Als je die functies iets logischer maakt en de overbodige functie reconnect() (+ bijbehorend commentaar) eruit haalt is het gewoon jouw class, maar een beetje verbeterd naar aanleiding van commentaar wat je gekregen hebt.
Je hebt die fout met verbinding/connection toch ook niet laten zitten omdat het anders niet meer jouw class is?? |
|
|
|
|
| Gepost op: 26 februari 2005 - 18:12 |
|
|
|
Crew algemeen
|
Citaat: Je hebt die fout met verbinding/connection toch ook niet laten zitten omdat het anders niet meer jouw class is??
Je hebt een punt....
Ik pas ze wel eens een keertje aan (naargelang de commentaar etc)  , maar, is ze voor de rest goed? |
|
|
|
|
| Gepost op: 26 februari 2005 - 19:07 |
|
|
|
Onbekend
|
Citaat: maar, is ze voor de rest goed?
zal dit bericht nog wel een keer editten als ik zin heb... |
|
|
|
|
| Gepost op: 26 februari 2005 - 23:02 |
|
|
|
PHP gevorderde
|
Ziet er inderdaad heel goed uit, alleen die or die() moet eruit.
En je moet de link identifier die mysql_connect() teruggeeft gebruiken! Probeer maar eens op 2 databases tegelijk in te loggen met twee instanties van je class. |
|
|
|
|
| Gepost op: 03 maart 2005 - 13:53 |
|
|
|
PHP interesse
|
| ik denk ook dat querys verkeerd is gespeld, het moet volgens mij met ie. -denk ik |
|
|
|
|
| Gepost op: 04 maart 2005 - 16:55 |
|
|
|
Crew algemeen
|
lol, ben geen perfect in nederlands, en aangezien query geen (correct) nederlands is (vermoed ik), pas ik ook maar mijn spelling toe  |
|
|
|
|
| Gepost op: 16 januari 2008 - 00:12 |
|
|
|
Nieuw lid
|
mss beetje zeer late post op dit script, maar bon, so be it, ik heb nu pas nood aan een dergelijke class.
Ik heb nooit nood gehad aan een database class omdat ik toch telkens maar bezig was met 1 database,...
echter nu ben ik bezig met een platform dat met meerdere databases moet kunnen verbinden, en op verschillende servers eventueel.
dit script is zeer goed om verbinding te maken met meerdere databases, op die op eenzelfde server staan, en waarbij je met een overkoepelende user zit, je hebt maar 1 instantie van de class nodig. ECHTER wanneer ik ook met een 2e server verbonden wil zijn, werkt het nie meer, tenzij ik de eerste instantie eerst close.
iemand een idee van hoe ik ervoor kan zorgen dat ik wel 2 instanties van de class (TEGELIJK) kan gebruiken (zonder eerst 1 van de 2 te moeten closen) die met verschillende servers verbonden zijn? |
|
|
|
|
| Gepost op: 16 januari 2008 - 16:15 |
|
|
|
Crew algemeen
|
| gaat niet met 1 instantie van deze classe... |
|
|
|
|
| Gepost op: 05 februari 2008 - 16:14 |
|
|
|
Nieuw lid
|
ehm jah, dat is inderdaad precies wat ik zeg hé, en dat is ook logisch dat dat niet kan met 1 instantie, maar MET 2 LUKT HET OOK NIET,.... DAT is mijn probleem,...
|
|
|
|
|
| Gepost op: 05 februari 2008 - 19:12 |
|
|
|
Crew algemeen
|
je kan een kleine aanpassing maken, en een array met alle connection resources defineren. Dmv een extra parameter (of een variabele) selecteer je dan een connectie die actief is (deze steek je in bvb de var $currentCon). Vervolgens maak je van mysql_query($query) dit: mysql_query($query, $this->currentCon)
Normaal moet het dan werken... |
|
|
| Enkel aanvullende informatie, vragen en antwoorden op vragen zijn welkom. |
|
|
|
Wat is jouw favoriete javascript framework? (S: 18, R: 2)
