{"action":"created","issue":{"url":"https://api.github.com/repos/aurusov/rdo-xtext/issues/47","labels_url":"https://api.github.com/repos/aurusov/rdo-xtext/issues/47/labels{/name}","comments_url":"https://api.github.com/repos/aurusov/rdo-xtext/issues/47/comments","events_url":"https://api.github.com/repos/aurusov/rdo-xtext/issues/47/events","html_url":"https://github.com/aurusov/rdo-xtext/issues/47","id":91443161,"number":47,"title":"Code Style","user":{"login":"bogachev-pa","id":6528492,"avatar_url":"https://avatars.githubusercontent.com/u/6528492?v=3","gravatar_id":"","url":"https://api.github.com/users/bogachev-pa","html_url":"https://github.com/bogachev-pa","followers_url":"https://api.github.com/users/bogachev-pa/followers","following_url":"https://api.github.com/users/bogachev-pa/following{/other_user}","gists_url":"https://api.github.com/users/bogachev-pa/gists{/gist_id}","starred_url":"https://api.github.com/users/bogachev-pa/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bogachev-pa/subscriptions","organizations_url":"https://api.github.com/users/bogachev-pa/orgs","repos_url":"https://api.github.com/users/bogachev-pa/repos","events_url":"https://api.github.com/users/bogachev-pa/events{/privacy}","received_events_url":"https://api.github.com/users/bogachev-pa/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-06-27T10:24:28Z","updated_at":"2015-06-27T10:43:34Z","closed_at":null,"body":"@aurusov, @k-alexandrovsky, @LeKaitoW, @kstrizhov\r\n\r\nНачало обсуждения тут: https://github.com/aurusov/rdo-xtext/issues/46, кто не видел, почитайте.\r\n\r\nТаска про обсуждение код-стайла.\r\nУже обсуждали форматированием, там остановились на дефолтном форматере.\r\nНо эклипс также позволяет автоматически править и код-стайл. В частности добавлять `final` где нужно, добавлять автоматические `@Override`, расставлять фигурные скобки в `if` блоках и тому подобное.\r\n\r\nНастройки по умолчанию такие:\r\n```\r\nChange non static accesses to static members using declaring type\r\nChange indirect accesses to static members to direct accesses (accesses through subtypes)\r\nRemove unused imports\r\nAdd missing '@Override' annotations\r\nAdd missing '@Override' annotations to implementations of interface methods\r\nAdd missing '@Deprecated' annotations\r\nRemove unnecessary casts\r\nRemove unnecessary '$NON-NLS$' tags\r\n```\r\n\r\nВопрос в том, хотим ли мы автоматизировать такие вещи как:\r\n- добавление `final` к константным полям класса\r\n- добавление `final` к константым параметрам методов\r\n- добавление фигурных скобок вокруг всех конструкций типа `if`, `for` и т.п.\r\n- т.п.\r\n\r\nМое мнение такое: не нужно делать это обязательным к использованию, но кто хочет, почему бы и не пользоваться настроенным код стайлером. Это ускорит код ревью и все такое. И, конечно, если использовать свой стиль, ничего из стиля по умолчанию убирать нельзя, только добавлять новые пункты. Т.е. свой стиль должен быть как минимум не менее строгий, чем дефолтный.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/aurusov/rdo-xtext/issues/comments/116003757","html_url":"https://github.com/aurusov/rdo-xtext/issues/47#issuecomment-116003757","issue_url":"https://api.github.com/repos/aurusov/rdo-xtext/issues/47","id":116003757,"user":{"login":"aurusov","id":5585202,"avatar_url":"https://avatars.githubusercontent.com/u/5585202?v=3","gravatar_id":"","url":"https://api.github.com/users/aurusov","html_url":"https://github.com/aurusov","followers_url":"https://api.github.com/users/aurusov/followers","following_url":"https://api.github.com/users/aurusov/following{/other_user}","gists_url":"https://api.github.com/users/aurusov/gists{/gist_id}","starred_url":"https://api.github.com/users/aurusov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aurusov/subscriptions","organizations_url":"https://api.github.com/users/aurusov/orgs","repos_url":"https://api.github.com/users/aurusov/repos","events_url":"https://api.github.com/users/aurusov/events{/privacy}","received_events_url":"https://api.github.com/users/aurusov/received_events","type":"User","site_admin":false},"created_at":"2015-06-27T10:43:34Z","updated_at":"2015-06-27T10:43:34Z","body":">Мое мнение такое: не нужно делать это обязательным к использованию, но кто хочет, почему бы и не пользоваться настроенным код стайлером.\r\n\r\nНе, всем надо будет настроить эклипс на единый стиль. Мы же можем попроваить переносы строк и т.д."}}