{"action":"opened","issue":{"url":"https://api.github.com/repos/l1feforce/RubicksCube/issues/3","repository_url":"https://api.github.com/repos/l1feforce/RubicksCube","labels_url":"https://api.github.com/repos/l1feforce/RubicksCube/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/l1feforce/RubicksCube/issues/3/comments","events_url":"https://api.github.com/repos/l1feforce/RubicksCube/issues/3/events","html_url":"https://github.com/l1feforce/RubicksCube/issues/3","id":310238827,"number":3,"title":"Upgrade impl.","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":0,"created_at":"2018-03-31T12:57:20Z","updated_at":"2018-03-31T12:57:20Z","closed_at":null,"author_association":"NONE","body":"У Вас получилось на удивление аккуратный код. Постарайтесь поддерживать его в таком состоянии, пока будете исправлять замечания. Javadoc тоже порадовал. Собственно замечания:\r\n- [ ] Уберите ненужные классы. \r\n- [ ] По условию задачи необходимо было реализовать кубик произвольного размера. Насколько я понял, Вы поддерживаете только вариант 3x3x3.\r\n- [ ] По условиям задачи должен быть метод, который предоставляет состояние произвольной грани. \r\n- [ ] По условиям задачи должен быть метода поворота всего кубика в пространстве (представьте, что Вы просто повернули кубик другой гранью к себе). \r\n- [ ] Подумайте, может ли метод `getCube` что-то \"сломать\" в Вашей реализации. То же самое касается метода `setCube`.\r\n- [ ] Для того, чтобы воспользоваться методом `generate` мне необходимо сначала создать экземпляр `Cube` и затем вызвать этот метод. Несколько странно, учитывая, что первый экземпляр больше никогда не понадобится. Суть в том, что для метода `generate` не важно состояние самого объекта `Cube`, поэтому имеет смысл сделать его статическим. \r\n- [ ] То же самое касается метода `randomize`. Либо уж действительно рандомизируйте имеющийся кубик.\r\n- [ ] Вместо того, чтобы каждый раз писать `Arrays.asList(RIGHT, LOWER, LEFT, UPPER, BACK, FRONT)`, можно использовать метод `values()`, которые автоматически генерируется для всех `enum` классов, и возвращает массив (не список) всех элементов `enum` класса.\r\n- [ ] В коде не должно быть комментариев, отличных от javadoc. Если Вам приходится писать комментарии для того, чтобы Ваш код был понятен, то это плохой код."}}
{"actor":{"display_login":"JohnKhandygo"}}