CLASSE HELPER POSTGRESQL

Commentaire de jsiAngers le 24/11/2006 21:26:54

Je crois qu'il faut tu revois tes classiques... il faut ajouter une référence vers cette librairie libre qu'est Npgsql, si tu souhaites utiliser cette classe avec une base de données PostGresql... donc de nouveau, Cher OneHacker, pas d'erreur...

Commentaire de Benouille le 06/03/2007 20:43:27

le code est ultra propre et ça fait plaisir.
même si j'ai toujours du mal avec les "If <boolean> = True Then" (pourquoi pas If <boolean> = True = True = True And True Or False ... Then" )

ou pire encore:
        Cn.Open()
        If Cn.State = ConnectionState.Closed Then
            ConnectDataBase = False
        ElseIf Cn.State = ConnectionState.Open Then
            ConnectDataBase = True
            Cn.Close()
        End If

soit quelquechose m'échappe soit c'est du code inutile.
Si le .open plante de toutes façons la suite ne sera pas lue (pas de try catch géré dans la fonction appelante donc c'est le catch de la fonction open qui lui même ne fait rien qui est sensé gérer l'erreur)

le If pue du cul aussi, pourquoi pas écrire simplement:
        Return Cn.State = ConnectionState.Open

d'ailleurs une meilleure gestion du try catch sur une ouverture de connection se fait apres le .open pour assurer sa fermeture dans le finally en cas de plantage dans la requête.

Je n'ai rien vu (pardon si ça m'a échapé) concernant les proc stockées et pourtant il me semble que postgres les gere (comme des fonctions utilisateurs même si je dis pas de bétises mais à confirmer).
A aucun moment on ne passe de paramètres permettant d'utiliser la base autrement qu'en lui refourgant une chaine sql.

chaine sql filtrée d'ailleurs,
sous couvert de sécurité on ne peut plus utiliser exec ou %, ni de ponctuation, ouais ok mais bon on peut avoir des choses a stocker qui parlent la france avec des bouts de mots non tronqués.


c'est donc tres beau mais je trouve le niveau un peu léger, beaucoup de code inutile et le tout peu exploitable sur une application plus ouverte.
Je pense que tu devras la modifier le jour ou tu auras besoin de t'en servir.


j'ai mis 7/10 pour la clarté du code