{"action":"opened","issue":{"url":"https://api.github.com/repos/nekosyakolya/Basics-of-project-work/issues/1","repository_url":"https://api.github.com/repos/nekosyakolya/Basics-of-project-work","labels_url":"https://api.github.com/repos/nekosyakolya/Basics-of-project-work/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/nekosyakolya/Basics-of-project-work/issues/1/comments","events_url":"https://api.github.com/repos/nekosyakolya/Basics-of-project-work/issues/1/events","html_url":"https://github.com/nekosyakolya/Basics-of-project-work/issues/1","id":182203505,"number":1,"title":"Замечания по 1.1","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":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-10-11T08:25:04Z","updated_at":"2016-10-11T08:25:04Z","closed_at":null,"body":"- подкаталог ipch следует убрать из-под контроля версий Git\r\n- подкаталог .vs также следует убрать\r\n- то же касается подкаталогов Debug\r\n\r\n- использовать `#define G 9.8f` не следует, [такой способ объявления констант считается устаревшим](http://stackoverflow.com/questions/1674032/static-const-vs-define-vs-enum)\r\n- после `using namespace std;` лучше оставить пустую строку\r\n- к глаголу лучше добавить окончание `ed` или приставочное слово `did`: `bool printMaxState = false;`\r\n\r\nЭто очень странное выражение:\r\n```cpp\r\nbool validityHeight;\r\n((heightJump <= 0) || (heightJump > INT_MAX)) ? validityHeight = false : validityHeight = true;\r\nreturn validityHeight;\r\n```\r\nВ чём смысл выполнять присваивание в тернарном операторе? Зачем в зависимости от истинности или ложности выражения выбирать false/true соответственно, если есть оператор отрицания?\r\n\r\n- Название `IsValidityHeight` лучше сменить на `IsValidHeight`\r\n- Вызов функции Pause и саму функцию следует убрать, в консольном приложении они создают лишнюю проблему (необходимость ввода чего-либо для закрытия приложения). При запуске из Visual Studio есть сочетание Ctrl+F5, в котором эмулятор консоли будет ждать нажатия клавиши после завершения приложения."}}
{"actor":{"display_login":"sergey-shambir"}}