もう少しややこしい話でした

相変わらず、ＡＶＲスプライトのスクロールプログラムです。

昨日寝る前にプログラムを眺めていたら、間違えを見つけて
しまいました。

しかもそれほど単純じゃなく、またしても処理時間が長くなる
ようなお話。

うーん、こまった。
と思って、どう直せばいいかを少しだけ考えてみるために、
ソースを眺めてみたところ…

あれ？なんでこんな処理しているんだろう？って疑問を
感じちゃう部分が。

ある数値を８で割って、それをその少し下で８を掛けて
いるんですよ。
同じ数を割って掛けるという必要は無いはずなのに…。
（下位３ビットの処理は別途考慮が必要だとしても）

そう思って、それぞれの計算式の意味を調べてみると、

・８で割る意味
　→１キャラクターは縦８ドットなので、キャラクタ数単位
　　のＹ座標を求めるために８で割る
・８掛ける意味
　→横方向は１行あたり１６キャラクターだが、１バイト
　　あたり２キャラクターなので、結果１行あたり８バイト。
　　なのでＹ座標を８倍するとＶＲＡＭ上のアドレスが求まる

といった具合。なるほど。それぞれに意味はあるんだな！

設計どおりのプログラムになっているので可読性が高く、
仕様変更するなどは都合が良いんだけども、ただ、ループの
最も内側なんだし、もっと最適化したロジックにしないと
処理時間に影響大だから、ここは見直しが必要だな…

それ以外にも探してみると、数値の範囲が１２７以下しか
ない数値をバカ正直に１６ビット幅で処理していたり…。
（１命令ですみそうなことを、１０倍以上の手間掛けて
　処理しております…バカだなぁ…）

そこらへんを見直したら、もしかしたら大幅に処理時間を
短縮できそうだなぁ！

見直しの観点だけ忘れないようにメモしておいて、あとは
時間が有る時に考えることにしよう。
それにしても、思わぬ収穫でした…