Erreur avec l'importation de module

Bonjour,

je m'adresse à vous aujourd'hui car j'ai un problème que je n'arrive pas a résoudre et que je ne comprend pas.
Je travaille sur une appli dans le cadre de mon boulot pour gérer mon parc d'ordinateurs (c'est perso pour tuer le temps et apprendre du python :P).
l'appli fonctionnait correctement mais celle-ci commençant à être dense j'ai voulu la découper en plusieurs fichier "py" pour faciliter la navigation et le codage entre mes différentes fenêtres.
sauf que depuis j'ai une erreur:

Citation:

---

<< ImportError: cannot import name 'RootWindow' from partially initialized module 'mainwindow' (most likely due to a circular import) (C:\Users\Jean-Philippe\PycharmProjects\PARC_2\mainwindow.py) >>

---

Voici une partie de mon code:

Fichier execution :

Code:

---

1 2 3 4

from mainwindow import RootWindow   global winpcpal winpcpal = RootWindow()

---

Fichier mainwindow :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34

from tkinter import * from settingutil import SettingUtil from settingparc import SettingParc     class RootWindow:   def __init__(self):   # Creation de la fenetre principale du logo et du titre self.master = Tk() # Creation de la frame principale et des sous-frames self.frame = Frame(self.master, width=1540, height=1360) self.frame.pack() self.lframe = Frame(self.frame, width=770, height=580) self.lframe.pack(side="left")   # Bouton self.btn_user_settings = Button(self.lframe, command=self.sh_gestion_util) self.btn_user_settings.pack()   self.flag = 0 self.master.mainloop()   def sh_gestion_util(self): global winsetutil if self.flag == 0: self.master.state('iconic') winsetutil = SettingUtil() self.flag = 1 #self.settings.swin.mainloop() else: self.settings.swin.destroy() self.flag = 0

---

Fichier settingutil:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

from tkinter import * from execution import winpcpal   # Classe pour fenetre de gestion des utilisateurs class SettingUtil:   def __init__(self): self.swin = Toplevel()   # Creation de la frame principale self.frame = Frame(self.swin, width=1500, height=1500) self.frame.pack(padx=5, pady=5)   self.swin.protocol("WM_DELETE_WINDOW", self.on_close)     def on_close(self): self.swin.destroy() self.settings = winpcpal self.settings.deiconify()

---

Voila, j'ai épuré un max pur ne mettre que l'utile le reste étant d'autres boutons , labels, etc ..
et cela me et l'erreur dès que je tente d'importer ma variable globale "winpcpal".

je comprend le principe de boucle de l'erreur mais je n'arrive pas à voir comment cela se produit dans mon code et encore moins comment le résoudre.
(je ne suis qu'un débutant qui s'est trouvé un projet pour occuper son temps et apprendre ^^).
Je suis disponible si des éclaircissement sont demandé :D.

merci d'avance a vous et bonne journée !

Salut,

Pour moi le problème vient de from execution import winpcpal dans le fichier settingutil.py.

On ne voit pas trop bien l'utilité de cette référence globale, mais, de toute façon, efface toutes les définitions de global de ton code, les global n'ont rien à faire dans de la programmation objet.

Bonjour

Citation:

---

Envoyé par jeanfly

sauf que depuis j'ai une erreur

<< ImportError: cannot import name 'RootWindow' from partially initialized module 'mainwindow' (most likely due to a circular import) (C:\Users\Jean-Philippe\PycharmProjects\PARC_2\mainwindow.py) >>

---

Import circulaire. Tu as "execution" qui importe "mainwindow" , "mainwindow" qui importe "settingutil" et "settingutil" qui importe "execution".

Citation:

---

Envoyé par jeanfly

Voici une partie de mon code

---

:tagcode:

Citation:

---

Envoyé par jeanfly

et encore moins comment le résoudre.

---

Tu crées un programme maitre qui importera X, Y et Z. Ensuite X pourra importer Y ou Z s'il en a besoin et etc mais aucun des trois ne peut importer le maitre.
Exemple
X.py

Code:

---

1 2 3 4 5 6

#!/usr/bin/env python3 # coding: utf-8   def X(): print("X") import Y Y.Y()

---

Y.py

Code:

---

1 2 3 4 5 6

#!/usr/bin/env python3 # coding: utf-8   def Y(): print("Y") import X X.X()

---

principal.py

Code:

---

1 2 3 4 5 6 7

#!/usr/bin/env python3 # coding: utf-8   import X import Y X.X() Y.Y()

