Huhu!
Erstmal will ich bevor ich anfang zu motzen ;) ein Lob aussprechen euer System gefllt mir im Grunde sehr gut.
Aber es gibt 2 1/2 Dinge die mir immer ziemlich "aufstoen"
1. Die tausend mysql_connects . Ihr habt praktisch fr jeden Query auch einen connect zur db den ihr imemr weider schliet. Soweit mir bekannt ist soll das Gift fr die Geschwindigkeit sein.
Ihr habt ein auf Klassen basierendes System, aber keine Datenbankklasse. Das ist mir sehr unverstndlich.
2. Ihr solltet eine Funktion einabauen, die es dem User erlaubt seinen Vor + Nachnamen auf der Gsteliste zu verbergen. Bei uns auf der Lan gab es etliche user denen das sehr sehr wichtig war, und die etwas schlecht gelaunt waren das das nicht mglich war.
3. (der halbe punkt :) Ich bitte / rate euch irgendwo in euerer CCore.php oder so das Error_reporting selbst zu setzen. Ich hab mein standard error_reporting auf E_ALL zu debug zwecken. DAs bedeutet aber bei Chrystal, das ich erstmal 2 Seiten Notices ber nicht initialisierte Variablen habe.
~bye~ StIllE
www.csvjm-zeltlager.de
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Servus, freut uns, dass dir das System gefllt.
Zu 1.: scand1sk hat eine neue Klasse CDB.php eingebaut (vorgestern), auerdem bin ich bereits dabei, einige Klassen umzustellen.
zu 2.: Werde ich sobald wie mglich einbauen
zu 3.: Hm... also quasi ein ini_set(), ja?
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
ja ist ein ini set (http://www.php.net/manual/en/function.error-reporting.php) mit z.b. der Einstellung: E_ALL ^ E_NOTICE
aber wie schon gesagt: is nur en halber punkt ;)
*cvs.client.such*
Danke fr das schnelle Beachten :)
~bye~
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
trotzdem muss ich zum error handling nochnal kritik abgeben: E_ALL + E_NOTICE sind von PHP gesetze Konstanten. das & und ~ sind die Bitoperatoren von php.
Wenn du aber mit ini_set('error_reporting', "E_ALL $ ~E_NOTICE"); den Eintrag setzt, bergibst du einen string, wo eigentlich eine Zahl bergeben werden sollte => es nimmt die einstellung nicht richtig. Wenn du einfach die Anfhrungszeichen um E_ALL $ ~E_NOTICE weglsst, funktioniert es problemlos :)
So Far
StIllE
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Anonymous
-
2003-04-21
nder ich eben
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Anonymous
-
2003-04-21
gendert
-> CVS
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Huhu!
Erstmal will ich bevor ich anfang zu motzen ;) ein Lob aussprechen euer System gefllt mir im Grunde sehr gut.
Aber es gibt 2 1/2 Dinge die mir immer ziemlich "aufstoen"
1. Die tausend mysql_connects . Ihr habt praktisch fr jeden Query auch einen connect zur db den ihr imemr weider schliet. Soweit mir bekannt ist soll das Gift fr die Geschwindigkeit sein.
Ihr habt ein auf Klassen basierendes System, aber keine Datenbankklasse. Das ist mir sehr unverstndlich.
2. Ihr solltet eine Funktion einabauen, die es dem User erlaubt seinen Vor + Nachnamen auf der Gsteliste zu verbergen. Bei uns auf der Lan gab es etliche user denen das sehr sehr wichtig war, und die etwas schlecht gelaunt waren das das nicht mglich war.
3. (der halbe punkt :) Ich bitte / rate euch irgendwo in euerer CCore.php oder so das Error_reporting selbst zu setzen. Ich hab mein standard error_reporting auf E_ALL zu debug zwecken. DAs bedeutet aber bei Chrystal, das ich erstmal 2 Seiten Notices ber nicht initialisierte Variablen habe.
~bye~ StIllE
www.csvjm-zeltlager.de
Servus, freut uns, dass dir das System gefllt.
Zu 1.: scand1sk hat eine neue Klasse CDB.php eingebaut (vorgestern), auerdem bin ich bereits dabei, einige Klassen umzustellen.
zu 2.: Werde ich sobald wie mglich einbauen
zu 3.: Hm... also quasi ein ini_set(), ja?
-> CVS
ja ist ein ini set (http://www.php.net/manual/en/function.error-reporting.php) mit z.b. der Einstellung: E_ALL ^ E_NOTICE
aber wie schon gesagt: is nur en halber punkt ;)
*cvs.client.such*
Danke fr das schnelle Beachten :)
~bye~
np *g*
trotzdem muss ich zum error handling nochnal kritik abgeben: E_ALL + E_NOTICE sind von PHP gesetze Konstanten. das & und ~ sind die Bitoperatoren von php.
Wenn du aber mit ini_set('error_reporting', "E_ALL $ ~E_NOTICE"); den Eintrag setzt, bergibst du einen string, wo eigentlich eine Zahl bergeben werden sollte => es nimmt die einstellung nicht richtig. Wenn du einfach die Anfhrungszeichen um E_ALL $ ~E_NOTICE weglsst, funktioniert es problemlos :)
So Far
StIllE
nder ich eben
gendert
-> CVS
Thx