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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Java

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

Android

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

Q&A

解決済

1回答

4006閲覧

AndroidのSurfaceViewの描画が遅い

Rio12882

総合スコア27

Java

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

Android

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

0グッド

0クリップ

投稿2016/10/27 08:43

現在アンドロイドアプリで円が放射運動をするSurfaceViewの開発をしたのですが、FPSが低すぎて、ボールが滑らかに運動しているように見えなくて困っています。

あまりAndroidについて知らないので、基本的なところの理解不足が原因かもしれませんがなにか改善方法があればご教示ください。
よろしくお願いいたします。

public class MySurfaceView extends SurfaceView implements SurfaceHolder.Callback, Runnable { private Paint mPaint = null; private Canvas mCanvas = null; static final long FPS = 20; static final long FRAME_TIME = 1000 / FPS; static final int BALL_R = 10; SurfaceHolder surfaceHolder; Thread thread; DisplayMetrics dm = Resources.getSystem().getDisplayMetrics(); int width = dm.widthPixels; int height = dm.heightPixels; float cx = 0, cy = 0; int screen_width, screen_height; float g = (float) 9.8; int temp1, temp2; double velocity = (width-60)/(35*Math.cos(Math.PI*4/9)); float vcos = (float) (velocity*Math.cos(Math.PI*4/9)); float vsin = (float) (velocity*Math.sin(Math.PI*4/9)); public MySurfaceView(Context context){ super(context); surfaceHolder = getHolder(); surfaceHolder.addCallback(this); } @Override public void surfaceCreated(SurfaceHolder holder) { thread = new Thread(this); thread.start(); } @Override public void surfaceChanged(SurfaceHolder surfaceHolder, int i, int i1, int i2) { } @Override public void surfaceDestroyed(SurfaceHolder surfaceHolder) { thread = null; } @Override public void run() { Canvas canvas = null; Paint paint = new Paint(); Paint bgPaint = new Paint(); // Background bgPaint.setStyle(Paint.Style.FILL); bgPaint.setColor(Color.WHITE); // Ball paint.setStyle(Paint.Style.FILL); paint.setColor(Color.BLUE); long startTime = System.currentTimeMillis(); float time = 0; float cvy; int temp1, temp2; temp1 = (int) (velocity * 10000); temp2 = temp1 % 10; temp1 -= temp2; velocity = ((float)temp1)/10000; temp1 = (int) (vcos * 10000); temp2 = temp1 % 10; temp1 -= temp2; vcos = ((float)temp1)/10000; temp1 = (int) (vsin * 10000); temp2 = temp1 % 10; temp1 -= temp2; vsin = ((float)temp1)/10000; while(thread != null){ time = (System.currentTimeMillis()-startTime)/1000; temp1 = (int) (time * 10000); temp2 = temp1 % 10; temp1 -= temp2; time = ((float)temp1)/10000; cvy = g*time - vsin; cx = vcos*time; cy = cvy*time/2-vsin*time/2; try{ canvas = surfaceHolder.lockCanvas(); canvas.drawCircle( cx+BALL_R, cy+height-200, BALL_R, paint); } catch(Exception e){} finally { surfaceHolder.unlockCanvasAndPost(canvas); } canvas = surfaceHolder.lockCanvas(); canvas.drawColor(0, PorterDuff.Mode.CLEAR); surfaceHolder.unlockCanvasAndPost(canvas); long end = System.currentTimeMillis(); Log.d(getClass().getName(), "measure: " + (end-startTime)); } } }

気になる質問をクリップする

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

abs123

2016/10/28 01:36

動作確認を行っている環境を追記してください。
guest

回答1

0

ベストアンサー

Java

1surfaceHolder.unlockCanvasAndPost(canvas);

上記の処理を一回のループで2回読んでいるから重いのかもしれないです。
下記にループ内の処理を変えてみてはどうですかね?

Java

1while(thread != null){ 2 time = (System.currentTimeMillis()-startTime)/1000; 3 temp1 = (int) (time * 10000); 4 temp2 = temp1 % 10; 5 temp1 -= temp2; 6 time = ((float)temp1)/10000; 7 cvy = g*time - vsin; 8 cx = vcos*time; 9 cy = cvy*time/2-vsin*time/2; 10 11 try{ 12 canvas = surfaceHolder.lockCanvas(); 13 canvas.drawColor(0, PorterDuff.Mode.CLEAR); 14 canvas.drawCircle( 15 cx+BALL_R, cy+height-200, BALL_R, 16 paint); 17 18 } 19 catch(Exception e){} 20 finally { 21 surfaceHolder.unlockCanvasAndPost(canvas); 22 } 23 24 long end = System.currentTimeMillis(); 25 Log.d(getClass().getName(), "measure: " + (end-startTime)); 26 27 }

投稿2016/10/27 12:01

退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Rio12882

2016/10/27 19:40

全く変わらないです。 自分は、cx、cyの計算がfloatの計算なので時間がかかるのかと思いtemp1やtemp2を用いて丸めて計算させたのですが変わりませんでした。 また、cx+=1のように簡単な計算にして描画するとおそらく通常の速さで描画されるのも確認しました。
退会済みユーザー

退会済みユーザー

2016/10/27 23:16 編集

これ処理が重いのではなく、1フレーム辺りのx座標の計算に問題がありそうです。ループ処理内にx座標の値のログを入れて確認してみてください。何ループも同じ値が表示されませんか。 演算処理は基本そんな重い処理ではなく重いのは描画処理なので、あまり考慮する必要はないはずです。
Rio12882

2016/10/28 05:53

なんループも同じxの値が表示されていました。 約20millisごとに更新されている感じでした。30ループごとに値が変更されていてその時間1.6秒程度だったので、まさにこれが問題のようです。 どのようにしたら改善されるのでしょうか。 System.currentTimeMillisを使わないで、一回のループあたりの時間を考慮して time += 0.03 としたところ、期待通りの動きができました。 端末や環境によって1ループの実行時間が違うと思うので、なにか一般性のある改善方法があったら教えていただきたいです。
退会済みユーザー

退会済みユーザー

2016/10/28 11:20

>>端末や環境によって1ループの実行時間が違うと思うので、なにか一般性のある改善方法があったら教えていただきたいです。 Thread.sleep()をループ処理の最後に実装して指定したFPSで描画できる様に調整するのが一般的だと思います。処理落ちしない限り端末差がでなくなります。 そんな感じでやってみてダメだったらまた質問してはどうですか。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問