MVVMの考え方／特異な項目の追加

Question

###概要
現在、C#でWPFアプリケーションを作成しています。一種のメモ帳なのですが、Evernoteのように、タグ管理ができるようにしたいと考えております。手始めに、「データベースから取得したタグ一覧をリストボックスに表示する」ようにしているのですが、MVVMの考え方が身についておらず、四苦八苦しています（むしろ、オブジェクト指向の理解が足りない可能性も）。
また、表示項目に「タグを指定しない」という選択肢をどう追加するかにも悩んでいます。
###質問①
以下のコード、TagListVMクラスが無駄に見えます。startupでTagListVMのコンストラクターを呼び出し、さらにその中でTagListのコンストラクターを呼ぶという構成も変です。また、変数名なども全体的に極めて分かりにくいです（tagListって何回出てくるねん！）。あと、publicだらけです。改善の指針を提示してくださるとありがたいです。
###質問②
以下のコードに、「タグを指定しない」選択肢を加えるには、どのようにするのがよいでしょうか。「Tagクラスにコンストラクターを追加→Tag notSpecify = new Tag(...)などとして生成→tagListにadd」という方法は考えましたが、「タグを指定しない」というのはタグではないので、ふさわしくないですよね。TagListをTagCommandListのようにして……というのも考えたのですが、無駄に複雑化する気がします。
###コード

<ListBox ItemsSource="{Binding tagList}" DisplayMemberPath="name" />

    public partial class App : Application
    {
        public void startup(object sender, StartupEventArgs e)
        {
            TagListVM tagListVM = new TagListVM();
        }
    }

    public class TagList
    {
        public List<Tag> tagList { get; private set; }

        public TagList()
        {
            tagList = new List<Tag>();

            using (SQLiteConnection connection = new SQLiteConnection("DataSource = xxxx.db"))
            {
                connection.Open();
                using (DataContext context = new DataContext(connection))
                {
                    Table<Tag> tagTable = context.GetTable<Tag>();
                    IQueryable<Tag> queryResult =
                        from t in tagTable
                        select t;
                    foreach (Tag t in queryResult)
                    {
                        tagList.Add(t);
                    }
                }
            }
        }
    }

    [Table(Name = "tag")]
    public class Tag
    {
        [Column(Name = "tagID", DbType = "INT")]
        public int id { get; private set; }

        [Column(Name = "name", DbType = "TEXT")]
        public string name { get; private set; }
    }

    public class TagListVM
    {
        public List<Tag> tagList { get; private set; }
        
        public TagListVM()
        {
            tagList = new TagList().tagList;
        }
    }

Answer 1

まず気になったのが、
class TagListVMのメンバのpublic List<Tag> tagList
これ、どうしてclass TagListのインスタンスはコンストラクタ内で変数に格納せずにプロパティのList<Tag> tagList だけをclass TagListVMに残そうとしたのか
自分ならclass TagListVMのprivateでclass TagListのインスタンスを格納して、public List<Tag> tagListプロパティはTagList.tagListを返す形にしてgetのみ

同様に、class TagListのpublic List<Tag> tagListもprivateで実体を持っておいてpublicにはgetのみでprivateのインスタンスを上げる形で
setまでpublicにすると、外から勝手に置き換えられてしまうので
class TagListVMに至ってはclass TagListをnewした後受けずに放置しているので、外部から差し替えられたらコンストラクタで作る意味が完全になくなってしまうしクラスのメンバが中身を把握できなく外部から好き勝手いじられるようになってしまうと、そのうち自分一人で作っていてもすっかり忘れて変なミスを引き起こすと思う

あと、List<Tag> tagList自身は他のwindowでも使いまわすのならいいけど、使いまわさないのであるなら、App.startupでTagListVM tagListVM = new TagListVM();せずに使用するwindowのcontextへnewしてぶち込むかなぁ
いまいちどう使うのかもよくわからないので曖昧なことしか書けないけど、見えてる内容だけだととりあえずはそんな感じに変更するかな

WPFって書いてあるのでTagListVMは他にもバインドする項目があるのならそれの詰め合わせにする感じの使い方だからこれ一つしかない現状だと無駄に見えるのは仕方がない

Answer 2

なんだか自分が20年前に書いたプログラムを見ているようです。
所感ですが、MVVM以前にClass設計について復習するのが良いかと思いますが、回答してみます。

質問1

ご自分でお気づきのようですが、思っていることは正しいです。
まず適切な変数名を付けること、それから変数のスコープをきちんとつけること（publicばっかりやめる）
そうすると、だいぶ読みやすくなるかと思います。
現状、行数の割にはめちゃくちゃ読みにくいコードなので、まずもう少し読み易くして欲しいです。

質問2

今のソースで自分ならTagListのコンストラクタでやるか、中身追加した後の最後に追加するとかですかね。

Comments

[cisdur]

ご回答ありがとうございます。
なるほど、やっぱり読みにくいですよね。
TagListVM（VMはViewModelの略です）さえ外せば、そこそこ見られるコードだとは思ったのですが……。

最初はTagListVMなんてクラスはなかったのですが、TagListとTagはModelに属するクラスですので、ViewにTagListの内容を反映させるためには、間をつなぐViewModelに属するクラスが必要かと思って追加しました。
これをどうするかが一番の悩みです。

publicの件については、どれをprivateにしても動かなくなってしまうのですが、どうするのが適切でしょうか。