Бусловский - 3 tasks competed (encrypted, decrypted, bruteforce)#22
Бусловский - 3 tasks competed (encrypted, decrypted, bruteforce)#22Kirill-Buslovskiy wants to merge 10 commits intodemologin:mainfrom
Conversation
add psvm
add alphabet and scanner
add methods read and write
# Conflicts: # src/main/java/com/javarush/buslovkii/Runner.java
Kirill-Buslovskiy
changed the title
demologin
left a comment
demologin
left a comment
There was a problem hiding this comment.
Решено в виде консольного предложения с тремя методами из четырёх возможных, однако практически нет никаких замечаний, поставил оценку B
| private final CaesarCipher cipher = new CaesarCipher(); | ||
| private final BruteForce bruteForce = new BruteForce(); | ||
|
|
||
| private static final String SOURCE_FILE = "C:/Users/ms/IdeaProjects/CryptoAnalyzerPantera/text/text.txt"; |
There was a problem hiding this comment.
Этот код не будет работать ни на одном компьютере кроме компьютера разработчика. Абсолютные пути никогда нельзя использовать в прикладных программах
| private static final String ALPHABET = "абвгдеёжзийклмнопрстуфхцчшщъыьэюя.,\"':-!?"; | ||
|
|
||
| public String process(String text, int key, boolean encrypt) { | ||
| StringBuilder result = new StringBuilder(); |
There was a problem hiding this comment.
процедура расшифрования и дешифрования производится целиком в памяти, если бы обрабатывал байтовый поток из одного файла в другой файл тогда не было бы никаких ограничений на размер этой памяти
| private int evaluateText(String text) { | ||
| int score = 0; | ||
|
|
||
| if (text.matches(".*[.,]\\s[а-яА-ЯЁё].*")) { |
There was a problem hiding this comment.
этот шаблон лучше вынести куда-то в константу и осмысленно её назвать
| } | ||
| } | ||
|
|
||
| System.out.println("Brute Force: Наиболее вероятный ключ: " + bestKey); |
There was a problem hiding this comment.
когда вывод находится прямо в методе это всегда плохо потому что заменить такой вывод на мобильный или десктопный будет очень непросто
2 participants
No description provided.