Petit ensemble de classes

Petit ensemble de classes

Anthor
Hors ligne

Manitou

Messages : 1 399
Sujets : 5
Inscription : Dec 2007

#15

04-11-2008, 07:46 PM

Ha ben attend, j'étais au boulot, faut quand même que je fasse semblant de travailler Smile

Code PHP :
<?php 

// On démarre la bufferisation

ob_start(); 

Code PHP :
<?php 

$buffer = ob_get_clean();

echo $buffer; 

C'est pas du tout cohérent, le premier est dans common et le deuxième dans l'index. C'est pas spécialement lisible, une fonction ouverte qui appelle une fermeture, les deux doivent être dans le même fichier.

Ou dans un fichier prepend et un postcon.

De plus je ne vois aucun intérêt à appeler la bufferisation dans un tel cas ou une variable revient au même et est plus léger.

Code PHP :
<?php 

$buffer = '';

$buffer .= $moduleGeneral->vue();

$buffer .= $moduleGeneral->vue();

echo $buffer;

Pourquoi ne pas avoir utilisé les exceptions pour la gestion d'erreur ?
Plutôt que de simple die ?

Pour la base de données :

Code PHP :
<?php 

// Pas de tests de validité du résultat, on le considère comme bon

return @mysql_num_rows($this->last); 

Pas d'accord, tu ne peux pas le considérer bon si la requête n'est pas intégré à la méthode.

Code PHP :
<?php 

return (is_ressource($this->last))?mysql_num_rows($this->last):false; 

Code PHP :
<?php 

public function update($val, $clef, $table)

{

    // On prépare la requête de base

    $sql = 'UPDATE ' . $table . ' SET ';

    // On passe en revu les différentes clefs

    foreach($val as $cl => $vl)

    {

        $sql .= '`' . $cl . '` = \'' . $vl . '\', ';

    }

    // On ajoute la fin

    $sql = substr($sql, 0, strlen($sql) - 2);

    $sql .= ' WHERE `' . $clef . '` = \'' . $val[$clef] . '\';';

    // On l'execute

    $this->exec($sql, 'Impossible de mettre à jour la table.');

}

C'est pas spécialement joli, ni intuitif, pourquoi ne pas faire quelque chose dans le genre ? Qui permet de toujours bien visualiser sa requête ?

Code PHP :
<?php 

// update("select name where user = '%s' and id = %u and visible = %u", "test", 5, 1);

public function update($requete, $arg1, $arg2)

{

    $args = func_get_args();

    $tmp = call_user_func_array("sprintf", array_merge($args))

    return $this->exec($tmp, 'Impossible de mettre à jour la table.');

}

C'est aussi applicable à query ^^ Qui en plus n'a pas à s'appeler query, puisque qu'elle fait la requête et le résultat sous forme de tableau Smile

C'est plus un queryFetchAssoc dans ton cas.
Et il me semble avoir raté la méthode exec ?

Voilà ce n'est que mon avis, j'avais vu quelques autres trucs mais j'ai oublié Smile

JE te les redonnerais si je retrouve.

Anthor | http://www.webgame-achievements.com/

Elite Dangerous - Trade Database | Elite Dangerous Star Map

« Sujet précédent | Sujet suivant »

Utilisateur(s) parcourant ce sujet : 4 visiteur(s)

Messages dans ce sujet

Petit ensemble de classes - par Ekilio - 28-10-2008, 08:55 AM

JeuWeb Créez votre jeu par navigateur !