2012-04-23 17 views
-3

私はアンドロイドアプリを構築していますが、日食は私には括弧のエラーで怒っています。閉じかっこがどこにある必要があるか誰にでも教えてもらえますか? 5月までにこの小さなプロジェクトを完了したいと思っていますが、5月20日は絶対最新です。ブラケットを完成させる方法を指摘してください

public class MainActivity extends Activity { 
/** Called when the activity is first created. */ 
@Override 
public void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
    setContentView(R.layout.main); 


final Button button = (Button) findViewById(R.id.toggleButton); 
button.setOnClickListener(new View.OnClickListener() { 
    public void onClick;{; 
      newPhoneImage = 
      getResources().getDrawable(R.drawable.i_dens); 
    } 
      public void onClick(View arg0) { 
       // TODO Auto-generated method stub 

      }; 
     ) 
     }; 
    }; 

}

+0

私はあなたのコードをより一貫してインデントすることをお勧めします。それはあなたの頭痛を救うでしょう。また、 'public void onClick; {;'? – climbage

+0

なぜ[javascript]タグですか? –

+0

インデントを一貫してインデントするか、まったくインデントしません。また、あなたのコードのいくつかは、引用符で囲まれたコードブロックにぶら下がっています – Hawken

答えて

1
public void onClick;{; 

;の文字は正確には何ですか?あなたのの最初のの手順は、それらを取り除き、再度試してみることです。

0

パブリッククラスMainActivityアクティビティを拡張{

/**活性が最初に作成されたときに呼び出さ。 */

@Override

公共ボイドのonCreate(バンドルsavedInstanceState){

super.onCreate(savedInstanceState)。

setContentView(R.layout.main); 

最終ボタンボタン=(ボタン)findViewById(R.id.toggleButton)。

button.setOnClickListener(新しいOnClickListener(){

@Override 
public void onClick(View arg0) { 
    // TODO Auto-generated method stub 
    newPhoneImage = 
     getResources().getDrawable(R.drawable.i_dens); 
} 

})。

}

}

+0

バストした書式設定 – sarnold

0

あなたは決算ブラケットが行くところ知っている必要があります。配置はではなく、ではありません。それらを追加するのが簡単だった場合は、それらを残しておき、それで済ませることもできます。しかし、彼らが重要なので、あなたは手でそれらを追加する必要があります。次のコードを考えてみましょう:

if (current_time() - time_since_last_contact() > 20) 
alert_moscow_to_launch(); 
open_silo_hatches(); 
sleep(20); 
response = check_for_message_from_moscow(); 
if (response == LAUNCH) 
launch_missiles(); 
else if (response == PEACE) 
close_silo_hatches(); 
else /* FALSE RESPONSE */ 
launch_missiles(); 

これは角かっこなしの無意味なジャムです。いくつかを追加することができます:

if (current_time() - time_since_last_contact() > 20) { 
    alert_moscow_to_launch(); 
} 
open_silo_hatches(); 
sleep(20); 
response = check_for_message_from_moscow(); 
if (response == LAUNCH) { 
    launch_missiles(); 
} else if (response == PEACE) { 
    close_silo_hatches(); 
} else /* FALSE RESPONSE */ { 
    launch_missiles(); 
} 

それは今一部理にかなっていますが、非常に親切ではありません - それは関係なく、それは指導者との接触があったことからきていますどのくらいのサイロを開きます。リーダーシップとの通信が完全に切断された場合

if (current_time() - time_since_last_contact() > 20) { 
    alert_moscow_to_launch(); 
    open_silo_hatches(); 
    sleep(20); 
    response = check_for_message_from_moscow(); 
    if (response == LAUNCH) { 
     launch_missiles(); 
    } else if (response == PEACE) { 
     close_silo_hatches(); 
    } else { /* no response or false response */ 
     launch_missiles(); 
    } 
} 

このバージョンでは、明らかにのみthe final steps行います。

プログラマーは、かっこがどこにあるか知っている必要があります。問題がある場合は、おそらくアルゴリズムの論文にflow chartを作成してください。ちょっと古い学校ですが、かっこがどこに行くのかを理解するのには良いスタートです。コールバックの世界にはうまく対応していませんが、中括弧の間のブロックをネストされた階層として見ると、かっこがどこに必要なのかを確認するのに役立ちます。

0

は、私はここでそれを固定思う:そう方法でなければなりません

public void onClick;{; 

public class MainActivity extends Activity { 
@Override 
public void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
    setContentView(R.layout.main); 

    final Button button = (Button) findViewById(R.id.toggleButton); 
    button.setOnClickListener(new View.OnClickListener(){ 
     public void onClick(){ 
      newPhoneImage=getResources().getDrawable(R.drawable.i_dens); 
     } 
     public void onClick(View arg0){} 
    }); 
} 

問題がでた

public void onClick(){ 

私はちょうど再構築残り、その一部は過度に使用されています

}; 

これは私が見る限り、決してプログラム内にはないはずです。

関連する問題