ジャバスクリプトで可変長の引数を処理するクラスを作りたいです。

Question

入力した数字を足すプログラムを書いたのですが、
0[object Object]と表示されます。

let cal=new Cal(su);
でクラスの引数にsuの配列が送られていないようなのですが、
どう書けばいいのか分からなくて質問させていただきました。

let fun6 = function () {
	let i=0;
	let su={};
	do{
		su[i]=prompt("数字")
	}while(su[i++]!="");
	**let cal=new Cal(su);**
	cal.tasu();
}

class Cal{
	constructor(...arg){
		this.arg=arg;
	}
	tasu(){
		let ans = 0;
		for(i=0;i<this.arg.length;i++){
			ans+=this.arg[i];
		}
		alert(ans);
	}
}

Comments

[y_waiwai]

で、しつもんはなんでしょうか

[ottopo]

入力した数字を足すプログラムを書いたのですが、
結果が正しく表示されません。
let cal=new Cal(su);
でクラスに入力したsuの配列が送られていないようなのですが、
どう書けばいいのか分からなくて質問させていただきました。

[退会済みユーザー]

引数を可変数にしたいんですか？
配列を渡したいんですか？

そもそも配列なんて渡されてませんが。

[ottopo]

su[i]=prompt("数字")
で入力された複数の数字をsuの配列で引数として渡し、
足した合計を表示したいです。

Answer 1

とりあえず動くようにだけしました。

let fun6 = function () {
    let i=0;
    let su=[]; // 配列でいいよね。
    do{
        su[i]=prompt("数字")
    }while(su[i++]!=""); // 後述するが、かならずsuの最後は空文字列になる
        console.log('su',su);
    let cal=new Cal(su); // ここで配列を渡しているので、⇒
    cal.tasu();
}

class Cal{
    constructor(arg){// ⇒引数は普通に受け取る
        this.arg=arg;
    }
    tasu(){
        let ans = 0;
        for(let i=0;i<this.arg.length - 1;i++){ // do while の結果、最後の空文字列も入っているので。（もっとまともな処理の仕方はあるが）
            ans+=parseInt(this.arg[i]); // promptは文字列として返ってくるので数値化
        }
        console.log(ans); // デベロッパーツールで確認するようにしましょう http://eng-entrance.com/javascript-display-error
    }
}
fun6();
```**動くサンプル：**[https://jsfiddle.net/xrgmfst3/](https://jsfiddle.net/xrgmfst3/)

Comments

[ottopo]

ありがとうございます。無事動かすことができました。
可変長の引数を利用したプログラムを作りたかったのですが、必要なかったみたいですね、、、
配列の[]は盲点でした。
for(let i=0;i<this.arg.length - 1;i++){ // do while の結果、最後の空文字列も入っているので。（もっとまともな処理の仕方はあるが）
が気になるのですがスマートな記述だとどういう書き方になるのですか？

[kei344]

> スマートな記述だと
先に数値以外をフィルタするとか、そういう感じです。（そもそも現在のコードは「A」とか文字列が入るとNaNを返すので）

[ottopo]

なるほど入力値のチェックは必要ですね。
親切にありがとうございました。