回答編集履歴

1 メソッド名の修正(setOn'C'lickListener)

swordone

swordone score 19705

2015/02/14 14:12  投稿

あなたがブラウザ起動させるためのインテントを`onClick()`メソッド内に書いていますが、
現状このメソッドを**どこからも呼び出すことがない**状態であると考えられます。
`onClick()`という名称のメソッドは、その名の通り「クリック(スマホ等では"タップ")した時」に呼び出される様に見えますが、
これは`View.OnClickListener`(以下リスナー)というインタフェース内のメソッドなのです。
ただ`onClick()`メソッドを書いても、コードのどこかで明示的に呼び出さない限り実行されません。
(リスナーを実装してViewにセットしなければただの"onClick"という名前のメソッドというだけ)  
クリックした時に何かさせたいのであれば、対象となるボタンなどのViewに対して(仮にButton btとする)
```lang-java
bt.setOnclickListener(/* リスナー実装型オブジェクト */);
bt.setOnClickListener(/* リスナー実装型オブジェクト */);
```
を実行し、どういった動作をするのかは、リスナーを実装したクラスで
`onClick()`をオーバーライドして動作を記述します。
こうすることで、対象のボタンがタップされた時に、**リスナーのonClick()メソッド**が呼び出され、処理が行われます。
通常はViewが置かれているActivity自体にリスナーを実装し、
```lang-java
bt.setOnclickListener(this);
bt.setOnClickListener(this);
```
とするか、無名クラスを使って、
```lang-java
bt.setOnclickListener(new View.OnClickListener(){
bt.setOnClickListener(new View.OnClickListener(){
   @Override
   public void onClick(View v){
       //処理を記述する
   }
});
```
と記述します。
尤も、あなたがどういう操作でブラウザを開きたいのかわからないのでこの回答が適切かはわかりませんが・・・

思考するエンジニアのためのQ&Aサイト「teratail」について詳しく知る