Отладка простого метода класса Ruby? - PullRequest
Новая
       11

Отладка простого метода класса Ruby?

0 голосов
/ 07 июля 2011 
class DobbsyKretts
  def initialize
    #Receive idea
    puts "Enter an idea, a secret or anything else you want to secretize; hit enter to stop typing and save the file"
    (@idea = gets).reverse.upcase
    #Filename and saving - to encrypt the file
    puts "Enter the file name you'd like to have this saved as; Type PLAN at the beginning for plans and REM for reminders"
    (@file_name = gets.chomp.upcase)
    File::open("DobbsyKrett-"+ @file_name + ".txt", "w") do |f|
      f << @idea
    end
  end

  def unzip
    puts "Do you want to withdraw PLAN or REM"
    response = gets.chomp.upcase!
    puts "Invalid" if !["PLAN","REM"].include?(response)
    file_contents = nil
    Dir['DobbsyKrett-'+response+"*.txt"].each do |file_nom|
      file_contents = File.read(file_nom)
    end
    puts file_contents
  end
end
somethingsomething1 = DobbsyKretts.new
somethingsomething1.unzip


  def unzip
    puts "Do you want to withdraw PLAN or REM"
    @response = gets.strip
    if @response.downcase != "plan" and @response.downcase != "rem"
      puts "Invalid" end
    Dir["DobbsyKrett-"+@response+".txt"].each do |file_nom|
      @value = file.read(file_nom)
    end
    puts @value
  end
end

1 Ответ

0 голосов
/ 07 июля 2011

Функция gets вернет строку с символом конца строки в конце, который не соответствует ожидаемому. Чтобы удалить его, используйте функцию chomp: 

@response = gets.chomp

Это нормально для метода (например, unzip) для создания новых переменных экземпляра (например, @valueholder). В общем случае для ваших переменных всегда лучше иметь наименьшую возможную область видимости, поэтому, если вам не нужно читать valueholder позже, вам следует просто использовать локальную переменную (уберите @ из имени): 

Dir["DobbsyKrett-"+@response+".txt"].each do |file_nom|
  valueholder = File.read(file_nom)
end
puts valueholder

Кроме того, valueholder - ужасное имя для переменной, но если вы сделали ее локальной переменной, которая могла бы быть оправдана.

Кроме того, ваши начальные / конечные блоки не совпадают. Вот исправленная версия вашей функции, которая не должна приводить к синтаксическим ошибкам: 

def unzip
  puts "Do you want to withdraw PLAN or REM"
  response = gets.chomp.downcase
  if !["plan","rem"].include? response
    puts "Invalid"
  else
    Dir["DobbsyKrett-#{response}.txt"].each do |file_nom|
      valueholder = file.read(file_nom)
    end
    puts valueholder
  end
end

Редактировать: Вы должны использовать File для правильного вызова File.read.

.
...