Merge pull request #5 from korarei/refactor/multi-return

Refactor: Make rotate function return multiple values and change laye…

Author: korarei

README.md

@@ -65,7 +65,11 @@ AviUtl ExEdit2のカメラ操作感を変更するスクリプト群．
 
 親オブジェクトの位置，回転，拡大率によってカメラの座標系を移動させる．
 
-- Use Relative Layer: `Parent Layer`で指定するレイヤー番号を相対値にする．
+- Layer Reference: `Parent Layer`で指定するレイヤー番号の参照方法を設定する．
+
+  - Absolute: 絶対指定
+
+  - Relative: 相対指定
 
 - PI: パラメータインジェクション．
 
@@ -80,7 +84,7 @@ AviUtl ExEdit2のカメラ操作感を変更するスクリプト群．
   rz = 0.0, -- カメラのZ回転 (number)
   rot_mode = 21, -- カメラの回転モード (number)
   parent_layer = 0, -- 親オブジェクトのレイヤー (number)
-  use_rel_layer = false -- 親オブジェクトのレイヤーを相対値で設定するかどうか (boolean or number)
+  layer_ref = 0 -- 親オブジェクトのレイヤーを相対値で設定するかどうか (number)
 }
 ```
 
@@ -154,7 +158,11 @@ AviUtl ExEdit2のカメラ操作感を変更するスクリプト群．
 
 - Target Layer: 目標オブジェクトのレイヤー．
 
-- Use Relative Layer: `Target Layer`で指定するレイヤー番号を相対値にする．
+- Layer Reference: `Target Layer`で指定するレイヤー番号の参照方法を設定する．
+
+  - Absolute: 絶対指定
+
+  - Relative: 相対指定
 
 - Influence: 影響度．
 
@@ -163,7 +171,7 @@ AviUtl ExEdit2のカメラ操作感を変更するスクリプト群．
 ```lua
 {
   target_layer = 1, -- 目標オブジェクトのレイヤー (number)
-  use_rel_layer = false, -- 目標オブジェクトのレイヤーを相対値で設定するかどうか (boolean or number)
+  layer_ref = 0 -- 親オブジェクトのレイヤーを相対値で設定するかどうか (number)
   influence = 100.0 -- 影響度 (number)
 }
 ```
@@ -208,7 +216,7 @@ local parent = {
   scale = 1.0 -- 根本スケール値 (number)
 }
 
-obj.setoption("camera_param",cam)
+local cam = obj.getoption("camera_param")
 local input = {
   x = cam.x,
   y = cam.y,
@@ -226,14 +234,13 @@ local input = {
 
 回転モードに応じて複数の3次元ベクトルを回転させる．
 
-`beta15`ではテーブル内テーブルや複数の値を返す関数の設定ができないため`key`に添え字をつける形にした．
-
 |位置|名前|型|説明|
 |:---|:---:|:---:|:---|
 |引数 #1|`rot`|table|設定値|
-|引数 #2|`vec`|table|3次元ベクトル|
-|…|…|…|同様に vec を追加可能|
-|戻り値 #1|`output`|table|回転後のベクトル群|
+|引数 #2|`vec_i`|table|3次元ベクトル|
+|…|…|…|同様に`vec_i`を追加可能|
+|戻り値 #1|`vec_o`|table|回転後の3次元ベクトル|
+|…|…|…|引数に加えた`vec_i`の数だけ順に回転後のベクトルが返る|
 
 ```lua
 local rot = {
@@ -249,18 +256,6 @@ local vec = {
   y = 0.0, -- Y要素
   z = 0.0 -- Z要素
 }
-
-local output = {
-  -- 1つ目のベクトルを回転させたもの
-  x0 = 0.0, 
-  y0 = 0.0,
-  z0 = 0.0,
-  -- 2つ目のベクトルを回転させたもの
-  x1 = 0.0, 
-  y1 = 0.0,
-  z1 = 0.0,
-  -- ...
-}
 ```
 
 ### encode_order 関数
@@ -309,6 +304,10 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 THE SOFTWARE.
 
 ## Change Log
+- **v1.0.4**
+  - `rotate`関数を複数値を返す関数に変更．
+  - `Transform`と`Track`における`Use Relative Layer`チェックボックスを`Layer Reference`リストに変更 (破壊的)
+
 - **v1.0.3**
   - `.mod2`化
   - `Parent`でアンカーによる仮想的な親オブジェクトを表示するように変更．

dll_src/main.cpp

@@ -80,32 +80,10 @@ rotate(SCRIPT_MODULE_PARAM *p) {
 
     auto output = Transform::rotate(input, rot);
 
-    std::size_t count = output.size();
-    std::size_t size = count * 3;
-    std::vector<double> vals(size);
-    std::vector<std::string> keys(size);
-    std::vector<LPCSTR> ckeys(size);
-
-    for (std::size_t i = 0; i < count; ++i) {
-        std::size_t x = i * 3;
-        std::size_t y = x + 1;
-        std::size_t z = x + 2;
-        std::string idx = std::to_string(i);
-
-        vals[x] = output[i].get_x();
-        vals[y] = output[i].get_y();
-        vals[z] = output[i].get_z();
-
-        keys[x] = "x" + idx;
-        keys[y] = "y" + idx;
-        keys[z] = "z" + idx;
-
-        ckeys[x] = keys[x].c_str();
-        ckeys[y] = keys[y].c_str();
-        ckeys[z] = keys[z].c_str();
-    }
+    LPCSTR keys[] = {"x", "y", "z"};
+    int size = std::size(keys);
 
-    p->push_result_table_double(ckeys.data(), vals.data(), static_cast<int>(size));
+    for (auto &v : output) p->push_result_table_double(keys, v.ptr(), size);
 }
 
 void

scripts/@CameraTransform_K_template.cam2

@@ -10,7 +10,7 @@
 --track6:Z Rotation,-3600,3600,0,0.01
 --select@s0:Rotation Mode=21,Quaternion=0,Axis Angle=1,XYZ Euler=5,XZY Euler=7,YXZ Euler=11,YZX Euler=15,ZXY Euler=19,ZYX Euler=21
 --track7:Parent Layer,-100,100,0,1
---check0:Use Relative Layer,0
+--select@s1:Layer Reference,Absolute=0,Relative=1
 --value@_0:PI,{}
 
 if (obj.getinfo("version") < 2001500) then
@@ -28,18 +28,13 @@ local ry = tonumber(_0.ry) or obj.track5
 local rz = tonumber(_0.rz) or obj.track6
 local rot_mode = tonumber(_0.rot_mode) or s0 s0 = nil
 local parent_layer = math.floor(tonumber(_0.parent_layer) or obj.track7)
-local use_rel_layer = obj.check0
-if (type(_0.use_rel_layer) == "boolean") then
-    use_rel_layer = _0.use_rel_layer
-elseif (type(_0.use_rel_layer) == "number") then
-    use_rel_layer = _0.use_rel_layer ~= 0
-end
+local layer_ref = tonumber(_0.layer_ref) or s1 s1 = nil
 _0 = nil
 
 local lib = obj.module("CameraTransform_K")
 local cam = obj.getoption("camera_param")
 
-local layer = use_rel_layer and math.max(obj.layer + parent_layer, 1) or math.max(parent_layer, 0)
+local layer = layer_ref == 1 and math.max(obj.layer + parent_layer, 1) or math.max(parent_layer, 0)
 local parent = {
     type = 0,
     x = 0.0,
@@ -160,24 +155,21 @@ local rot = {
 }
 
 local elem = math.min(obj.screen_w, obj.screen_h) * 0.25 * scale
-local vx = Vec:new(elem, 0.0, 0.0)
-local vy = Vec:new(0.0, elem, 0.0)
-local vz = Vec:new(0.0, 0.0, elem)
+local dx = Vec:new(elem, 0.0, 0.0)
+local dy = Vec:new(0.0, elem, 0.0)
+local dz = Vec:new(0.0, 0.0, elem)
 
 local cam = obj.getoption("camera_param")
 local pos = Vec:new(x - cam.x, y - cam.y, z - cam.z)
-local col = Vec:new(0xf05050, 0x208020, 0x5050f0)
+local col = {0xf05050, 0x208020, 0x5050f0}
 
 local lib = obj.module("CameraTransform_K")
-local output = lib.rotate(rot, vx, vy, vz)
-
-vx = Vec:new(output.x0, output.y0, output.z0) + pos
-vy = Vec:new(output.x1, output.y1, output.z1) + pos
-vz = Vec:new(output.x2, output.y2, output.z2) + pos
+dx, dy, dz = lib.rotate(rot, dx, dy, dz)
 
-obj.setanchor({pos.x, pos.y, pos.z, vx.x, vx.y, vx.z}, 2, "line", "xyz", "color", col.x)
-obj.setanchor({pos.x, pos.y, pos.z, vy.x, vy.y, vy.z}, 2, "line", "xyz", "color", col.y)
-obj.setanchor({pos.x, pos.y, pos.z, vz.x, vz.y, vz.z}, 2, "line", "xyz", "color", col.z)
+for i, v in ipairs({dx, dy, dz}) do
+    local t = v + pos
+    obj.setanchor({pos.x, pos.y, pos.z, t.x, t.y, t.z}, 2, "line", "xyz", "color", col[i])
+end
 
 local name = obj.getoption("script_name", 1, true)
 if (name == "Transform@CameraTransform_K") then
@@ -202,18 +194,18 @@ end
 --infomation:Track${SCRIPT_NAME} ${VERSION} by ${AUTHOR}
 --label:${LABEL}
 --track0:Target Layer,-100,100,1,1
---check0:Use Relative Layer,0
+--select@s0:Layer Reference,Absolute=0,Relative=1
 --track1:Influence,0,100,100,0.01
 --value@_0:PI,{}
 
+if (obj.getinfo("version") < 2001500) then
+    debug_print("AviUtl ExEdit2 beta15 or later is required.")
+    return
+end
+
 _0 = _0 or {}
 local target_layer = math.floor(tonumber(_0.target_layer) or obj.track0)
-local use_rel_layer = obj.check0
-if (type(_0.use_rel_layer) == "boolean") then
-    use_rel_layer = _0.use_rel_layer
-elseif (type(_0.use_rel_layer) == "number") then
-    use_rel_layer = _0.use_rel_layer ~= 0
-end
+local layer_ref = tonumber(_0.layer_ref) or s0 s0 = nil
 local influence = (tonumber(_0.influence) or obj.track1) * 0.01
 _0 = nil
 
@@ -223,7 +215,7 @@ end
 
 local cam = obj.getoption("camera_param")
 local tx, ty, tz = 0.0, 0.0, 0.0
-local layer = math.max(use_rel_layer and obj.layer + target_layer or target_layer, 1)
+local layer = math.max(layer_ref == 1 and obj.layer + target_layer or target_layer, 1)
 if (layer ~= obj.layer and obj.getvalue("layer" .. layer .. ".x")) then
   tx = obj.getvalue("layer" .. layer .. ".x")
   ty = obj.getvalue("layer" .. layer .. ".y")