{"action":"created","issue":{"url":"https://api.github.com/repos/askmike/gekko/issues/261","labels_url":"https://api.github.com/repos/askmike/gekko/issues/261/labels{/name}","comments_url":"https://api.github.com/repos/askmike/gekko/issues/261/comments","events_url":"https://api.github.com/repos/askmike/gekko/issues/261/events","html_url":"https://github.com/askmike/gekko/pull/261","id":54359061,"number":261,"title":"Small Performance Optmizations","user":{"login":"fsbatista","id":7033810,"avatar_url":"https://avatars.githubusercontent.com/u/7033810?v=3","gravatar_id":"","url":"https://api.github.com/users/fsbatista","html_url":"https://github.com/fsbatista","followers_url":"https://api.github.com/users/fsbatista/followers","following_url":"https://api.github.com/users/fsbatista/following{/other_user}","gists_url":"https://api.github.com/users/fsbatista/gists{/gist_id}","starred_url":"https://api.github.com/users/fsbatista/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fsbatista/subscriptions","organizations_url":"https://api.github.com/users/fsbatista/orgs","repos_url":"https://api.github.com/users/fsbatista/repos","events_url":"https://api.github.com/users/fsbatista/events{/privacy}","received_events_url":"https://api.github.com/users/fsbatista/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-01-14T18:39:57Z","updated_at":"2015-01-15T16:02:54Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/askmike/gekko/pulls/261","html_url":"https://github.com/askmike/gekko/pull/261","diff_url":"https://github.com/askmike/gekko/pull/261.diff","patch_url":"https://github.com/askmike/gekko/pull/261.patch"},"body":"Since performance is such a big thing in trading bots, I thought it would be nice to do some small code refactoring, like changing the variable declaration style (http://jsperf.com/variable-declaration5) and replacing if/else expressions for ternary operator. (http://jsperf.com/if-else-vs-arrays-vs-switch-vs-ternary/24). Let me know what you think!"},"comment":{"url":"https://api.github.com/repos/askmike/gekko/issues/comments/70108533","html_url":"https://github.com/askmike/gekko/pull/261#issuecomment-70108533","issue_url":"https://api.github.com/repos/askmike/gekko/issues/261","id":70108533,"user":{"login":"fsbatista","id":7033810,"avatar_url":"https://avatars.githubusercontent.com/u/7033810?v=3","gravatar_id":"","url":"https://api.github.com/users/fsbatista","html_url":"https://github.com/fsbatista","followers_url":"https://api.github.com/users/fsbatista/followers","following_url":"https://api.github.com/users/fsbatista/following{/other_user}","gists_url":"https://api.github.com/users/fsbatista/gists{/gist_id}","starred_url":"https://api.github.com/users/fsbatista/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fsbatista/subscriptions","organizations_url":"https://api.github.com/users/fsbatista/orgs","repos_url":"https://api.github.com/users/fsbatista/repos","events_url":"https://api.github.com/users/fsbatista/events{/privacy}","received_events_url":"https://api.github.com/users/fsbatista/received_events","type":"User","site_admin":false},"created_at":"2015-01-15T16:02:54Z","updated_at":"2015-01-15T16:02:54Z","body":"These are small modifications in the coding style that, if expanded towards the whole app, could result in interesting (even if small) gains of performance. I do agree that ternary operators are not as straightforward as if else statements though, it's a tradeoff."}}