[python] otázka ohladom mojho štýlu písania v OOP

Jan Bednařík jan.bednarik na gmail.com
Sobota Leden 12 18:23:49 CET 2013


Ahoj,

v prvé řadě bych doporučil nastudovat si PEP-8, což je doporučení (de-facto
standard) jak psát kód v Pythonu. http://www.python.org/dev/peps/pep-0008/

Honza



2013/1/12 Roman Beno <romanbeno273 na gmail.com>

> <http://programujte.com/anonymni-profil/170094/>
>
> Dobrý deň,
>
> chcel by som sa spýtať na váš názor ohľadne môjho štýlu písania programov
> v OOP.
> Rád prijmem akukolvek kritiku, mojim cielom je sa v tomto ohľade zlepšit a
> v tejto oblasti som si nie som príliš istý.
>
> Tu sú ukážky mojich kódov:
>
> # -*- coding: utf-8 -*-
> #################Definícia triedy#######################
> class Fibonacci:
>     "Fibonacciho trieda"
>     def __init__(self,a=0,b=1,pocet_cisiel=10,default=0):
>         "Vyrobíme stavebný materiál pre fibonaccciho postupnosť."
>         self.a=a
>         self.b=b
>         self.pocet_cisiel=pocet_cisiel
>         self.default=default
>     def proces(self):
>         print(self.a,self.b,end=" ")
>         while True:
>             self.a=(self.a+self.b)
>             self.default+=1
>             if self.default>=self.pocet_cisiel:
>                 print("\n")
>                 break
>             print(self.a,end=" ")
>             # postupnost bude a,b,a,b,a,b... tento blok je pre všetky a
>             self.b=(self.a+self.b)
>             self.default+=1
>             if self.default>=self.pocet_cisiel:
>                 print("\n")
>                 break
>             print(self.b,end=" ")
>             # a tento pre všetky b
> #################Hlavný program##########################
> MojaInstancia=Fibonacci(pocet_cisiel=20)
> MojaInstancia.proces()
>
>
> ************************************************************************************************************
> Tu je další:
>
> # -*- coding: utf-8 -*-
> # program Záznamník adries
> # tréning Tkinter + OOP
> # autor JA, dnes
> # ver. 1.0
> # * opravená chyba pri vytvárani poľa pre zadávaní adresy, použitá
> neexistujúca farba okna
> # * opravená chyba pri vytváraní tlačítka ukončujúceho okno zadávania,
> použitý neexistujúci názov prvku
> # * zmenené umiestnenie a veľkosť tlačítka pre zavretie okna pri pridávaní
> adries
> # ver. 1.1
> # * opravená chyba pri potvrdení voľby mena pri pridávaní mena, použitá
> neexistujúca premenná
> # * opravená chyba pri potvrdení voľby mena pri pridávaní adresy, použitá
> neexistujúca premenná
> # ver. 1.2
> # * polia pre zadávaní mena aj adresy sa po zadaní oboch údajov teraz
> vymažú (ich hodnota)m
> #   uživatel ich teraz nemusí zadávať sám
> # * opravená chyba pri  načítaní údajov zo zoznamu, použitá neexistujúca
> premenná
> # ver. 1.2.1
> # * opravená chyba pri zobrazovaní načítaných údajov,použitá neexistujúca
> premenná
> # ver. 1.3
> # * opravená chyba pri vytváraní tlačítka na zavretie okna načítania
> údajov, použitá neexistujúca
> #   premenná
> # * opravená chyba pri zobrazovaní načítaných údajov (druhá chyba v
> tomto), použitá neexistujúca premenná
> # * zmenený text niektorých tlačítok v úvodnom menu
> # * v popisku okna sa teraz zobrazuje aj aktuálna verzia programu
> # ver. 1.3.1
> # * pri načítaní údajov upravená oddelovacia čiara údajov
> # * zmenené rozmery tlačítka ukončujúceho okno načítania údajov
> # ver. 1.3.2
> # * v okne ukladania súboru zmenená farba informačného textu na
> prehľadnejšiu
> # * opravená chyba pri ukladaní súborov, použitá neexistujúca premenná
> # ver. 1.4
> # * opravená chyba pri ukladaní súborov, použitá ďalšia neexistujúca
> premenná
> # * zmenená farba riadku pre chybové hlásenia v okne ukladania dát na
> prirodzenú bledomodrú
> # * pridaná možnosť odstránenia položiek zo zoznamu
>
>  from tkinter import *
> import sys
> import os
>
> class Zaznamnik:
>     "Toto je trieda môjho záznamníku"
>
>     def __init__(self):
>         self.hlavneOkno=Tk()
>         self.hlavneOkno.title("Záznamník, Ver. 1.4")
>         self.adresy=[]
>         self.zoznamMena=[]
>
>     def program(self):
>         "Táto časť programu bude mať za úlohu čisto len grafiku hlavného
> menu (nie logiku)."
>
> self.HOuvodnypopisokLabel=Label(self.hlavneOkno,bg="lightblue",text="Vitajte
> v mojom záznamníku adries.\nDúfam, že sa vám bude páčiť.")
>         self.HOuvodnypopisokLabel.grid(row=0,column=0,sticky=W+E+N+S)
>         # uvodny popisok - label - ktorý nás privíta v programe
>
> self.HOtlacitkoZadavatAdresy=Button(self.hlavneOkno,bg="lightblue",activebackground="lightgreen",text="Pridať
> adresy do záznamníka")
>         self.HOtlacitkoZadavatAdresy.grid(row=1,column=0,sticky=W+E+N+S)
>         # tlačítko, ktoré by malo umožniť zadávanie adries
>
> self.HOtlacitkoOdstranitAdresy=Button(self.hlavneOkno,bg="lightblue",activebackground="lightgreen",text="Odstrániť
> adresu zo záznamníka")
>         self.HOtlacitkoOdstranitAdresy.grid(row=2,column=0,sticky=W+E+N+S)
>         # tlačítko, ktoré by malo umožniť odstránenie adries
>
> self.HOtlacitkoUlozit=Button(self.hlavneOkno,bg="lightblue",activebackground="lightgreen",text="Uložiť
> do súboru")
>         self.HOtlacitkoUlozit.grid(row=3,column=0,sticky=W+E+N+S)
>         # tlačítko, ktoré by malo umožnit uloženie do súboru
>
> self.HOtlacitkoNacitat=Button(self.hlavneOkno,bg="lightblue",activebackground="lightgreen",text="Načítať
> dáta zo zoznamu")
>         self.HOtlacitkoNacitat.grid(row=4,column=0,sticky=W+E+N+S)
>         # tlačítko, ktoré by malo umožniť načítanie dát zo súboru
>
> self.HOtlacitkoNapoveda=Button(self.hlavneOkno,bg="lightblue",activebackground="lightgreen",text="Zobrazenie
> nápovedy")
>         self.HOtlacitkoNapoveda.grid(row=5,column=0,sticky=W+E+N+S)
>         # tlačítko, ktoré by malo zobraziť nápovedu
>
> self.HOtlacitkoUkoncit=Button(self.hlavneOkno,bg="lightblue",activebackground="lightgreen",text="Ukončiť
> program",command=self.hlavneOkno.quit)
>         self.HOtlacitkoUkoncit.grid(row=6,column=0,sticky=W+E+N+S)
>         # tlačítko, ktoré by malo ukončiť program
>
>         def zadavanie_adries():
>             "Táto časť bude zachytávať logiku programu. Kvoli velkému
> rozsahu rozdelenú na niekoľko častí. Konkrétne zadávanie adries."
>             # prvá časť tejto funkcie
>             # vzhľad podokna na zadávanie
>             self.OknoZadavanie=Toplevel()
>             self.OknoZadavanie.title("Zadávanie adries")
>
> self.OknoZadavanieLabel=Label(self.OknoZadavanie,bg="lightblue",text="
> Aktuálna činnosť: Zadávanie adries                 \nPre potvrdenie údaju z
> okna stlačte ENTER.\n")
>
> self.OknoZadavanieLabel.grid(row=0,column=0,columnspan=4,sticky=W+E+N+S)
>             # Vytvorenie podokna na zadávanie adries, nastavený titulok +
> úvodný Label
>             self.VarPPCH=StringVar()
>
> self.OknoZadavanieLabelChybHlasky=Label(self.OknoZadavanie,bg="lightblue",textvariable=self.VarPPCH)
>
> self.OknoZadavanieLabelChybHlasky.grid(row=1,column=0,columnspan=4,sticky=W+E+N+S)
>             # Vytvorená textová premenná pre prípadné chybové hlášky. Pre
> začiatok sa bude tváriť ako
>             # neviditeľná súčasť okna
>
> self.RamecZadajMeno=Frame(self.OknoZadavanie,bd=3,bg="lightblue")
>
> self.RamecZadajMeno.grid(row=2,column=0,columnspan=4,sticky=W+E+N+S)
>             self.VarPZM=StringVar()
>             # Vytvorený rámec pre zadanie mena + popisok k nemu. Taktiež
> premenná pre políčko
>             # pre vstup
>             self.RamecPopisokPola=Label(self.RamecZadajMeno,text="Zadajte
> meno dotyčného:",bg="lightblue")
>             self.RamecPopisokPola.grid(row=2,column=0,sticky=W+E+N+S)
>
> self.PolePreVstupMena=Entry(self.RamecZadajMeno,bg="lightyellow",textvariable=self.VarPZM,font="Arial
> 8")
>
> self.PolePreVstupMena.grid(row=2,column=1,columnspan=3,sticky=W+E+N+S)
>             # Vytvorený popisok pre pole pre vstup + políčko samotné
>
> self.RamecZadajAdresu=Frame(self.OknoZadavanie,bd=4,bg="lightblue")
>
> self.RamecZadajAdresu.grid(row=3,column=0,columnspan=4,sticky=W+E+N+S)
>
> self.LabelPreDruhePole=Label(self.RamecZadajAdresu,bg="lightblue",text="Zadajte
> adresu dotyčného: ")
>             self.LabelPreDruhePole.grid(row=3,column=0,sticky=W+E+N+S)
>             self.VarPPA=StringVar()
>
> self.PolePreAdresu=Entry(self.RamecZadajAdresu,bg="lightyellow",textvariable=self.VarPPA,font="Arial
> 8")
>             self.PolePreAdresu.grid(row=3,column=1,columnspan=3)
>             # Vytvorený rámec pre zadanie adresy, popis pre neho, premennú
> pre neho a taktiež pole
>             # samé.
>
> self.ZadavacieTlacitkoUkoncit=Button(self.OknoZadavanie,bg="lightblue",activebackground="lightgreen",command=self.OknoZadavanie.destroy,text="Zavrieť
> toto okno")
>
> self.ZadavacieTlacitkoUkoncit.grid(row=4,column=0,columnspan=4,sticky=W+E+N+S)
>             self.OverovaciToken=0
>             # Vytvorené tlačítko pre zavretie tohto okna pre zadanie
> adresy.
>             def enter(BoloStlacene):
>                 BoloStlacene.widget.configure(state=DISABLED)
>                 self.udajMeno=str(self.PolePreVstupMena.get())
>                 self.OverovaciToken+=1
>                 if self.OverovaciToken==2:
>                     self.PolePreVstupMena.configure(state=NORMAL)
>                     self.PolePreAdresu.configure(state=NORMAL)
>                     self.zoznamMena.append(self.udajMeno)
>                     self.VarPZM.set("")
>                     self.VarPPA.set("")
>                     self.adresy.append([self.udajMeno,self.udajAdresu])
>                     self.OverovaciToken=0
>             # funkcia, čo sa má stať po zdarnom zadaní informácie do
> políčka Meno. Dané políčko
>             # sa disabluje, údaj z neho sa uloží. + špecialita overovací
> token. Ak dosiahne hodnotu 2,
>             # program zistí, že má spravit ďalšiu "obrátku" pri pridávaní
> do zoznamu
>             def entri(StlaceneBolo):
>                 StlaceneBolo.widget.configure(state=DISABLED)
>                 self.udajAdresu=str(self.PolePreAdresu.get())
>                 self.OverovaciToken+=1
>                 if self.OverovaciToken==2:
>                     self.PolePreVstupMena.configure(state=NORMAL)
>                     self.PolePreAdresu.configure(state=NORMAL)
>                     self.zoznamMena.append(self.udajMeno)
>                     self.VarPZM.set("")
>                     self.VarPPA.set("")
>                     self.adresy.append([self.udajMeno,self.udajAdresu])
>                     self.OverovaciToken=0
>             # táto funkcia zase slúži na definovanie činnosti políčka
> Adresy. Zapíše údaj
>             # do zoznamu a disabluje políčko. Malo by to zabezpečiť
> zadávanie.
>             self.PolePreVstupMena.bind("<Return>",enter)
>             self.PolePreAdresu.bind("<Return>",entri)
>             # čo sa má stať po stlačení Entrov v rôznych okienkach
>
>         def nacitanie_dat():
>             "Táto funkcia by mala zabezpečiť zobrazenie, resp. načítanie
> dát zo zoznamu."
>             self.OknoNacitanie=Toplevel()
>             self.OknoNacitanie.title("Načítanie dát")
>             # vytvorené vlastné okno pre načítanie dát, kde by sa
> teoreticky mali zobrazovať
>             # dáta uložené v zozname adries
>
> self.oknoNacitanieZobrazenie=Label(self.OknoNacitanie,bg="lightblue",text="")
>             self.oknoNacitanieZobrazenie.grid(row=0,column=0)
>
> self.premenna="""\tMená\t\tAdresy\n***************************************\n"""
>             if len(self.adresy)==0:
>                 self.oknoNacitanieZobrazenie.configure(text="Zoznam je
> prázdny.")
>             else:
>                 for p in self.adresy:
>                     self.coNacitaj="\t{0}\t\t{1}\n".format(p[0],p[1])
>                     self.premenna+=self.coNacitaj
>                 self.oknoNacitanieZobrazenie.configure(text=self.premenna)
>             # toto by malo zabezpečiť zdarné zobrazovanie položiek zoznamu
>
>
> self.tlacitkoUkoncitON=Button(self.OknoNacitanie,bg="lightblue",activebackground="lightgreen",text="Zavrieť
> toto okno",command=self.OknoNacitanie.destroy)
>             self.tlacitkoUkoncitON.grid(row=1,column=0,sticky=N+S+E+W)
>             # toto tvorí tlačítko, ktoré by mohlo zavrieť okno
>
>         def ulozenie_do_suboru():
>             "Toto má na starosti zdarné uloženie dát do súboru. Trošku nám
> pomôže predošlá funkcia, načítanie dát."
>             self.OknoUlozenie=Toplevel()
>             self.OknoUlozenie.title("Uloženie do súboru")
>
> self.OknoUlozenieLabelVyber=Label(self.OknoUlozenie,text="Vyberte meno
> súboru, do ktorého chcete uložiť dáta zo súboru.",bg="lightblue")
>
> self.OknoUlozenieLabelVyber.grid(row=0,column=0,columnspan=2,sticky=W+E+N+S)
>             # vytvorené ukladacie okno, nastavený popisok pre neho +
> popisok pre políčko pre zadanie mena ukladacieho súboru
>             self.OUPPV=StringVar()
>
> self.OUPolePreVstup=Entry(self.OknoUlozenie,bg="lightyellow",font="Arial
> 8",textvariable=self.OUPPV)
>
> self.OUPolePreVstup.grid(row=0,column=2,columnspan=2,sticky=W+E+N+S)
>             # pole, kam môžeme zadávať samotný názov súboru
>
> self.OUPosudzovaciaHlaska=Label(self.OknoUlozenie,bg="lightblue",text="")
>
> self.OUPosudzovaciaHlaska.grid(row=1,column=0,columnspan=4,sticky=W+E+N+S)
>             # tu sa vypíš prípadná chybová hláška, v závislosti od toho,
> čo zadáme do zmieneného políčka
>             def entricek(ppv):
>                 self.result=str(self.OUPolePreVstup.get())
>                 if len(self.result)==0:
>                     self.OUPosudzovaciaHlaska.configure(text="Nezadali ste
> meno žiadneho súboru.")
>                     self.OUPPV.set("")
>                     return
>                 # ak ste nezadali nič, tak vypíše hlášku, znuluje hodnotu
> pola a zavrie funkciu
>                 if len(self.adresy)==0:
>                     self.OUPosudzovaciaHlaska.configure(text="Zoznam
> adries je prázdny.")
>                     self.OUPPV.set("")
>                     return
>                 # podobné ako v predchádzajúcom prípade, ibaže ak je
> zoznam adries prázdny
>                 self.subor=open(self.result,"wt")
>                 try:
>                     self.subor.write(self.premenna)
>                 except:
>                     self.OUPosudzovaciaHlaska.configure(text="Zatial ste
> nenačítali žiadne dáta. Urobte tak cez hlavné menu.")
>                     self.subor.close()
>                     self.OUPPV.set("")
>                     return
>                     # stale je možnost že uživatel nenačítal požadované
> dáta na vloženie
>                 self.OUPosudzovaciaHlaska.configure(text="Zatial ste
> nenačítali žiadne dáta. Urobte tak cez hlavné menu.")
>                 self.subor.close()
>                 self.OUPolePreVstup.configure(state=DISABLED)
>                 # toto sa stane v pripade že chyba nenastala,
>             self.OknoUlozenie.bind("<Return>",entricek)
>
> self.OUTlacitkoZavri=Button(self.OknoUlozenie,bg="lightblue",activebackground="lightgreen",text="Zavrieť
> toto okno",command=self.OknoUlozenie.destroy)
>
> self.OUTlacitkoZavri.grid(row=2,column=0,columnspan=4,sticky=W+E+N+S)
>             # urobené tlačítko, ktoré zavrie okno
>
>         def odstranenie_poloziek():
>             "Odstránenie jedného záznamu zo zoznamu adries a mien"
>             self.OknoOdstranenie=Toplevel()
>             self.OknoOdstranenie.title("Odstranenie poloziek")
>             self.OOpripravenytext=[]
>             # vytvorené okno odstránenia, nastavený popisok + náradíčko =
> prázdny zoznam
>             for x in self.zoznamMena:
>                 self.OOpripravenytext.append(x)
>
> self.OOodstranenieLabel=Label(self.OknoOdstranenie,bg="lightblue",text="Vyberte
> si položku zoznamu,ktorú\nchcete odstrániť:")
>             self.OOodstranenieLabel.grid(row=0,column=0)
>             # pripraví zoznam položiek pre neskoršie odstranovanie +
> vytvorenie labelu na odstranenie
>             # položky
>             self.OOPPO=StringVar()
>
> self.OOPolePreOdstranenie=Entry(self.OknoOdstranenie,bg="lightyellow",textvariable=self.OOPPO,font="Arial
> 8")
>
> self.OOPolePreOdstranenie.grid(row=0,column=1,columnspan=2,sticky=W+E+S+N)
>
> self.OOLabelChybovaHlaska=Label(self.OknoOdstranenie,bg="lightblue",text="")
>
> self.OOLabelChybovaHlaska.grid(row=1,column=0,columnspan=3,sticky=W+E+S+N)
>             # samotné políčko, kam možete zadat meno položky (meno
> konk.osoby, ktorú chcete odstrániť
>             def stlacil_entri(gangnam):
>                 coBoloStlacenee=str(self.OOPolePreOdstranenie.get())
>                 if coBoloStlacenee in self.zoznamMena:
>
> indexCoBoloStlacenee=self.zoznamMena.index(coBoloStlacenee)
>                 else:
>                     self.OOLabelChybovaHlaska.configure(text="Neexistujúce
> meno.")
>                     self.OOPPO.set("")
>                 del self.adresy[indexCoBoloStlacenee]
>                 del self.zoznamMena[self.zoznamMena.index(coBoloStlacenee)]
>                 self.OOPPO.set("")
>                 # zoberieme to z políčka pre vstup, ak je v zozname mien,
> odstrán z oboch zoznamov
>                 # ak nie, vyhod chybove hlasky. Samozrejmostou je nasledne
> premazanie policka
>             self.OOPolePreOdstranenie.bind("<Return>",stlacil_entri)
>
> self.OOZavriOkno=Button(self.OknoOdstranenie,bg="lightblue",activebackground="lightgreen",text="Zavrieť
> toto okno",command=self.OknoOdstranenie.destroy)
>
> self.OOZavriOkno.grid(row=2,column=0,columnspan=3,sticky=W+E+S+N)
>             # zabezpečená akcia pre pole pre vstup po stlačení Enteru.
>             # vytvorené tlačítko na zavretie okna
>
>
>         self.HOtlacitkoNacitat.configure(command=nacitanie_dat)
>         self.HOtlacitkoUlozit.configure(command=ulozenie_do_suboru)
>         self.HOtlacitkoZadavatAdresy.configure(command=zadavanie_adries)
>
> self.HOtlacitkoOdstranitAdresy.configure(command=odstranenie_poloziek)
>
>         self.hlavneOkno.mainloop()
>
> #########################################################
>
> inst=Zaznamnik()
> inst.program()
>
> Ďakujem za akúkolvek odpoved
>
> _______________________________________________
> Python mailing list
> Python na py.cz
> http://www.py.cz/mailman/listinfo/python
>
------------- další část ---------------
HTML příloha byla odstraněna...
URL: <http://www.py.cz/pipermail/python/attachments/20130112/2a4cf2ec/attachment.html>


Další informace o konferenci Python