Das deutsche Python-Forum

Hallo Ich schreibe momentan ein Script bei dem man zwischn verschiedenen knöpfen auswählen kann. (mann kann nur einen konpf auf einmal auswählen). um das verwirklichen muss ich enen string umwandel in einen befehl. Sorry ich weiss die frage ist schlecht formuliert aber seht euch am besten den Code selber an. Es geht dabe um dei Variable unselectbuttonvalue




# -*- coding: utf-8 -*-

# Form implementation generated from reading ui file 'cape-select-gui.ui'
#
# Created by: PyQt5 UI code generator 5.15.6
#
# WARNING: Any manual changes made to this file will be lost when pyuic5 is
# run again. Do not edit this file unless you know what you are doing.

from PyQt5 import QtCore, QtGui, QtWidgets

def vergleichen():
filew = open('./capevalue.txt','r')

global readcapevalue
readcapevalue=filew.read()



print(readcapevalue)
filew.close()
vergleichen()








class Ui_MainWindow(object):
def setupUi(self, MainWindow):
MainWindow.setObjectName("MainWindow")
MainWindow.resize(631, 482)
self.centralwidget = QtWidgets.QWidget(MainWindow)
self.centralwidget.setObjectName("centralwidget")
self.pushButton = QtWidgets.QPushButton(self.centralwidget)
self.pushButton.setGeometry(QtCore.QRect(9, 99, 91, 141))
self.pushButton.setMinimumSize(QtCore.QSize(0, 141))
self.pushButton.setText("")
icon = QtGui.QIcon()
icon.addPixmap(QtGui.QPixmap("ghg-chape-zickzack-v3-unselected.png"), QtGui.QIcon.Normal, QtGui.QIcon.Off)
self.pushButton.setIcon(icon)
self.pushButton.setIconSize(QtCore.QSize(150, 150))
self.pushButton.setCheckable(False)
self.pushButton.setChecked(False)
self.pushButton.setAutoDefault(False)
self.pushButton.setFlat(True)
self.pushButton.setObjectName("pushButton")
self.pushButton_3 = QtWidgets.QPushButton(self.centralwidget)
self.pushButton_3.setGeometry(QtCore.QRect(257, 99, 91, 141))
self.pushButton_3.setMinimumSize(QtCore.QSize(91, 141))
self.pushButton_3.setText("")
self.pushButton_3.setObjectName("pushButton_3")
self.pushButton_4 = QtWidgets.QPushButton(self.centralwidget)
self.pushButton_4.setGeometry(QtCore.QRect(380, 99, 91, 141))
self.pushButton_4.setMinimumSize(QtCore.QSize(91, 141))
self.pushButton_4.setText("")
self.pushButton_4.setObjectName("pushButton_4")
self.pushButton_5 = QtWidgets.QPushButton(self.centralwidget)
self.pushButton_5.setGeometry(QtCore.QRect(488, 99, 91, 141))
self.pushButton_5.setMinimumSize(QtCore.QSize(0, 141))
self.pushButton_5.setText("")
self.pushButton_5.setObjectName("pushButton_5")
self.pushButton_2 = QtWidgets.QPushButton(self.centralwidget)
self.pushButton_2.setGeometry(QtCore.QRect(133, 99, 91, 141))
self.pushButton_2.setMinimumSize(QtCore.QSize(91, 141))
self.pushButton_2.setText("")
icon1 = QtGui.QIcon()
icon1.addPixmap(QtGui.QPixmap("ghg-chape-zickzack-v4-unselected.png"), QtGui.QIcon.Normal, QtGui.QIcon.Off)
self.pushButton_2.setIcon(icon1)
self.pushButton_2.setIconSize(QtCore.QSize(150, 150))
self.pushButton_2.setFlat(True)
self.pushButton_2.setObjectName("pushButton_2")
self.pushButton_7 = QtWidgets.QPushButton(self.centralwidget)
self.pushButton_7.setGeometry(QtCore.QRect(180, 380, 231, 61))
self.pushButton_7.setObjectName("pushButton_7")
MainWindow.setCentralWidget(self.centralwidget)
self.statusbar = QtWidgets.QStatusBar(MainWindow)
self.statusbar.setObjectName("statusbar")
MainWindow.setStatusBar(self.statusbar)

