루비 블록이 <a href="http://railscasts.com/episodes/396-importing-csv-and-excel" rel="nofollow">screencast</a>을

Question

업 후, 그는 어떻게든지 2..3 대신 Product 경우 레코드

def save 
    puts "--- imported_products: #{imported_products.inspect}" 
    // --- imported_products: 2..3 
    if imported_products.map(&:valid?).all? 
     imported_products.each(&:save!) 
     true 
    else 
     imported_products.each_with_index do |product, index| 
     product.errors.full_messages.each do |message| 
      errors.add :base, "Row #{index+2}: #{message}" 
     end 
     end 
     false 
    end 
end 

def imported_products 
    @imported_products ||= load_imported_products 
end 

def load_imported_products 
    spreadsheet = open_spreadsheet 
    header = spreadsheet.row(1) 
    (2..spreadsheet.last_row).each do |i| 
    row = Hash[[header, spreadsheet.row(i)].transpose] 
    product = Product.find_by_id(row['id']) || Product.new 
    product.attributes = row.to_hash.slice(*accessible_attributes) 
    product 
    end 
end 

Answer 1

내 load_imported_products 방법은 each 블록을 포함 복귀. 이 블록은 메소드의 마지막 '라인'이므로 블록의 반환 값은 메소드의 반환 값이됩니다. 당신이 load_imported_products 방법에 대해 얘기하고 있다면, 다음 배열을 반환하는 대신 배열에 각 제품을 추가하려고

def load_imported_products 
    spreadsheet = open_spreadsheet 
    header = spreadsheet.row(1) 
    products = [] 
    (2..spreadsheet.last_row).each do |i| 
    row = Hash[[header, spreadsheet.row(i)].transpose] 
    product = Product.find_by_id(row['id']) || Product.new 
    product.attributes = row.to_hash.slice(*accessible_attributes) 
    products << product 
    end 
    products 
end 

Answer 2

:

은 다음보십시오.

정확히 어떤 메소드가 반환되는지는 확실하지 않지만 제품 컬렉션을 명시 적으로 반환해야합니다.

그래서

def load_imported_products 
    products = [] 
    spreadsheet = open_spreadsheet 
    header = spreadsheet.row(1) 
    (2..spreadsheet.last_row).each do |i| 
    row = Hash[[header, spreadsheet.row(i)].transpose] 
    product = Product.find_by_id(row['id']) || Product.new 
    product.attributes = row.to_hash.slice(*accessible_attributes) 
    products << product 
    end 
    return products 
end 

Answer 3

또는 사용지도

def load_imported_products 
    spreadsheet = open_spreadsheet 
    header = spreadsheet.row(1) 
    products = (2..spreadsheet.last_row).map do |i| 
    row = Hash[[header, spreadsheet.row(i)].transpose] 
    product = Product.find(row['id'].to_i) || Product.new 
    product.attributes = row.to_hash.slice(*accessible_attributes) 
    product 
    end 
end 

또한 나는 그것이 nil 또는 string로 저장했다 경우 정수로 강제 있지만 find 방법은 id를 사용하여 필요하지 않습니다 find_by_id.