Чем заменить тернарный оператор?

Question

[Philipp]

Есть говнокод вида:

margin = @supply.supplier.margin ? @supply.supplier.margin : 0

Если есть какой-то общий подход, например проверить наличие поставщика, наценки и вернуть 0 в случае их отсутствия, подскажите. Затупляю в общем.

Answer 1

[Viktor Vsk]

@supply.supplier.margin || 0
Даст не то, что вы хотите, когда margin будет равняться "" (что будет, если меняете через форму на пустое значение)

Для этого в Rails сделали метод
presence
margin = @supply.supplier.margin.presence || 0

Что эквивалентно:

margin = @supply.supplier.margin.present? ? @supply.supplier.margin : 0

present? лучше проверяет пустые значения, чем неявное приведение типов

Comments

[Philipp]

Спасибо за presence. Я сегодня на чистую голову глянул, можно ведь в модели сделать валидацию плюс при создании подставлять дефолтное значение. Совсем вчера упоролся.

Answer 2

[anyd3v]

margin = @supply.supplier.margin || 0
как то так