質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.35%

  • Java

    14414questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Android

    6730questions

    Androidは、Google社が開発したスマートフォンやタブレットなど携帯端末向けのプラットフォームです。 カーネル・ミドルウェア・ユーザーインターフェイス・ウェブブラウザ・電話帳などのアプリケーションやソフトウェアをひとつにまとめて構成。 カーネル・ライブラリ・ランタイムはほとんどがC言語/C++、アプリケーションなどはJavaSEのサブセットとAndroid環境で書かれています。

  • Android Studio

    3896questions

    Android Studioは、 Google社によって開発された、 Androidのネイティブアプリケーション開発に特化した統合開発ツールです。

【android studio】 ブロックを0.5秒ごとに落下させたいのですがThread.sleep(500);を使ってもうまくいきません原因は何でしょうか

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 272

buelur

score 5

 質問

Thread.sleep(500);を使うとsleepを使う分だけ起動の時間が遅くなっているように感じます。
原因は何か教えて頂けると幸いです。

 ソースコード

import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.widget.TextView;

public class MainActivity extends AppCompatActivity {
    String screen;
    int random, i, j, x, y, z;

    int field[][] =                                             //パズルの中身
            {     // 0  1  2  3  4  5  6  7  8  9  10 11 12
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //0
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //1
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //2
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //3
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //4
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //5
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //6
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //7
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //8
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //9
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //10
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //11
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //12
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //13
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //14
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //15
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //16
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //17
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //18
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //19
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //20
            };

    int blocktype[][][][] = {             //ブロックタイプ 7種 回転 0度、90度、180度、360度
            {
                    {                     //0-0
                            {0, 1, 0, 0},
                            {0, 1, 0, 0},
                            {0, 1, 0, 0},
                            {0, 1, 0, 0}
                    },
---------------------------------------------------------------------省略
                    {                     //6-3
                            {0, 1, 0, 0},
                            {1, 1, 1, 0},
                            {0, 1, 0, 0},
                            {0, 0, 0, 0}
                    }
            }
    };

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

        blockoutput();
    }

    protected void display() {                 //画面に表示
        TextView textView = findViewById(R.id.puzzle);
        screen = "";

        for (i = 0; i <= 20; i++) {
            for (j = 0; j <= 12; j++) {
                if (field[i][j] == 1) {
                    screen = screen + '■';
                } else if (field[i][j] == 0) {
                    screen = screen + '□';
                } else {
                    screen = screen + '\n';
                }
            }
        }
        textView.setText(screen);
    }

    protected void blockoutput() {              //7種のブロックをrandomに1つ取り出して表示
        random = (int) (Math.random() * 10 % 6);

        display();
        for (z = 0; z + 3 <= 20; z++) {
            for (x = 0; x <= 3; x++) {
                for (y = 0; y <= 3; y++) {
                    if (blocktype[random][0][x][y] == 1) {
                        field[z + x][4 + y] = blocktype[random][0][x][y];
                    }
                }
            }
            try {
                Thread.sleep(500);
                display();
            } catch (InterruptedException e) {
            }
        }
    }
}

修正後

import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.widget.TextView;


public class MainActivity extends AppCompatActivity {
    String screen;
    int random, i, j, x, y, z;

    int field[][] =                                             //パズルの中身
            {     // 0  1  2  3  4  5  6  7  8  9  10 11 12
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //0
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //1
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //2
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //3
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //4
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //5
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //6
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //7
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //8
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //9
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //10
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //11
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //12
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //13
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //14
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //15
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //16
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //17
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //18
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //19
                    {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2},    //20
            };

    int blocktype[][][][] = {             //ブロックタイプ 7種 回転 0度、90度、180度、360度
            {
                    {                     //0-0
                            {0, 1, 0, 0},
                            {0, 1, 0, 0},
                            {0, 1, 0, 0},
                            {0, 1, 0, 0}
                    },
                   -------------------------------------省略
                    {                     //6-3
                            {0, 1, 0, 0},
                            {1, 1, 1, 0},
                            {0, 1, 0, 0},
                            {0, 0, 0, 0}
                    }
            }
    };

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

        blockoutput();
    }

    protected void display() {                 //画面に表示
        TextView textView = findViewById(R.id.puzzle);
        screen = "";

        for (i = 0; i <= 20; i++) {
            for (j = 0; j <= 12; j++) {
                if (field[i][j] == 1) {
                    screen = screen + '■';
                } else if (field[i][j] == 0) {
                    screen = screen + '□';
                } else {
                    screen = screen + '\n';
                }
            }
        }
        textView.setText(screen);
    }

    protected void blockoutput() {              //7種のブロックをrandomに1つ取り出して表示
        random = (int) (Math.random() * 10 % 6);
        time tm = new time();

        display();
        for (z = 0; z + 3 <= 20; z++) {
            for (x = 0; x <= 3; x++) {
                for (y = 0; y <= 3; y++) {
                    if (blocktype[random][0][x][y] == 1) {
                        field[z + x][4 + y] = blocktype[random][0][x][y];
                    }
                }
            }
            display();
            tm.start();
            for (x = 0; x <= 3; x++) {
                for (y = 0; y <= 3; y++) {
                    field[z + x][4 + y] = 0;
                }
            }
        }
    }
}


class time extends Thread {
    public void run() {
        try {
            Thread.sleep(500);
        } catch (InterruptedException e) {
        }
    }
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • y_waiwai

    2018/07/29 20:20

    うまくいかないとは具体的にどうなるんですか?

    キャンセル

  • buelur

    2018/07/29 20:32