self.retranslateUi(MainWindow)
QtCore.QMetaObject.connectSlotsByName(MainWindow)
self.pushButton.clicked.connect(self.zickzack_v3_cape_select)
self.pushButton_2.clicked.connect(self.zickzack_v4_cape_select)
self.pushButton_7.clicked.connect(QtWidgets.qApp.quit)
def retranslateUi(self, MainWindow):
_translate = QtCore.QCoreApplication.translate
MainWindow.setWindowTitle(_translate("MainWindow", "MainWindow"))
self.pushButton_7.setText(_translate("MainWindow", "Zurück"))



def zickzack_v3_cape_select(self):
self.buttonunselect()
global buttonvalue
buttonvalue= self.pushButton
self.unselect_cape()

global buttonvalueunselected
buttonvalueunselected = self.pushButton_2

print("ghg-zickzav-v3-cape")
filew = open('./capevalue.txt','w')
filew.write("ghg-chape-zickzack-v3")
filew.close()


global selected_cape
selected_cape = str("ghg-chape-zickzack-v3-selected")
self.selcet_cape()
vergleichen()
filew = open('./unselectbuttonvalue.txt','w')
filew.write("self.pushButton")
filew.close()

def unselect_cape(self):

filew = open('./capevalue.txt','r')

global unselectvalue
unselectvalue= str(filew.read())
print(unselectvalue+"gg")



print(unselectvalue)
filew.close()

def buttonunselect(self):

filew = open('./unselectbuttonvalue.txt','r')

global unselectbuttonvalue
unselectbuttonvalue=filew.read()
print(unselectbuttonvalue+"gg")



print(unselectbuttonvalue)
filew.close()



def zickzack_v4_cape_select(self):
self.buttonunselect()
self.unselect_cape()
global buttonvalue
buttonvalue= self.pushButton_2
#muss updatet werden (bei drei button nicht funktionieren) in datei bei drücken namen das konpfesschreiben

global buttonvalueunselected
buttonvalueunselected = self.pushButton

print("ghg-zickzav-v4-cape")
filew = open('./capevalue.txt','w')
filew.write("ghg-chape-zickzack-v4")
filew.close()

global selected_cape
selected_cape = str("ghg-chape-zickzack-v4-selected")
vergleichen()
self.selcet_cape()
filew = open('./unselectbuttonvalue.txt','w')
filew.write("self.pushButton_2")
filew.close()


def selcet_cape(self):
icon_selected_cape = QtGui.QIcon()
icon_selected_cape.addPixmap(QtGui.QPixmap(unselectvalue+"-unselected"+".png"), QtGui.QIcon.Normal, QtGui.QIcon.Off)
print(unselectbuttonvalue)

# farage
unselectbuttonvalue.setIcon(icon_selected_cape)

icon_selected_cape = QtGui.QIcon()
icon_selected_cape.addPixmap(QtGui.QPixmap(selected_cape+".png"), QtGui.QIcon.Normal, QtGui.QIcon.Off)
buttonvalue.setIcon(icon_selected_cape)








def close(self):
print("hi")

if __name__ == "__main__":
import sys
app = QtWidgets.QApplication(sys.argv)
MainWindow = QtWidgets.QMainWindow()
ui = Ui_MainWindow()
ui.setupUi(MainWindow)
MainWindow.show()
sys.exit(app.exec_())

Danke im Voraus für die Hilfe

@tryyan: Als erstes mal hättest Du da in dem generierten Code nicht herumeditieren sollen. Das steht doch auch recht deutlich in dem Kommentar von dem generierten Quelltext als Warnung drin.

Und man generiert eigentlich auch keinen Quelltext mehr aus *.ui-Dateien sondern lädt die GUI mit Hilfe des `PyQt5.uic`-Moduls in das Programm.

Die GUI verwendet absolute Positionen und Grössen — das funktioniert nicht wirklich, sondern nur auf dem Rechner wo das so platziert wurde und Systemen mit ähnlichen Bildschirmgrösse-/auflösung/einstellungen. Verwende Layouts, damit das überall funktioniert.

Dann vergiss bitte das es ``global`` gibt. Man verwendet keine globalen Variablen. Funktionen und Methoden bekommen alles was sie ausser Konstanten benötigen, als Argument(e) übergeben.

Mir ist das mit den globalen Variablen hier deutlich zu wirr als das ich Lust hätte zu versuchen zu verstehen was da passiert oder passieren soll.

