Traduttore Binario

[imported_FireSkull]

Allora ragazzi, questo è il mio primo programma che creo. Non ha un utilità, più che altro l'ho fatto per esecizio, ma ci tenevo a farvelo guardare.

#Questo e' un traduttore in binario by FireSkull
def TraduzioneI_B(parola):
    Indice = 0
    while Indice < len(parola):
        Lettera = parola[Indice]
        for Caretteri in parola[Indice]:
            if parola[Indice] == 'A':
                print "1000001",'\t', " = ",'\t'," A "
            elif parola[Indice] == 'B':
                print "1000010",'\t', " = ",'\t'," B "

            elif parola[Indice] == 'C':
                print "1000011",'\t', " = ",'\t'," C "

            elif parola[Indice] == 'D':
                print "1000100",'\t', " = ",'\t'," D "

            elif parola[Indice] == 'E':
                print "1000101",'\t', " = ",'\t'," E "
                
            elif parola[Indice] == 'F':
                print "1000110",'\t', " = ",'\t'," F "

            elif parola[Indice] == 'G':
                print "1000111",'\t', " = ",'\t'," G "

            elif parola[Indice] == 'H':
                print "1001000",'\t', " = ",'\t'," H "

            elif parola[Indice] == 'I':
                print "1001001",'\t', " = ",'\t'," I "

            elif parola[Indice] == 'J':
                print "1001010",'\t', " = ",'\t'," J "

            elif parola[Indice] == 'K':
                print "1001011",'\t', " = ",'\t'," K "

            elif parola[Indice] == 'L':
                print "1001100",'\t', " = ",'\t'," L "
                
            elif parola[Indice] == 'M':
                print "1001101",'\t', " = ",'\t'," M "

            elif parola[Indice] == 'N':
                print "1001110",'\t', " = ",'\t'," N "

            elif parola[Indice] == 'O':
                print "1001111",'\t', " = ",'\t'," O "

            elif parola[Indice] == 'P':
                print "1010000",'\t', " = ",'\t'," P "

            elif parola[Indice] == 'Q':
                print "1010001",'\t', " = ",'\t'," Q "

            elif parola[Indice] == 'R':
                print "1010010",'\t', " = ",'\t'," R "

            elif parola[Indice] == 'S':
                print "1010011",'\t', " = ",'\t'," S "

            elif parola[Indice] == 'T':
                print "1010100",'\t', " = ",'\t'," T "

            elif parola[Indice] == 'U':
                print "1010101",'\t', " = ",'\t'," U "

            elif parola[Indice] == 'V':
                print "1010110",'\t', " = ",'\t'," V "

            elif parola[Indice] == 'W':
                print "1010111",'\t', " = ",'\t'," W "

            elif parola[Indice] == 'X':
                print "1011000",'\t', " = ",'\t'," X "
                
            elif parola[Indice] == 'Y':
                print "1011001",'\t', " = ",'\t'," Y "

            elif parola[Indice] == 'Z':
                print "1011010",'\t', " = ",'\t'," Z "

            elif parola[Indice] == 'a':
                print "1100001",'\t', " = ",'\t'," a "

            elif parola[Indice] == 'b':
                print "1100010",'\t', " = ",'\t'," b "

            elif parola[Indice] == 'c':
                print "1100011",'\t', " = ",'\t'," c "

            elif parola[Indice] == 'd':
                print "1100100",'\t', " = ",'\t'," d "

            elif parola[Indice] == 'e':
                print "1100101",'\t', " = ",'\t'," e "

            elif parola[Indice] == 'f':
                print "1100110",'\t', " = ",'\t'," f "

            elif parola[Indice] == 'g':
                print "1100111",'\t', " = ",'\t'," g "

            elif parola[Indice] == 'h':
                print "1101000",'\t', " = ",'\t'," h "

            elif parola[Indice] == 'i':
                print "1101001",'\t', " = ",'\t'," i "

            elif parola[Indice] == 'j':
                print "1101010",'\t', " = ",'\t'," j "

            elif parola[Indice] == 'k':
                print "1101011",'\t', " = ",'\t'," k "

            elif parola[Indice] == 'l':
                print "1101100",'\t', " = ",'\t'," l "

            elif parola[Indice] == 'm':
                print "1101101",'\t', " = ",'\t'," m "

            elif parola[Indice] == 'n':
                print "1101110",'\t', " = ",'\t'," n "

            elif parola[Indice] == 'o':
                print "1101111",'\t', " = ",'\t'," o "

            elif parola[Indice] == 'p':
                print "1110000",'\t', " = ",'\t'," p "

            elif parola[Indice] == 'q':
                print "1110001",'\t', " = ",'\t'," q "

            elif parola[Indice] == 'r':
                print "1110010",'\t', " = ",'\t'," r "

            elif parola[Indice] == 's':
                print "1110011",'\t', " = ",'\t'," s "

            elif parola[Indice] == 't':
                print "1110100",'\t', " = ",'\t'," t "

            elif parola[Indice] == 'u':
                print "1110101",'\t', " = ",'\t'," u "

            elif parola[Indice] == 'v':
                print "1110110",'\t', " = ",'\t'," v "

            elif parola[Indice] == 'w':
                print "1110111",'\t', " = ",'\t'," w "

            elif parola[Indice] == 'x':
                print "1111000",'\t', " = ",'\t'," x "

            elif parola[Indice] == 'y':
                print "1111001",'\t', " = ",'\t'," y "

            elif parola[Indice] == 'z':
                print "1111010",'\t', " = ",'\t'," z "
        Indice = Indice + 1

