[update] Improved specs of ScheduledController's protected methods.

sjplayer/src/sjplayer/CircleElementScheduledController.d

@@ -1,6 +1,8 @@
 /// License: MIT
 module sjplayer.CircleElementScheduledController;
 
+import std.typecons;
+
 import gl4d;
 
 import sjscript;
@@ -32,16 +34,15 @@ class CircleElementScheduledController :
     element_.weight       = 1;
     element_.smooth       = 0.01;
     element_.color        = vec4(1, 1, 1, 1);
+    super.PrepareOperation(params);
   }
   override void FinalizeOperation(ref in ParametersBlock params) {
     element_.alive = false;
   }
 
-  override float GetVariable(string name) const {
+  override void SetParameter(Nullable!float time, ref in Parameter param) {
-    throw new Exception("not implemented");  // TODO:
+    // TODO:
-  }
+    super.SetParameter(time, param);
-  override void SetParameter(ref in Parameter param) {
-    throw new Exception("not implemented");  // TODO:
   }
 
  private:

sjplayer/src/sjplayer/ScheduledControllerInterface.d

@@ -4,7 +4,8 @@ module sjplayer.ScheduledControllerInterface;
 import std.algorithm,
        std.array,
        std.exception,
-       std.format;
+       std.format,
+       std.typecons;
 
 import sjscript;
 
@@ -34,7 +35,7 @@ abstract class AbstractScheduledController : ScheduledControllerInterface {
 
       const last_operation = &operations_[next_operation_index_-1];
       if (IsTimeInPeriod(time, last_operation.period)) {
-        ProcessOperation(*last_operation);
+        ProcessOperation(time, *last_operation);
         return;
       }
       FinalizeOperation(*last_operation);
@@ -45,7 +46,7 @@ abstract class AbstractScheduledController : ScheduledControllerInterface {
     const next_operation = &operations_[next_operation_index_];
     if (IsTimeInPeriod(time, next_operation.period)) {
       PrepareOperation(*next_operation);
-      ProcessOperation(*next_operation);
+      ProcessOperation(time, *next_operation);
       ++next_operation_index_;
     }
   }
@@ -53,7 +54,7 @@ abstract class AbstractScheduledController : ScheduledControllerInterface {
  protected:
   abstract void PrepareOperation(ref in ParametersBlock params);
 
-  abstract void ProcessOperation(ref in ParametersBlock params);
+  abstract void ProcessOperation(float time, ref in ParametersBlock params);
 
   abstract void FinalizeOperation(ref in ParametersBlock params);
 
@@ -79,30 +80,39 @@ abstract class AbstractScheduledControllerWithOperationImpl :
   static struct VarStore {
    public:
     float opIndex(string name) {
-      float result = void;
+      if (!time_.isNull && name == "time") return time_.get;
-      if (!this_.GetVariable(name).collectException(result)) return result;
+      return this_.GetVariable(name);
-      if (!this_.varstore_[name]  .collectException(result)) return result;
-      if (!this_.user_vars_[name] .collectException(result)) return result;
-      throw new Exception("unknown variable %s".format(name));
     }
    private:
     AbstractScheduledControllerWithOperationImpl this_;
+    Nullable!float time_;
   }
 
-  override void ProcessOperation(ref in ParametersBlock params) {
+  override void PrepareOperation(ref in ParametersBlock params) {
-    foreach (const ref param; params.parameters) {
+    params.parameters.
-      if (param.name.length >= 2 && param.name[0..2] == "__") {
+      filter!(x => x.type == ParameterType.OnceAssign).
-        user_vars_[param.name[2..$]] =
+      each  !(x => SetParameter(Nullable!float.init, x));
-          param.rhs.CalculateExpression(VarStore(this));
-        continue;
-      }
-      SetParameter(param);
   }
+  override void ProcessOperation(float time, ref in ParametersBlock params) {
+    params.parameters.
+      filter!(x => x.type != ParameterType.OnceAssign).
+      each  !(x => SetParameter(time.nullable, x));
   }
 
-  abstract float GetVariable(string name) const;
+  float GetVariable(string name) const {
+    if (name in user_vars_) return user_vars_[name];
+    return varstore_[name];
+  }
+  void SetParameter(Nullable!float time, ref in Parameter param) {
+    (param.name.length >= 2 && param.name[0..2] == "__").
+      enforce("user defined variables must be prefixed '__'");
 
-  abstract void SetParameter(ref in Parameter param);
+    auto value = param.rhs.CalculateExpression(VarStore(this, time));
+    if (param.type == ParameterType.AddAssign) {
+      value += user_vars_[param.name];
+    }
+    user_vars_[param.name] = value;
+  }
 
  private:
   const VarStoreInterface varstore_;