Почему две кнопки перестали работать?

Question

Почему две кнопки изображения перестали работать после добавления кода для намерения электронной почты. Я играл с Public Void onCreate bundle, и когда я перемещаю этот раздел кода ниже, он снова работает. Я вложил их неправильно? Может ли кто-то его исправить?

My OnClickListener в основном не работает, когда кнопка ничего не делает при нажатии.

import android.app.Activity; 
import android.content.Intent; 
import android.os.Bundle; 
import android.view.View; 
import android.view.View.OnClickListener; 
import android.webkit.WebView; 
import android.widget.Button; 
import android.widget.ImageButton; 

public class contactActivity extends Activity implements OnClickListener { 

    private WebView webView; 
    private WebView webView1; 

    @Override 
    public void onCreate(Bundle savedInstanceState) { 
     super.onCreate(savedInstanceState); 
     setContentView(R.layout.contact); 
     Button mail = (Button)findViewById(R.id.Sendbutton); 
     mail.setOnClickListener(this); 
    } 

    public void onClick(View v) { 
     switch(v.getId()) 
     { 
      case R.id.Sendbutton: 
       Intent emailIntent = new Intent(android.content.Intent.ACTION_SEND); 
       String[] recipients = new String[]{"email add", "",}; 
       emailIntent.putExtra(android.content.Intent.EXTRA_EMAIL, recipients); 
       emailIntent.putExtra(android.content.Intent.EXTRA_SUBJECT, "Sample mail"); 
       emailIntent.putExtra(android.content.Intent.EXTRA_TEXT, "This is a sample mail.."); 
       emailIntent.setType("text/plain"); 
       startActivity(Intent.createChooser(emailIntent, "Send mail client :")); 
       finish(); 
       break; 
     } 

     ImageButton ViewFacebookFeed = (ImageButton) findViewById(R.id.ViewFacebookFeed); 
     ImageButton ViewTwitterFeed = (ImageButton) findViewById (R.id.ViewTwitterFeed); 

     //Add a listener to ImageButton1 
     ViewFacebookFeed.setOnClickListener(new OnClickListener() {      
      @Override 
      public void onClick(View v) { 
       openBrowser1(); 
      }   
     });        
     ViewTwitterFeed.setOnClickListener(new OnClickListener() {      
      @Override 
      public void onClick(View v) { 
       openBrowser2(); 
      }   
     }); 
    } 
    private void openBrowser1() 
    {  
     //this intent is used to open other activity which contains another webView 
     Intent intent = new Intent(contactActivity.this,fbtwitterfeedActivity.class); 
     startActivity(intent); 
    } 
    private void openBrowser2() { 
     // TODO Auto-generated method stub 
     Intent intent1 = new Intent(contactActivity.this,twitterfeedActivity.class); 
     startActivity(intent1); 
    } 
} 

Answer 1

, как вы были это реализовать, вы только зарегистрировали щелчки кнопки после того, как вы нажали на кнопку «отправить». Вы вызываете finish() в send onClick, чтобы слушатели никогда не регистрировались. Зарегистрируйте своих слушателей в onCreate, а затем прослушайте клики по определенному идентификатору вида в методе onClick. Я исправил свой код, чтобы вы зарегистрировать клик слушатель в нужное время и ваш метод OnClick обрабатывает все щелчки от всех кнопок

public class contactActivity extends Activity implements OnClickListener { 

    private WebView webView; 
    private WebView webView1; 

    @Override 
    public void onCreate(Bundle savedInstanceState) { 
     super.onCreate(savedInstanceState); 
     setContentView(R.layout.contact); 
     Button mail = (Button)findViewById(R.id.Sendbutton); 
     mail.setOnClickListener(this); 

     ImageButton viewFacebookFeed = (ImageButton) findViewById(R.id.ViewFacebookFeed); 
     viewFacebookFeed.setOnClickListener(this); 
     ImageButton viewTwitterFeed = (ImageButton) findViewById (R.id.ViewTwitterFeed); 
     viewTwitterFeed.setOnClickListener(this); 
    } 