n = 0
print "------ Traduttore Italiano-Binario ------"
print "Indice:",'\n'," Premi 0 per usare il traduttore.",'\n' " Premi 1 per uscire dal programma."
n = int(raw_input("Opzione : "))
while n != 1:
    if n == 0:
        parola = raw_input("Inserisci la parola da tradurre:")
        TraduzioneI_B(parola)
        n = int(raw_input("Opzione : "))
        

    elif n >= 1:
        print "Il programma e' chiuso."

    elif n <= -1:
        print "Il programma e' chiuso."

Consigli da darmi? Sapete dirmi qualche modo per accorciare il codice?

 

[!R~]

Invece di stampare il rispettivo binario per ogni carattere usando una montagna di elif, fai la codifica dec -> bin e vinci, ti risparmi 700 righe di codice

Ah, non usare input, è bastardo Se proprio ti serve un intero fai int(raw_input()).

 

[imported_FireSkull]

Ho sostituito input con int(raw_input)), ma non riesco a capire come usare la codifica dec -> bin.

 

[RedSkull]

tramite moltiplicazioni successive
e per fare da bin -> dec le divisioni successive
guarda quì per avere maggiori info: http://www.diit.unict.it/users/michele/didattica/fondamenti/fondamenti/conv.html
sono cose basilari che bisogna sapere

 

[fojeaf]

allora, se tu usassi python3 imho sarebbe meglio, e dovresti fare int(input()).

Anyway:

1) perché usi Indice ed il while? e perché crei la variabile Lettera e poi non la usi? e perché continui a fare parola[indice] invece di "sfruttare" il for? e perché fai un for su una cosa di 1 elemento (parola[Indice])?
2) non vorrei deluderti, ma il python offre già tutto asd:
3) non serve che chiedi l'opzione così.. è un while...
4) il modo con cui usi la variabile n è racapricciante se permetti.. anche perché se io metto -1 mica si chiude: basta mettere un bel ==0 e un else

def Traduzione_B(parola):
    for char in parola:
        print bin(ord(char)),"\t = \t",char

n = 0
print "------ Traduttore Italiano-Binario ------"
print "Indice:",'\n'," Premi 0 per usare il traduttore.",'\n' " Premi 1 per uscire dal programma."

while n==0:
    n = int(raw_input("Opzione: ")
    if n==0:
        parola = raw_input("Inserisci la parola da tradurre:")
        TraduzioneI_B(parola)
    else:
         print "Il programma è chiuso"

Come primo codice non c'è male cmq, solo ti consiglio di pensare con + logica a dove il codice può essere "riassunto".

 

[!R~]

IMHO essendo un esercizio svolto da uno che è alle prime armi sarebbe meglio se l'algoritmo se lo scrivesse by his self.
È una cazzata si, ma almeno si esercita.

 

[imported_FireSkull]

Ho capito grazie ragazzi

 

[fojeaf]

IMHO essendo un esercizio svolto da uno che è alle prime armi sarebbe meglio se l'algoritmo se lo scrivesse by his self.
È una cazzata si, ma almeno si esercita.

sì ok, quellò è vero, cmq era per spiegargli come usare meglio i cicli.

 

[Pythoner]

Consiglio al volo, perché non usi un dizionario al posto di tutti quegli elif?

 

[fojeaf]

Consiglio al volo, perché non usi un dizionario al posto di tutti quegli elif?

il consiglio in sè sarebbe giusto, normalmente si può ovviare agli elif con un dizionario, MA in questo caso specifico basta fare una funzione che converte le lettere nei rispettivi bin, invece che scriverseli tutti a mano.

 

[Pythoner]

Ah bè io mi riferivo al suo codice, poi tanto basta usare bin() e int(num, 2) per convertire da binario a decimale e viceversa.

 

[fojeaf]

Ah bè io mi riferivo al suo codice, poi tanto basta usare bin() e int(num, 2) per convertire da binario a decimale e viceversa.

o farsi una funzione che con *1* ciclo fa tutto