Javascript配列中の特定条件の要素のみ処理したい

Question

前提・実現したいこと

以下のような配列があるとします。

let array = ["2020", null, "19,1189", 7, "ほげ", "", 0, 1,24, ""];

この配列内の要素で文字列中のカンマを全角（，）に変換したいです。
空、nullになっている物は変換後消したりせずそのまま維持する必要があります。

試したこと

for(var i = 0; i < array.length; i ++){
  if(typeof array[i] === 'string'){
    array[i] = array[i].replace(before, after);
  }
}

上記のコードで成功はしたものの、いまいちパッとしない書き方かなと思っています・・・。
ベターな書き方はありますでしょうか？
理想としては下記のように書きたいです。

const replacedArr = array.map(str => str.replace(before, after));

ただ、やはり上記だと要素が空のときにエラーになってしまいました。

以上よろしくお願い致しますm(_ _)m

Answer 1

いまいちパッとしない書き方

いまいちパッとしない書き方

感覚的な言葉ではなく、論理的に具体的に問題点を捉える事から始めてください。
プログラミングは論理的に行うものです。　

コード

for(var i = 0; i < array.length; i ++){
  if(typeof array[i] === 'string'){
    array[i] = array[i].replace(before, after);

xdmv> }
x> }x

String値だけを対象に置換するなら、typeof 演算子が必要なのは自明です。
何か無駄な処理があるように読めますか。
私は全て必要な処理に読めますが。

var array = [{toString:()=>'a,b'}];

このような配列があった場合、置換処理を行わない為には、どのような実装手段が考えられますか。

String.prototype.replace

String.prototype.replace を持つのは、String型か new Stringなので、判定条件が不十分という指摘なら、あり得ます。

var i = array.length;

while (i--) {
  var value = array[i];
  if (typeof value === 'string' || string instanceof String) array[i] = value.replace(/,/g, '，');
}

Re: Sutasyu127 さん

Comments

[Sutasyu127]

ありがとうございます。
論理的に具体的に問題点を捉えるという点、今後心に留めて行きたいと思います。
ご回答ありがとうございました。

Answer 2

let array = ["2020", null, "19,1189", 7, "ほげ", "", 0, 1,24, ""];
const reg=/,/g;
array=array.map(x=>reg.test(x)?x.replace(reg,'，'):x);
console.log(array);

Comments

[Sutasyu127]

ご回答ありがとうございました。
無駄のない書き方で大変参考になりました。