NetBeans GUIアプリケーションは、実行しようとするとフリーズし続けます

Question

NetBeans 6.9.1プログラミングを使用していますが、現在はSwing GUIを使用しています。プログラムを実行し、結果を決定するために「計算」ボタンをクリックするたびに、プログラムがフリーズします。ここではそれがフリーズする原因と私のコードの塊がある：

private void calculateButtonActionPerformed(java.awt.event.ActionEvent evt) { 
    String backTrouble; 
    String heartTrouble; 
    int riderHeight = Integer.parseInt(inputHeight.getText()); 

    backTrouble = inputBack.getText(); 
    heartTrouble = inputHeart.getText(); 

    while ((riderHeight >= 122) && (riderHeight <= 188)){ 

     if ((backTrouble.equals("N")) && (heartTrouble.equals("N"))){ 

      responseField.setText("It is OK for you to ride this roller coaster. Have fun!"); 
     } 

     else if ((backTrouble.equals("Y")) && (heartTrouble.equals("N"))){ 

      responseField.setText("Sorry, it is not safe for you to ride this roller coaster."); 
     } 

     else if ((backTrouble.equals("N")) && (heartTrouble.equals("Y"))){ 

      responseField.setText("Sorry, it is not safe for you to ride this roller coaster."); 
     } 
     else{ 

      responseField.setText("Sorry, it is not safe for you to ride this roller coaster."); 
     } 

    while ((riderHeight < 122) || (riderHeight > 188)){ 

     responseField.setText("Sorry, it is not safe for you to ride this roller coaster."); 
    } 
    } 


} 

それが凍結し続ける理由を私は本当に理解していない、といくつかの助けをいただければ幸いです、ありがとうございました。

Answer 1

while (such and such)ループが連続して実行され、Swingイベントスレッドをブロックし、スレッドがGUIにペイントするのを防ぎ、ユーザーと対話してアプリケーションを効果的にフリーズさせます。これらのループはコンソールプログラムでは意味がありますが、イベント駆動型のSwing GUIではありません。それらを取り除くと、凍結は不凍液になります。おそらく、ここではifブロックを代わりに使用したいと思うかもしれません。

だから、もう一度、おそらくブロックが良い仕事かどう：

private void calculateButtonActionPerformed(java.awt.event.ActionEvent evt) { 
    String backTrouble; 
    String heartTrouble; 
    int riderHeight = Integer.parseInt(inputHeight.getText()); 
    backTrouble = inputBack.getText(); 
    heartTrouble = inputHeart.getText(); 

    // get rid of the while loop, 
    // while ((riderHeight >= 122) && (riderHeight <= 188)){ 
    if ((riderHeight >= 122) && (riderHeight <= 188)){ 
     if ((backTrouble.equals("N")) && (heartTrouble.equals("N"))){ 
      responseField.setText("It is OK for you to ride this roller coaster. Have fun!"); 
     } 
     else if ((backTrouble.equals("Y")) && (heartTrouble.equals("N"))){ 
      responseField.setText("Sorry, it is not safe for you to ride this roller coaster."); 
     } 
     else if ((backTrouble.equals("N")) && (heartTrouble.equals("Y"))){ 
      responseField.setText("Sorry, it is not safe for you to ride this roller coaster."); 
     } 
     else{ 
      responseField.setText("Sorry, it is not safe for you to ride this roller coaster."); 
     } 

     // again, no while loops here 
     // while ((riderHeight < 122) || (riderHeight > 188)){ 
     if ((riderHeight < 122) || (riderHeight > 188)){ 
      responseField.setText("Sorry, it is not safe for you to ride this roller coaster."); 
     } 
    } 
} 

しかし、私は100％確実ではないです。

Answer 2

私は問題が間違った場所に近いかっこを持っていると思います。

else{ 

    responseField.setText("Sorry, it is not safe for you to ride this roller coaster."); 
} 

while ((riderHeight < 122) || (riderHeight > 188)){ 

この前には、かっこを閉じてください。

また、riderHeightは決して変更しないので、ループ内に永遠に留まります。

この2つのwhileループをif..thenステートメントにすることもできますが、whileループで目的がないように思われる場合があります。あなたは

while(stillOnBike) { 
} 

を持っていた場合

次に、あなたは最終的には人間としての出口が、最終的に自転車を降りるなりますロジックを持っている可能性がありますが、基本的にwhile(true)を持っています。