Ohodnotenie, vytknutie niektorych veci v programe – Python – Fórum – Programujte.com

Zdravím všetkých, takže moj prvý "program" je na svete. Je to moj prvý jazyk a preto by som rád požiadal o kritiku, dake hodnotenie. Je to len preto, aby som vedel, že to určite tak robiť nemožem v budúcnosti. Skutočne budem rad každej kritike :) Viem o jednej veci, ktorá tam riešena nie je, no to spravím, hned ako na to pridem :) Do súboru sa to uklada pre buducu historiu, kde by boli tí najlepši, resp. najlepšie vysledky. Ďakujem

print '                                Vitajte v hre Keno :)					 '

print

print ' Tip zapiste v tvare prve cislo [ciarka] druhe cislo [ciarka] tretie cislo [ciarka] atd!!!'

print '          Maximalny pocet tipov je 10, cisla je mozne tipovat od 1 do 80 		 '

print '*'*90

print

print '                       Pre zaciatok hry Vam darujem 100 SK :D				 '

print

import random

konto=100

spustenie="A"

print

while spustenie is "A" or spustenie is "a":

    print

    spustenie=raw_input('Pre zacatie hry stlacte >A<, pre ukoncenie >N< ! ---> ')

    print

    if spustenie is "A" or spustenie is "a":

        while 1:

            try:

                stavka=input('Zadaj vysku stavky: ')

                while konto < stavka or stavka < 1:

                    stavka=input('Zadaj vysku stavky: ')

                break

            except (NameError, SyntaxError, TypeError):

                print "Stavku zadavaj pozorne, ide o vsetko :)!"

            else:

                continue

        while 1:

            try:

                poletip=input('Zadaj svoj tip: ')

                if len(poletip) not in xrange(1,11):

                    print

                    print " Maximalny pocet tipov je 10, vsetky tipy po 10. tipe budu zmazane !!!"

                poletip2=list(poletip[0:10])

                del poletip

                break

            except NameError:

                print "Precitajte si pozorne pravidla a zopakujte Vas tip!!! "

            except SyntaxError:

                print "Precitajte si pravidla a skuste znova!!!"

            except TypeError:

                print "Aj v pripade zadania len jedneho tipu, piste za tipom ciarku!!! "

            else:

                continue

        konto=konto-stavka

        poletip2.sort()

        polekeno=xrange(1,81)

        polevylosovane=random.sample(polekeno, 20)

        del polekeno

        polevylosovane.sort()

        vysledok=[]

        zmenapoletip2=len(poletip2)

        for pozicia in xrange(0,(len(poletip2))):

            if min(poletip2) < 1:

                poletip2.remove(min(poletip2))    

            elif max(poletip2) >80:

               poletip2.remove(max(poletip2))

        else:

            for pozicia in xrange(0,(len(poletip2))):

                vysledok.append(int(poletip2[pozicia] in polevylosovane))

        del pozicia

        if len(poletip2) < zmenapoletip2:

            print

            print " Tipy zadane mimo rozsah 1 az 80 sa neberu do uvahy pri zrebovani!!!"

        del zmenapoletip2

        print

        print '-'*90

        print ' Pocet uhadnutych cisiel: ',vysledok.count(1)

        print

        print ' Vylosovane boli tieto cisla:'

        print '--->',polevylosovane

        del polevylosovane

        print

        print ' Vas tip bol: '

        if len(poletip2)==0:

            print ' Nezadali ste ani jedno cislo v rozsahu 1 az 80 !!!'

        else:

            print '--->',poletip2 

        print

        konto=stavka*(vysledok.count(1))+konto

        del stavka

        if konto==0:

            spustenie="N"

            print "               Zial, prehrali ste vsetky svoje peniaze. Prajeme pekny den :D "

        else:

            print ' Vyska Vaseho konta je:'

            print '--->',konto,' SK' 

        print '-'*90

    else:

        print

        historia=('Historia.txt')

        subor=file(historia,'a')

        zapis=raw_input('Zadajte svoje meno: ')+(' : ')+(str(konto))

        subor.write((zapis) + '\n')

        subor.close()        

        print       

        print       "    '||''|.    ..|''||   '||'  '|' '||' '||''|.   '||''''|  '|.   '|' '||'     |        "

        print       "     ||   ||  .|'    ||   '|.  .'   ||   ||   ||   ||  .     |'|   |   ||     |||       "

        print       "     ||    || ||      ||   ||  |    ||   ||    ||  ||''|     | '|. |   ||    |  ||      " 

        print       "     ||    || '|.     ||    |||     ||   ||    ||  ||        |   |||   ||   .''''|.     "

        print       "    .||...|'   ''|...|'      |     .||. .||...|'  .||.....| .|.   '|  .||. .|.  .||.    "

        print                     

        print               

        print                             

        print       "     '|.   '|'     |      '||''|.   '||'  '|' '||''|.   '||'  '|'   ..|'''.| '||''''|   "

        print       "      |'|   |     |||      ||   ||   ||    |   ||   ||   ||    |  .|'     '   ||  .     "

        print       "      | '|. |    |  ||     ||'''|.   ||    |   ||    ||  ||    |  ||          ||''|     " 

        print       "      |   |||   .''''|.    ||    ||  ||    |   ||    ||  ||    |  '|.      .  ||        "

        print       "     .|.   '|  .|.  .||.  .||...|'    '|..'   .||...|'    '|..'    ''|....'  .||.....|  "

        print      

        import time

        time.sleep(3)

