実現したいこと
デバッグ出力でcommentTest: 文字列2 となるようにしたい。
発生している問題・分からないこと
デバッグ出力でcommentTest: 文字列2 という風になることを期待しているのですが、
デバッグ出力の内容は次の通りになってしまいます。原因は何でしょうか?
該当のソースコード
以下はデバッグ出力の内容です。 //ファイルを読み込んだら、そこから呼び出される一連のメソッドでAddCommentが呼び出される。 AddComment: Comment added for key 'Key1'. New comment: 文字列1 AddComment: Comment added for key 'Key2'. New comment: 文字列2 //AddCommentはここまで。 //ユーザーがMouseDownしたときや、その他のイベントでChangedが呼び出される。 GetComment: No comment found for key 'Key2' commentTest: GetComment: No comment found for key 'Key2' commentTest: GetComment: No comment found for key 'Key2' commentTest: GetComment: No comment found for key 'Key2' commentTest: GetComment: No comment found for key 'Key2' commentTest:
C#
1 2namespace NameSpace 3{ 4 public partial class Manager 5 { 6 public class CommentManager 7 { 8 private Dictionary<string, Q> comments = new Dictionary<string, Q>(); 9 10 public class Q 11 { 12 public string Comment { get; set; } = ""; 13 } 14 15 public void AddComment(string key, string comment) 16 { 17 if (!comments.ContainsKey(key)) 18 { 19 comments[key] = new Q(); 20 } 21 else 22 { 23 Debug.WriteLine($"AddComment: Existing q found for key '{key}'"); 24 } 25 comments[key].Comment += comment; 26 Debug.WriteLine($"AddComment: Comment added for key '{key}'. New comment: {comments[key].Comment}"); 27 28 } 29 30 public string GetComment(string key) 31 { 32 if (comments.ContainsKey(key)) 33 { 34 Debug.WriteLine($"GetComment: Comment found for key '{key}': {comments[key].Comment}"); 35 return comments[key].Comment; 36 } 37 else 38 { 39 Debug.WriteLine($"GetComment: No comment found for key '{key}'"); 40 return string.Empty; 41 } 42 } 43 } 44 45
C#
1 2namespace NameSpace 3{ 4 public partial class ControlA : UserControl 5 { 6 public ControlA() 7 {} 8 private void Changed(PropertyChangedEventArgs args) 9 { 10 var list = args.value as List<ListRow>; 11 var manager = new Manager(); 12 var commentManager = new Manager.CommentManager(); 13 foreach(var e in list) 14 { 15 string str = "Key2"; 16 Debug.WriteLine("commentTest:" + commentManager.GetComment( str ) ); 17 } 18 } 19
試したこと・調べたこと
次のような書き方も試しましたが、同様にダメでした。書き方が間違っているのでしょうか。
C#
1namespace NameSpace 2{ 3 public partial class ControlA : UserControl 4 { 5 6 private Manager manager; 7 private Manager.CommentManager commentManager; 8 public ControlA( Manager.CommentManager commentManager ) 9 { 10 this.commentManager = commentManager; 11 } 12 private void Changed(PropertyChangedEventArgs args) 13 { 14 foreach(var e in list) 15 { 16 string str = "Key2"; 17 Debug.WriteLine("commentTest:" + commentManager.GetComment( str ) ); 18 } 19 }
C#
1partial class Screen 2 { 3 private System.ComponentModel.IContainer components = null; 4 protected override void Dispose(bool disposing) 5 { 6 if (disposing && (components != null)) 7 { 8 components.Dispose(); 9 } 10 base.Dispose(disposing); 11 } 12 private void InitializeComponent() 13 { 14 Manager Manager = new Manager(); 15 KifuManager.CommentManager commentManager = new Manager.CommentManager(); 16 this.ControlA = new NameSpace.ControlA(commentManager);
ScreenにControlAをControls.Addしている。
エラーは出ていません。
上記の詳細・結果
エラーは出ていません。
補足
visual studio2019.
.NET FrameWork
Changedがいつ呼ばれるのかは分かりませんがこの中でManagerやCommentManagerをいちいちnewしているのは意図している動作なのですか?
またChangedのローカル変数として定義されているのでChangedを抜けると消えてしまうのではないですか?
あとAddCommentが呼ばれているコードもないのでどのタイミングでAddCommentされどのタイミングでログ出力して値を確認しようとしているかも読み取れません。
そのあたりを整理してはどうでしょうか。
意図はありません。
プログラミングに明るくないのでそうなっているだけです。
ControlA内でprivateなどとして書いた方が良いのでしょうか?
情報が足りないようで申し訳ありません。
AddCommentはファイルを読み込んだことで呼び出されるメソッドの中で呼び出されています。
なので、デバッグ出力の情報は上から下へと時系列順に出力されています。
なので、というのはおかしいですね。すみません。
ではnewしたら新しくインスタンスが作成されるというのは理解されていますか?
また変数のスコープ(有効範囲)は理解されていますか?
上記のことが理解できていないのであればそれらを学習すべきだと思います。
commentManager のインスタンスは別のところで設定するべき感じですね。 ControlA のコンストラクタで渡す想定ですか?
Changedを抜けたら、そこで定義した変数は消えてしまうということでしょうか?しかし、インスタンスそのものはヒープ上に残り、それに割り当てられたメモリも解放されないから、今回のコメントが取得できないこととは関係がないと思っていました。 どのように書き直せばいいのでしょう?また各場所はpublic ControlA()の中でしょうか?見当違いなことを言っているかも知れませんが、どうぞよろしくお願いします。
CommentManagerのAddCommentが呼ばれたのがどこのインスタンスかって話になりますね。
例えば、
var managerA = new Manager.CommentManager();
var managerB = new Manager.CommentManager();
managerA.AddComment("key1", "value1");
とすれば、managerA.comment には登録されていますが、managerB.comment は空っぽです。
また、ローカル変数で作成したらメソッドを抜けた時点でインスタンスは破棄されるので、クラスメンバに保持する必要があるでしょう。既に他の方が指摘されていますが、毎回newしてたら当然インスタンスも毎回作り直されて空っぽのままなので、コンストラクタで一回だけ初期化するとか考える必要があります。
コンストラクタって何だよ!とかそこからだと、C#の言語仕様の基本的な話になってくるので、入門書などで地道に段階を踏んだ学習から始める事をお勧めします。じゃないと、説明されても理解できないと思います。さすがに質問サイトで1から10まで全部説明する訳にはいかないので。
質問を編集いたしました。
試したことについての情報を追加しました。
コンストラクタに一度記述する方法という認識で記述したつもりです。皆さんのおっしゃっていることは、これのことだろうと認識しています。
もし全然違うということでしたら、あきらめて基礎から勉強することにします。よければ、基礎を勉強する前に、事前に意識しておいた方が良い点を教えてください。
全般的に勉強していると、今回のことに関係する知識を関係ないと思い込んで、なんとなくわかった気になって終わってしまうことがありそうなので、その為にも、なにか事前知識や考え方などを教えてほしいのです。
AddCommentのときのcommentManagerとGetCommentのときのcommentManagerが別物だということですね。なぜかGetCommentの方ばかり気にしていました。自分の頭の中がどうなっていたのか、という感じです。これから修正しようと思います。それでも解決しなかった場合のことを踏まえてしばらく、ベストアンサは保留にしておこうかと思います。おやすみなさい。
Markdownのソース記述がおかしいのか、表示が崩れています。投稿前にきちんとプレビューで確認しましょう。
また、C#のソースで <!-- --> こんなコメントは許されていません。C#のコードなら、C#のコメントの書き方で書き、なるべく回答者が自環境で即再現できるように、実際にそのままビルド可能な必要最低限のコードで記述するようにした方がよいです。
コードの記載がおかしい為インデントが崩れてに読みにくくなっています。
また、CommentManagerに対してAddしている箇所もありませんからControlAのコンストラクタに渡したCommentManagerと同一インスタンスに対してAddしているのかも判断できません。
学習しなおすのであればインスタンスについて学習されることをお勧めします。
せめてコンパイルの通るコードでお願いします。