納得できないactionscript2.0のサンプルコード

質問

納得できないactionscript2.0のサンプルコード

http://oshige.com/flash/f8/chap01/で紹介されている、「01-3フレーム移動を行うクラス」
のサンプルコードなのですが（リンク先でダウンロードしていただけるとありがたいです）、
理解できない部分があります。

サンプルコードによると、
doll.asで変数loopを宣言しており、
またNaviFrame.as内ではloopの値を用いた処理を記述しているのですが、
変数loopに関係する部分を全て削除しても挙動に変化は見られませんでした。
（手元にあるテキストの説明によると、「loopの値がtrueならフレーム１に戻り、
falseならフレーム移動を行わない」とありますが、
フレーム１に戻る／戻らないの処理は、

if(mc._currentframe == mc._totalframes) {……}

の条件分岐だけで事足りるし、変数loopを宣言して条件分岐に用いたところで、
何の意味もなしていないと思うのです）

私には変数loopに関する一連の記述の必然性が全く分からないのですが、
どなたか教えていただけないでしょうか？
意味が無いように見えて、実は意味があるのでしょうか。
それとも、単なる誤植の類いなのでしょうか。
初心者ですので、できるだけ平易な解説でお願いします。

回答No.2

BlurFiltan · ベストアンサー

ご質問の意味の方が納得できませんが...。

> 手元にあるテキストの説明によると、「loopの値がtrueならフレーム１に戻り、
> falseならフレーム移動を行わない」とありますが、

実際にそうなるでしょう？
dolls.fla の フレーム1 のスクリプトの冒頭

--------------
var target_mc:MovieClip = dolls_mc;
target_mc.stop();
var loop:Boolean = true;
//abcMorphムービークリップを動作の対象にするnaviObjインスタンスを作る
var naviObj:NaviFrame = new NaviFrame(target_mc, loop);
～略～
--------------

この３行目
var loop:Boolean = true;
をこののままにしておけばループしますし
var loop:Boolean = false;
に変更すればループしません。
変数 loop はちゃんと生きて動作してます。

通常，クラスファイルは使うたびにその中身をイジって使うものではなくて
イジらなくて良い汎用的なファイルとしてそのまま使い回しができるように作ります。
※例えば最初からすでに定義されてあるクラス
　 (例：MovieClipクラス や Mathクラス や LoadVarsクラスなど)の
　 クラスファイルの中など一々見ませんし作り替えないでしょう？
それがクラスのメリットです（Documentクラスを除く）。

dolls.fla の
var loop:Boolean の値を true や false にするだけで，
「NaviFrame.as」自体は触ることなく全くそのまま同じ物が使えますよね。
そうするためにあるのです。

「NaviFrame.as」の21行目より引用し注釈を書き直します。
---------------------
　// 次のフレームへ移動するメソッド「next」を定義
　function next() {

　　　// もし現在の登録MCのフレームが最終フレームであれば
　　　if (mc._currentframe == mc._totalframes) {

　　　　　// なおかつ変数 loop が true であるなら
　　　　　if (loop) {
　　　　　　　// 登録MCをフレーム1に戻して停止
　　　　　　　mc.gotoAndStop(1);
　　　　　}
　　　　　//↑★変数 loop が false であるなら何もしない
　　　　　//　 ということです(フレーム移動もしないということ)。

　　　// もし登録MCのフレームが最終フレームに達していなければ
　　　} else {
　　　　　//次のフレームへ移動させて停止
　　　　　mc.nextFrame();
　　　}

　}
---------------------

↑変数 loop の存在意味は大アリです。

ただ最初から，
「ループ再生するものだけ限定としてクラスを作る」
または
「ループ再生しないものだけ限定としてクラスを作る」
というのでしたら変数 loop は不要です。

ただしその場合，ループするしないを変更しようと思うと，
「NaviFrame.as」 を作り替えるとか
「NaviFrameDoLoop」 クラスと 「NaviFrameNotLoop」 クラスのような２つを作っておかなければならないという事態になってしまいます。
つまり汎用性に欠ける出来の良くないクラスとなります。

お礼

>変数 loop が false であるなら何もしない
>ということです(フレーム移動もしないということ)。

まずif構文などの基本的な部分の理解がたりなかったみたいですね。
コメント付きのスクリプトで丁寧に解説していただき、ありがとうございます。

クラスの拡張性を考慮していなかったので、
わざわざloopでの条件分岐を付け加える意味が分からなかったのですが、
ようやく理解できました。

回答No.1

cspl

結論から言うと変数loopに意味はあります。

仰る通り、ループのするしない、という仕様が最初からどちらかに決まっていて、
この先もどちらか一方しか使用しない、ということが前提であれば

if(mc._currentframe == mc._totalframes) {……}

この部分の条件分岐だけで事足りますね。

しかし、こういった類のクラスファイル(ここではNaviframe.as)は
後の仕様変更によっても極力いじらなくても済むように作る必要があります。

たとえば、「今は最後の画像の次に１枚目の画像を表示させてループさせているけど、後からループさせないように変更することになった」場合、クラスファイルをいじらずに、fla内に定義されている変数loopの値をfalseにするだけで、クラスファイルの中身を知らない人でも簡単に更新することができるわけです。
また、これは配布されているファイルなので、いろんな人のニーズに合うように汎用的に作らなければならない、ということもあります。

要はクラスファイルをいじることなく、外側からカスタマイズするための変数だと思っていただければいいと思います。

納得いただけましたでしょうか。

お礼

ありがとうございます。
例題のスクリプトを理解するので一杯で、
先々変更を加えたりする可能性までは考えが及んでいませんでした。
言われてみると、loop変数があったほうが変更はしやすそうですね。
自分が同じようなものを一から作ることになったとき、こういう考えができるか心配ですが。。