Conversation
| } | ||
|
|
||
| private static void digestFile(MessageDigest digest, Path file) throws IOException { | ||
| FileInputStream fin = new FileInputStream(file.toString()); |
| digestFile(messageDigest, path); | ||
| } | ||
| } catch (IOException e) { | ||
| Error e1 = new Error(); |
There was a problem hiding this comment.
Error не стоит кидать самому, это исключения, которые возникают, когда заканчивается оперативная память или что-то равноценно ужасное, на что мы не можем никак повлиять. то, что на диске файл не нашёлся, в эту категорию явно не попадает
There was a problem hiding this comment.
вполне можно было обойтись обычным циклом по Files.walk(filePath) и не париться с исключениями в внутри лямбды
|
|
||
| private static void digestFile(MessageDigest digest, Path file) throws IOException { | ||
| FileInputStream fin = new FileInputStream(file.toString()); | ||
| boolean read = false; |
There was a problem hiding this comment.
булевы переменные обычно называют, начиная с is
| digestFile(digest, curPath); | ||
| } else { | ||
| int lastNameIndex = curPath.getNameCount() - 1; | ||
| digest.update(curPath.getName(lastNameIndex). |
| protected void compute() { | ||
| try { | ||
| if (Files.isDirectory(path)) { | ||
| int lastNameIndex = path.getNameCount() - 1; |
There was a problem hiding this comment.
эти две строчки вполне можно было не раскопировать и выделить в отдельный метод
|
|
||
| public class Main { | ||
| public static void main(String[] args) { | ||
| switch (args[0]) { |
There was a problem hiding this comment.
хорошо бы проверить, что аргумент есть вообще
| import java.security.NoSuchAlgorithmException; | ||
| import java.util.Arrays; | ||
|
|
||
| public class Main { |
There was a problem hiding this comment.
по-хорошему доки нужно писать ко всему публичному
| case "multithread": { | ||
| System.out.println("Calculating MD5 hash for " + args[1] + "..."); | ||
| try { | ||
| System.out.println(new BigInteger( |
|
5 баллов |
2 participants
No description provided.