Ruby 方法变量声明

Question

我正在尝试定义解析 apache 日志文件并提取 IP 地址、URL、每小时请求数和错误代码的方法。我已经在方法之外完成了所有工作，但是在尝试将该代码放入方法时，我不断收到错误消息“堆栈级别太深”。这是有问题的代码。

   class CommonLog

    def initialize(logfile)
        @logfile = logfile
    end

    def readfile
         @readfile = File.readlines(@logfile).map { |line|
    line.split()
  }
  @readfile = @readfile.to_s.split(" ")
    end

    def ip_histogram
        @ip_count = 0
        @readfile.each_index { |index|
            if (@readfile[index] =~ /\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3}/ )
                puts @readfile[index]
                puts @ip_count += 1
            end
        }
    end

    def url_histogram
        url_count = 0 
        cleaned_file.each_index { |index|
            if (cleaned_file[index] =~ /\/{1}(([a-z]{4,})|(\~{1}))\:{0}\S+/ )
                puts cleaned_file[index]
                puts url_count += 1
            end
        }
    end

    def requests_per_hour
    end

    def sorted_list
    end

end

my_file = CommonLog.new("test_log")
cleaned_file = my_file.readfile
puts cleaned_file.ip_histogram

Answer 1

看来问题出在你CommonLog#readfile方法上：

def readfile
  @readfile = File.readlines(@logfile).map { |line|
    line.split()
  }
  @readfile = readfile.to_s.split(" ")
end

注意到readfile 的实现内部你递归调用readfile？当它执行时，它从文件中读取行，映射它们并将结果分配给@readfile；然后它调用readfile 并再次开始执行该方法；这种情况会一直持续下去，直到由于递归方法调用过多而导致堆栈炸毁。

我假设你的实际意思是：

@readfile = @readfile.to_s.split(" ")