Reacties op het script OpenTrivia
|
|
 Gepost op: 12 februari 2007 - 17:56 |
|
|
|
PHP expert
|
Goed systeem. Ik ben er 2 minuten mee bezig en heb 2 grote opmerkingen!
1) Ik zou in je eerste stap de mysql gegevens opvragen. Niet iedereen is zo bekwaam als jij en ik om deze aan te passen in het script. Dit zou ik veranderen.
2) Je kan modules inactief maken. Ook het admin panel kan je inactief maken! Dat is niet de bedoeling van een blog. Je moet kunnen posten. Nu kan ik niet meer in de admin panel omdat ik "per ongeluk" op inactief geklikt heb. Let op welke modules je kan uitschakelen hé.
Dus tip voor de volgende versie: werk aan je gebruiksvriendelijkheid.
stijn
*edit*
Wat ik wel grappig vind (ik zit in je database class te lezen) is dat je niet gebruikt maakt van je class.
<?
//regel 110
if(mysql_num_rows($results) > 0) // Check if there are results
?>
en enkele regels zie ik dit staan:
function query_num_rowsDB($query)
{
$count = mysql_num_rows($this->query_execDB($query));
return $count;
}
function query_num_rowsDB($query) { return $count; }
dus waarom maak je daar geen gebruik van?
<?
//regel 110
if( $this->query_num_rows($results) > 0 )
?>
<? //regel 110 if( $this->query_num_rows($results) > 0 ) ?>
|
|
|
|
|
| Gepost op: 13 februari 2007 - 19:55 |
|
|
|
Nieuw lid
|
Dankje voor je reactie, Stijn.
1) Misschien wel een goed idee van de MySQL gegevens en de gebruiksvriendelijkheid.
2) Het is inderdaad vreemd dat ik zo dom ben geweest mijn classe niet te gebruiken. Aan de andere kant ben ik niet echt een OOP-kenner :-) . |
|
|
|
|
| Gepost op: 14 februari 2007 - 17:27 |
|
|
|
PHP expert
|
Ik gok erop dat dit een opensource project is ?
Dan gok ik er ook op dat je zoveel mogelijk mensen je systeem wilt laten gebruiken. Nu is het alleen de MySQL gebruikers die je systeem zullen gebruiken.
Een oplossing kan zijn: een DAL (Database Abstraction Layer, een class maken dat meerdere database ondersteund (zoek maar op google)) maken of de DAL van pear.php.net gebruiken (ook hiervoor kun je zoeken). |
|
|
|
|
| Gepost op: 14 februari 2007 - 23:17 |
|
|
|
PHP expert
|
@siliecom14: DAL wordt gebruikt in PHP5 classes. Hij gebruikt PHP4 classes. De meest gebruikte database is nog steeds mysql als over php praten. Iedere php project is bijna opensource, want die valt onder php licensie  |
|
|
|
|
| Gepost op: 15 februari 2007 - 07:52 |
|
|
|
Nieuw lid
|
@ Siliecom14: Volgens mij is het niet echt een goed idee om aan DAL te gaan doen. Als ik een beetje juist denk, dan is het toch zo dat 98% van de sites die met PHP werken ook een MySQL-database hebben  ?
@ Stijn: Juist, dus dat valt al naast de kwestie. Ik vind het ook een beetje stom om de PHP4 gebruikers dan uit te sluiten, waaronder ikzelf ook val  ! (Aan jouw goede ideeën wordt trouwens gewerkt )
Toch bedankt allebei !
[edit]typo[/edit] |
|
|
|
|
| Gepost op: 04 maart 2009 - 23:21 |
|
|
|
Lid
|
|
|
|
|
| Gepost op: 07 mei 2009 - 17:58 |
|
|
|
PHP interesse
|
|
|
| Enkel aanvullende informatie, vragen en antwoorden op vragen zijn welkom. |
|
|
|
Wat is jouw favoriete javascript framework? (S: 18, R: 2)
