ImageViewのgetX(),getY()ができない。

Question

すみません。
分かりにくいところや、間違えて書いていることろがあったので、質問をもう一度書き換えさせてもらいます。

androidstudioでゲーム作りをしています。
kaki[0]～kaki[23]のImageViewと、saru1,saru2のImageViewがmain_activityに設定してあります。
アニメーションが繰り返されるようにコードを書いたつもりですが、アニメーションされず、ImageViewがとまったままになります。

そこで、どこに原因があるのか探すためTextViewに、

ma_tv1.setText("kaki["+kaki_num+"]="+kaki[kaki_num]);
ma_tv2.setText("kaki"+(kaki_num+1)+".x="+ x+" : kaki"+(kaki_num+1)+".y="+y);
ma_tv3.setText("saru1="+saru1);
ma_tv4.setText("saru1.x"+saru1.getX()+" : saru1.y="+saru1.getY());
ma_tv5.setText("list_kaki24"+list_kaki24);
ma_tv6.setText("");
ma_tv7.setText("kazu="+kazu+" : list_kaki_select="+list_kaki_select.size());
ma_tv8.setText("list_kaki_select="+list_kaki_select);

を表示してみました。

ここから見ると

ma_tv2.setText("kaki"+(kaki_num+1)+".x="+ x+" : kaki"+(kaki_num+1)+".y="+y);

に問題があり、
kaki20.x=0.0 : kaki20.y=0.0
となっています。
他の部分には問題はありません。

x = kaki[kaki_num].getX();
y = kaki[kaki_num].getY();
が取得されていないということです。
実際は0.0ではなく違う場所に設置してあるので、そのx座標とy座標を正しく取得したいです。

ImageViewでは、getX(),getY()ではだめなのでしょうか。
どうしたら取得できるか教えていただきたいです。
よろしくお願いします。

import androidx.appcompat.app.AppCompatActivity;
import androidx.constraintlayout.widget.ConstraintLayout;
import androidx.lifecycle.ViewModelProvider;

import android.animation.Animator;
import android.animation.AnimatorInflater;
import android.animation.AnimatorSet;
import android.animation.ObjectAnimator;
import android.content.Intent;
import android.media.Image;
import android.os.Bundle;
import android.os.Handler;
import android.os.Looper;
import android.util.DisplayMetrics;
import android.view.View;
import android.view.animation.Animation;
import android.view.animation.AnimationUtils;
import android.widget.ImageView;
import android.widget.TextView;

import java.util.ArrayList;
import java.util.List;
import java.util.Random;

public class MainActivity extends AppCompatActivity implements Runnable {
    TextView ma_tv1;
    TextView ma_tv2;
    TextView ma_tv3;
    TextView ma_tv4;
    TextView ma_tv5;
    TextView ma_tv6;
    TextView ma_tv7;
    TextView ma_tv8;
    ImageView saru1,saru2;    
    static int w,h;//画面サイズ取得
    ImageView kaki[]=new ImageView[24];//柿全部24こ
    int kakiImageId[]=new int[]{R.id.kaki1, R.id.kaki2,R.id.kaki3, R.id.kaki4,R.id.kaki5, R.id.kaki6,R.id.kaki7, R.id.kaki8,R.id.kaki9, R.id.kaki10,R.id.kaki11, R.id.kaki12,R.id.kaki13, R.id.kaki14,R.id.kaki15, R.id.kaki16,R.id.kaki17, R.id.kaki18,R.id.kaki19, R.id.kaki20,R.id.kaki21, R.id.kaki22,R.id.kaki23, R.id.kaki24};;//柿24このImageId
    boolean flag_animator;
    List<Integer> list_kaki24 = new ArrayList<>();
    List<Integer> list_kaki_select = new ArrayList<>();
    int kazu;//サルがとる柿の数
    int px;//サル画像の左側のマージン
    int kaki_num;//とる柿ナンバー（配列インデックス）
    float x;//とる柿のX座標
    float y;//とる柿のy座標

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        saru1 = findViewById(R.id.saru1);
        saru2 = findViewById(R.id.saru2);

        ma_tv1 = findViewById(R.id.ma_tv1);
        ma_tv2 = findViewById(R.id.ma_tv2);
        ma_tv3 = findViewById(R.id.ma_tv3);
        ma_tv4 = findViewById(R.id.ma_tv4);
        ma_tv5 = findViewById(R.id.ma_tv5);
        ma_tv6 = findViewById(R.id.ma_tv6);
        ma_tv7 = findViewById(R.id.ma_tv7);
        ma_tv8 = findViewById(R.id.ma_tv8);

        for(int i=0;i<24;i++){
           kaki[i]=(ImageView)this.findViewById(kakiImageId[i]);
           list_kaki24.add(i);
        }
        kimeru_kaki();//どの柿をいくつとるか決めておく
        flag_animator = true;
        px = convertDPtoPX(100);//サル画像の左側のマージンが100
        kaki_num=list_kaki_select.get(0);//最初の要素を取り出す
        list_kaki_select.remove(0);
        x = kaki[kaki_num].getX();
        y = kaki[kaki_num].getY();