Vielleicht beschreibst Du auch mal was Du da erreichen willst, statt in Code zu zeigen wie Du denkst wie man das erreicht. Denn ich habe da den Verdacht, dass Du selbst irgendwas basteln willst, was es bereits fertig gibt. Buttons können ja beispielsweise auch einen ”checked”-Zustand haben, und man kann Buttons zu einer Gruppe (QButtonGroup) zusammenfassen, in der nur *einer* diesen Zustand haben kann. Und darum kümmert sich Qt — das muss man nicht selbst zu Fuss programmieren.

Auf Modulebene sollte nur Code stehen der Konstanten, Funktionen, und Klassen definiert. Das Hauptprogramm steht üblicherweise in einer Funktion die `main()` heisst.

Dateien öffnet man wo es geht mit der ``with``-Anweisung zusammen, damit das schliessen garantiert ist, auch wenn Ausnahmen auftreten oder der Programmfluss aus anderen Gründen nicht am `close()`-Aufruf vorbeikommen sollte.

Beim öffnen von Textdateien sollte man immer explizit die Kodierung mit angeben, statt sich darauf zu verlassen, dass der systemabhängige Defaultwert richtig ist/funktioniert. Das dient zudem der Dokumentation für den Leser, damit der weiss wie die Texte kodiert sein müssen. Wenn man die Kodierung selbst in der Hand hat, bietet sich UTF-8 an, weil das alle Unicode-Zeichen repräsentieren kann.

Der Name `vergleichen()` ist falsch. Die Funktion vergleicht überhaupt nichts, die lädt den Inhalt einer Datei.

Werte sollte man nicht wiederholen. Wenn Du den Dateinamen (oder den Pfad) von "capevalue.txt" mal ändern willst/musst, dann musst Du das gesamte Programm danach absuchen und jede Kopie davon ändern. Für so etwas definiert man *einmal* am Programmanfang, nach den Importen eine Konstante. Vorzugsweise hier auch nicht als Zeichenkette, sondern als `pathlib.Path`-Objekte, weil dann das einlesen und schreiben einfacher wird — dafür haben diese Objekte nämlich eine Methode. Damit wird `vergleichen()` so trivial, dass man da im Grunde keine eigene Funktion mehr für braucht.

Der Code sieht so ein bisschen danach aus als wäre die einzige objektorientierte Komponente der generierte Quelltext. Bei GUI-Programmierung allgemein, und bei Qt im besonderen muss man objektorientierte Programmierung (OOP) drauf haben und anwenden können. Das sollte man eventuell lernen und üben *bevor* man sich in das wirklich grosse Qt-Rahmenwerk stürzt. Das ist sehr wahrscheinlich einfacher das getrennt zu lernen als beides zusammen.

@__blackjack__ Danke für die Antwort. Es sollte ein Fenster für Einstellungen sein darum heisst es nicht Main da es von einem anderen Programm geöffnet werden wird. Das mit dem Pfad werde ich mir zu Herzen nehmen. Mit dem Button werde ich überarbeiten und checked machen. Ich habe nur noch eine Frage wenn ich eine Datei auslese also Test=file.read() dann ist Test ja der Inhalt des Dokumentes z.b. „Inhalt von Test“ aber das ist ja in Anführungszeichen geschrieben wie kann ich machen das der Text vormartiert wird zu Inhalt von Test ohne Anführungszeichen. Herzlichen Dank für die schnelle Antwort und ich werde mit dir Tipps zu Herzen nehmen sorry ich bin erst 14 und ich programmiere erst seit 1 Jahr Python und dass war alles ohne Hilfe des Internets Liebe Grüsse und danke für alles

@tryyan: Wenn Code auf Modulebene der nicht nur Konstanten, Klassen, und Funktionen definiert nicht das Hauptprogramm ist, dann gehört der halt in eine Funktion die anders als `main()` heisst. Der wichtige Punkt ist, dass der Code nichts auf Modulebene zu suchen hat und nicht einfach beim Import ausgeführt werden sollte.

Beim Einlesen mit `file.read()` hat man nur Anführungszeichen wenn die auch in der Datei stehen. Das liest nämlich einfach nur den Inhalt der Datei aus. Hier: keine Anführungszeichen:

