JavaScript配列連結が機能していません。どうして?


92

そこで、このjqueryuiウィジェットを作成しました。エラーをストリーミングできるdivを作成します。ウィジェットのコードは次のようになります。

$.widget('ui.miniErrorLog', {
   logStart: "<ul>",   // these next 4 elements are actually a bunch more complicated.
   logEnd:   "</ul>",
   errStart: "<li>",
   errEnd:   "</li>",
   content:  "",
   refs:     [],

   _create: function() { $(this.element).addClass( "ui-state-error" ).hide(); },

   clear: function() { 
      this.content = ""; 
      for ( var i in this.refs )
         $( this.refs[i] ).removeClass( "ui-state-error" );
      this.refs = [];
      $(this.element).empty().hide(); 
   }, 

   addError: function( msg, ref ) {
      this.content += this.errStart + msg + this.errEnd; 
      if ( ref ) {
         if ( ref instanceof Array )
            this.refs.concat( ref );
         else
            this.refs.push( ref );
         for ( var i in this.refs )
            $( this.refs[i] ).addClass( "ui-state-error" );
      }
      $(this.element).html( this.logStart + this.content + this.logEnd ).show();
   }, 

   hasError: function()
   {
      if ( this.refs.length )
         return true;
      return false;
   },
});

エラーメッセージを追加でき、エラー状態になるページ要素への参照を追加できます。ダイアログの検証に使用します。「addError」メソッドでは、次のように単一のIDまたはIDの配列を渡すことができます。

$( "#registerDialogError" ).miniErrorLog( 
   'addError', 
   "Your passwords don't match.", 
   [ "#registerDialogPassword1", "#registerDialogPassword2" ] );

しかし、idの配列を渡すと機能しません。問題は次の行にあります(私は思う):

if ( ref instanceof Array )
   this.refs.concat( ref );
else
   this.refs.push( ref );

なぜその連結が機能しないのですか?this.refsとrefはどちらも配列です。では、なぜ連結が機能しないのでしょうか。

おまけ:このウィジェットで他に何かばかげたことはありますか?初めてです。


回答:


260

concatメソッドは元の配列を変更しないため、再割り当てする必要があります。

if ( ref instanceof Array )
   this.refs = this.refs.concat( ref );
else
   this.refs.push( ref );

5
それはそれでした。オブジェクトのconcatメソッドがオブジェクトに追加されると思ったでしょう。しかし、私はそれがどのように機能するかではないと思います。
Rafael Baptista

3
@Rafael:pushメソッドはそれを行います、あなたはそうすることができます[].push.apply(this.refs, ref)
Bergi

78

理由は次のとおりです。

定義と使用法

concat()メソッドは、2つ以上の配列を結合するために使用されます。

このメソッドは既存の配列を変更しませんが、結合された配列の値を含む新しい配列を返します。

連結の結果を配列に割り当てる必要があります。


2
なぜ、ああ、なぜ、いつもこれを忘れなければならないのですか?
ジェフロウリー

9

Konstantin Dinevを拡張するには:

.concat()現在のオブジェクトには追加されないため、これは機能しません

foo.bar.concat(otherArray);

この意志:

foo.bar = foo.bar.concat(otherArray);

4

=を使用して値を配列に再割り当てする必要があり、連結した値を取得したい

let array1=[1,2,3,4];
let array2=[5,6,7,8];

array1.concat(array2);
console.log('NOT WORK :  array1.concat(array2); =>',array1);

array1= array1.concat(array2);
console.log('WORKING :  array1 = array1.concat(array2); =>',array1);


弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.