Ruby объект клонирование / копирование - PullRequest
Новая
       26

Ruby объект клонирование / копирование

1 голос
/ 01 августа 2011

Обзор

Я создаю объекты в своем скрипте ruby ​​из запросов к базе данных, которые генерируют файлы XML.Я сделал так, чтобы за один раз обрабатывался только один XML-файл, и все теги были общими, чтобы можно было легко добавлять другие запросы.

Проблема

Я создаю один объект за раз, затем добавляю его в список, например: 

#create a new BarChart
bar_chart = BarChart.new(title, data, labels, x_axis, y_axis);

#add the chart to the chart list
charts.push(bar_chart)

Но каждый разЯ обрабатываю XML-файл и хочу создать BarChart. Я повторно использую переменную bar_chart, которая приводит к перезаписи данных моих объектов.Я ищу способ обойти это.

Что я пробовал

Я пытался передать копию объекта в список, но это все ещеперезапись данных. 

#create a new BarChart
bar_chart = BarChart.new(title, data, labels, x_axis, y_axis);

#add the chart to the chart list
charts.push(bar_chart.clone)

и 

#create a new BarChart
bar_chart = BarChart.new(title, data, labels, x_axis, y_axis);

#add the chart to the chart list
charts.push(bar_chart.dup)

Любая помощь / идеи будут великолепны.Спасибо.

РЕДАКТИРОВАТЬ, больше информации Вот метод, в котором я выполняю обработку XML. 

def self.process_xml_files2(filenames)
    labels = []
    data  = []
    charts = []
    title    = nil
    type   = nil
    x_axis  = nil
    y_axis  = nil

    #retrieve needed data from the XML file
    filenames.each do |filename|
        f = File.new(filename)
        #create a document
     doc = Document.new(f)
         doc.elements.each("//row/field") do |e|
            tag = e.attributes['name']
            text = e.text

            #search for tags and append correct data to lists
            if tag.casecmp('Type') == 0
            type = text
        elsif tag.casecmp('Title') == 0
                title = text
            elsif tag.casecmp('Labels') == 0
                labels.push(text)
            elsif tag.casecmp('Data') == 0
                data.push(text)
            elsif tag.casecmp('X-Axis') == 0
                x_axis = text
            elsif tag.casecmp('Y-Axis') == 0
                y_axis = text
            end
        end
        f.close()

        #test for correct chart parameters
        raise "Not Enough Arguments" 
            if title == nil or type == nil or data.empty? or labels.empty?

        #process the raw chart data 
        if type.casecmp('Bar') == 0
            #test for labels
            raise "Bar Charts require X and Y axis labels" 
                    if x_axis == nil or y_axis == nil

            #format the data for the bar chart
            data = BarChart.barify_data(data)

            #create a new BarChart
            bar_chart = BarChart.new(title, data, labels, x_axis, y_axis);

            #add the chart to the chart list
            charts.push(bar_chart)
        elsif type.casecmp('Pie') == 0
            #format data and labels for the pie chart
            data = PieChart.pieify_data(data)

            #create a new Pie Chart
            pie_chart = PieChart.new(title, data, labels)

            #add the pie chart to the chart list
            charts.push(pie_chart.clone)
            else
            raise "Invalid Chart Type: Not Pie or Bar"
        end
    end

        #write all the charts to the images directory
    charts.each do |ch|
        puts ch.url + "\n\n"
        ch.download_image(ch.url, ch.title)
    end
end

1 Ответ

2 голосов
/ 01 августа 2011

Из того, что я вижу в коде, вы повторно используете labels и data объекты (будьте осторожны: объекты, а не переменные!) Для каждой диаграммы, которую вы добавляете в список. Похоже, вы должны двигаться 

labels = []
data = []

инициализации внутри цикла filenames.each.

...