---

Pour le reste, le mot clef "global" ne s'utilise pas comme tu le fais et surtout on n'utilise pas de variable globale sans en avoir une absolue nécessité ce qui n'est clairement pas le cas ici.

Citation:

---

Envoyé par VinsS

Salut,

Pour moi le problème vient de from execution import winpcpal dans le fichier settingutil.py.

On ne voit pas trop bien l'utilité de cette référence globale, mais, de toute façon, efface toutes les définitions de global de ton code, les global n'ont rien à faire dans de la programmation objet.

---

Merci pour ce commentaire, cela ne m'aide pas trop, j'avais déjà vu que l'utilisation de var globale n'est pas top mais avec peu de connaissance je fais "au mieux" et juste me dire que ca ne sert à rien ne m'aidera pas beaucoup.

Citation:

---

Envoyé par Sve@r

Bonjour

Import circulaire. Tu as "execution" qui importe "mainwindow" , "mainwindow" qui importe "settingutil" et "settingutil" qui importe "execution".


:tagcode:


Tu crées un programme maitre qui importera X, Y et Z. Ensuite X pourra importer Y ou Z s'il en a besoin et etc mais aucun des trois ne peut importer le maitre.

---

Merci pour cela !!

J'ai essayé de m'en aider pour mon code mais je ne comprend pas trop la hiérarchisation dans ton exemple :

Mon fichier "execution" permet de lancer ma fenêtre maitre (mainwindow)

Depuis cette dernière je peux obtenir une nouvelle fenêtre (settingutil)

j'ai essayé de créer une méthode me retournant le nom de la fenêtre "mainwindow" afin que lorsque je ferme la fenêtre "settingutil" la fentre "mainwindow" s'affiche mais cela ne fonctionne pas.

Dans settingutil.py

Code:

---

1 2 3 4

def get_wpcpal(self): import execution return execution.winpcpal

---

je pensais pouvoir récupérer ensuite la variable dans ma méthode "on_close " du même fichier mais sans succès pou l'instant, je vais continuer a chercher car je pense bien que ma méthode n'est pas bonne mai cela va m'aider a avancer.

J'aimerai surtout comprendre l'ordre dans lequel ton exemple fonctionne.

Citation:

---

Envoyé par jeanfly

J'ai essayé de m'en aider pour mon code mais je ne comprend pas trop la hiérarchisation dans ton exemple :

---

Il y a un maitre "principal" et 2 modules "X" et "Y". Les modules peuvent s'importer (presque) à loisir mais surtout ne peuvent pas importer "principal".
Et c'est "principal" qui fait le lien et qui est exécuté.

Pour ton cas, c'est à peu près pareil. Puisque c'est "execution" qui fait le lien, alors aucun module ne peut l'importer. Tu ne dois pas écrire import execution quelles que soient les circonstances. Et comme tu ne l'importes que pour avoir accès à la variable globale winpcpal, on retombe sur la remarque de VinsS qui, contrairement à ce que tu crois, t'aide beaucoup puisque ça te dit ce qu'il ne faut pas faire donc à toi ensuite de trouver le moyen de ne pas l'importer.
Bon je ne connais pas tk (mon IHM de prédilection étant PyQt) mais d'après ce que je vois, winpcpal contient la RootWindow. Cette RootWindow crée ensuite une SettingUtil et cette SettingUtil utilise ensuite winpcpal (ie la RootWindow) pour lui faire un truc.
Donc de là, pourquoi, quand tu crées ton SettingUtil tu ne lui passes pas la RootWindow à ce moment là (qui se trouve donc être "self" puisque cela se passe dans ledit RootWindow)?
Ensuite ton SettingUtil recevant ce "RootWindow" en paramètre, le stocke dans un de ses attributs ce qui lui permettra de l'utiliser quand il en aura besoin (surtout qu'en plus il le stockait déjà dans ses attributs via self.settings = winpcpal !!!)

Grosso-modo

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31

class RootWindow: def sh_gestion_util(self): if self.flag == 0: self.master.state('iconic') winsetutil = SettingUtil(self) self.flag = 1 #self.settings.swin.mainloop() else: self.settings.swin.destroy() self.flag = 0 # if # sh_gestion_util() # class RootWindow   class SettingUtil: def __init__(self, rootWindow): self.__rootWindow=rootWindow self.swin = Toplevel()   # Creation de la frame principale self.frame = Frame(self.swin, width=1500, height=1500) self.frame.pack(padx=5, pady=5)   self.swin.protocol("WM_DELETE_WINDOW", self.on_close) # __init__()   def on_close(self): self.swin.destroy() self.__rootWindow.deiconify() # on_close # class SettingUtil

