Discussion :

[Fred973]

Bonjour à tous,

J'ai un programme de gestion de base de donnée avec une bdd SQLite3 qui comporte 3 tables. je peux les afficher chacune via un menubar dans un treeview. Quand je sélectionne une ligne dans le treeview et que je clic sur le bouton "delete" elle est supprimée de la bdd.
Je n'ai pas de problème tous fonctionne parfaitement. Sauf que j'aimerai savoir si quelqu'un pourrait arranger mon code que je trouve un peu redondant sur les lignes 5, 6 et 7 ........

Code :

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

1
2
3
4
5
6
7
8
9
10
def delete_button():
    idSelect = tree.item(tree.selection())["values"][0]
    conn = sqlite3.connect('data/database.db')
    cursor = conn.cursor()
    cursor.execute("DELETE FROM parts_inventory WHERE id = {}".format(idSelect))
    cursor.execute("DELETE FROM tools_inventory WHERE id = {}".format(idSelect))
    cursor.execute("DELETE FROM consumables_inventory WHERE id = {}".format(idSelect))
    conn.commit()
    tree.delete(tree.selection())
    conn.close()

merci d'avance pour votre aide.

[flapili]

bonjour,

il faut éviter d'utiliser format ou de concaténer les requêtes toi même, ça évitera les injections sql.

Il serait plus approprié d'utiliser des relations et de faire un delete on cascade, pour vulgarisé c'est comme si vous aviez un sac avec un numéro dessus, et des patates avec une étiquette indiquant le numéro de sac, quand on supprimé un sac ça supprime les patates associées ... bon bien sur cette analogie est limitée puisque par ce que un sac avec des patates "virtuellement" dedans via des étiquettes mais pas physiquement c'est totalement con, mais en SQL ça ne l'ai pas, ça se traduit par des relation avec des foreign key.

[Fred973]

Merci beaucoup pour la réponse je vais aller voir ça de plus près et reviendrait poster ce qui fonctionne avant de mettre le sujet "résolu". Par contre si tu as un exemple suis preneur .....

[flapili]

pour éviter les injections SQL: https://docs.python.org/3.8/library/sqlite3.html (4ème codeblock, pas d'ancre ....)



pour ce qui est de la structure de la base de donnée il faudrait savoir la structure actuelle de ta base, les relations entres les tables

[Fred973]

pour ce qui est de la structure de la base de donnée il faudrait savoir la structure actuelle de ta base, les relations entres les tables

elles n'ont aucune liaison je me sert d'une GUI pour les gérer une après l'autre jamais en même temps

[wiztricks]

Salut,

elles n'ont aucune liaison je me sert d'une GUI pour les gérer une après l'autre jamais en même temps

Certes mais balayer toutes les tables pour y supprimer les lignes qui ont un identifiant donné (lorsqu'il en existe)... fait partie de "gérer" et montre un "en même temps"...

Quelque part, vous avez un soucis de conception... mais vous pouvez très bien vivre avec sans que votre vie soit en péril.

- W

[Fred973]

oui je comprends en effet les lignes avec le même id seront supprimées de toutes les tables ....... il va falloir y remédier merci beaucoup pour votre aide. Je vais séparer par des fonctions différentes pour gérer chaque tables uniquement.

[flapili]

ce n'est pas la question, il sera plus judicieux de nous donner le contexte, voir même une capture d'écran de votre programme ça pourrait nous aider à comprendre.