        if(flag_animator==true&&list_kaki_select.size()!=0){
            kaki_saru_keisan_animation();
        }

        ma_tv1.setText("kaki["+kaki_num+"]="+kaki[kaki_num]);
        ma_tv2.setText("kaki"+(kaki_num+1)+".x="+ x+" : kaki"+(kaki_num+1)+".y="+y);
        ma_tv3.setText("saru1="+saru1);
        ma_tv4.setText("saru1.x"+saru1.getX()+" : saru1.y="+saru1.getY());
        ma_tv5.setText("list_kaki24"+list_kaki24);
        ma_tv6.setText("");
        ma_tv7.setText("kazu="+kazu+" : list_kaki_select="+list_kaki_select.size());
        ma_tv8.setText("list_kaki_select="+list_kaki_select);
    }
    
    public int convertDPtoPX(int dp) {
        DisplayMetrics metrics = this.getResources().getDisplayMetrics();
        int px = (int)(dp*metrics.density);
        return px;
    }

    public void onWindowFocusChanged(boolean hasFocus) {
        super.onWindowFocusChanged(hasFocus);
        View view=findViewById(R.id.ma_app_screen);
        h= view.getHeight();
        w= view.getWidth();
    }
       
    private void kimeru_kaki(){
        Random rand = new Random();
        kazu = rand.nextInt(4) + 5;//5～9の数が持ち去る柿の数
        Random rand2 = new Random();
        for (int i = 0; i < kazu; i++) {//全柿の数24の中からどの柿を持ち去るか決める
            int num = rand2.nextInt(list_kaki24.size());
            list_kaki_select.add(list_kaki24.get(num));
            list_kaki24.remove(num);
        }
    }
    private void kaki_getXY(){
        kaki_num=list_kaki_select.get(0);//最初の要素を取り出す
        x = kaki[kaki_num].getX();
        y = kaki[kaki_num].getY();
    }
    private void kaki_saru_keisan_animation() {
        Random rand=new Random();
        int r=rand.nextInt(3000)+2000;
        float from_x=0;
        float to_x=0;
        float from_xx=0;
        float to_xx=0;
        int jikan=0;
        saru1.setY(y - 48);//48は位置調整
        saru2.setY(y - 48);
        if(kaki_num<12){  //上の段の柿
            jikan = (int) (r *(w-x) / (w + px-20));//移動時間の計算//pxはサル画像のマージン分
            from_x=w+px;
            to_x=x+px-15;
            from_xx=x + px - 15;
            to_xx=w+px;
        }else{  //下の段の柿
            jikan = (int)( r * x / (w + px-20));//移動時間の計算
            from_x=-px;
            to_x=x + px - 15;
            from_xx=x + px - 15;
            to_xx=-px;
        }
        ObjectAnimator animator_yoko_idou = ObjectAnimator.ofFloat(saru1, "translationX", from_x,to_x).setDuration(jikan);//15は位置調整
        ObjectAnimator animator_idou_modoru1 = ObjectAnimator.ofFloat(saru1, "translationX", from_xx, to_xx).setDuration(jikan);
        ObjectAnimator animator_idou_modoru2 = ObjectAnimator.ofFloat(saru2, "translationX", from_xx, to_xx).setDuration(jikan);
        ObjectAnimator animator_toumei = ObjectAnimator.ofFloat(kaki[kaki_num], "alpha", 0f, 0f).setDuration(jikan);
        Random rand2 = new Random();
        int start_ofset = rand2.nextInt(4000);//スタートオフセットの時間
        animator_toumei.setStartDelay(start_ofset + jikan);
        AnimatorSet animatorset_idou_modoru12 = new AnimatorSet();
        animatorset_idou_modoru12.playTogether(animator_idou_modoru1, animator_idou_modoru2);
        AnimatorSet animatorset_ouhuku = new AnimatorSet();
        animatorset_ouhuku.playSequentially(animator_yoko_idou, animatorset_idou_modoru12);
        animatorset_ouhuku.setStartDelay(start_ofset);
        animator_toumei.start();
        animatorset_ouhuku.start();
        flag_animator=false;
        ma_tv6.setText("kaki"+(kaki_num+1)+".x="+x+" : kaki"+(kaki_num+1)+".y="+y);
        animatorset_ouhuku.addListener(new Animator.AnimatorListener() {
            @Override
            public void onAnimationStart(Animator animation) { }
            @Override
            public void onAnimationEnd(Animator animation) {
                flag_animator=true;
                list_kaki_select.remove(0);
                kaki_getXY();
            }
            @Override
            public void onAnimationCancel(Animator animation) { }
            @Override
            public void onAnimationRepeat(Animator animation) { }
        });
    }
}

Comments

[jimbe]

メソッドがパラメータ・戻り値を使用していないため、データがどこでどう設定・参照・変更されているのかが非常に分かり難く読み難くなっています。これはコードとして悪い傾向と見られます。

