Kod cezara problem

Kod cezara problem
maxmiks
MA
maxmiks
MA
  • Rejestracja:ponad 7 lat
  • Ostatnio:ponad 6 lat
  • Postów:72
0

Mam tutaj taki kod do szyfru cezara, który najpierw przygotowuje tekst do szyfrowania zostawiając jedynie małe litery. Coś nie chce mi działać i podejrzewam, że problem może leżeć w linijce 17. Np. dla przesunięcia równego 5 wyświetla Stsj bez względu na wprowadzony tekst. Nie wiem czy w taki sposób mogę uznać tekst2 jako string. Dodam, że nie chcę raczej zmieniać kodu, ponieważ chodzi mi o to, aby był dla mnie zrozumiały. Pomocy ;D

Kopiuj
def przygotowanie(tekst):
    dlugosc=len(tekst)
    for i in range(0, dlugosc):
        if (ord(tekst[i])>=97 and ord(tekst[i])<=122):
            print(tekst[i], end='')
        elif (ord(tekst[i])>=65 and ord(tekst[i])<=90):
            print(chr(ord(tekst[i])+32), end='')
        else:
            print(end='')        
            
def cezar(tekst, przesuniecie):
    dlugosc1=len(tekst)
    for i in range (0, dlugosc1):
        print(chr(ord(tekst[i])+przesuniecie), end='')
        
tekst1=str(input("Wprowadz tekst: "))
tekst2=przygotowanie(tekst1)
x=int(input("Wprowadz wartosc przesuniecia (od 1 do 26): "))
cezar(tekst2, x)
Udostępnij
Kopiuj link do postu Kopiuj link do postu jako Markdown
Komentuj
Guaz
Guaz
Guaz
Guaz
  • Rejestracja:około 8 lat
  • Ostatnio:ponad 4 lata
  • Lokalizacja:Częstochowa
  • Postów:221
1

W sumie to ciężko wymienić wszystkie błędy. Ale jest jeden kluczowy:

Kopiuj
def przygotowanie(tekst):
    dlugosc=len(tekst)
    for i in range(0, dlugosc):
        if (ord(tekst[i])>=97 and ord(tekst[i])<=122):
            print(tekst[i], end='')
        elif (ord(tekst[i])>=65 and ord(tekst[i])<=90):
            print(chr(ord(tekst[i])+32), end='')
        else:
            print(end='')
    return tekst #< ===== MUSISZ zwrócić wartość którą przypisujesz na dole do zmiennej, inaczej funkcja zwróci None. A wartość None, nie ma długości którą próbujesz odczytać:

def cezar(tekst, przesuniecie):
    dlugosc1=len(tekst) # < ========= Tutaj. Jako zmienna tekst przekazujesz None.
    for i in range (0, dlugosc1):
        print(chr(ord(tekst[i])+przesuniecie), end='')

tekst1=str(input("Wprowadz tekst: "))
tekst2=przygotowanie(tekst1) # < ==== Przez brak returna, tutaj właśnie przypisujesz None
x=int(input("Wprowadz wartosc przesuniecia (od 1 do 26): "))
cezar(tekst2, x) # < ========= A tu tego None przekazujesz do funkcji cezar.

Przy okazji, unikaj printów w funkcjach, lepiej je robić poza nimi chyba że debugujesz. Poza tym, nie zmieniasz elementów tekst[i] w funkcji przygotowanie. Tylko drukujesz na ekran po przesunięciu.
Aby zmienić dane to musisz zrobić deklarację typu:

Kopiuj
tekst[i] = chr( ord( tekst[i] )+32 )
#Zwiększasz wartość ord'a, a następnie zamieniasz na char'a. (w sposób lekko naiwny w przypadku np. "Z")
#Ale chodzi tu głównie o to, aby go w stringu podmienić tak jak w powyższej logice (bo ten przykład nie zadziała, chodzi o zobrazowanie), chociaż można to zrobić lepiej, szybciej, generatorem i zrozumialej, niemniej prosiłeś by kod został czytelny.
#To błędy kluczowe dla działania.

Z tym że nie zmieniasz tu o wartość przesunięcia, ale o stałą wartość, co ci uniemożliwi rozczytanie bez wczytania słownika w danym języku i dobrej funkcji sprawdzającej :P.

Wybacz za krytykę, ale mam nadzieję że pomoże :). Jeśli będziesz potrzebował dalszej pomocy, to pisz śmiało, nie gryziemy, tylko chcemy wpoić dobre nawyki ;d

@Edit:

Kopiuj
Wprowadz tekst: aZsG
azsg
Wprowadz wartosc przesuniecia (od 1 do 26): 1
bath

Kodem:

Kopiuj
def cezar(tekst, przesuniecie):
    tekst = list(tekst)
    for i in range (len(tekst)):
        temp = ord(tekst[i])
        while temp+przesuniecie > 122:
            temp = 96+(temp+przesuniecie-123)
        tekst[i] = chr(temp+przesuniecie)

        #~ print(chr(ord(tekst[i])+przesuniecie), end='')
    tekst = "".join(i for i in tekst)
    return tekst

tekst1="".join(str(input("Wprowadz tekst: "))).lower()
print(tekst1)

x=int(input("Wprowadz wartosc przesuniecia (od 1 do 26): "))
print( cezar(tekst1, x) )

