{"action":"created","issue":{"url":"https://api.github.com/repos/MayneKeen/Rubiks_Cube/issues/1","repository_url":"https://api.github.com/repos/MayneKeen/Rubiks_Cube","labels_url":"https://api.github.com/repos/MayneKeen/Rubiks_Cube/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/MayneKeen/Rubiks_Cube/issues/1/comments","events_url":"https://api.github.com/repos/MayneKeen/Rubiks_Cube/issues/1/events","html_url":"https://github.com/MayneKeen/Rubiks_Cube/issues/1","id":306234760,"number":1,"title":"Замечания","user":{"login":"maximpetrov","id":2346982,"avatar_url":"https://avatars0.githubusercontent.com/u/2346982?v=4","gravatar_id":"","url":"https://api.github.com/users/maximpetrov","html_url":"https://github.com/maximpetrov","followers_url":"https://api.github.com/users/maximpetrov/followers","following_url":"https://api.github.com/users/maximpetrov/following{/other_user}","gists_url":"https://api.github.com/users/maximpetrov/gists{/gist_id}","starred_url":"https://api.github.com/users/maximpetrov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maximpetrov/subscriptions","organizations_url":"https://api.github.com/users/maximpetrov/orgs","repos_url":"https://api.github.com/users/maximpetrov/repos","events_url":"https://api.github.com/users/maximpetrov/events{/privacy}","received_events_url":"https://api.github.com/users/maximpetrov/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":8,"created_at":"2018-03-18T12:09:49Z","updated_at":"2018-03-26T15:05:24Z","closed_at":null,"author_association":"NONE","body":"1. Вместо строк для хранения значений цветов лучше использовать `enum`.\r\n1. Хранить временные переменные в полях класса - плохая идея (`whiteCount` и т.д.).\r\n1. Зачем Вам `switch` по `times` в `colourCounter`, если можно просто вычесть `times`?\r\n1. Можно было бы избежать и `switch` по `colour`, если вместо 6 переменных сделать `Map`.\r\n1. Здесь и везде `switch` Вы используете неправильно: если не писать `break` в конце каждого случая, то все последующие также выполнятся. Кстати, фигурные скобки внутри `case` лишние.\r\n1. `randomize()`, даже если правильно воспользоваться `switch`, принципиально неверно пытается собрать кубик: у кубика элементы, соседние через ребро (находящиеся на одном маленьком кубике), не могут быть одного цвета, а Вы специально это делаете.\r\n1. `randomColour` тоже очень странно выглядит. Во-первых, зачем Вы генерируете случайное число от 0 до 99, а потом проверяете его принадлежность к 1 из 6 примерно равных диапазонов? Почему нельзя сразу сгенерировать число от 0 до 5? Во-вторых, рекурсию лучше заменить циклом, иначе можно получить переполнение стека вызовов, если долго не будет получаться нужное число. А вообще, можно сделать так, чтобы случайный цвет выбирался только из доступных.\r\n1. Какой из углов для каждой грани вы считаете за (0,0)? Судя по всему, для разных поворотов Вы считаете по-разному.\r\n1. Не хватает поворотов вокруг третьей оси.\r\n1. Тесты должны использовать `assert...` методы, а не кидать исключения."},"comment":{"url":"https://api.github.com/repos/MayneKeen/Rubiks_Cube/issues/comments/376198834","html_url":"https://github.com/MayneKeen/Rubiks_Cube/issues/1#issuecomment-376198834","issue_url":"https://api.github.com/repos/MayneKeen/Rubiks_Cube/issues/1","id":376198834,"user":{"login":"MayneKeen","id":31849708,"avatar_url":"https://avatars0.githubusercontent.com/u/31849708?v=4","gravatar_id":"","url":"https://api.github.com/users/MayneKeen","html_url":"https://github.com/MayneKeen","followers_url":"https://api.github.com/users/MayneKeen/followers","following_url":"https://api.github.com/users/MayneKeen/following{/other_user}","gists_url":"https://api.github.com/users/MayneKeen/gists{/gist_id}","starred_url":"https://api.github.com/users/MayneKeen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MayneKeen/subscriptions","organizations_url":"https://api.github.com/users/MayneKeen/orgs","repos_url":"https://api.github.com/users/MayneKeen/repos","events_url":"https://api.github.com/users/MayneKeen/events{/privacy}","received_events_url":"https://api.github.com/users/MayneKeen/received_events","type":"User","site_admin":false},"created_at":"2018-03-26T15:05:24Z","updated_at":"2018-03-26T15:05:24Z","author_association":"OWNER","body":"А есть ли смысл еще исправлять этот класс или лучше начать делать следующее задание?"}}
{"actor":{"display_login":"MayneKeen"}}