Vidim, že som tým svojim programom očaril z reakcii :) No nic, tak sem davam pokročilejsiu verziu, ošetrenu myslim že už voči všetkým možným chybám na vstupe, hodnotenie je taktiež viac vicibrené. Mám z toho skutočne radosť, že to ide tak ako má a tomu chápem. Ale mám predsa len otázku:
1. Ako prinutit prostredie prikazového riadku, aby po sputení programu malo presne 90 znakov na širku? Kludne ma odporučte aj na dáky článok, alebo podobne, pretože ja neviem ani čo vlastne mám hladať. Vdaka

Tu je ten moj krásavec :D :

print "                                Vitajte v hre Keno :)					 "

print

print " Tip zapiste v tvare prve cislo [ciarka] druhe cislo [ciarka] tretie cislo [ciarka] atd!!!"

print "          Maximalny pocet tipov je 10, cisla je mozne tipovat od 1 do 80 		 "

print "*"*90

print

print "                        Pre zaciatok hry Vam darujem 100 SK :D				 "

print

import random

konto=100

spustenie="A"

print

while spustenie is "A" or spustenie is "a":

    print

    spustenie=raw_input("Pre zacatie hry stlacte >A<, pre ukoncenie >N< ! ---> ")

    print

    if spustenie is "A" or spustenie is "a":

        while 1:

            try:

                stavka=input("Zadaj vysku stavky: ")

                while konto < stavka or stavka < 1:

                    print "Stavku zadavajte pozorne, skontrolujte si vysku svojho konta!!!"

                    stavka=input("Zadaj vysku stavky: ")

                break

            except (NameError, SyntaxError, TypeError):

                print "Stavku zadavaj pozorne, ide o vsetko :)!!!"

            else:

                continue

        while 1:

            try:

                poletip=input('Zadaj svoj tip: ')

                if len(poletip) not in xrange(1,11):

                    print

                    print " Maximalny pocet tipov je 10, vsetky tipy po 10. tipe budu zmazane !!!"

                poletip2=list(poletip[0:10])

                del poletip

                break

            except NameError:

                print "Precitajte si pozorne pravidla a zopakujte Vas tip!!! "

            except SyntaxError:

                print "Precitajte si pravidla a skuste znova!!!"

            except TypeError:

                print "Aj v pripade zadania len jedneho tipu, piste za tipom ciarku!!! "

            else:

                continue

        konto=konto-stavka

        poletip2.sort()

        polekeno=xrange(1,81)

        polevylosovane=random.sample(polekeno, 20)

        del polekeno

        polevylosovane.sort()

        vysledok=[]

        zmenapoletip2=len(poletip2)

        for pozicia in xrange(0,(len(poletip2))):

            if min(poletip2) < 1:

                poletip2.remove(min(poletip2))    

            elif max(poletip2) >80:

                poletip2.remove(max(poletip2))

            for pozicia in xrange(0,(len(poletip2))):

                if pozicia < len(poletip2):

                    if poletip2.count(poletip2[pozicia]) > 1:

                        poletip2.remove(poletip2[pozicia])

        else:

            for pozicia in xrange(0,(len(poletip2))):

                vysledok.append(int(poletip2[pozicia] in polevylosovane))

        del pozicia

        if len(poletip2) < zmenapoletip2:

            print

            print " Tipy zadane mimo rozsah 1 az 80 sa neberu do uvahy pri zrebovani!!!"

        del zmenapoletip2

        print

        print "-"*90

        print " Pocet uhadnutych cisiel: ",vysledok.count(1)

        print

        print " Vylosovane boli tieto cisla:"

        print "--->",polevylosovane

        del polevylosovane

        print

        if len(poletip2)==0:

            print "                    Nezadali ste ani jedno cislo v rozsahu 1 az 80 !!!"

        else:

            print " Vas tip bol: "

            print "--->",list(poletip2)

        print

        if len(poletip2)==1:

            moznosti=(0,2)

            konto=stavka*(int(moznosti[(vysledok.count(1))]))+konto

        elif len(poletip2)==2:

            moznosti=(0,0,8)

            konto=stavka*(int(moznosti[(vysledok.count(1))]))+konto

        elif len(poletip2)==3:

            moznosti=(0,0,2,16)

            konto=stavka*(int(moznosti[(vysledok.count(1))]))+konto

        elif len(poletip2)==4:

            moznosti=(0,0,0,8,32)

            konto=stavka*(int(moznosti[(vysledok.count(1))]))+konto

        elif len(poletip2)==5:

            moznosti=(0,0,0,2,16,40)

            konto=stavka*(int(moznosti[(vysledok.count(1))]))+konto

        elif len(poletip2)==6:

            moznosti=(1,0,0,1,2,20,60)

            konto=stavka*(int(moznosti[(vysledok.count(1))]))+konto

        elif len(poletip2)==7:

            moznosti=(1,0,0,0,2,10,40,80)

            konto=stavka*(int(moznosti[(vysledok.count(1))]))+konto

        elif len(poletip2)==8:

            moznosti=(1,0,0,0,1,4,40,60,100)

            konto=stavka*(int(moznosti[(vysledok.count(1))]))+konto

        elif len(poletip2)==9:

            moznosti=(1,0,0,0,0,3,20,30,60,120)

            konto=stavka*(int(moznosti[(vysledok.count(1))]))+konto

        elif len(poletip2)==10:

            moznosti=(1,0,0,0,0,3,10,20,50,100,200)

            konto=stavka*(int(moznosti[(vysledok.count(1))]))+konto

        del vysledok 

        del stavka

        if konto==0:

            spustenie="N"

            print "               Zial, prehrali ste vsetky svoje peniaze. Prajem pekny den :D "

            print

            print "-"*90

            import time

            time.sleep(3)

        else:

            print " Vyska Vaseho konta je:"

            print "--->",konto," SK" 

            print "-"*90

    else:

        print

        historia=('Historia.txt')

        subor=file(historia,'a')

        zapis=raw_input('Zadajte svoje meno: ')+(' : ')+(str(konto))

        subor.write((zapis) + '\n')

        subor.close()        

        print       

        print       "    '||''|.    ..|''||   '||'  '|' '||' '||''|.   '||''''|  '|.   '|' '||'     |        "

        print       "     ||   ||  .|'    ||   '|.  .'   ||   ||   ||   ||  .     |'|   |   ||     |||       "

        print       "     ||    || ||      ||   ||  |    ||   ||    ||  ||''|     | '|. |   ||    |  ||      " 

        print       "     ||    || '|.     ||    |||     ||   ||    ||  ||        |   |||   ||   .''''|.     "

        print       "    .||...|'   ''|...|'      |     .||. .||...|'  .||.....| .|.   '|  .||. .|.  .||.    "

        print                     

        print               

        print                             

        print       "     '|.   '|'     |      '||''|.   '||'  '|' '||''|.   '||'  '|'   ..|'''.| '||''''|   "

        print       "      |'|   |     |||      ||   ||   ||    |   ||   ||   ||    |  .|'     '   ||  .     "

        print       "      | '|. |    |  ||     ||'''|.   ||    |   ||    ||  ||    |  ||          ||''|     " 

        print       "      |   |||   .''''|.    ||    ||  ||    |   ||    ||  ||    |  '|.      .  ||        "

        print       "     .|.   '|  .|.  .||.  .||...|'    '|..'   .||...|'    '|..'    ''|....'  .||.....|  "

        print      

        import time

        time.sleep(3)

