Discussion :

[AgriPacTe]

Bonjour,

La "mécanique" de mon édition de plusieurs tables est toute entière contenue de la ligne 64 à la ligne 86 (sauf outData pour récupérer les données éditées dans self.EnCoursEdition; mais je ne récupère pas ce que je voudrais.

Je pense que c'est la dernière ligne 86 self.EnCoursEdition=tableDatas qui ne fonctionne pas comme je voudrais, c'est à dire avoir dans EnCoursEdition l'objet tableDatas pour pouvoir le modifier avec les nouvelles données éditées.
(En C, j'utiliserai un pointer, et j'ai l'impression que ce n'est pas lui que j'ai dans EnCoursEdition)

Je n'ai pas réussi à mettre cela en évidence avec les deux print qui suivent en commentaire
Edition: Je viens de découvrir id() ; les lignes 87 à 90 deviennent:

Code :

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

1
2
3
4
        print(type(tableDatas), id(tableDatas))
        # donne : <class 'list'> 140616917357064
        print(type(self.EnCoursEdition), id(self.EnCoursEdition))
        # donne : <class 'list'> 140616917357064

Le code:

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
# Classes de base du UI de AptBureau
# Début: 23 nov 2019
 
from PySide2 import Qt, QtCore, QtWidgets, QtGui
import operator
 
import mainwindow
import Archive
import BandesPourAptQGis
import CodeDate
 
class AptBureauWin(QtWidgets.QMainWindow, mainwindow.Ui_MainWindow):
    def __new__(self):
        if not hasattr(self, 'instance'):
            self.instance = super(AptBureauWin, self).__new__(self)
            self.instanceinit=0
        else: self.instanceinit=1 # ligne inutile (à tester)
        return self.instance
 
    def __init__(self, parent=None):
        if not self.instanceinit==0: return
        super(AptBureauWin, self).__init__(parent)
        self.instanceinit=1
        self.setupUi(self)
        self.setWindowIcon(QtGui.QIcon('AptVerger.svg'))
 
        # Connections des Menus
        self.actionEnregistrer.triggered.connect(Archive.ArchiveDonnees)
        self.actionOpen.triggered.connect(Archive.ChargeDonnees)
        #self.actionContr_le_des_bandes.triggered.connect(Archive.CohérenceBandes)
        self.action_Export_GeoJason_des_bandes.triggered.connect(BandesPourAptQGis.ArchiveShapeFileGeoJason)
 
        # Edition des Tables
        self.action_Bandes_2.triggered.connect(self.EditeBandes)
        self.actionCultures_2.triggered.connect(self.EditeCultures)
        self.action_Fa_ons_2.triggered.connect(self.EditeFaçonsK)
        self.actionActions_2.triggered.connect(self.EditeActions)
        self.action_Productions_2.triggered.connect(self.EditedocProd)
 
        # Import .CSV
        self.actionOpen.triggered.connect(Archive.LireCSV)
 
        # Boutons de la barre de status
        insertBtn=QtWidgets.QPushButton('A')
        insertBtn.setObjectName('insertBtn')
        insertBtn.clicked.connect(self.InsereLigne)
        removeBtn=QtWidgets.QPushButton('R')
        removeBtn.setObjectName('removeBtn')
        removeBtn.clicked.connect(self.SupprimeLigne)
        self.statusbar.addPermanentWidget(insertBtn)
        self.statusbar.addPermanentWidget(removeBtn)
        #self.button.setIcon(QtGui.QIcon('myImage.jpg'))
        #self.button.setIconSize(QtCore.QSize(24,24))
 
 
        header = self.tableView.horizontalHeader()
        header.setSectionResizeMode(QtWidgets.QHeaderView.ResizeToContents)       
        header.setStretchLastSection(True)
        self.tableView.setSortingEnabled(True)
        # futur : pas de sort sur dernière colonne
        # ? : pourquoi un sort sur la première colonne (en ordre descendant) s'execute au démarrage ?
        # sans cliquer sur l'entête de colonne 
 
        Archive.ChargeDonnees
        self.flagsDataChanged = False #[False, True, False, False, False]
        self.EnCoursEdition= None
        self.modelTTEdition= None
        self.idDatas = 4 #{'Bandes':0,'Cultures':1,'FaçonsK':2, 'docProd':3, 'Actions':4}
        self.EditeTouteTable(Archive.Actions)
 
    def EditeBandes(self):  self.idDatas = 0; self.EditeTouteTable(Archive.Bandes)
    def EditeCultures(self):self.idDatas = 1; self.EditeTouteTable(Archive.Cultures)
    def EditeFaçonsK(self): self.idDatas = 2; self.EditeTouteTable(Archive.FaçonsK)
    def EditedocProd(self): self.idDatas = 3; self.EditeTouteTable(Archive.docProd)
    def EditeActions(self): self.idDatas = 4; self.EditeTouteTable(Archive.Actions)
 
    def EditeTouteTable(self, tableDatas):
        if self.flagsDataChanged: # Stockage des données modifiées
            self.modelTTEdition.outData() # rempli EnCoursEdition avec les nouvelles lignes éditées
            print(tableDatas)           # contient le nouveau à éditer
            print(self.EnCoursEdition)  # correct, mais tableDatas ancien perdu!!!
            Archive.ArchiveDonnees()    # d'où pas d'archivage du EnCoursEdition
            self.flagsDataChanged = False
        self.modelTTEdition= TouteTableModel(tableDatas, self)
        self.tableView.setModel(self.modelTTEdition)
        self.EnCoursEdition=tableDatas
        #print(type(tableDatas), tableDatas.__instance__)
        # donne : # comment avoir l'instance comme : <__main__.Citron object at 0x7ff2193a20f0>
        #print(type(self.EnCoursEdition), self.EnCoursEdition.__instance__)
        # donne : 
 
    def InsereLigne(self):
        #print('Insere ligne')
        self.modelTTEdition.addRow()
 
    def SupprimeLigne(self):
        #print('Supprime ligne')
        indexes=self.tableView.selectionModel().selectedIndexes()
        for index in indexes:
            if not index.isValid(): continue
            self.modelTTEdition.removeRow(index.row())
 
 
class TouteTableModel(QtCore.QAbstractTableModel):
    def __init__(self, datain, parent=None):
        QtCore.QAbstractTableModel.__init__(self, parent)
        self.headers = datain[0]
        self.arraydata = datain[1:]
        self.types = tuple(type(item).__name__ for item in self.arraydata[0])
        # donne (pour Bandes) : ('str', 'float', 'int', 'int', 'int', 'str', 'str')
        self.UIParent= parent
 
    def outData(self):
        self.UIParent.EnCoursEdition= self.headers+self.arraydata
 
    def rowCount(self, parent=None): return len(self.arraydata)
 
    def columnCount(self, parent=None): return len(self.headers)
 
    def data(self, index, role):
        if not index.isValid(): return QVariant()
        elif role == QtCore.Qt.EditRole: return None
        elif role != QtCore.Qt.DisplayRole: return None
        for i, ligne  in enumerate(self.arraydata):
            if i == index.row(): return ligne[index.column()]
 
    def headerData(self, section, orientation, role):
        if role != QtCore.Qt.DisplayRole: return None
        if orientation == QtCore.Qt.Horizontal: return self.headers[section]
        else: return "{}".format(section)
 
    def sort(self, col, order): #trier la table sur la colonne col selon order
        self.layoutAboutToBeChanged.emit()
        self.arraydata = sorted(self.arraydata,key=operator.itemgetter(col),
            reverse=(order == QtCore.Qt.SortOrder.DescendingOrder))
        self.layoutChanged.emit()
 
    def addRow(self):
        idD= self.UIParent.idDatas
        NouvelleLigne= Archive.Modèles[idD]
        if idD==4: NouvelleLigne[0]=CodeDate.EncodeDate()
        print(NouvelleLigne)
        self.layoutAboutToBeChanged.emit()
        self.arraydata.append(NouvelleLigne)
        self.layoutChanged.emit()
 
    def removeRow(self, row):
        if 0 <= row < self.rowCount():
            self.layoutAboutToBeChanged.emit()
            del self.arraydata[row]
            self.layoutChanged.emit()
 
    def setData(self, index, value, role = QtCore.Qt.EditRole):
        if index.isValid():
            if role == QtCore.Qt.EditRole:
                row = index.row()
                col = index.column()
                if self.types[col] in ('int', 'float'): #['int', 'float']:
                    self.arraydata[row][col] = float(value.replace(",","."))
                elif self.types[col] == 'str': self.arraydata[row][col] = str(value)
                else: print('type incompatible') # futur : liste de coordonnées pour longueur
                self.UIParent.flagsDataChanged= True
                return True
            else: return False
        else: return False
 
    def flags(self, index):
        return QtCore.Qt.ItemIsSelectable|QtCore.Qt.ItemIsEditable|QtCore.Qt.ItemIsEnabled
 
"""
        indexes=tableView.selectionModel().selectedIndexes()
 
        for index in indexes:
            if not index.isValid(): continue
            if button.objectName()=='removeButton':
                tableView.model().removeRows(index.row(), 1, QModelIndex())
 
            elif button.objectName()=='insertButton':
                tableView.model().insertRows(index.row(), 1, QModelIndex())
 
class MyDelegate(QtWidgets.QItemDelegate):
    def __init__(self, parent = None):
        super(MyDelegate, self).__init__(parent)
        self.view = parent
 
    def eventFilter(self, editor, event):
        # there is a lot of checking in order to identify the desired situation and avoid errors
        if isinstance(event, QtGui.QKeyEvent):
            if event.type() == QtCore.QEvent.KeyPress:
                if event.key() == QtCore.Qt.Key_Escape:
                    # we should have a list here of length one (due to selection restrictions on the view)
                    index = self.view.selectedIndexes()
                    if index:
                        if index[0].isValid():
                            row = index[0].row()
                            self.view.model().removeRow(row)
"""

[Sve@r]

Bonjour

Je pense que c'est la dernière ligne 86 "self.EnCoursEdition=tableDatas" qui ne fonctionne pas comme je voudrais, c'est à dire avoir dans EnCoursEdition l'objet tableDatas pour pouvoir le modifier avec les nouvelles données éditées.
(En C, j'utiliserai un pointer, et j'ai l'impression que ce n'est pas lui que j'ai dans "EnCoursEdition")

C'est à peu près ça (bon j'ai pas examiné ton code mais je me doute de ce qui doit se passer).
Python ne copie pas les choses, il les fait juste pointer au même endroit. Exemple a=567; b=a si tu regardes id(a) et id(b) tu auras les mêmes résultats.

Alors autant pour les int (et float et autres trucs "simples" qu'on appelle "immutables") ça ne pose pas de souci. Tu peux ensuite changer "b" sans que "a" s'en trouve affecté.
Mais pour les trucs "complexes" (listes, dicos et objets), qu'on appelle "mutables", là ça pose souci car changer le contenu de "a" ou "b" se produira dans la même et unique zone. Donc (exemple) a=[1, 2, 3]; b=a; a.append(4) et tu affiches "b" et tu vois [1, 2, 3, 4].

Cela ne t'interdit pas d'écrire b=a bien sûr, mais faut juste que tu saches ce que ça entraine. C'est effectivement un truc basé sur l'utilisation des pointeurs (plus facile de copier une adresse qu'un contenu).

Il y a toutefois des solutions. Si vraiment tu veux créer "b" à partir de "a" mais avec sa vie propre, tu peux utiliser le slice (b=a[:]) ou une liste créée à chaud (b=list(a)).

Je pense que c'est un truc analogue qui se passe en ligne 86.

[AgriPacTe]

Oui; merci; je crois que j'ai compris; en fait je veux avoir une référence; or en python "tout est référence"
Seulement dans outData avec le code

Code :

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

self.UIParent.EnCoursEdition= self.headers+self.arraydata

je change la référence qu'est self.UIParent.EnCoursEdition pour un autre objet self.headers+self.arraydata.

C'est ce que je voudrais ne pas faire; je voudrais que ce soit toujours le même objet (en remontant on voit que c'est Archive.Actions) qui contienne self.headers+self.arraydata.