{"action":"closed","issue":{"url":"https://api.github.com/repos/AnatolyKorablyov/TP_lab7.2/issues/1","labels_url":"https://api.github.com/repos/AnatolyKorablyov/TP_lab7.2/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/AnatolyKorablyov/TP_lab7.2/issues/1/comments","events_url":"https://api.github.com/repos/AnatolyKorablyov/TP_lab7.2/issues/1/events","html_url":"https://github.com/AnatolyKorablyov/TP_lab7.2/issues/1","id":120534694,"number":1,"title":"Замечания по 7.2","user":{"login":"sergey-shambir","id":1856842,"avatar_url":"https://avatars.githubusercontent.com/u/1856842?v=3","gravatar_id":"","url":"https://api.github.com/users/sergey-shambir","html_url":"https://github.com/sergey-shambir","followers_url":"https://api.github.com/users/sergey-shambir/followers","following_url":"https://api.github.com/users/sergey-shambir/following{/other_user}","gists_url":"https://api.github.com/users/sergey-shambir/gists{/gist_id}","starred_url":"https://api.github.com/users/sergey-shambir/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sergey-shambir/subscriptions","organizations_url":"https://api.github.com/users/sergey-shambir/orgs","repos_url":"https://api.github.com/users/sergey-shambir/repos","events_url":"https://api.github.com/users/sergey-shambir/events{/privacy}","received_events_url":"https://api.github.com/users/sergey-shambir/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-12-05T06:26:40Z","updated_at":"2015-12-05T23:15:34Z","closed_at":"2015-12-05T23:15:34Z","body":"- в ```const int HEIGHT = 50, DEPTH = 20;``` две константы объявлены на одной строке, не надо так\r\n- желательно глобальные константы помечать словом ```static```, чтобы другие константы с таким же именем в других файлах не могли вызвать ошибок компоновки (linker errors)\r\n- вместо ```std::list``` лучше использовать ```std::vector``` \r\n- для обхода что std::list, что std::vector хорошо подходит range for: http://en.cppreference.com/w/cpp/language/range-for\r\n- в структуре ```init_shapes``` 4 поля объявлены на одной строке\r\n- в функции ```start_move_time``` список лучше передавать по константной ссылке, иначе будет происходить постоянное копирование из одной области памяти в другую\r\n- переменные следует объявлять непосредственно перед их использованием, в функции ```start_move_time``` это соблюдается для ```time```, но не соблюдается для ```count``` (её можно объявить там же, где она инициализируется нулём)"}}