стоит ли заморачиваться с дублированием строк?

Сообщения: 17 Репутация: N Группа: Кто попало

23 мая 2016 г. 11:18

как думаете, если вместо такого:


var volume = bla bla,
    result;

if(APP.helper.isNumCheck(volume)) { 
  result = volume;
} else {
  result = undefined;
};  

this.model.set({shippOptionsVolume: result});

писать такое:


var volume = bla bla;

if(APP.helper.isNumCheck(volume)) { 
  this.model.set({shippOptionsVolume: volume});
} else {
  this.model.set({shippOptionsVolume: undefined});
};

,то коллеги обзывать индусом не начнут?

какой вариант вы использовали бы?

master

Сообщения: 3244 Репутация: N Группа: Джедаи

23 мая 2016 г. 12:13, спустя 55 минут 18 секунд

нормально

не всё полезно, что в swap полезло
phpdude

Сообщения: 26646 Репутация: N Группа: в ухо

23 мая 2016 г. 15:45, спустя 3 часа 31 минуту 30 секунд

оба варианта хорошо

Сапожник без сапог
adw0rd

Сообщения: 22959 Репутация: N Группа: в ухо
23 мая 2016 г. 15:51, спустя 6 минут 5 секунд
var volume = bla bla; this.model.set({ shippOptionsVolume: APP.helper.isNumCheck(volume) ? volume : undefined });
https://smappi.org/ - платформа по созданию API на все случаи жизни

Пожалуйста, авторизуйтесь, чтобы написать комментарий!

стоит ли заморачиваться с дублированием строк?

Войти