{"action":"created","issue":{"url":"https://api.github.com/repos/Alignak-monitoring/alignak/issues/81","labels_url":"https://api.github.com/repos/Alignak-monitoring/alignak/issues/81/labels{/name}","comments_url":"https://api.github.com/repos/Alignak-monitoring/alignak/issues/81/comments","events_url":"https://api.github.com/repos/Alignak-monitoring/alignak/issues/81/events","html_url":"https://github.com/Alignak-monitoring/alignak/issues/81","id":88427141,"number":81,"title":"private functions of class in top of class (it's cosmetics & better to read the class)","user":{"login":"ddurieux","id":239744,"avatar_url":"https://avatars.githubusercontent.com/u/239744?v=3","gravatar_id":"","url":"https://api.github.com/users/ddurieux","html_url":"https://github.com/ddurieux","followers_url":"https://api.github.com/users/ddurieux/followers","following_url":"https://api.github.com/users/ddurieux/following{/other_user}","gists_url":"https://api.github.com/users/ddurieux/gists{/gist_id}","starred_url":"https://api.github.com/users/ddurieux/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ddurieux/subscriptions","organizations_url":"https://api.github.com/users/ddurieux/orgs","repos_url":"https://api.github.com/users/ddurieux/repos","events_url":"https://api.github.com/users/ddurieux/events{/privacy}","received_events_url":"https://api.github.com/users/ddurieux/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Alignak-monitoring/alignak/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-06-15T13:27:40Z","updated_at":"2015-08-10T16:34:25Z","closed_at":null,"body":"For better read of code, put all private functions (like __str__) immediately after class declaration avec __init__ .\r\n\r\nWe have some classes where we have these functions anywhere in class"},"comment":{"url":"https://api.github.com/repos/Alignak-monitoring/alignak/issues/comments/129519158","html_url":"https://github.com/Alignak-monitoring/alignak/issues/81#issuecomment-129519158","issue_url":"https://api.github.com/repos/Alignak-monitoring/alignak/issues/81","id":129519158,"user":{"login":"gst","id":38055,"avatar_url":"https://avatars.githubusercontent.com/u/38055?v=3","gravatar_id":"","url":"https://api.github.com/users/gst","html_url":"https://github.com/gst","followers_url":"https://api.github.com/users/gst/followers","following_url":"https://api.github.com/users/gst/following{/other_user}","gists_url":"https://api.github.com/users/gst/gists{/gist_id}","starred_url":"https://api.github.com/users/gst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gst/subscriptions","organizations_url":"https://api.github.com/users/gst/orgs","repos_url":"https://api.github.com/users/gst/repos","events_url":"https://api.github.com/users/gst/events{/privacy}","received_events_url":"https://api.github.com/users/gst/received_events","type":"User","site_admin":false},"created_at":"2015-08-10T16:34:25Z","updated_at":"2015-08-10T16:34:25Z","body":"I agree it's better (for readability and understand-ability of a class behavior) to keep most of those little (mostly usually) functions near the top of a class definition.\r\n\r\nwe have no enforced nor defined rule about that I think; \r\n\r\nBy the way aren't there some pylint / pep8 rule(s) about this kind of \"code placement\" ??\r\n\r\nNow this isn't a fun task at all.."}}