    起動時に(sleep(500)×繰り返しの数)暗い画面が続き、表示されると落下処理が終了した後になっている。という状況です、説明になっているでしょうか?

    キャンセル

  • asahina1979

    2018/07/29 22:14

    メインスレッドをとめてるからでは?

    キャンセル

  • buelur

    2018/07/29 23:08

    for (x = 0; x < 1000000000; x++){}に変えてみたのですが起動が遅くなるだけでしたので、仕様上の問題なのでしょうか?

    キャンセル

回答 2

checkベストアンサー

+2

たぶん、以下のようにすると想定しているような動きになると思います。

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

    new Thread(new Runnable() {
        @Override
        public void run() {
            blockoutput();
        }
    }).start();
}


みなさんがおっしゃっている通り、ただThread.sleep(500);としてもそこで処理するのを待つだけになります。
一連の処理が終わってから画面が描画されるので途中で止めてもその間に描画されることはありません。
そこで別のスレッドで処理を行うことで、メインの処理が止まらないので描画が行われます。
スレッドの概念などについては調べてみてください。

コメントへの回答:
アプリが落ちた場合にはエラー内容を貼っていただけるとありがたいです。
tm.start();の部分は特にこのように処理する必要もなく、動かなかったのでtime tm = new time();も消して、以下に書き換えて単純にsleepだけするようにしていました。

try {
    Thread.sleep(500);
} catch (InterruptedException e) {
}

これで私の環境では動いていました。
もしこれで動かなかった際には以下のように書き換えてみてください。
textView.setText(screen)のみmainスレッドで処理するようにしています。

public class MainActivity extends AppCompatActivity {

Handler mHandler;

// 省略

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

    mHandler = new Handler();

    new Thread(new Runnable() {
        @Override
        public void run() {
            blockoutput();
        }
    }).start();
}

   protected void display() {

        screen = "";

        for (i = 0; i <= 20; i++) {
            for (j = 0; j <= 12; j++) {
                if (field[i][j] == 1) {
                    screen = screen + '■';
                } else if (field[i][j] == 0) {
                    screen = screen + '□';
                } else {
                    screen = screen + '\n';
                }
            }
        }

        mHandler.post(new Runnable() {
            @Override
            public void run() {
                TextView textView = findViewById(R.id.puzzle);
                textView.setText(screen);
            }
        });
    }
// 省略

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/30 20:15

    ありがとうございます。
    Thread.sleep(500);で実行すると、fieldとbockの最初の表示はうまくいくのですが落下処理に入ってしまうとアプリが落ちてしまいます。
    Thread.sleep(500);の値を100や3000に変えてみると、ブロックの位置が落下処理1回目の位置や4回目の位置が表示され、その状態からThread.sleep();の値の秒数経過するとアプリが落ちてしまいます。
    このことから、onCreateの処理が終わるまでにtextに代入された文字列が表示されているように感じますが、私のプログラムはどこか間違っているのでしょうか?

    キャンセル

  • 2018/07/31 10:55

    ありがとうございます。
    ご回答の通りに実行すると想定した動きになりました。
    長文に加えソースコードまで書いて下さり、ありがとうございました。

    キャンセル

  • 2018/07/31 11:42

    想定通り動いたのであればよかったです。ゲームのようなものを実装するとなると通常のアプリ開発とはまた違った感覚が必要になったりすると思いますので少々大変かもしれませんね。(スレッドに関しては一般的な知識ではありますが、ちょっと難しいですよね。)

    キャンセル

+2

onCreateでblockoutputを呼び出しているせいです。
起動時にonCreateから呼ばれたblockoutputがループごとに0.5秒停止することになるため、
onCreateの完了に時間がかかり、起動が遅くなるのです。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/29 23:18

    秒数を稼ぐためにThread.sleep(500);からfor (x = 0; x < 1000000000; x++){}に変えてみたのですが、症状は変わらず、起動の時間が遅くなるだけでしたので、別の所に原因があると思うのですがどうでしょうか?

    キャンセル

  • 2018/07/29 23:24

    だからそれはonCreateでやってるからですってば。
    onCreateはActivity起動時の画面構成を担ってて、全部終わらないと表示しないんです。

    アニメーションの部類に入るため、SurfaceView(だったっけ?)などを使わないといけないのでは?

    キャンセル

  • 2018/07/29 23:43

    テキストでテトリミノを描いているだけのようなので、SurfaceViewとかは必要なさそうですね。blockoutputを別スレッド化すればいいと思うのですが、さてどうしたものか・・・

    キャンセル

  • 2018/07/30 17:31

    すみません。一通りjavaは学んだのですが実践的なブログラムはこれが初めてで、知識が及ばずご迷惑をおかけしました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.35%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る

  • Java

    14414questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • Android

    6730questions

    Androidは、Google社が開発したスマートフォンやタブレットなど携帯端末向けのプラットフォームです。 カーネル・ミドルウェア・ユーザーインターフェイス・ウェブブラウザ・電話帳などのアプリケーションやソフトウェアをひとつにまとめて構成。 カーネル・ライブラリ・ランタイムはほとんどがC言語/C++、アプリケーションなどはJavaSEのサブセットとAndroid環境で書かれています。

  • Android Studio

    3896questions

    Android Studioは、 Google社によって開発された、 Androidのネイティブアプリケーション開発に特化した統合開発ツールです。

  • トップ
  • Javaに関する質問
  • 【android studio】 ブロックを0.5秒ごとに落下させたいのですがThread.sleep(500);を使ってもうまくいきません原因は何でしょうか