@__blackjack__ Okey danke ich probiere gerade im Qt Designer zu machen das wenn der einte Knopf gedrückt wird der andere wechselt also checkable aber ich schaffe das nicht mit welcher Funktion muss ich das machen ? Danke für die schnelle Antwort der Support ist super und das mit dem Import werde ich mir merken also sollte ich die Funktionen in eine speperate Datei schreiben und diese dann importieren in die ui

@tryyan: Wie gesagt `QButtonGroup` ist dafür da um „checkable“ Buttons zu einer Gruppe zusammen zu fassen in der nur einer zur gleichen Zeit im „checked“-Zustand sein kann. Im Designer wählt man dazu alle Buttons aus und legt im Kontextmenü eine neue QButtonGroup an zu der die dann gehören. Das ist kein Widget sondern existiert quasi parallel zu den Widgets. Die Buttons müssen „checkable“ sein, und man sollte sich Gedanken darüber machen ob es den Zustand „keiner ist gecheckt“ geben darf und falls nicht wie man das sicherstellt.

Bei Icons könnte es Sinn machen für „Normal“ je eines für „On“ und „Off“ zu setzen, falls man mit dem vorhandenen Verhalten der Darstellung nicht zufrieden ist. Und wenn da nur Icons angezeigt werden und man überall „flat“ setzt, könnte man auch überlegen stattdessen QToolButton zu verwenden und dort „autoRaise“ zu setzen.

@__blackjack__ entschuldigung die farge war falsch formuliert


# frage
unselectbuttonvalue.setIcon(icon_selected_cape)

icon_selected_cape = QtGui.QIcon()
icon_selected_cape.addPixmap(QtGui.QPixmap(selected_cape + ".png"), QtGui.QIcon.Normal, QtGui.QIcon.Off)
selected_button.setIcon(icon_selected_cape)

es geht dabei um unselectbuttonvalue.
wenn ich in eine datei schreibe self.pushButton
und die dann auslese
with open("test.txt", "r", encoding="utf-8") as file:
global unselectbuttonvalue
unselectbuttonvalue = file.read()
print(file.read())
dann kommt self.pushbutton
aber unselectbuttonvalue gibt einen error zurück, da es eine str in "" ist
jetzt ist die frage wie kann ich das umwandeln das es kein str ist.
so das am schluss es eigentlich so ist

icon_selected_cape = QtGui.QIcon()
icon_selected_cape.addPixmap(QtGui.QPixmap(unselect_cape + "-unselected"+".png"), QtGui.QIcon.Normal, QtGui.QIcon.Off)
# frage
#asugelesene variable
self.pushbutton.setIcon(icon_selected_cape)

icon_selected_cape = QtGui.QIcon()
icon_selected_cape.addPixmap(QtGui.QPixmap(selected_cape + ".png"), QtGui.QIcon.Normal, QtGui.QIcon.Off)
selected_button.setIcon(icon_selected_cape)
entschuldigung nochamls und danke viel mals vür die antwort

@tryyan: Das macht alles keinen Sinn was Du da machst und auch wie Du das Problem beschreibst. Du kannst keine Button-Objekte in eine Datei schreiben und dementsprechend auch nicht aus einer Datei lesen. Das hat nichts mit Anführungszeichen zu tun. Und die Zeichenkette "self.pushbutton" ist etwas anderes als der Code ``self.pushbutton`` der ja bevor das Programm überhaupt gelaufen ist mal am Compiler vorbeigekommen ist, der daraus Bytecode gemacht hat, den CPython dann ausführt. Da ist immer noch ``global`` drin, und die Idee irgendwelchen Code zu generieren und in einer Funktion in eine Datei zu schreiben, um den in einer anderen Funktion dann zu laden und ausführen zu wollen ist ziemlich wirr, und noch schlimmer als ``global``. Denn das ist dann ja nicht nur eine globale Variable, die dazu auch noch Code enthält (oder enthalten soll), sondern auch noch persistenter globaler Zustand, wo auch noch externe Programme zugreifen könnten, man also noch weniger Überblick hat wer da wann drauf zugreift.

@__blackjack__ Ah okey und wie sollte ich die globalen variablen ersetzten. Es tut mir wirklich sehr leid das ich so schlecht bin

PS:
Ich ahbe es hinbekommen aber deine Haare werden warscheindlich zu Bergestehen :




from PyQt5 import QtCore, QtGui, QtWidgets








Danke für den super support und die Hilfe ich habe wieder vieles gelernt

@__blackjack__ heute probiere ich das mit der Buttongroup aus danke für die Auskunft