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

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

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

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

Eclipse

Eclipseは、IBM社で開発された統合開発環境のひとつです。2001年11月にオープンソース化されました。 たくさんのプラグインがあり自由に機能を追加をすることができるため、開発ツールにおける共通プラットフォームとして位置づけられています。 Eclipse自体は、Javaで実装されています。

Q&A

解決済

1回答

1335閲覧

自作アプリを動作させるとエミュレータが落ちてしまいます。

eldorado

総合スコア17

Android

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

Eclipse

Eclipseは、IBM社で開発された統合開発環境のひとつです。2001年11月にオープンソース化されました。 たくさんのプラグインがあり自由に機能を追加をすることができるため、開発ツールにおける共通プラットフォームとして位置づけられています。 Eclipse自体は、Javaで実装されています。

0グッド

0クリップ

投稿2014/10/28 13:03

EclipseにてJava,Androidの製作を行っているのですが連続して当たり判定を行う際にフリーズを起こしてしまいます。
LogCatに
D/dalvikvm(900): GC_CONCURRENT freed 503K, 7% free 10097K/10759K, paused 15ms+9ms
が延々と表示されてしまい、調べてみても核心的な何かを見つけ出せずにいます。
もしわかる方いましたら教えてください!
アプリ内では複数の円をランダムに生成しており円の生成時に他の円との衝突判定をみて重ならないようにしています。
円をタップして円を再生成させるタイミングの衝突判定を呼び出しているところを削除すると動くようになるのですが何が原因でしょうか?

問題のコードの一部を添付いたします。

lang

1class BallView extends View { 2 ArrayList<Ball> ball; 3 4 Point size; 5 Display disp; 6 Random rn; 7 8 public BallView(Context con) { 9 10 super(con); 11 rn = new Random(); 12 WindowManager wm = (WindowManager) getSystemService(WINDOW_SERVICE); 13 // ディスプレイのインスタンス生成 14 disp = wm.getDefaultDisplay(); 15 size = new Point(); 16 disp.getSize(size); 17 ball = new ArrayList<Ball>(); 18 for (int i = 0; i &lt; cnt; i++) { 19 Ball b = new Ball(); 20 b = ReBall(b); 21 ball.add(b); 22 } 23 } 24 25 public Ball ReBall(Ball b) { 26 do { 27 // ランダムで位置を生成 28 b.x = rn.nextInt(size.x); 29 b.y = rn.nextInt(size.y); 30 31 } while (HitBall(b));//ここの当たり判定を消すと動くようになる。 32 33 return b; 34 } 35 36 public boolean HitTouch(int TouchX, int TouchY) { 37 for (Ball b : ball) { 38 // タッチの当たり判定 39 if ((((b.x - TouchX) * (b.x - TouchX)) + ((b.y - TouchY) * (b.y - TouchY))) &lt;= (Radius * Radius)) { 40 // タッチしたらスコアを++して再生成 41 Score++; 42 b = ReBall(b); 43 invalidate(); 44 return true; 45 } 46 } 47 return false; 48 } 49 50 public boolean HitBall(Ball bb) { 51 for (Ball b : ball) { 52 // ボールとの当たり判定 53 if ((((b.x - bb.x) * (b.x - bb.x)) + ((b.y - bb.y) * (b.y - bb.y))) <= ((Radius + Radius) * (Radius + Radius))) { 54 return true; 55 } 56 } 57 return false; 58 } 59 }

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

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

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

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

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

guest

回答1

0

ベストアンサー

実際に動かしてみました。
コメントにある消すと動くあたり判定が悪さをしているのは確実ですね。
もしかしたらReBallの引数でリスト内のBallを渡していますが
渡されたBall(ランダムで変更される)ものとList内のBallが同じ実体を持っているので
変更される→変更されたもの同士を比べる→変更する。
を繰り返して無限ループに陥ってしまっているのではないでしょうか
ReBallは引数をなしにして新たなBallを返すようにしたらよいと思います。

投稿2014/10/28 13:52

codingforfun

総合スコア72

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問