android ボタンがクリックされたら画像が変わるようにするには？OnClick imageView

Question

buttonがクリックされたら、画像がapple4(リンゴ）というものからeatenapple（かじられたリンゴ）にかわるソースです。

もう一度クリックするとapple4に戻り、またクリックするとeatenapple4に戻したいのですが、理屈は下記であってますか？

下記の、

if(imageView2の画像が R.drawable.apple4の時)

の部分をandroid studioでどう表記したらいいでしょうか？

if(imageView2の画像が R.drawable.apple4の時){
        button.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                imageView2.setImageResource(R.drawable.eatenapple);
       }
        });
   
} else {
        button.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                imageView2.setImageResource(R.drawable.apple4);

}

元ソース（ワンクリックしたら画像が変わるが、もう一度クリックしても何も起こらない）

@Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar);
        setSupportActionBar(toolbar);

        Button button = (Button) findViewById(R.id.button);
        final ImageView imageView2 = (ImageView) findViewById(R.id.imageView2);

        button.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                imageView2.setImageResource(R.drawable.eatenapple);
                


            }
        });


    }

Answer 1

ベーシックな方法はこちらだと思いますが、今回はコードで実装するということでしょうか？

Comments

[yona]

Viewがアクティブかどうかを使うのは副作用が多すぎてオススメではないですよ。

Answer 2

質問のコードをどこに書くかによってできるかできないかは変わります。

やりたいことは２つの画像をボタンを押すたびに交互に切り替えることですよね。
どこかにどの画像を表示しているかを保持する必要があります。
今回はフィールド変数を使いました。

@Override
public void onClick(View v) {
if(i == 1){
  i = 0;
    imageView2.setImageResource(R.drawable.apple4);
}else{
   i = 1;
    imageView2.setImageResource(R.drawable.eatenapple);
}

}

Comments

[masaakitsuyoshi]

ありがとうございます！

imageViewに表示されている画像のidを取得する方法を探してましたが、
こういうやり方もあるんですね。

onCreateの中に書くと、エラーは出ないのですが、アプリ実行の時に"has stopped"
となるのですが、これは何でしょうか。

[yona]

前調べた記憶が正しければ、ImageViewはリソースやファイルなどいろいろな画像が表示できます。そのため、さまざまな画像はBitmapに変換されてしまうため、ImageViewに何のリソース由来の画像かは情報として持っていないと思われます。

また、エラーについてはログを見ないとわからないですよ。

[masaakitsuyoshi]

onClick(View v)
の引数の部分に、

cannot resolve symbol "v"
expected ")", ";"

と出てます。

@override
はoncreateの中で使えません、というエラーも出てます。

onClickメソッドの部分に、method call expected と出ています。

※imageview2 をimageViewに変えました

@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);
Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar);
setSupportActionBar(toolbar);

Button button = (Button) findViewById(R.id.button);
final ImageView imageView = (ImageView) findViewById(R.id.imageView);
int i = 0;

public void onClick (View v) {
if(i == 1){
i = 0;
imageView.setImageResource(R.drawable.apple4);
}else{
i = 1;
imageView.setImageResource(R.drawable.eatenapple);
}

}
onClick(button);

}

[yona]

ごめんなさい、プログラムは省略して書いてあるので、質問の元ソースのbutton.setOnClickListnerの中にあるらonClickに修正を加えてください。
あと、iはフィールド変数にしないとダメですよ。

[masaakitsuyoshi]

package com.example.dddd;

import android.os.Bundle;
import android.support.v7.app.AppCompatActivity;
import android.support.v7.widget.Toolbar;
import android.view.View;
import android.view.Menu;
import android.view.MenuItem;
import android.widget.Button;
import android.widget.ImageView;

public class MainActivity extends AppCompatActivity {

Button button = (Button) findViewById(R.id.button);
final ImageView imageView = (ImageView) findViewById(R.id.imageView);

@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);
Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar);
setSupportActionBar(toolbar);

button.setOnClickListener(new View.OnClickListener() {
int i = 0;

@Override
public void onClick(View v) {

if(i == 1){
i = 0;
imageView.setImageResource(R.drawable.apple4);
}else{
i = 1;
imageView.setImageResource(R.drawable.eatenapple);
}
}
});

}

でしょうか？？
どのクラスのフィールド変数にしたらいいのかがよくわかりません。

[yona]

MainActivityのフィールド変数にしてください。
また、buttonを初期化するときに、Activity#findViewByIdはインスタンスメソッドなのでフィールド変数の初期化には使えないと思いますよ。

[masaakitsuyoshi]

動作するようになりました！

[yona]

今回はとりあえずフィールド変数にしましたが、ケースによってはView#setTagを使う方法などもあるので調べてみた方がいいでしょう。

[masaakitsuyoshi]

調べてみます！ありがとうございました！