disabled の記述について

javascript
1testBtn.onclick = function(){
2   testBtn.disabled = true;
3   if(条件①){
4      testBtn.disabled = false
5      return
6   }
7   if(条件②){
8      testBtn.disabled = false
9      return
10   }
11   ----------
12  処理
13   ----------
14    //処理が正常に完了した場合
15    testBtn.disabled = false
16}

上記のようにボタンを押したら処理が終わるまで操作不能にしてるのですが、途中で処理を抜けさせるようなif文があったりして
testBtn.disabled = falseの記述が多くなってしまっています。こういった記述は少し汚いと感じてしまうでしょうか？
これでも問題ないのでいいのですが、もっときれいにまとめる方法があれば教えていただけると嬉しいです。

行動規範の内容に同意します

回答3件

まず「testBtn」をどうやって掴むのかを決めてください
その上で.onclick()は実行した状態なのでfunctionを指定するのは無意味です
あえていえば.onclick=function()ですが、これも推奨されていません。
addEventlistenerで処理するのが妥当では？

またdisableを切り替えるか、clickを無視するか同じように見えて
ちがうので仕様をよく検討したほうがよいでしょう

投稿2016/12/05 02:27

yambejp

総合スコア114769

条件①、条件②・・・がそれぞれtrueだろうがfalseだろうが、最後まで処理が流れるようにして
最後でtestBtn.disabled = false;すればいいと思います。

投稿2016/12/05 02:48

deigo

総合スコア200

coko1

2016/12/05 03:29

その中間の処理を行いたくないのですが

deigo

2016/12/05 03:44

だから ``` testBtn.onclick = function(){ testBtn.disabled = true; if(条件①){ //testBtn.disabled = false //return } else if(条件②){ //testBtn.disabled = false //return } else { ---------- 処理 ---------- } //処理が正常に完了した場合 testBtn.disabled = false } ```

行動規範の内容に同意します

JavaScript
1testBtn.Clicked = function() {
2  testBtn.disabled = true;
3  var flgExec = true;
4  if (条件①） {
5    flgExec = false;
6  }
7  if (条件②) {
8    flgExec = false;
9  }
10  ...
11
12  if (flgExec) {
13    実際の処理
14  }
15  testBtn.disabled = false;
16}