Gravity

Question

package com.example.takamasaawai.myapplication;

import android.view.Gravity;

import java.util.Random;

public class TextViewGravity {
    int[] g;
    Random r;
    
    void gravity() {
        g = new int[]{Gravity.CENTER, Gravity.LEFT, Gravity.RIGHT, Gravity.TOP,
                Gravity.BOTTOM, Gravity.FILL, Gravity.FILL_VERTICAL, Gravity.FILL_HORIZONTAL,
                Gravity.CENTER_VERTICAL, Gravity.CENTER_VERTICAL,
                Gravity.TOP | Gravity.CENTER, Gravity.TOP | Gravity.CENTER_HORIZONTAL, Gravity.TOP | Gravity.RIGHT,
                Gravity.TOP | Gravity.LEFT, Gravity.BOTTOM | Gravity.LEFT, Gravity.BOTTOM | Gravity.RIGHT};


        int[] newlist = new int[g.length];
        for (int i = 0; i < g.length; i++) {
            while (true) {
                int a = r.nextInt(g.length);
                newlist[i] = g[a];
                break;
            }
        }

        g = r.nextInt(g.length);
    }
    int[] getG() {
        return g;
    }
    }

いま、こんなオブジェクトでランダムなgravityを生成しているのですが、 g = r.nextInt(g.length);がエラーになります。どうしてでしょうか？また、その他修正点ありましたらお願いします。
ちなみに、メインのクラスで設定しているところは
textView = (TextView) findViewById(R.id.t);
TextViewGravity tvg = new TextViewGravity();
tvg.gravity();
textView.setGravity(tvg.getG());

Answer 1

今gはint型の配列で，それに対してint型のr.nextInt(g.length)を代入しようとしているためです．

他には

        int[] newlist = new int[g.length];
        for (int i = 0; i < g.length; i++) {
            while (true) {
                int a = r.nextInt(g.length);
                newlist[i] = g[a];
                break;
            }
        }

これが配列の並び替えなら，これではダメです．同じ要素が重複して並ぶ可能性があります．

      textView.setGravity(tvg.getG());

getG()はint型配列を返すので，int型を引数に取るsetGravityには渡せません．

      textView.setGravity(tvg.getG()[0]);

こういう形なら可能ですが．

Gravity値をランダムに出したいだけならわざわざ並び替えをする必要もなく，

package com.example.takamasaawai.myapplication;

import android.view.Gravity;

import java.util.Random;

public class TextViewGravity {
    int[] g = {Gravity.CENTER, Gravity.LEFT, Gravity.RIGHT, Gravity.TOP,
                Gravity.BOTTOM, Gravity.FILL, Gravity.FILL_VERTICAL, Gravity.FILL_HORIZONTAL,
                Gravity.CENTER_VERTICAL, Gravity.CENTER_VERTICAL,
                Gravity.TOP | Gravity.CENTER, Gravity.TOP | Gravity.CENTER_HORIZONTAL, Gravity.TOP | Gravity.RIGHT,
                Gravity.TOP | Gravity.LEFT, Gravity.BOTTOM | Gravity.LEFT, Gravity.BOTTOM | Gravity.RIGHT};
    Random r = new Random();
    
    int getRndGravity() {
        return g[r.nextInt(g.length)];
    }
}

これで事足ります．
また，個人的にはわざわざインスタンス化する必要もない気がします．
int g[]→private static int g[] にして，
Random rの行を消して，

    public static int getRndGravity() {
        return g[(int)(Math.random() * g.length)];
    }

これで呼び出すときTextViewGravity.getRndGravity()とすれば，
gから毎回ランダムに値が取り出されます．