92 lines
2.7 KiB
Markdown
92 lines
2.7 KiB
Markdown
|
# VM比較処理のリファクタリング
|
|||
|
|
Status: Pending (80%実装済み)
|
|||
|
|
Created: 2025-08-25
|
|||
|
|
Priority: Medium
|
|||
|
|
Related-Code: src/backend/vm_instructions.rs::execute_compare(), src/backend/vm_values.rs::execute_compare_op()
|
|||
|
|
|
|||
|
|
## 現状（80%実装）
|
|||
|
|
- BoxRef比較のTypeErrorは解決済み
|
|||
|
|
- 正規化処理が2箇所に分散（execute_compare, execute_compare_op）
|
|||
|
|
- デバッグログが複数の環境変数に依存
|
|||
|
|
- 基本動作は完全に正常
|
|||
|
|
|
|||
|
|
## 問題点
|
|||
|
|
1. **重複した正規化ロジック**
|
|||
|
|
- vm_instructions.rs: BoxRef → Integer変換
|
|||
|
|
- vm_values.rs: 同様の変換処理
|
|||
|
|
|
|||
|
|
2. **デバッグの複雑さ**
|
|||
|
|
- NYASH_VM_DEBUG
|
|||
|
|
- NYASH_VM_DEBUG_CMP
|
|||
|
|
- 複数箇所でのログ出力
|
|||
|
|
|
|||
|
|
3. **エラーパスの複雑さ**
|
|||
|
|
- "[BoxRef-BoxRef]", "[BoxRef-Integer]", "[Integer-BoxRef]", "[Default]"
|
|||
|
|
- どこでエラーが出るか予測困難
|
|||
|
|
|
|||
|
|
## 改善案（残り20%）
|
|||
|
|
|
|||
|
|
### 1. 正規化を単一関数に統一
|
|||
|
|
```rust
|
|||
|
|
fn canonicalize_for_comparison(value: VMValue) -> VMValue {
|
|||
|
|
match value {
|
|||
|
|
VMValue::BoxRef(b) => {
|
|||
|
|
// IntegerBox → Integer
|
|||
|
|
if let Some(ib) = b.as_any().downcast_ref::<IntegerBox>() {
|
|||
|
|
return VMValue::Integer(ib.value);
|
|||
|
|
}
|
|||
|
|
// String parse fallback
|
|||
|
|
if let Ok(n) = b.to_string_box().value.trim().parse::<i64>() {
|
|||
|
|
return VMValue::Integer(n);
|
|||
|
|
}
|
|||
|
|
VMValue::BoxRef(b)
|
|||
|
|
}
|
|||
|
|
other => other,
|
|||
|
|
}
|
|||
|
|
}
|
|||
|
|
```
|
|||
|
|
|
|||
|
|
### 2. 比較処理の階層化
|
|||
|
|
```rust
|
|||
|
|
impl VM {
|
|||
|
|
// エントリーポイント
|
|||
|
|
pub fn execute_compare(...) {
|
|||
|
|
let left = self.canonicalize_value(lhs)?;
|
|||
|
|
let right = self.canonicalize_value(rhs)?;
|
|||
|
|
let result = self.compare_canonical(op, &left, &right)?;
|
|||
|
|
self.set_value(dst, VMValue::Bool(result));
|
|||
|
|
}
|
|||
|
|
|
|||
|
|
// 正規化
|
|||
|
|
fn canonicalize_value(&self, id: ValueId) -> Result<VMValue, VMError> {
|
|||
|
|
let raw = self.get_value(id)?;
|
|||
|
|
Ok(canonicalize_for_comparison(raw))
|
|||
|
|
}
|
|||
|
|
|
|||
|
|
// 比較実行
|
|||
|
|
fn compare_canonical(&self, op: &CompareOp, left: &VMValue, right: &VMValue) -> Result<bool, VMError> {
|
|||
|
|
// シンプルな比較ロジックのみ
|
|||
|
|
}
|
|||
|
|
}
|
|||
|
|
```
|
|||
|
|
|
|||
|
|
### 3. デバッグトレースの統一
|
|||
|
|
```rust
|
|||
|
|
struct ComparisonTrace {
|
|||
|
|
original_left: VMValue,
|
|||
|
|
original_right: VMValue,
|
|||
|
|
canonical_left: VMValue,
|
|||
|
|
canonical_right: VMValue,
|
|||
|
|
result: Result<bool, VMError>,
|
|||
|
|
}
|
|||
|
|
```
|
|||
|
|
|
|||
|
|
## 実装タイミング
|
|||
|
|
- [ ] 次の大きなバグが出たとき
|
|||
|
|
- [ ] Phase 10の最適化フェーズ
|
|||
|
|
- [ ] 新しい比較演算子追加時
|
|||
|
|
|
|||
|
|
## メリット
|
|||
|
|
- デバッグが容易になる
|
|||
|
|
- 新しい型の比較追加が簡単
|
|||
|
|
- テストが書きやすくなる
|