配列初歩 ブロック崩しのブロックの配列がうまく描画されない android Java

Question

androidでブロック崩しを作っています。
配列の知識が乏しく、ブロックが正常に描画されないためご教授いただけると幸いです。

下記にてBlockクラスの配列をblock[]として作成、
その後 値をそれぞれ入れました。

SurfaceView内のrunメソッド内にてforし、drawRectしています。
エラーはないのですが、ブロックが描画されません。

MainActivity

//ーーーーー中略ーーーーー

class CustomView extends SurfaceView implements SurfaceHolder.Callback, Runnable {

        // スレッドクラス
        Thread mainLoop = null;
        // 描画用
        Paint paint = null;
        Paint paint2 = null;
        Paint paint3 = null;


        // 円のX,Y座標
        private int circleX = 50;
        private int circleY = 50;
        private int circle2X = 30;
        private int circle2Y = 30;
        // 円の移動量
        private int circleVx = 15;
        private int circleVy = 15;
        private int circle2Vx = 8;
        private int circle2Vy = 15;

        //今は6個だが、Block[ここに変数を入れて] 画面サイズによって表示できる個数を変えたい
        Block block[] = new Block[6];


        int view_w, view_h; //画面サイズ
        Resources res = this.getContext().getResources();
        Bitmap balor = BitmapFactory.decodeResource(res, R.drawable.balor);
        Bitmap sraimKing = BitmapFactory.decodeResource(res, R.drawable.sraim_king);


//ーーーーー中略ーーーーー

//コンストラクタ
        public CustomView(Context context) {
            super(context);
            // SurfaceView描画に用いるコールバックを登録する。
            getHolder().addCallback(this);
            // 描画用の準備
            paint = new Paint();
            paint2 = new Paint();
            paint3 = new Paint();
            //ボール
            paint.setColor(Color.GREEN);
            paint2.setColor(Color.CYAN);
            //ブロック
            paint3.setColor(Color.RED);



        }

//ーーーーー中略ーーーーー

@Override
        public void run(){

            while (true) {
                Canvas canvas = getHolder().lockCanvas();
                if (canvas != null)
                {
                    view_h = canvas.getHeight();
                    view_w = canvas.getWidth();

                    block[0]= new Block(view_w/10, view_h - view_h +300);
                    block[1]= new Block(view_w/10 + 100, view_h - view_h +300);
                    block[2]= new Block(view_w/10 + 200, view_h - view_h +300);
                    block[3]= new Block(view_w/10 + 300, view_h - view_h +300);
                    block[4]= new Block(view_w/10 + 400, view_h - view_h +300);
                    block[5]= new Block(view_w/10 + 500, view_h - view_h +300);

                // max720 1230
                    for(int i=0; i < 6;i+=1){

                        canvas.drawRect(block[i].x -40,block[1].y, block[i].x +40, block[i].y +20,paint3);

                }

//ーーーーー中略ーーーーー

Blockクラス

package com.example.canvastest;


public class Block {

    float x;
    float y;
    int margin = 10; //ブロックの間のマージン
    int i; //カウンター

    //コンストラクタ
    public Block (int _x,int _y){
        this.x = (float)_x;
        this.y = (float)_y;
    }

}

![

Answer 1

yonaさんの意見を参考に、コード自体にエラーがないなら描画に問題があるんじゃね？と思い。

解決方法
canvas.drawColor(Color.BLACK);
背景を描くメソッドをrunの一番上に持ってくる。
その後にブロックやバー、ボール（モンスター）の描画メソッドを描く

基本事項
プログラムって絶対に上から下に処理していくので、背景描画処理などは最初に行う。

Comments

[fuzzball]

スライムの綴りはSlimeです。

[moredeep]

回答が参考になったならばその回答をベストアンサーにし、この回答の内容はコメントに書くべきです。

[fuzzball]

>>moredeepさん
「べき」の根拠を教えて下さい。どこかで明文化されているでしょうか？

[masaakitsuyoshi]

>>fuzzball さん
スペルミスでした！
>>moredeepさん
直接的に「書く順序を入れ替えるだけで直る」ということを示したかったため自己解決をベストアンサーにしました。
その間の悩んでいる過程やデバッグの過程はあくまで参考なのかな、と。
念のためyonaさんのアンサーをベストアンサーにしました。

Answer 2

・スレッドはスタートしていますか？
・view_h - view_h + 300は常に300ですよね。
・for文の中の第二引数の配列の要素番号が1になっている。

また、drawRectの直前にログ出力処理を追加して各値を出力して見てください。

Comments

[masaakitsuyoshi]

view_h - view_h + 300
画面サイズの違うスマホでも同じ位置に表示させたかったのですが、書き方が間違っていたようです。
block[0]= new Block(view_w/10, view_h/10*3);
このようにしました。

forの第２引数、確かに１でした。。。

・スレッドはスタートしていますか？
　おそらくスタートしていると思います. surfaceCreatedの中でスレッドをnewし、スタートしています。デバッガで見ても、スレッドスタートされた後、@Override runの処理に正常にうつっているようです。

>>drawRectの直前にログ出力処理を追加して各値

block[3]　Block@3847
x=372 y=369(一定）
block[4] Block@3850
x=472 y=369
このように一応値は入っているようです。

[yona]

確かに、コードは問題なさそうですね。
今度はpaint3をどのように初期化しているかを教えてください。

[masaakitsuyoshi]

ご回答ありがとうございます。

CustomViewのメンバ変数にて
Paint paint3 = null;

CustomViewコンストラクタ内で
paint3 = new Paint();
paint3.setColor(Color.RED);

としています。

MainActivityに上記コード追加しました。

[masaakitsuyoshi]

surfaceviewの背景をコメントアウトしてみたら、ブロックが表示されました。
その代わりボール（画像でモンスターにしています）の挙動がおかしくなってしまいましたが。。。

[masaakitsuyoshi]

yonaさんの意見を参考に、自己解決しました！

解決方法
canvas.drawColor(Color.BLACK);
背景を描くメソッドをrunの一番上に持ってくる。
その後にブロックやバー、ボール（モンスター）の描画メソッドを描く

基本事項
プログラムって絶対に上から下に処理していくので、背景描画処理などは最初に行う。