Jaj, tohle mi nějak uniklo ;-) Takže pár mých připomínek:

* pro lepší čitelnost doporučuji nastudovat print se třemi uvozovkami na začátku i konci
* pro lepší udržovatenost doporučuji do elif nedávav stejné opakující se věci - dát to mimo celý if, když to platí pro všechno. Mluvím o konto=stavka*(int(moznosti[(vysledok.count(1))]))+konto
* pro lepší čitelnost i udržovatelnost bych se nebál funkcí
* všechny importy na začátek
* použivání funkce del je v Pythonu v této úrovni dost zbytečné. Nic tím neušetříš a ztrácíš na čitelnosti
* pokud chceš hodnotit, neškodí u význačných či složitějších částí komentář

Jinak je to fajn, funguje. Jo a to "přesně 90 znaků na šířku" - to nejde. Leda pak jokenní programování, tedy gui, tedy tkinter, wx, gtp, qt, atd.

To geon : Geon, ty si boh :) Vdaka tebe a tvojmu serialu som sa naučil tieto veci a teraz hodnotiš moj prvý pokus v živote v čomkolvek niečo spraviť. Tak dakujem. Chyby si poopravujem. Tie printy sa ani mne nepacili, že ich je také kvantum, takže to určite poriešim. Potom sa pustím konečne do tej historie. Skrátka nejaký prehľad najlepších vysledkov a možno ešte dačo ma napadne. Tak zatím a maj sa fajn.

Zdravim vsetkych. Takze k dnesnemu dnu som dal dokopy ten moj program. V prilohe je program pre uzivatelov windowsov, ktory nemaju python nainstalovany a taktiez aj pre python pozitivnych a uzivatelov linuxovych distier a asi aj macov, kde je python tiez uz nainstalovany. Neprisiel som na to, ci sa sem da vlozit aj viac archivov, takze treba stiahnut cely archiv, ktory obsahuje aj exe. Som sa to snazil co najlepsie, tak snad to aspon na chvilku niekoho zabavi. V archive pre windows sa exe subor nachadza v zlozke dist.
Su tam poopravovane dake chyby, spravena historia. V reakcii na geonovo hodnotenie napisem tolkoto. Print s tromi uvodzovkami som sa tam snazil dat, ale neviem, ci to ma byt tak. Hladal som dake vzorove priklady, ale zial. Funkcie som nepouzil, to si necham na dalsi program, bo to najprv musim zvladnut. Funkciu del som viac nepouzival, ale tam kde bola, som ju nechal, aj ked ako vravis zbytocne. Ale som ti vdacny a tie rady si v dalsej tvorbe urcite vezmem k srdcu.

p.s.: Ak sa najdu dake chyby, budem iba rad, ak mi ich oznamite.