{"comment":{"url":"https://api.github.com/repos/ilya89forrails/MKDEV_RUBY/comments/16724295","html_url":"https://github.com/ilya89forrails/MKDEV_RUBY/commit/a2444443e3a3fd236b4b749938fad1dbce352b1c#commitcomment-16724295","id":16724295,"user":{"login":"zverok","id":129656,"avatar_url":"https://avatars.githubusercontent.com/u/129656?v=3","gravatar_id":"","url":"https://api.github.com/users/zverok","html_url":"https://github.com/zverok","followers_url":"https://api.github.com/users/zverok/followers","following_url":"https://api.github.com/users/zverok/following{/other_user}","gists_url":"https://api.github.com/users/zverok/gists{/gist_id}","starred_url":"https://api.github.com/users/zverok/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zverok/subscriptions","organizations_url":"https://api.github.com/users/zverok/orgs","repos_url":"https://api.github.com/users/zverok/repos","events_url":"https://api.github.com/users/zverok/events{/privacy}","received_events_url":"https://api.github.com/users/zverok/received_events","type":"User","site_admin":false},"position":7,"line":7,"path":"movies_lesson2.rb","commit_id":"a2444443e3a3fd236b4b749938fad1dbce352b1c","created_at":"2016-03-16T09:10:02Z","updated_at":"2016-03-16T09:10:02Z","body":"Во-первых, then тут лишний (работает и с ним.\r\nВо-вторых, оборачивать весь рабочий код в один общий if — нехорошо (всё время читая «саму программу», надо помнить, что где-то там вокруг был ещё if, и к нему будет else). Намного лучше — проверить условия, в которых программа не может работать, и если это так, сразу выйти. А потом уже тело основного кода."}}