配列から前後の文字列だけを削除したい

Question

下記コードの流れでわからないところがあります。

まず対象の配列がこうあります。
Array [ "[いぬ[の] しっぽ]", "[ねこのしっぽ]" ]
ここから前後の括弧だけを削除したいと考えています。

つまりこうしたいのです。
Array [ "いぬ[の] しっぽ", "ねこのしっぽ" ]

そのために下記コードでは.slice()を使い、前後の括弧を削除しているつもりです。

// 対象の配列
let array = [ "[いぬ[の] しっぽ]", "[ねこのしっぽ]" ];

// 前後の括弧だけ削除する
let result = [];
array.forEach(function(x){
  array = x.slice( 0, -1 ).slice( 0, 1 ) ;
  result.push(array);
});

// Array [ "いぬ[の] しっぽ", "ねこのしっぽ" ] としたい
console.log( result );  

わからないのは、括弧を削除しているつもりが、括弧だけが残ってしまうという現象です。

目的は、Array [ "いぬ[の] しっぽ", "ねこのしっぽ" ]なのに
結果は、Array [ "[", "[" ]となってしまうのです。

どうすれば目的のようになるでしょうか？

Answer 1

forEach内部でarrayを呼んでいるため、もとの配列を上書きしてしまっています。

別の変数名にするか、直接result.pushを呼ぶかして、arrayを上書きしないようにしましょう。

array.forEach(function(x){
  result.push(x.slice( 0, -1 ).slice( 0, 1 ));
});

なお、このような「配列の各要素に操作を加えて別な配列を得る」というのには、.mapというメソッドも便利です。

var result = array.map(function(x){
  return x.slice( 0, -1 ).slice( 0, 1 );
});

Comments

[kamekamekame]

ありがとうございます。yambejp様のおかげで解決したものの原因がわからないままでしたので、ご説明に感謝申し上げます。

Answer 2

var a = [ "[いぬ[の] しっぽ]", "[ねこのしっぽ]" ];
a=a.map(x=>x.substr(1,x.length-2));
console.log(a);

Comments

[kamekamekame]

たびたびのご回答に心より感謝申し上げます！おかげ様で考えたことがひととおりできそうです。本当にありがとうございました。

Answer 3

まずは、
sliceの仕様を勘違いされているかと思います。

Array.prototype.slice() - JavaScript | MDN

sliceが返すのは、抜いた残りを返すのではなく、
抜いた文字自体を返します。

よって、array変数には、質問者さんが書いた通りのものが入っているわけです。

また、元配列である、arrayは破壊的に、代入し直すのは後々の処理も面倒になるし、あまりよろしくないかと。

下記でいかがでしょうか。

array.forEach(function(x){
  let str = x.slice(1, x.length - 1);
  result.push(str);
});

ようは、文字列の、二文字目から、最後から二番目の文字を取得するって結果です。
ですが、この形なら、replaceを使って、正規表現で文字を抜き出した方がいいかと思います。
本質問の要件ではないので、replaceと正規表現に関してはご自分でお調べください。

Answer 4

slice(0, 1)で文字列の先頭から一文字を切り出しているため先頭の括弧だけが切り出されています。
sliceを使用して先頭の文字を切り取る場合はslice(1)のように記述します。

Comments

[kamekamekame]

なるほど、私の書き方は「削除」ではなく「切り出し（取得）」だったのですね。覚えておきます。