相変わらず、ＡＶＲスプライトのスクロールプログラムです。
昨日寝る前にプログラムを眺めていたら、間違えを見つけて
しまいました。
しかもそれほど単純じゃなく、またしても処理時間が長くなる
ようなお話。
うーん、こまった。
と思って、どう直せばいいかを少しだけ考えてみるために、
ソースを眺めてみたところ…
あれ？なんでこんな処理しているんだろう？って疑問を
感じちゃう部分が。
ある数値を８で割って、それをその少し下で８を掛けて
いるんですよ。
同じ数を割って掛けるという必要は無いはずなのに…。
（下位３ビットの処理は別途考慮が必要だとしても）
そう思って、それぞれの計算式の意味を調べてみると、
・８で割る意味
→１キャラクターは縦８ドットなので、キャラクタ数単位
のＹ座標を求めるために８で割る
・８掛ける意味
→横方向は１行あたり１６キャラクターだが、１バイト
あたり２キャラクターなので、結果１行あたり８バイト。
なのでＹ座標を８倍するとＶＲＡＭ上のアドレスが求まる
といった具合。なるほど。それぞれに意味はあるんだな！
設計どおりのプログラムになっているので可読性が高く、
仕様変更するなどは都合が良いんだけども、ただ、ループの
最も内側なんだし、もっと最適化したロジックにしないと
処理時間に影響大だから、ここは見直しが必要だな…
それ以外にも探してみると、数値の範囲が１２７以下しか
ない数値をバカ正直に１６ビット幅で処理していたり…。
（１命令ですみそうなことを、１０倍以上の手間掛けて
処理しております…バカだなぁ…）
そこらへんを見直したら、もしかしたら大幅に処理時間を
短縮できそうだなぁ！
見直しの観点だけ忘れないようにメモしておいて、あとは
時間が有る時に考えることにしよう。
それにしても、思わぬ収穫でした…
|
おすすめブログ
@goo_blog
