{"action":"created","issue":{"url":"https://api.github.com/repos/CrazyTosser/Politech/issues/9","repository_url":"https://api.github.com/repos/CrazyTosser/Politech","labels_url":"https://api.github.com/repos/CrazyTosser/Politech/issues/9/labels{/name}","comments_url":"https://api.github.com/repos/CrazyTosser/Politech/issues/9/comments","events_url":"https://api.github.com/repos/CrazyTosser/Politech/issues/9/events","html_url":"https://github.com/CrazyTosser/Politech/issues/9","id":310243005,"number":9,"title":"Upgrade impl. 2.0","user":{"login":"JohnKhandygo","id":2545754,"avatar_url":"https://avatars3.githubusercontent.com/u/2545754?v=4","gravatar_id":"","url":"https://api.github.com/users/JohnKhandygo","html_url":"https://github.com/JohnKhandygo","followers_url":"https://api.github.com/users/JohnKhandygo/followers","following_url":"https://api.github.com/users/JohnKhandygo/following{/other_user}","gists_url":"https://api.github.com/users/JohnKhandygo/gists{/gist_id}","starred_url":"https://api.github.com/users/JohnKhandygo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JohnKhandygo/subscriptions","organizations_url":"https://api.github.com/users/JohnKhandygo/orgs","repos_url":"https://api.github.com/users/JohnKhandygo/repos","events_url":"https://api.github.com/users/JohnKhandygo/events{/privacy}","received_events_url":"https://api.github.com/users/JohnKhandygo/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":10,"created_at":"2018-03-31T14:02:46Z","updated_at":"2018-04-10T19:15:43Z","closed_at":null,"author_association":"NONE","body":"- [x] Уберите ненужные файлы из проекта. Ну серьезно, сколько раз надо повторять?\r\n- [x] Если значение переменной/поля не меняется, лучше использовать `val`, а не `var`.\r\n- [x] Зачем `program` на уровне класса объявлять? Я бы понял, если Вы его переиспользовать собираетесь, но так нет же, Вы его внутри метода инициализируете и используете один раз.\r\n- [x] Секцию `init` лучше писать в самом начала объявления класса. А то я сидел, ломал голову, как это у Вас вообще работает, пока не увидел его где-то в самом низу.\r\n- [x] Какой-то бред вытворяете при парсинге командной строки. Библиотека позволяет получить всю необходимую информацию из командной строки. RTFM, как говорится.\r\n- [x] В `Cli::parse` Вы бросаете исключение для того, чтобы потом его самому же и поймать. Серьезно? \r\n- [x] Вы не правильно выбираете путь, в котором по умолчанию должен происходить поиск. В `user.dir` хранится путь, из которого был запущен java процесс. То есть, если я буду дергать Вашу программу из скрипта, не напрямую, то ничего не получится. \r\n- [x] Не понял, зачем держите в `Program` два метода, если один сводится к вызову второго? \r\n- [x] Вместо того, чтобы использовать `File::listFiles`, лучше воспользовать extension из kotlin [`File::walk`](https://kotlinlang.org/api/latest/jvm/stdlib/kotlin.io/java.io.-file/walk.html). Если правильно сконфигурируете [`FileTreeWalk`](https://kotlinlang.org/api/latest/jvm/stdlib/kotlin.io/-file-tree-walk/index.html), то очень легко получится реализовать логику по рекурсивному обходу. \r\n- [x] Есть еще такой замечательный extension в kotlin [`File::relativeTo`](https://kotlinlang.org/api/latest/jvm/stdlib/kotlin.io/java.io.-file/relative-to.html). По-моему, Вам пригодится.\r\n- [x] Ваша программа вообще ничего не делает, если ее запустить из консоли. На консоль вывод кто будет делать?)\r\n- [x] Нет скриптов сборки/запуска. Я не могу нормально проверить, что Ваша программа действительно работает из командной строки."},"comment":{"url":"https://api.github.com/repos/CrazyTosser/Politech/issues/comments/380216178","html_url":"https://github.com/CrazyTosser/Politech/issues/9#issuecomment-380216178","issue_url":"https://api.github.com/repos/CrazyTosser/Politech/issues/9","id":380216178,"user":{"login":"JohnKhandygo","id":2545754,"avatar_url":"https://avatars3.githubusercontent.com/u/2545754?v=4","gravatar_id":"","url":"https://api.github.com/users/JohnKhandygo","html_url":"https://github.com/JohnKhandygo","followers_url":"https://api.github.com/users/JohnKhandygo/followers","following_url":"https://api.github.com/users/JohnKhandygo/following{/other_user}","gists_url":"https://api.github.com/users/JohnKhandygo/gists{/gist_id}","starred_url":"https://api.github.com/users/JohnKhandygo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JohnKhandygo/subscriptions","organizations_url":"https://api.github.com/users/JohnKhandygo/orgs","repos_url":"https://api.github.com/users/JohnKhandygo/repos","events_url":"https://api.github.com/users/JohnKhandygo/events{/privacy}","received_events_url":"https://api.github.com/users/JohnKhandygo/received_events","type":"User","site_admin":false},"created_at":"2018-04-10T19:15:43Z","updated_at":"2018-04-10T19:15:43Z","author_association":"NONE","body":"Сгодится. Главное, чтобы Вы чему-то научились в конце концов."}}
{"actor":{"display_login":"JohnKhandygo"}}