{"comment":{"url":"https://api.github.com/repos/IvanRud95/Ivanrepository/comments/30352049","html_url":"https://github.com/IvanRud95/Ivanrepository/commit/0edfceb128850135ba772eac1eda2b641b165f9a#commitcomment-30352049","id":30352049,"node_id":"MDEzOkNvbW1pdENvbW1lbnQzMDM1MjA0OQ==","user":{"login":"max-underthesun","id":13752607,"node_id":"MDQ6VXNlcjEzNzUyNjA3","avatar_url":"https://avatars3.githubusercontent.com/u/13752607?v=4","gravatar_id":"","url":"https://api.github.com/users/max-underthesun","html_url":"https://github.com/max-underthesun","followers_url":"https://api.github.com/users/max-underthesun/followers","following_url":"https://api.github.com/users/max-underthesun/following{/other_user}","gists_url":"https://api.github.com/users/max-underthesun/gists{/gist_id}","starred_url":"https://api.github.com/users/max-underthesun/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/max-underthesun/subscriptions","organizations_url":"https://api.github.com/users/max-underthesun/orgs","repos_url":"https://api.github.com/users/max-underthesun/repos","events_url":"https://api.github.com/users/max-underthesun/events{/privacy}","received_events_url":"https://api.github.com/users/max-underthesun/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":null,"commit_id":"0edfceb128850135ba772eac1eda2b641b165f9a","created_at":"2018-08-31T05:04:06Z","updated_at":"2018-08-31T05:04:06Z","author_association":"NONE","body":"замечания по коду следующие:\r\n1. ты опять копипастишь код на гитхаб. вместо пуша через гит\r\n2. снова проблемы с отступами в нескольких классах (видимо это следствие предыдущего пункта)\r\n3. программа не запускается:\r\n    ```\r\n    $ ruby main.rb \r\n    main.rb:87: syntax error, unexpected end-of-input, expecting keyword_end\r\n    ```\r\n4. судя по методам \"инишиалайз\" в классах КаргоТрейн и ПассанджерТрейн ты не очень хорошо понимаешь что такое аргументы метода - эти методы написаны с ошибкой и код работать не будет (попробуй разобраться с тем, что такое аргументы - будут вопросы - задавай в слаке в личку)\r\n5. КаргоВэгон: инишиалайз не должен принимать аргумент \"тайп\" совсем, даже с дефолтным значением, потому что для этого класса \"тайп\" не может меняться\r\n6. ПэссенджерВэгон: строка номер 2 меня очень интересует - объясни пожалуйста что она делает?\r\n7. Оба подкласса КаргоВэгон и ПассанджерВэгон наследуют от класса Вэгон... но тут есть одна проблемма - у тебя нет класса Вэгон (также оба класса обращаются к родителю в инишиалайзе)\r\n8. в файле main.rb код \"висит в воздухе\"... будет гораздо лучше, если интерфейс сделать в объектном стиле:\r\n    - сделать класс, у которого в конструкторе будут инициализироваться все данные (сейчас они как-то в вакууме инициализируются)\r\n    - этот класс будет иметь один публичный метод (например  `start`)\r\n    - этот метод запускает цикл меню, а все остальные методы сделать приватными\r\n9. кода в файле main.rb явно мало.... в меню перечислено 8 пунктов, а хоть как-то реализовано 5 и эта реализация вызывает сомнения\r\n10. лучше разделить логику и интерфейс, т.е. логику оставить в том же классе Мейн, а все инпуты и аутпуты вынести в отдельный класс, к примеру Интерфейс... соотвественно в методах класса Мейн обращаться к интерфейсу примерно так: `@interface.show_main_menu` и т.п.\r\n\r\n"}}
{"actor":{"display_login":"max-underthesun"}}