[Processing] 一秒ごとにインスタンス化をしたい

Question

Processingを使って、ゲームを製作しています。1秒ごとにモンスターオブジェクトをインスタンス化し、モンスターを増殖させたいと思い、以下のプログラムを組みました。

Monstar[] monstars = new Monstar[10];
int prevMs = 0;
int cnt = 0;
void setup(){
  size(1200, 800);
  smooth();
  frameRate(100);
}

void draw(){
  background(255);
  /* 一秒ごとに実行 */
  if( millis() > prevMs + 1000 ){
    monstars[cnt] = new Monstar(100, 200);
    monstars[cnt].display();
    cnt++;
    prevMs = millis();
  }
}

/* モンスタークラス */
class Monstar{
  float x, y;
  PImage img;

  Monstar(float _x, float _y){
    img = loadImage("./img/Monstar.png");
    x = _x;
    y = _y;
  }

  void display(){
    image(img, x, y);
  }

ですが、実行すると、一体もモンスターが表示されません。
一秒ごとに実行させるif文内の処理を外に出して実行させると、表示されます。
どなたか、ご教授頂けると助かります。

Comments

[ironya]

> 一秒ごとに実行させるif文内の処理を外に出して実行させると、表示されます。
if 文の中にある時に。if 文の中の処理が動いているのに出ない、のですか？
それとも if 文の中に入らないのですか？

[Siroyan]

if文の中の処理は動きます。(printf(cnt)で確認いたしました)

[ironya]

そうですか、今添付されているソースコードだけ見ていると、特に落ち度は見当たらないので、ここに書かれている以外のところに原因が潜んでいるのでしょうか。。。
あ、すでに自己解決されていたようですね。良かったです。

Answer 1

原因が判明いたしました。
掲載したソースコードですと、一秒に一回しかdisplay()が呼び出されず、すぐにbackground()によって上書きされてしまうため、見た目上表示できてない模様です。

Answer 2

画像ファイルの読み出しが出来なくて止まってしまっているのではないですか?
実行時にIDEの下の方になにかメッセージが出ていませんか?

Comments

[thkana]

おっと、
> 一秒ごとに実行させるif文内の処理を外に出して実行させると、表示されます。
か。ちょっと考え直し。

[thkana]

解決したみたいなんで別件言っとくと、
配列の範囲外に飛び出るのはまぁ把握してるとして、

コンストラクタの中でファイルを読み込んでいるのはどうなんでしょう。
別にコンストラクタで読むこと自体は悪くないのですが、ディスクの読み出しという時間がかかる処理を1秒毎にやらなきゃいけないのかしら。絵柄が変わらないなら、setup()の中とかで一度読み込んでおいて、それをコンストラクタに渡したほうが良くないかしら。

[Siroyan]

ご回答ありがとうございます。
確かに、仰る通りですね。改善してみたいと思います。