Rails. Как добавлять элементы в массив в хеше?

Question

[kramidev]

Привет всем.
Есть хэш. В хеше есть массив например 'pictures'. Как добавлять в массив элементы?

hash = {'pictures' => [] }
someArray.each do |a|
   hash['pictures'].push(a) # не работает :/
end

=================== все получилось, добавил немного логики. всем спасибо.

def check_docs(docs)
    if docs.nil?
      false
    else
      hash = {}
      docs.each do |doc|
        name = doc.original_filename.split('.').last
        if pic(name) != false
          hash['pictures'] = [] unless hash['pictures']
          hash['pictures'] << doc
        end
        if doc(name) != false
          hash['docs'] = [] unless hash['docs']
          hash['docs'] << doc
        end
      end
      if hash.size > 0
        hash
      else
        false
      end
    end
  end

Comments

[Виктор Маслов]

При чем здесь рельсы? Где на TOSTER кнопка минус?

[kramidev]

Виктор Маслов: там же где ваше самомнение и курс нефти.

Answer 1

[Дмитрий]

Проверил ваш вариант -- у меня работает. Вот альтернативный синтаксис:

hash = {'pictures' => [] }
someArray.each do |a|
   hash['pictures'] << a 
end

Comments

[kramidev]

Может проблема с обьявлении. Если мы зададим hash = {}. То hash['pictures'] << a не работает

[kramidev]

def check_docs(docs)
if docs.nil?
false
else
hash = {}
docs.each do |doc|
hash['pictures'] << doc
end
hash
end
end

docs - массив с фотками
[#, @original_filename="6014866-8969339.jpg", @content_type="image/jpeg", @headers="Content-Disposition: form-data; name=\"gov[docs][]\"; filename=\"6014866-8969339.jpg\"\r\nContent-Type: image/jpeg\r\n">, #, @original_filename="220px-RIAN_archive_100306_Vladimir_Putin,_Federal_Security_Service_Director.jpg", @content_type="image/jpeg", @headers="Content-Disposition: form-data; name=\"gov[docs][]\"; filename=\"220px-RIAN_archive_100306_Vladimir_Putin,_Federal_Security_Service_Director.jpg\"\r\nContent-Type: image/jpeg\r\n">, #, @original_filename="200px-Turgenev_by_Repin.jpg", @content_type="image/jpeg", @headers="Content-Disposition: form-data; name=\"gov[docs][]\"; filename=\"200px-Turgenev_by_Repin.jpg\"\r\nContent-Type: image/jpeg\r\n">]

[Дмитрий]

kramidev: Поясните, что значит "не работает" и покажите содержимое someArray. Ваша конструкция написана правильно -- должна работать.

[kramidev]

Дмитрий: выше привел пример.
Результат undefined method `<<' for nil:NilClass указывающий на строку hash['pictures'] << doc

[Дмитрий]

kramidev: При объявлении hash = {} в hash нет элемента 'pictures'. Добавлять некуда. В вашем вопросе -- правильное объявление.

[kramidev]

Дмитрий:

def check_docs(docs)
if docs.nil?
false
else
hash = {}
docs.each do |doc|
hash['pictures'] << doc
end
hash
end
end

docs - массив с фотками
[#, @original_filename="6014866-8969339.jpg", @content_type="image/jpeg", @headers="Content-Disposition: form-data; name=\"gov[docs][]\"; filename=\"6014866-8969339.jpg\"\r\nContent-Type: image/jpeg\r\n">, #, @original_filename="220px-RIAN_archive_100306_Vladimir_Putin,_Federal_Security_Service_Director.jpg", @content_type="image/jpeg", @headers="Content-Disposition: form-data; name=\"gov[docs][]\"; filename=\"220px-RIAN_archive_100306_Vladimir_Putin,_Federal_Security_Service_Director.jpg\"\r\nContent-Type: image/jpeg\r\n">, #, @original_filename="200px-Turgenev_by_Repin.jpg", @content_type="image/jpeg", @headers="Content-Disposition: form-data; name=\"gov[docs][]\"; filename=\"200px-Turgenev_by_Repin.jpg\"\r\nContent-Type: image/jpeg\r\n">]

[Дмитрий]

kramidev: При объявлении hash = {} в hash нет элемента 'pictures'. Добавлять некуда. В вашем вопросе -- правильное объявление.

[kramidev]

Дмитрий: И как обьявить при пустом hash = {} элемент pictures?

[Дмитрий]

kramidev: ПОСМОТРЕТЬ СОБСТВЕННЫЙ ВОПРОС!!!!
В вашем вопросе -- правильное объявление.
hash = {'pictures' => [] }
или
hash = {}
hash ['pictures'] = []

[Дмитрий]

kramidev: hash['docs'] = [] unless hash['docs'] можно заменить на hash['docs'] ||= []

Answer 2

[Viktor Vsk]

Попробуйте, вот так полегче читается, вроде:

def check_docs(docs)
  return false unless docs.present?
  hash = {}
  docs.each do |doc|
    name = doc.original_filename.split('.').last
    hash['pictures']  << doc if pic(name) != false and hash['pictues'] ||= []
    hash['docs']      << doc if doc(name) != false and hash['docs'] ||= []
  end
  hash.presence
end

Разве что, две оговорки:
1. Первая и последняя строчка подразумевают, что используется Rails (ну, или хотябы ActiveSupport)
2. Если hash будет пустой, вернется не false, а nil

P.S. Кстати, какая-то плохая идея в блоке each использовать переменную doc и там же обращаться к какому-то методу doc() (с одинаковыми именами)

Comments

[kramidev]

Поправил. Спасибо за 2 пункт. Поправил