Discussion :

[figuedi]

Bonjour,
J'ai créé un fichier Mes_Classes.py, dans lequel j'ai mis mes Classes

quand je veux les utiliser dans un script
je peux faire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

from Mes_Classes import Acces_Base, Personnage

Et après je peux les utiliser comme ça

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
Acces_Base.Add_Perso(perso)
perso = Personnage()

Si je fais l'import comme ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

import Mes_Classes as MC

je dois les utiliser comme ceci

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
perso = MC.Personnage()
MC.Acces_Base.Add_Perso(perso)

Du coup c'est vachement plus long à écrire quand je veux utiliser une fonction qui est dans ma classe.
Il n'y aurait pas un moyen de définir l'import, pour que je puisse utiliser la fonction Add_Perso directement sans lui dire que Add_Perso est une fonction de la classe Acces_Base, ou en tout cas que je le lui dise une fois pour toute au début et qu'il se débrouille avec pour la suite?

[wiztricks]

Du coup c'est vachement plus long à écrire quand je veux utiliser une fonction qui est dans ma classe.
Il n'y aurait pas un moyen de définir l'import, pour que je puisse utiliser la fonction Add_Perso directement sans lui dire que Add_Perso est une fonction de la classe Acces_Base, ou en tout cas que je le lui dise une fois pour toute au début et qu'il se débrouille avec pour la suite?

On crée des "class" parce qu'on va fabriquer des objets similaires qui s'appellent "instances". Et ils seront similaires parce qu'ils auront les mêmes attributs/méthodes.
Pour fabriquer une collections de fonctions, un module suffit (pas besoin de créer une "class" dans le "module" puisqu'il n'y a pas d'instances).
Et avec cette histoire d'instances, on écrira plutôt:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
ab = MC.Acces_Base()
perso = MC.Personnage()
ab.Add_Perso(perso)

ce qui est beaucoup plus court.

Il y a d'autres techniques mais il va falloir prendre le temps d'apprendre les bases avant pour que ce soit intéressant.

- W

[figuedi]

oui pour les classes
Dans la classe mère Personnage, il y a les enfants, guerriers, magicien, Clerc, ....
J'avais dans l'idée de faire la classe Acces_Base
ou il se connecte à la BDD et ouvre le curseur et le connecteur
puis des fonctions qui comme ça ne m'oblige pas à appeler la connexion à la base ni l'ouverture du connecteur et du curseur lors de l'appel de chaque fonction

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
class Acces_Base():
    global cursor
    global cnx
    from mysql.connector import (connection)
    cnx = connection.MySQLConnection(user='root', password='',
                                     host='127.0.0.1',
                                     database='mysql')
    cursor = cnx.cursor()
    def Lire_Perso(nom):
        sql_query = pd.read_sql('SELECT * FROM jeu WHERE Nom="%s"' % nom, cnx)
        print(sql_query)
 
    def Delete_Perso(nom):
        SQL_Querry = ("DELETE FROM jeu where Nom = %s")
        cursor.execute(SQL_Querry, (nom,))
        cnx.commit()

L'inconvénient majeur dont je viens de me rendre compte à l'instant c'est que je ne ferme pas le connecteur ni le curseur, ça n'a pas l'air de le déranger outre mesure mais bon ...

Si je n'utilise pas la classe je dois rajouter l'appel à ma fonction Acces_Base en effet l'écriture sera plus rapide et surement plus correcte vis à vis des bonnes pratiques

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
    def Lire_Perso(nom):
        Acces_Base()
        sql_query = pd.read_sql('SELECT * FROM jeu WHERE Nom="%s"' % nom, cnx)
        print(sql_query)
        cursor.close()
        cnx.close()

Pour une fois j'ai peut etre compris du premier coup tes explications

[wiztricks]

ou il se connecte à la BDD et ouvre le curseur et le connecteur
puis des fonctions qui comme ça ne m'oblige pas à appeler la connexion à la base ni l'ouverture du connecteur et du curseur lors de l'appel de chaque fonction

Trouver "global" dans une "class" est que vous n'avez pas assez potassé les "class" avant de les utiliser.
Rien n'empêche de créer un objet connection à la BDD qui initialise ce qu'on veut pour que les méthodes en profitent.

Mais relisez votre fonction Lire_Perso: elle se contente d'afficher (via print) sans rien retourner à l'appelant. Aucun intérêt!


- W

[Sve@r]

Bonjour

Dans la classe mère Personnage, il y a les enfants, guerriers, magicien, Clerc, ....

Pourquoi pas une classe "personnage" qui contient le type (guerrier, magicien, clerc) ? Parce que dans les jeux de rôle, guerrier et clerc c'est la même chose (le premier frappe fort mais a un QI d'huitre tandis que le second a mémorisé les 40 grimoires de la magie mais peine à soulever son propre corps)...

J'avais dans l'idée de faire la classe Acces_Base
ou il se connecte à la BDD et ouvre le curseur et le connecteur
puis des fonctions qui comme ça ne m'oblige pas à appeler la connexion à la base ni l'ouverture du connecteur et du curseur lors de l'appel de chaque fonction

Oui c'est l'idée de base de MVC. Mais pourquoi en rester juste au curseur? Tu crées une fonction style "sauvegarde_bdd" qui se charge de se connecter, ouvrir le curseur, faire la sauvegarde puis fermer le curseur et fermer la bdd.

L'inconvénient majeur dont je viens de me rendre compte à l'instant c'est que je ne ferme pas le connecteur ni le curseur, ça n'a pas l'air de le déranger outre mesure mais bon ...

Ouaip. Tu peux aussi quitter ta maison sans fermer la porte, ça ne va pas te déranger outre mesure jusqu'au jour où...

On ferme ses ressources ouvertes, c'est une des bases de la programmation "propre" et ça ne se discute pas. Ceci dit, créer une classe pour l'accès à la bdd je comprends, créer une classe pour le curseur je comprends beaucoup moins. C'est pas la mort de placer le curseur dans les fonctions qui en ont besoin.
Donc au début du programme tu ouvres la bdd, à la fin du programme tu la fermes et les curseurs tu les crées quand tu en as besoin. Et là tout est ok.