Skip to content

Instantly share code, notes, and snippets.

@fee1good

fee1good/movies.rb

Last active August 2, 2016 10:48
Show Gist options
  • Save fee1good/6468c4f6e4c2cbbfeca866d0321d72e0 to your computer and use it in GitHub Desktop.
Save fee1good/6468c4f6e4c2cbbfeca866d0321d72e0 to your computer and use it in GitHub Desktop.
Download ZIP
Raw
movies.rb
good_film = 'Matrix'
bad_film = 'Titanic'
definition = ARGV[0]
if definition == good_film
puts good_film.to_s + ' is a good movie'
elsif definition == bad_film
puts bad_film.to_s + ' is a bad movie'
else
puts 'Haven\'t seen ' + ARGV[0].to_s
end
Raw
movies_array.rb
puts 'Ввведите название фильма:'
film_name = gets.chomp
name = ['Matrix','Matrix2','Matrix3']
if name.include?(film_name.to_s)
puts film_name + ' is a good movie'
else
puts 'Haven\'t seen ' + film_name
end
@zverok
Copy link

zverok commented Aug 1, 2016

К первой программе:

  • названия плохого и хорошего фильма положить в переменные в начале программы, чтобы их можно было легко заменить, если у тебя сменились вкусы :)
  • вместо вложенных if'ов лучше использовать elsif
  • название фильма по условиям должно передаваться в командной строке при вызове скрипта, а не спрашиваться у пользователя в процессе

Ко второй:

  • а если фильмов будет 5? а если 10? а если 20? :) должен быть какой-то способ попроще проверить, входит ли фильм в список хороших, м?

@zverok
Copy link

zverok commented Aug 2, 2016

  • переменные принято называт вот_так (маленькими буквами, слова разделяет подчерк)
  • обрати внимание, что ты за программу несколько раз вызываешь ARGV[0],может положить его сразу в переменную?
  • зачем столько to_s?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment