{"action":"created","issue":{"url":"https://api.github.com/repos/tomudding/Player-Visibility/issues/2","repository_url":"https://api.github.com/repos/tomudding/Player-Visibility","labels_url":"https://api.github.com/repos/tomudding/Player-Visibility/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/tomudding/Player-Visibility/issues/2/comments","events_url":"https://api.github.com/repos/tomudding/Player-Visibility/issues/2/events","html_url":"https://github.com/tomudding/Player-Visibility/issues/2","id":184698538,"number":2,"title":"Code Readability ","user":{"login":"bramhaag","id":5541521,"avatar_url":"https://avatars.githubusercontent.com/u/5541521?v=3","gravatar_id":"","url":"https://api.github.com/users/bramhaag","html_url":"https://github.com/bramhaag","followers_url":"https://api.github.com/users/bramhaag/followers","following_url":"https://api.github.com/users/bramhaag/following{/other_user}","gists_url":"https://api.github.com/users/bramhaag/gists{/gist_id}","starred_url":"https://api.github.com/users/bramhaag/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bramhaag/subscriptions","organizations_url":"https://api.github.com/users/bramhaag/orgs","repos_url":"https://api.github.com/users/bramhaag/repos","events_url":"https://api.github.com/users/bramhaag/events{/privacy}","received_events_url":"https://api.github.com/users/bramhaag/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-10-23T16:07:49Z","updated_at":"2016-10-23T17:22:00Z","closed_at":null,"body":"A quick glance over the [Commands ](https://github.com/tomudding/Player-Visibility/blob/master/bukkit/src/nl/tomudding/plugins/visibility/commands/Commands.java) class reveals an issue. The readability of the code.\r\n\r\nTo be more specific: nested if-statements (if statements inside of an if statement). It's extremely messy, but we've all nested if statements in the past. \r\n\r\nI recently came across [this video](https://www.youtube.com/watch?v=ldqDpmMkXgw) by codecourse. He gives great examples in this vide. While the language he shows this in is PHP, it also applies to Java and other programming languages. \r\n\r\nI'd recommend preventing nested if-statements as much as possible."},"comment":{"url":"https://api.github.com/repos/tomudding/Player-Visibility/issues/comments/255601234","html_url":"https://github.com/tomudding/Player-Visibility/issues/2#issuecomment-255601234","issue_url":"https://api.github.com/repos/tomudding/Player-Visibility/issues/2","id":255601234,"user":{"login":"tomudding","id":4983571,"avatar_url":"https://avatars.githubusercontent.com/u/4983571?v=3","gravatar_id":"","url":"https://api.github.com/users/tomudding","html_url":"https://github.com/tomudding","followers_url":"https://api.github.com/users/tomudding/followers","following_url":"https://api.github.com/users/tomudding/following{/other_user}","gists_url":"https://api.github.com/users/tomudding/gists{/gist_id}","starred_url":"https://api.github.com/users/tomudding/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomudding/subscriptions","organizations_url":"https://api.github.com/users/tomudding/orgs","repos_url":"https://api.github.com/users/tomudding/repos","events_url":"https://api.github.com/users/tomudding/events{/privacy}","received_events_url":"https://api.github.com/users/tomudding/received_events","type":"User","site_admin":false},"created_at":"2016-10-23T17:22:00Z","updated_at":"2016-10-23T17:22:00Z","body":"I'll (try to) clean up the code with the next update."}}
{"actor":{"display_login":"tomudding"}}