Не работает if, что делаю не так?

Question

[lavezzi1]

Здравствуйте.
Есть хелпер

module ArticlesHelper
	def image_from_amazon(amazon_id)
    	image_tag "http://images.amazon.com/images/P/#{amazon_id}.01.ZTZZZZZZ.jpg"
	end
end

Пробую так

module ArticlesHelper
	def image_from_amazon(amazon_id)
		if amazon_id == nil
			image_tag "http://vk.com/images/deactivated_100.png"
		else
    		image_tag "http://images.amazon.com/images/P/#{amazon_id}.01.ZTZZZZZZ.jpg"
    	end
	end
end

В итоге картинка не ставится, а на странице выдает

<img src="http://images.amazon.com/images/P/.01.ZTZZZZZZ.jpg" alt=".01.ztzzzzzz">

Answer 1

[ТыжСисАдмин]

А вы уверены что amazon_id имеет значение nil?
== ''

Comments

[lavezzi1]

Да, если есть значение то выглядит так amazon_id: "0140283293", если нет то amazon_id: nil

[lavezzi1]

add_column :articles, :amazon_id, :string

[ТыжСисАдмин]

где выглядит? В БД чтоли?
Вы убидитесь что в метод у вас передаётся nil после запроса.
Но мне кажется что будет не nil а ''

[lavezzi1]

Через rails console посмотрел, а как надо?

[Евгений Ткаченко]

А если просто проверять не на нил, а: amazon_id.strip.length == 0

[Viktor Vsk]

Евгений Ткаченко: что?)
есть метод #present?

[Евгений Ткаченко]

Виктор Ablebeam: есть конечно, есть и blank? и т.д., но куда лучше чем проверять стрингу на нилл

[Viktor Vsk]

Евгений Ткаченко: просто нужно запомнить, что есть #present? (и как следстсвтие #presence) для начала - можно даже без #blaml?, #empty? и т.д.
Ваш пример делает частично то, что делает #present? но делает это плохо (не на всех случаи) и еще и упадет на nil, array и т.д.

[Евгений Ткаченко]

Виктор Ablebeam: согласен

Answer 2

[Анатолий Егоров]

Если используется Rails, то вместо amazon_id == nil разумнее использовать amazon_id.blank?