[h00k]

Napisałem sobie prosty program do szyfru Vigenere w #python
Jakby komuś z was chciało się zrobić #codereview to bardzo chętnie bym wysłuchał/

link do repozytorium

#programowanie trochę #kryptografia

[s.....e]

@h00k:
##############################
# XD
##############################

[L.....k]

Poczytaj o https://docs.python.org/3.8/library/argparse.html

[a231]

@h00k: zdecydowanie zobacz jak działa argParse bo się później przyda

[ProfesorBigos]

@h00k: Hej. Jeśli chodzi o wczytywanie argumentów z linii poleceń, to używa się https://docs.python.org/3/library/argparse.html Powtarzasz kod do budowy tablicy do szyfrowania w obu plikach, lepiej go napisać raz i zaimportować tu i tu. A jeszcze lepiej importować gotowy wynik tego kodu, żeby nie budować tej samej tablicy przy każdym wywołaniu funkcji. Poczytaj trochę o list comprehensions i pomyśl jak ich użyć w twoim kodzie. Nie wiem czemu zostawiasz pustą linię

[ProfesorBigos]

@h00k: Aha i zastanów się, czy używanie dict zamiast list nie uprościłoby Ci trochę programu.

[asterix61]

@h00k: Czekam na info o aktualizacji kodu. Na ten moment dostałeś dużo uwag, i nie wiem, czy jest sens dorzucać kolejne. Generalnie kod wygląda bardzo niskopoziomowo

[b.....t]

@h00k: nie widzę nigdzie operacji modulo
http://www.algorytm.org/kryptografia/szyfr-cezara/przesun-1-py.html

[b.....t]

@h00k: nie warto tworzyć własne implementacje czegoś co ma już implementacje modelowe, bo tylko się nauczysz źle
https://www.geeksforgeeks.org/vigenere-cipher/