Starałem się jak najmniej ingerować w twój kod, z wyjątkiem kilku kruczków. (jak "".join) powinno działać, spróbuj zrozumieć logikę programu i skorygować swoje błędy :).

Linux Mint
Arduino / Python 3.5.2
edytowany 3x, ostatnio: Guaz
MA
Dzięki wielkie za pomoc :D
Udostępnij
Kopiuj link do postu Kopiuj link do postu jako Markdown
Komentuj
vpiotr
vpiotr
vpiotr
vpiotr
  • Rejestracja:ponad 13 lat
  • Ostatnio:prawie 3 lata
1

Ludzie to liczyli na tabliczkach glinianych, jaki może być problem z szyfrem Cezara?
Może przyda się to:

Udostępnij
Kopiuj link do postu Kopiuj link do postu jako Markdown
Komentuj
maxmiks
MA
maxmiks
MA
  • Rejestracja:ponad 7 lat
  • Ostatnio:ponad 6 lat
  • Postów:72
0

Mam coś takiego, ale nie wiem co zrobić ze spacjami i innymi znakami, ponieważ jak daję remove() to poźniej wyskakuje mi błąd: list assignment index out of range
Czyli mam za mało elementów w tablicy. Jak rozwiązać problem? :D

Udostępnij
Kopiuj link do postu Kopiuj link do postu jako Markdown
Komentuj
Guaz
Guaz
Guaz
Guaz
  • Rejestracja:około 8 lat
  • Ostatnio:ponad 4 lata
  • Lokalizacja:Częstochowa
  • Postów:221
0

Zgaduje że usuwasz spacje po stworzeniu instrukcji która biegnie po zasięgu stringa posiadającym spacje. Co wywołuje index error gdy skrócisz string'a.
Więc polecam ci wykorzystać dodatkowo instrukcję try albo tylko while zamiast for'a.

Linux Mint
Arduino / Python 3.5.2
MA
while i<=długość czy jakoś inaczej?
Guaz
Dobry wstęp, tylko musisz opamiętać indeksy :).
Udostępnij
Kopiuj link do postu Kopiuj link do postu jako Markdown
Komentuj
maxmiks
MA
maxmiks
MA
  • Rejestracja:ponad 7 lat
  • Ostatnio:ponad 6 lat
  • Postów:72
0

moj kod

Kopiuj
def przygotowanie(tekst):
    dlugosc=len(tekst)
    tekstKoncowy=list(tekst)
    i=0
    while i<dlugosc:
        if (ord(tekst[i])>=97 and ord(tekst[i])<=122):
            tekstKoncowy[i]=tekst[i]
        elif (ord(tekst[i])>=65 and ord(tekst[i])<=90):
            tekstKoncowy[i]=chr(ord(tekst[i])+32)
        else:
            tekstKoncowy.remove(tekstKoncowy[i])
    return tekstKoncowy
Udostępnij
Kopiuj link do postu Kopiuj link do postu jako Markdown
Komentuj
Guaz
Guaz
Guaz
Guaz
  • Rejestracja:około 8 lat
  • Ostatnio:ponad 4 lata
  • Lokalizacja:Częstochowa
  • Postów:221
1

Wstępnie tak, ale musisz jeszcze w tej pętli zwiększać i, a jeśli usuwasz znak z tekstu początkowego, to nasze i wzrośnie, a znak następujący będzie poraz kolejny na miejscu naszego znaku który usuwaliśmy. To jest problem logiczny, nie braku wiedzy, więc sam musisz go rozwiązać, narzędzia do tego już wszystkie w kodzie masz, z wyjątkiem jednej linijki i+=1.
Powodzenia :)

Linux Mint
Arduino / Python 3.5.2
Udostępnij
Kopiuj link do postu Kopiuj link do postu jako Markdown
Komentuj
Leszek Seretny
Leszek Seretny
Leszek Seretny
Leszek Seretny
  • Rejestracja:ponad 8 lat
  • Ostatnio:ponad 4 lata
  • Postów:3
0

Tak problem logiczny :) czyli jak interpretować 0lub1 też OR no

Udostępnij
Kopiuj link do postu Kopiuj link do postu jako Markdown
Komentuj
maxmiks
MA
maxmiks
MA
  • Rejestracja:ponad 7 lat
  • Ostatnio:ponad 6 lat
  • Postów:72
0

W końcu się udało :) Co prawda z trochę innej strony, ale jest.

Kopiuj
def przygotowanie(tekst):
    dlugosc=len(tekst)
    tekstKoncowy=[]
    for i in range(0, dlugosc):
        if (ord(tekst[i])>=97 and ord(tekst[i])<=122):
            tekstKoncowy.append(tekst[i])
        elif (ord(tekst[i])>=65 and ord(tekst[i])<=90):
            tekstKoncowy.append(chr(ord(tekst[i])+32))
        else:
            print(end='')
    return tekstKoncowy

Dzięki za pomoc :D

edytowany 1x, ostatnio: maxmiks
enedil
No i po co ten indeks Tobie? Mogłaś for znak in tekst
enedil
A ten else z pustym printem? Na co on?
Udostępnij
Kopiuj link do postu Kopiuj link do postu jako Markdown
Komentuj

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.

Utwórz konto