どの個所で x,y が 0.0 になっているのが確認出来て、どの個所でgetX,getY をされたいのでしょうか。

[jimbe]

編集ありがとうございます。
ma_tv4 に saru1 の x,y は表示されているのですね。

x = kaki[kaki_num].getX();
y = kaki[kaki_num].getY();

と

ma_tv2.setText("kaki"+(kaki_num+1)+".x="+ x+" : kaki"+(kaki_num+1)+".y="+y);

の間に if 文と kaki_saru_keisan_animation の実行がありますが、 kaki_saru_keisan_animation は実行されているのでしょうか。

可能でしたらレイアウトもご提示頂けますでしょうか。

[aruko]

はい、実行できているので、
ma_tv4.setText("saru1.x="+saru1.getX()+" : saru1.y="+saru1.getY());
が、
saru1.x=0.0 : saru1.y=48.0
と表示されます。
これは、kaki_saru_keisan_animationの中の、
saru1.setY(y - 48);//48は位置調整　　　があるから48.0となります。
ですので、最初の投稿でsaru1はxとyが取得できていると書きましたが間違いで、saru1も取得できていないことが分かりました。
結局、ImageViewのx座標y座標がどれも取得できないというシンプルな内容であることに気づきました。
ところが、ここに至るまでの間に色々試しながらアニメーションさせようとしました。
threadを使う中でrunの中でこのアニメーションをスタートさせたところ、アニメーションの連続ができましたが、1つ目のアニメーションだけが今と同じ状態でxyが取得できない状態で始まりましたが、続いて2つ目から最後までは正しくアニメーションできていました。（2つ目からはxyが取得できていた）
サルがランダムに柿をとって戻っていくというアニメーションです。
その1つ目を何とかしようと原因を探っていると、1つ目だけがxyが取得できない状態になっていることが分かったので、runをなくして今の状態にしています。
レイアウトを載せます。
よろしくお願いします。

Answer 1

View の位置 ( やサイズ等 ) は、最終的に Activity の View ツリーが完全に構築されてからでないと取り出せないものと思います。
これは、画面サイズが onCreate 中では取れなかった (0) のと同じ理由です。
ですので、画面サイズと同じく onWindowFocusChanged() の実行されたタイミング(以降)で取り出す必要があるものと思います。

Comments

[aruko]

先週、horizontaiBiasの取得の仕方を教えてもらったように行い、できましたが、そこから計算してxyを出したら、実際の位置とわずかにずれが出たので、xyを取得する方法に変えている状態です。
レイアウトですが、
多分文字数の関係だと思いますが、かなり省略しても載せられないので、こちらに載せます。

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:app="http://schemas.android.com/apk/res-auto"
xmlns:tools="http://schemas.android.com/tools"
android:id="@+id/ma_app_screen"
android:layout_width="match_parent"
android:layout_height="match_parent"
android:visibility="visible">

<ImageView
android:id="@+id/saru2"
android:layout_width="wrap_content"
android:layout_height="wrap_content"

android:layout_marginStart="-100dp"
android:layout_marginLeft="-100dp"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintHorizontal_bias="0.0"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toTopOf="parent"
app:layout_constraintVertical_bias="0.489"
app:srcCompat="@drawable/saru_kaki" />

<ImageView
android:id="@+id/saru1"
android:layout_width="wrap_content"
android:layout_height="wrap_content"

android:layout_marginStart="-100dp"
android:layout_marginLeft="-100dp"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintHorizontal_bias="0.0"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toTopOf="parent"
app:layout_constraintVertical_bias="0.489"
app:srcCompat="@drawable/saru" />

<ImageView
android:id="@+id/ma_gamen"
android:layout_width="0dp"
android:layout_height="0dp"

android:scaleType="fitXY"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintHorizontal_bias="1.0"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toTopOf="parent"
app:layout_constraintVertical_bias="0.0"
app:srcCompat="@drawable/ma_gamen" />

<ImageView
android:id="@+id/kaki24"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintHorizontal_bias="0.958"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toTopOf="parent"
app:layout_constraintVertical_bias="0.923"
app:srcCompat="@drawable/kaki" />

・・・kaki2～kaki23とTextViewは文字数の関係で省略しました。・・・

<ImageView
android:id="@+id/kaki1"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintHorizontal_bias="0.291"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toTopOf="parent"
app:layout_constraintVertical_bias="0.341"
app:srcCompat="@drawable/kaki" />

<ImageView
android:id="@+id/jiisan"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:scaleType="fitXY"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toTopOf="parent"
app:layout_constraintVertical_bias="0.499"
app:srcCompat="@drawable/jiisan_l_mae" />

</androidx.constraintlayout.widget.ConstraintLayout>

[aruko]

いつも解決できるご指導ありがとうございます。
ばっちり連続アニメーションできました。

[jimbe]

>かなり省略しても載せられない
すいませんでした、考えてみれば ImageView だけで 24 個あるのですから長大になるのは間違いなかったですね。