Discussion :

[melwin]

Bonjour

première question qui porte sur les bons usages sur l'appel de méthode d'un autre module.
Laquelle doit être utilisée?

Première façon de faire:

Code :

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

1
2
3
from module2 import *
...
methode_module2(arg)

Deuxième façon:

Code :

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

1
2
3
import module2
.....
module2.methode_module2(arg)

Ma deuxième question est par rapport à la gestion du module de connexion à la base de données.
Pour éviter de reprendre plusieurs fois une partie du code j'ai crée une methode:

Code :

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

1
2
3
4
def connexion():
    " Connexion à la base de données BD_GESMAT.db3 "
    conn = sqlite3.connect("BD_GESMAT.db3")
    c = conn.cursor()

Ensuite quand je fais une requête, j'ai un message d'horreur:

Code :

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

1
2
3
4
5
6
def ajt_materiel(itemajt1, itemajt2):
    " Méthode pour ajouter du matériel dans la base de données  "
    connexion()
    c.execute("INSERT INTO MATERIEL (modele, marque) VALUES ('?', '?')", (itemajt1, itemajt2))
    conn.commit()
    conn.close()

c.execute("INSERT INTO MATERIEL (modele, marque) VALUES ('?', '?')", (itemajt1, itemajt2))
NameError: global name 'c' is not defined

comment faire pour gérer cette connexion sans devoir ajouter systématiquement les lignes de connexion à la base avant une requête ?

[melwin]

Pour la deuxième partie je pense avoir compris comment faire:

Code :

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

1
2
3
4
5
6
7
def connexion(requete):
    " Connexion à la base de données BD_GESMAT.db3 "
    conn = sqlite3.connect("BD_GESMAT.db3")
    c = conn.cursor
    c.execute(requete)
    conn.commit()
    conn.close()

Code :

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

1
2
3
def ajt_materiel(itemajt1, itemajt2):
    " Méthode pour ajouter du matériel dans la base de données "
    connexion("INSERT INTO MATERIEL (modele, marque) VALUES ('?', '?')", (itemajt1, itemajt2))

[VinsS]

Salut,

L'importation anonyme n'est pas appréciée pour deux raisons, d'abord tu importes tout le contenu du module et peut-être beaucoup de choses pour rien et ensuite c'est justement cet anonymat qui est gênant.

Lorsque l'on rencontre:

Code :

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

1
2
3
 
...
methode_module2(arg)

[/code]
on ne sait pas d'où sort cette méthode et tu obliges le lecteur de ton code à ouvrir tous les modules importés avec l'* pour savoir ce qu'est cette fonction, ce qu'elle fait, etc.

Pour ta base de données ta première méthode était bonne il suffisait de faire:

Code :

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

1
2
3
4
5
 
def connexion():
    " Connexion à la base de données BD_GESMAT.db3 "
    conn = sqlite3.connect("BD_GESMAT.db3")
    return conn.cursor()

Code :

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

1
2
3
4
5
6
def ajt_materiel(itemajt1, itemajt2):
    " Méthode pour ajouter du matériel dans la base de données  "
    c = connexion()
    c.execute("INSERT INTO MATERIEL (modele, marque) VALUES ('?', '?')", (itemajt1, itemajt2))
    conn.commit()
    conn.close()

[melwin]

merci pour ta réponse.

effectivement je trouve aussi que l'importation anonyme n'est terrible.

Pour la base de donnée, je me doutais qu'il fallait faire un return, mais je ne le faisait pas comme il faut.
Je vais essayer ça.

[melwin]

j'ai un message d'erreur en faisant comme ça:

in ajt_caisse
conn.commit()
NameError: global name 'conn' is not defined

[melwin]

J'avais essayé comme ça

Code :

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

1
2
3
4
5
 connexion():
    " Connexion à la base de données BD_GESMAT.db3 "
    conn = sqlite3.connect("BD_GESMAT.db3")
    c = connexion()
    return c

Code :

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

1
2
3
4
def ajt_materiel(itemajt1, itemajt2):
    " "
    c.execute("INSERT INTO MATERIEL (modele, marque) VALUES (?, ?)",(itemajt1, itemajt2))
    conn.commit()

mais j'ai le même type d'erreur

in ajt_caisse
c.execute("INSERT INTO MATERIEL (modele, marque) VALUES (?, ?)",(itemajt1, itemajt2))
NameError: global name 'c' is not defined

[VinsS]

Code :

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

1
2
3
4
 
def connexion():
    " Connexion à la base de données BD_GESMAT.db3 "
    return sqlite3.connect("BD_GESMAT.db3")

et:

Code :

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

1
2
3
4
5
6
7
8
 
def ajt_materiel(itemajt1, itemajt2):
    " Méthode pour ajouter du matériel dans la base de données  "
    conn = connexion()
    c = conn.cursor()
    c.execute("INSERT INTO MATERIEL (modele, marque) VALUES ('?', '?')", (itemajt1, itemajt2))
    conn.commit()
    conn.close()

[melwin]

Dans ce cas, juste pour économiser une ligne, je vais me passer de cette méthode de connexion(). je vais essayer de voir pour fouiller l'autre façon que j'avais envisagé.

mais marci quand même pour ta réponse et pour ton temps.