1. ホーム
  2. javascript

[解決済み] JavascriptのArray Concatが動作しません。なぜですか?

2022-09-02 04:14:21

質問

この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 );

なぜconcatがうまくいかないのか this.refsとrefはどちらも配列です。それなのになぜconcatがうまくいかないのでしょうか?

ボーナス: 私はこのウィジェットで何か他の馬鹿なことをしていますか? これは私の最初のものです。

どのように解決するのですか?

concatメソッドでは元の配列は変更されないので、再代入する必要があります。

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