    public void onClick(View v) { 
     switch(v.getId()) 
     { 
      case R.id.Sendbutton: 
       Intent emailIntent = new Intent(android.content.Intent.ACTION_SEND); 
       String[] recipients = new String[]{"email add", "",}; 
       emailIntent.putExtra(android.content.Intent.EXTRA_EMAIL, recipients); 
       emailIntent.putExtra(android.content.Intent.EXTRA_SUBJECT, "Sample mail"); 
       emailIntent.putExtra(android.content.Intent.EXTRA_TEXT, "This is a sample mail.."); 
       emailIntent.setType("text/plain"); 
       startActivity(Intent.createChooser(emailIntent, "Send mail client :")); 
       finish(); 
       break; 
      case R.id.ViewFacebookFeed: 
       openBrowser1(); 
       break; 
      case R.id.ViewTwitterFeed: 
       openBrowser2(); 
       break; 

     } 
    } 
    private void openBrowser1() 
    {  
     //this intent is used to open other activity which contains another webView 
     Intent intent = new Intent(contactActivity.this,fbtwitterfeedActivity.class); 
     startActivity(intent); 
    } 
    private void openBrowser2() { 
     // TODO Auto-generated method stub 
     Intent intent1 = new Intent(contactActivity.this,twitterfeedActivity.class); 
     startActivity(intent1); 
    } 
} 

Answer 2

Лучше не реализовывать интерфейс из активности, а непосредственно из объекта. Я rewritted своего кода, то лучше так:

import android.app.Activity; 
import android.content.Intent; 
import android.os.Bundle; 
import android.view.View; 
import android.view.View.OnClickListener; 
import android.webkit.WebView; 
import android.widget.Button; 
import android.widget.ImageButton; 

public class contactActivity extends Activity { 

    private WebView webView; 
    private WebView webView1; 

    @Override 
    public void onCreate(Bundle savedInstanceState) { 
     super.onCreate(savedInstanceState); 
     setContentView(R.layout.contact); 
     Button mail = (Button) findViewById(R.id.Sendbutton); 
     ImageButton ViewFacebookFeed = (ImageButton) findViewById(R.id.ViewFacebookFeed); 
     ImageButton ViewTwitterFeed = (ImageButton) findViewById(R.id.ViewTwitterFeed); 
     // Add a listener to ImageButton1 
     ViewFacebookFeed.setOnClickListener(new OnClickListener() { 
      @Override 
      public void onClick(View v) { 
       openBrowser1(); 
      } 
     }); 
     ViewTwitterFeed.setOnClickListener(new OnClickListener() { 
      @Override 
      public void onClick(View v) { 
       openBrowser2(); 
      } 
     }); 
     mail.setOnClickListener(new OnClickListener() { 
      @Override 
      public void onClick(View v) { 
       Intent emailIntent = new Intent(android.content.Intent.ACTION_SEND); 
       String[] recipients = new String[] { "email add", "", }; 
       emailIntent.putExtra(android.content.Intent.EXTRA_EMAIL, recipients); 
       emailIntent.putExtra(android.content.Intent.EXTRA_SUBJECT, "Sample mail"); 
       emailIntent.putExtra(android.content.Intent.EXTRA_TEXT, "This is a sample mail.."); 
       emailIntent.setType("text/plain"); 
       startActivity(Intent.createChooser(emailIntent, "Send mail client :")); 
       finish(); 
      } 

     }); 
    } 

    private void openBrowser1() { 
     // this intent is used to open other activity which contains another 
     // webView 
     Intent intent = new Intent(contactActivity.this, fbtwitterfeedActivity.class); 
     startActivity(intent); 
    } 

    private void openBrowser2() { 
     // TODO Auto-generated method stub 
     Intent intent1 = new Intent(contactActivity.this, twitterfeedActivity.class); 
     startActivity(intent1); 
    } 

}