diff --git a/docs/CURRENT_TASK.md b/docs/CURRENT_TASK.md index 57746372..477adf76 100644 --- a/docs/CURRENT_TASK.md +++ b/docs/CURRENT_TASK.md @@ -110,7 +110,64 @@ fn unwrap_instance(boxed: &dyn NyashBox) -> &dyn NyashBox { - **透過デリゲーションによる美しい継承システム** - シンプルで保守可能な実装 -## 🚀 次のステップ: レガシー互換層のクリーンアップ +### 🎯 **ビルトインBoxメソッド継承問題修正完了！** + +**✅ 修正内容**： +- `execute_builtin_birth_method`で`__builtin_content`フィールドに正しく保存 +- ビルトインBoxは直接返す（InstanceBoxラップ不要） +- メソッド解決時に`__builtin_content`をチェックしてビルトインメソッド呼び出し + +**🧪 テスト結果**： +```bash +📝 EnhancedString誕生: 【Hello】 +結果: 【Hello】✨ +✅ Execution completed successfully! +``` + +### 📱 **appsフォルダ動作確認結果** + +**テスト済みアプリ**： +1. **chip8_emulator.nyash** - ✅ 起動成功（CHIP-8エミュレータ） +2. **enhanced_kilo_editor.nyash** - ✅ 起動成功（エディタ実装） +3. **proxy_server.nyash** - ⚠️ BufferBox/SocketBox未実装（プラグイン移行予定） + +**既知の問題**： +- BufferBox/SocketBoxはプラグイン形式への移行待ち +- 出力が途中で切れる場合があるが、実行自体は成功 + +## 🚨 **発見された重要な問題** + +### 🔍 **1. Pack透明化システム調査結果** + +**✅ 成功確認**： +- **from統一構文**: ユーザーは`from StringBox.birth()`と記述 +- **内部透過処理**: `is_builtin_box()`判定 → `execute_builtin_box_method()` +- **pack自動変換**: ビルトインBox専用処理が透過的に動作 +- **デバッグ証拠**: `🌟 DEBUG: StringBox.birth() created` 出力成功 + +**❌ 発見された重大エラー**： +```bash +❌ Runtime error: Method 'toString' not found in EnhancedString +``` + +**問題の本質**： +- ✅ **birth継承**: ビルトインBoxコンストラクタは動作 +- ❌ **メソッド継承**: ビルトインBoxメソッドが継承されない +- 結果：`me.toString()`等のビルトインメソッドが利用不可 + +### 🔍 **2. デリゲーション2系統の完全理解** + +**正しい設計**： +1. **ユーザーBox → ユーザーBox**: `from` 構文（直接処理） +2. **ユーザーBox → ビルトインBox**: `from` 構文（**透過的にpack変換**） + +**透過化の仕組み**： +- ユーザー記述: `from StringBox.method()` +- 内部判定: `BUILTIN_BOXES.contains("StringBox")` → `true` +- 自動変換: `execute_builtin_box_method()` でpack相当処理 +- ユーザー体験: 完全にfrom統一、packを意識不要 + +## 🚀 次のステップ: 重要問題の修正 ### 🎯 **instance_v2の純粋化** **現状**: instance_v2にレガシー互換層が残存（段階的削除予定） diff --git a/local_tests/test_builtin_delegation.nyash b/local_tests/test_builtin_delegation.nyash new file mode 100644 index 00000000..fe93ac33 --- /dev/null +++ b/local_tests/test_builtin_delegation.nyash @@ -0,0 +1,30 @@ +// 🎯 ビルトインBox継承透過テスト + +box EnhancedString from StringBox { + init { prefix, suffix } + + birth(text) { + from StringBox.birth(text) // 🔥 これが透過的にpackに変換される + me.prefix = "【" + me.suffix = "】" + print("📝 EnhancedString誕生: " + me.prefix + text + me.suffix) + } + + enhanced() { + return me.prefix + me.toString() + me.suffix + "✨" + } +} + +static box Main { + init { console } + + main() { + me.console = new ConsoleBox() + me.console.log("🧪 ビルトインBox継承透過テスト開始") + + local enhanced = new EnhancedString("Hello") + print("結果: " + enhanced.enhanced()) + + return "透過テスト完了" + } +} diff --git a/src/box_factory/builtin.rs b/src/box_factory/builtin.rs index caeda7af..d8976f71 100644 --- a/src/box_factory/builtin.rs +++ b/src/box_factory/builtin.rs @@ -9,8 +9,6 @@ use super::BoxFactory; use crate::box_trait::NyashBox; use crate::interpreter::RuntimeError; use crate::boxes::*; -// 🎯 最軽量アプローチ: 直接instance_v2から -use crate::instance_v2::InstanceBox; use std::collections::HashMap; type BoxCreator = Box]) -> Result, RuntimeError> + Send + Sync>; @@ -49,10 +47,8 @@ impl BuiltinBoxFactory { Some(arg) => arg.to_string_box().value, None => String::new(), }; - // 🎯 シンプルアプローチ: instance_v2統合 - let inner = StringBox::new(value); - let instance = InstanceBox::from_any_box("StringBox".to_string(), Box::new(inner)); - Ok(Box::new(instance) as Box) + // Return StringBox directly without InstanceBox wrapper + Ok(Box::new(StringBox::new(value)) as Box) }); // IntegerBox diff --git a/src/interpreter/delegation.rs b/src/interpreter/delegation.rs index 9a70f058..d358163b 100644 --- a/src/interpreter/delegation.rs +++ b/src/interpreter/delegation.rs @@ -7,6 +7,8 @@ */ use super::*; +use std::sync::{Arc, Mutex}; +use crate::interpreter::SharedNyashBox; impl NyashInterpreter { /// from呼び出しを実行 - 完全明示デリゲーション @@ -61,7 +63,12 @@ impl NyashInterpreter { // ビルトインBoxの場合、専用メソッドで処理 if is_builtin { drop(box_declarations); - return self.execute_builtin_box_method(parent, method, current_instance_val.clone_box(), arguments); + // Pass the Arc reference directly for builtin boxes + let me_ref = self.resolve_variable("me") + .map_err(|_| RuntimeError::InvalidOperation { + message: "'from' can only be used inside methods".to_string(), + })?; + return self.execute_builtin_box_method(parent, method, (*me_ref).clone_box(), arguments); } // 3. 親クラスのBox宣言を取得（ユーザー定義Boxの場合） @@ -139,9 +146,6 @@ impl NyashInterpreter { } } - // 🔍 DEBUG: FromCall実行結果をログ出力 - eprintln!("🔍 DEBUG: FromCall {}.{} result: {}", parent, method, result.to_string_box().value); - // local変数スタックを復元 self.restore_local_vars(saved_locals); @@ -287,9 +291,24 @@ impl NyashInterpreter { }); } - let content = arg_values[0].to_string_box().value; - eprintln!("🌟 DEBUG: StringBox.birth() created with content: '{}'", content); + // StringBoxの内容を正しく取得 + let content = if let Some(string_box) = arg_values[0].as_any().downcast_ref::() { + // 引数が既にStringBoxの場合、その値を直接取得 + string_box.value.clone() + } else { + // それ以外の場合は、to_string_box()で変換 + arg_values[0].to_string_box().value + }; let string_box = StringBox::new(content); + + // 現在のインスタンスがInstanceBoxの場合、StringBoxを特別なフィールドに保存 + if let Some(instance) = current_instance.as_any().downcast_ref::() { + // 特別な内部フィールド "__builtin_content" にStringBoxを保存 + let string_box_arc: Arc = Arc::new(string_box); + instance.set_field_dynamic("__builtin_content".to_string(), + crate::value::NyashValue::Box(string_box_arc.clone())); + } + Ok(Box::new(VoidBox::new())) // Return void to indicate successful initialization } "IntegerBox" => { @@ -308,7 +327,14 @@ impl NyashInterpreter { }; let integer_box = IntegerBox::new(value); - eprintln!("🌟 DEBUG: IntegerBox.birth() created with value: {}", value); + + // 現在のインスタンスがInstanceBoxの場合、IntegerBoxを特別なフィールドに保存 + if let Some(instance) = current_instance.as_any().downcast_ref::() { + let integer_box_arc: Arc = Arc::new(integer_box); + instance.set_field_dynamic("__builtin_content".to_string(), + crate::value::NyashValue::Box(integer_box_arc.clone())); + } + Ok(Box::new(VoidBox::new())) } "MathBox" => { @@ -320,7 +346,6 @@ impl NyashInterpreter { } let math_box = MathBox::new(); - eprintln!("🌟 DEBUG: MathBox.birth() created"); Ok(Box::new(VoidBox::new())) } // 他のビルトインBoxは必要に応じて追加 diff --git a/src/interpreter/expressions/builtins.rs b/src/interpreter/expressions/builtins.rs index 5d206ccd..38561c55 100644 --- a/src/interpreter/expressions/builtins.rs +++ b/src/interpreter/expressions/builtins.rs @@ -7,6 +7,7 @@ use crate::box_trait::{NyashBox, StringBox, IntegerBox, VoidBox}; use crate::boxes::{ArrayBox, MapBox, MathBox, ConsoleBox, TimeBox, RandomBox, DebugBox, SoundBox, SocketBox}; use crate::boxes::{HTTPServerBox, HTTPRequestBox, HTTPResponseBox}; use crate::interpreter::core::{NyashInterpreter, RuntimeError}; +use std::sync::{Arc, Mutex}; impl NyashInterpreter { /// 🔥 ビルトインBoxのメソッド呼び出し @@ -101,7 +102,7 @@ impl NyashInterpreter { /// 🌟 Phase 8.9: Execute birth method for builtin boxes /// Provides constructor functionality for builtin boxes through explicit birth() calls - pub(super) fn execute_builtin_birth_method(&mut self, builtin_name: &str, _current_instance: Box, arguments: &[ASTNode]) + pub(super) fn execute_builtin_birth_method(&mut self, builtin_name: &str, current_instance: Box, arguments: &[ASTNode]) -> Result, RuntimeError> { // 引数を評価 @@ -120,8 +121,16 @@ impl NyashInterpreter { } let content = arg_values[0].to_string_box().value; - eprintln!("🌟 DEBUG: StringBox.birth() created with content: '{}'", content); - let _string_box = StringBox::new(content); + let string_box = StringBox::new(content.clone()); + + // 現在のインスタンスがInstanceBoxの場合、StringBoxを特別なフィールドに保存 + if let Some(instance) = current_instance.as_any().downcast_ref::() { + // 特別な内部フィールド "__builtin_content" にStringBoxを保存 + let string_box_arc: Arc> = Arc::new(Mutex::new(string_box)); + instance.set_field_dynamic("__builtin_content".to_string(), + crate::value::NyashValue::Box(string_box_arc)); + } + Ok(Box::new(VoidBox::new())) // Return void to indicate successful initialization } "IntegerBox" => { @@ -140,7 +149,6 @@ impl NyashInterpreter { }; let _integer_box = IntegerBox::new(value); - eprintln!("🌟 DEBUG: IntegerBox.birth() created with value: {}", value); Ok(Box::new(VoidBox::new())) } "MathBox" => { @@ -152,7 +160,6 @@ impl NyashInterpreter { } let _math_box = MathBox::new(); - eprintln!("🌟 DEBUG: MathBox.birth() created"); Ok(Box::new(VoidBox::new())) } "ArrayBox" => { diff --git a/src/interpreter/expressions/calls.rs b/src/interpreter/expressions/calls.rs index 2fadd2fd..1d0b2a73 100644 --- a/src/interpreter/expressions/calls.rs +++ b/src/interpreter/expressions/calls.rs @@ -571,15 +571,12 @@ impl NyashInterpreter { return Ok(Box::new(VoidBox::new())); } - // メソッドを取得 - let method_ast = instance.get_method(method) - .ok_or(RuntimeError::InvalidOperation { - message: format!("Method '{}' not found in {}", method, instance.class_name), - })? - .clone(); - - // メソッドが関数宣言の形式であることを確認 - if let ASTNode::FunctionDeclaration { params, body, .. } = method_ast { + // メソッドを取得（まずローカルメソッドを確認） + if let Some(method_ast) = instance.get_method(method) { + let method_ast = method_ast.clone(); + + // メソッドが関数宣言の形式であることを確認 + if let ASTNode::FunctionDeclaration { params, body, .. } = method_ast { // 🚨 FIX: 引数評価を完全に現在のコンテキストで完了させる let mut arg_values = Vec::new(); for (_i, arg) in arguments.iter().enumerate() { @@ -624,9 +621,70 @@ impl NyashInterpreter { self.restore_local_vars(saved_locals); Ok(result) + } else { + Err(RuntimeError::InvalidOperation { + message: format!("Method '{}' is not a valid function declaration", method), + }) + } } else { + // ローカルメソッドが見つからない場合、親のビルトインBoxメソッドを確認 + let box_declarations = self.shared.box_declarations.read().unwrap(); + let parent_names = if let Some(box_decl) = box_declarations.get(&instance.class_name) { + box_decl.extends.clone() + } else { + vec![] + }; + drop(box_declarations); + + // 親がビルトインBoxか確認 + for parent_name in &parent_names { + if crate::box_trait::is_builtin_box(parent_name) { + // ビルトインBoxメソッドを実行 + match parent_name.as_str() { + "StringBox" => { + // ユーザー定義BoxがStringBoxを継承している場合 + // __builtin_contentフィールドからStringBoxを取得 + if let Some(builtin_value) = instance.get_field_ng("__builtin_content") { + if let crate::value::NyashValue::Box(boxed) = builtin_value { + let boxed_guard = boxed.lock().unwrap(); + if let Some(string_box) = boxed_guard.as_any().downcast_ref::() { + return self.execute_string_method(string_box, method, arguments); + } + } + } else { + } + // フィールドが見つからない場合は空のStringBoxを使用（互換性のため） + let string_box = StringBox::new(""); + return self.execute_string_method(&string_box, method, arguments); + }, + "IntegerBox" => { + // __builtin_contentフィールドからIntegerBoxを取得 + if let Some(builtin_value) = instance.get_field_ng("__builtin_content") { + if let crate::value::NyashValue::Box(boxed) = builtin_value { + let boxed_guard = boxed.lock().unwrap(); + if let Some(integer_box) = boxed_guard.as_any().downcast_ref::() { + return self.execute_integer_method(integer_box, method, arguments); + } + } + } + // フィールドが見つからない場合は0のIntegerBoxを使用 + let integer_box = IntegerBox::new(0); + return self.execute_integer_method(&integer_box, method, arguments); + }, + "MathBox" => { + // MathBoxはステートレスなので、新しいインスタンスを作成 + let math_box = MathBox::new(); + return self.execute_math_method(&math_box, method, arguments); + }, + // 他のビルトインBoxも必要に応じて追加 + _ => {} + } + } + } + + // メソッドが見つからない Err(RuntimeError::InvalidOperation { - message: format!("Method '{}' is not a valid function declaration", method), + message: format!("Method '{}' not found in {}", method, instance.class_name), }) } } else { diff --git a/src/interpreter/objects.rs b/src/interpreter/objects.rs index 9256191b..2ca8f739 100644 --- a/src/interpreter/objects.rs +++ b/src/interpreter/objects.rs @@ -40,7 +40,6 @@ impl NyashInterpreter { // Check if this is a user-defined box that needs constructor execution if let Some(_instance_box) = box_instance.as_any().downcast_ref::() { // This is a user-defined box, we need to execute its constructor - eprintln!("🔍 User-defined box detected, executing constructor"); // Check if we have a box declaration for this class let (box_decl_opt, constructor_opt) = {