JavaScript - нужно ли сначала проверить состояние кондиционирования

Question

Мне было интересно, нужно ли проверять состояние в этом конкретном примере.

Состояние, о котором я говорю, это if (Id.indexOf("_")).

Он просто проверяет, имеет ли символ Id символ подчеркивания, и если да, то установите переменную, чтобы снять знак подчеркивания и заменить дефис.

Я знаю, что могу просто использовать Id = Id.split("_").join("-"); без инструкции if, чтобы проверить, является ли условие истинным, но просто интересно, правильно ли в этом случае проверить состояние или нет?

В каком направлении вы бы это сделали? И объясните, почему, пожалуйста.

Id = "My_ID"; 
    var brand = "The Brand";   
    var b = brand.trim().toLowerCase(); 
    var page = b.split(/\W+/g).join("-"); 

    if (Id.indexOf("_")) { 
     Id = Id.split("_").join("-"); 
    } 

Answer 1

Это может ответить на ваш вопрос:

console.log("mytext".split("_").join("-")); // mytext 
console.log("my_text".split("_").join("-")); // my-text 

Кроме того условия в вашем коде if (Id.indexOf("_")) не работает, как вы хотели. Вы не должны использовать if (Id.indexOf("_") > -1) или (~Id.indexOf("_"))

Answer 2

Если нет подчеркивания вашего split() не будет делать ничего, так нет - нет необходимости в if здесь. Берите с чем-то в этом роде, и вы прекрасно:

Id = Id.split("_").join("-"); 

или

Id = Id.replace(/_/g, '-'); 

, чтобы избежать создания массива первых.