ФорумПрограммированиеОбщие вопросы программирования → стоит ли заморачиваться с дублированием строк?

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

  • zlodiak

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

    Spritz Май 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 Группа: Джедаи

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

    нормально

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

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

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

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

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

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

    Spritz Май 23, 2016, 3:51 п.п., спустя 6 минут 5 секунд

    
    var volume = bla bla;
    this.model.set({
        shippOptionsVolume: APP.helper.isNumCheck(volume) ? volume : undefined
    });
    
    https://smappi.org/ - платформа по созданию API на все случаи жизни

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