Можно не писать в ветке if и else по два раза exit. Можно 1 раз и после него. Пустая строка после def не нужна. Чтобы не делать экранирование двойных кавычек, лучше писать их внутри одинарных
1. Оно работает в полной мере (все случаи как надо отрабатывают) 2. Спросить у более опытных колег 3. Подождать пару лет/дождаться дня, когда надо будет внести в него измененияи посмотреть, насколько просто это будет сделать :)
Doug Wright "I learned to code with resources found on the web. This little book about Ruby is to help the younger generation find the joy in learning to code." ( http://mrdougwright.gitbooks.io )