---

C'est une première façon.

Une seconde (c'est celle que j'utilise en Qt) c'est de créer toutes les fenêtres en invisible dès le départ, puis ensuite selon les circonstances j'affiche telle ou telle fenêtre et masque telle ou telle autre. Ca évite de faire "création/suppression/création/suppression/création/suppression" à répétition. Et donc puisque les fenêtres sont crées dès le départ, elles se connaissent toutes (en fait je les range dans un dico et le dico est accessible à chacune) et peuvent alors s'interfacer.

Et dans tous les cas, winpcpal disparait (enfin reste la variable qui reçoit le RootWindow mais n'est pas nécessaire d'être connue/importée ailleurs).

Cool merci je suis arrivé a me dépatouiler et cela donne ca ^^

Code:

---

1 2 3 4 5 6 7

class RootWindow: def sh_gestion_util(self): if self.flag == 0: self.master.state('iconic') self.flag = 1 self.winsetutil = SettingUtil(self)

---

Code:

---

1 2 3 4 5 6 7 8 9 10

class SettingUtil:   def __init__(self, rootWindow): # recuperation de l'instance de RootWindow self.__rootWindow = rootWindow   def on_close(self): self.swin.destroy() self.__rootWindow.master.deiconify()

---

Merci beaucoup pour ton aide !!! :?:?

Histoire d'abuser un peu j'ai une dernière question concernant ce passage de paramètre :mrgreen:
Grace a toi j'ai pu passer une instance d'un module à une autre mais je me demandais s'il était possible de le faire d'un sens puis de l'autre.
comme par exemple pour passer le paramètre "flag" de "1" à "0" sur le "on_close" du "settingutil" puis le renvoyer sur le module "rootwindow".

Je ne demande pas forcément la solution toute cuite mais à minima un aiguillage pour mes recherches futures ^^. Je pense que cela me servira pour lus tard ^^

Citation:

---

Envoyé par jeanfly

comme par exemple pour passer le paramètre "flag" de "1" à "0" sur le "on_close" du "settingutil" puis le renvoyer sur le module "rootwindow".

---

Bah oui, puisque ton SettingUI connait le RootWindow, il suffit de créer dans ledit RootWindow une méthode pour positionner son flag.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

class RootWindow: def setFlag(self, f): self.flag=f # class RootWindow   class SettingUtil: def __init__(self, rootWindow): self.__rootWindow=rootWindow self.swin = Toplevel()   # Creation de la frame principale self.frame = Frame(self.swin, width=1500, height=1500) self.frame.pack(padx=5, pady=5)   self.swin.protocol("WM_DELETE_WINDOW", self.on_close) # __init__()   def on_close(self): self.swin.destroy() self.__rootWindow.deiconify() self.__rootWindow.setFlag(1) # on_close # class SettingUtil

---

En plus, avec les property, tu peux même cacher une méthode derrière une affectation (en rendant alors l'attribut dédié à cette affectation privé cause conflit de nom)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34

class RootWindow: def __setFlag(self, f): self.__flag=f flag=property(None, __setFlag, None)   def __init__(self): self.__flag=0 # __init__()   def sh_gestion_util(self): if self.__flag: return self.master.state('iconic') self.__flag = 1 self.winsetutil = SettingUtil(self) # sh_gestion_util() # class RootWindow   class SettingUtil: def __init__(self, rootWindow): self.__rootWindow=rootWindow self.swin = Toplevel()   # Creation de la frame principale self.frame = Frame(self.swin, width=1500, height=1500) self.frame.pack(padx=5, pady=5)   self.swin.protocol("WM_DELETE_WINDOW", self.on_close) # __init__()   def on_close(self): self.swin.destroy() self.__rootWindow.deiconify() self.__rootWindow.flag=1 # On croit que c'est une affectation mais en réalité ça appelle "MainWindow.__setFlag" # on_close() # class SettingUtil

---

Citation:

---

Envoyé par jeanfly

mais à minima un aiguillage pour mes recherches futures ^^

---

Tutoriel Python (dans ma signature). Tout ça y est expliqué (chapitre XIV paragraphe 6.b pour le property)

Merci encore pour toutes ces infos ! c'est top je vais pouvoir m'en servir pour améliorer ma compréhension du langage grâce a tout ca !!