C# 他クラスからフォームのListBoxの表示ができない。

Question

前提・実現したいこと

C#にて
“「実行ファイル」or「ListBox」にドラッグ＆ドロップしたフォルダ内の特定フォルダを削除し、ListBoxにパスを表示する。”
C#の勉強のため作成しているのですがうまくいきません。

発生している問題・エラーメッセージ

①エラーが発生していませんが、ListBoxへの表示がうまくいきません。

②実行ファイルへドラック＆ドロップされた際の書き方や、パスの渡し方これで良いのかわかりません。

C#を始めてまだ1か月程度なので、未だにインスタンスなどが理解していませんので変なコードを書いているかもしれませんが、エラーも発生していないので直し方もわかりません。
以下に書いたソースを示しますのでが教授お願いします。
特定フォルダを消す処理はまだできていませんので、コメントで書いています。

###　Program.cs

 namespace WindowsFormsApp1
{
    static class Program
    {
        /// <summary>
        /// アプリケーションのメイン エントリ ポイントです。
        /// </summary>
        [STAThread]
        static void Main()
        {
            string[] files = System.Environment.GetCommandLineArgs();

            if (files.Length > 1)
            {
                for (int i = 1; i < files.Length; i++)
                {
                    Process processObj = new Process();
                    processObj.Other_Process(files);
                }
            }
            else
            {
                Application.EnableVisualStyles();
                Application.SetCompatibleTextRenderingDefault(false);
                Application.Run(new Form1());
            }

            
        }
    }
}

###　Form1.cs

namespace WindowsFormsApp1
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
            this.listBox1.AllowDrop = true;
        }

        private void listBox1_DragEnter(object sender, DragEventArgs e)
        {
            if (e.Data.GetDataPresent(DataFormats.FileDrop))
            {
                e.Effect = DragDropEffects.All;
            }
            else
            {
                e.Effect = DragDropEffects.None;
            }
        }

        private void listBox1_DragDrop(object sender, DragEventArgs e)
        {
            string[] fileName =
                (string[])e.Data.GetData(DataFormats.FileDrop, false);

            Process processObj = new Process(); 
            processObj.Other_Process(fileName);
        }

        public void listbox_display(string [] item)
        {
            listBox1.Items.AddRange(item);
        }
    }
}

###　Process.cs

namespace WindowsFormsApp1
{
    class Process
    {
       public void Other_Process(string [] Path)
        {
            /*～～特定フォルダなどを消す処理の処理～～*/

            Form1 frm1Obj = new Form1();
            frm1Obj.listbox_display(Path);
        }
    }
}

補足情報（FW/ツールのバージョンなど）

・Microsoft Visual Studio Professional 2019　Version 16.4.5
・Windows 10 64bit

Answer 1

ProcessクラスのOther_Processメソッドで新たにForm1クラスのインスタンスを生成し、そのインスタンスでlistbox_displayメソッドを実行しても、それは今現在画面上に表示しているForm1のインスタンスではないのですから、listBox1へは作用せず、従って今表示しているリストボックスに表示されません。

ProcessクラスのOther_Processメソッドの引数でForm1を渡せるようにすれば、要望する動きになると思います。

// Process.cs
public void Other_Process(Form1 frm1Obj, string[] Path)
{
    /*～～特定フォルダなどを消す処理の処理～～*/
    frm1Obj.listbox_display(Path);
}

と定義して、Form1クラスでは

// Form1.cs
private void listBox1_DragDrop(object sender, DragEventArgs e)
{
    string[] fileName =
               (string[])e.Data.GetData(DataFormats.FileDrop, false);

    Process processObj = new Process();
    processObj.Other_Process(this, fileName);
}

とします。

違う話になりますが、ProcessクラスはSystem.Diagnostics.Processクラスと言うものが既にあるので、要注意です。
Process クラス - Microsoft Docs

---

追記しました：

いくつか他の回答もいただきましたように様々な考え方、対応方法があると思いますが、「コマンドラインオプションからファイル名を指定した場合、最初の画面表示の際に自動でリストボックスに表示する」例を書いてみましたので以下に示します。コマンドラインオプションで指定したものはForm1に新設したstring[] Filesプロパティにセットして扱うようにします。

// Program.cs

using System;
using System.Windows.Forms;

namespace WindowsFormsApp1
{
    static class Program
    {
        [STAThread]
        static void Main()
        {
            Application.EnableVisualStyles();
            Application.SetCompatibleTextRenderingDefault(false);

            var form1 = new Form1();
            string[] files = System.Environment.GetCommandLineArgs();
            if (files.Length > 1)
            {
                // コマンドラインオプション指定があれば、args[1]以降を取り込む。
                var list = new System.Collections.Generic.List<string>(files);
                list.RemoveAt(0);
                form1.Files = list.ToArray();
            }

            Application.Run(form1);
        }
    }
}

// Form1.cs
using System.Windows.Forms;

namespace WindowsFormsApp1
{
    public partial class Form1 : Form
    {
        // 処理対象のファイル配列プロパティ
        public string[] Files { get; set; }

        // 使いまわすので、フィールドにする。
        private Process processObj = null;

        public Form1()
        {
            InitializeComponent();
            listBox1.AllowDrop = true;
            processObj = new Process();
        }

        private void Form1_Load(object sender, System.EventArgs e)
        {
            if (Files != null && Files.Length > 0)
            {
                // Filesプロパティにセット済みであれば
                // フォームの初期表示(Load)時に処理をする。
                processObj.Other_Process(this, Files);
            }
        }

        private void listBox1_DragEnter(object sender, DragEventArgs e)
        {
            if (e.Data.GetDataPresent(DataFormats.FileDrop))
            {
                e.Effect = DragDropEffects.All;
            }
            else
            {
                e.Effect = DragDropEffects.None;
            }
        }

        private void listBox1_DragDrop(object sender, DragEventArgs e)
        {
            Files = (string[])e.Data.GetData(DataFormats.FileDrop, false);
            processObj.Other_Process(this, Files);
        }

        public void listbox_display(string[] item)
        {
            listBox1.Items.Clear(); // クリアしても良いかも
            listBox1.Items.AddRange(item);
        }
    }
}

// Process.cs

namespace WindowsFormsApp1
{
    class Process
    {
        public void Other_Process(Form1 frm1Obj, string[] Path)
        {
            /*～～特定フォルダなどを消す処理の処理～～*/
            frm1Obj.listbox_display(Path);
        }
    }
}

こんな方法もある、と言う程度に参考にしてみてください。（エラーケースはほとんど考慮していません）
※他回答者であるTN8001さんも書かれていますが、質問者さんオリジナルのコードについて、もしC#含めプログラミング歴1ヶ月でそこまでできたのであれば大したものでは、と私も最初に思いました。引き続きがんばってください。

Comments

[doro-ro]

回答ありがとうございます。確かにインスタンスさらに新しく作成してしまうと意味ないですね。
おかげさまで表示できるようにできました。

さらに質問をして申し訳ないのですが、『Program.cs』のMainメソッドに同様に
　　　　　Process processObj = new Process();
processObj.Other_Process(this, files);
と記入したところ
エラーCS0026 キーワード 'this' は、静的プロパティ、静的メソッド、または静的フィールド初期化子では無効です。
と表示されてしまいました。
Mainメソットはstaticなので確かにそうなのですが、同様にドラッグ＆ドロップした場合でも同じような処理をしたい場合はどうすればよいでしょう？

[dodox86]

プログラムの設計としてそもそも問題があるのですが、Mainの方を見ると、コマンドラインでファイル名を複数指定した場合は、Form1は表示されません。Form1画面を表示させるには、Application.Run(new Form1()); まで実行する必要があります。質問者さんの要望は、コマンドラインでファイル名を指定した場合、それらをドラッグ＆ドロップと同様に表示させ、削除したいということでしょうか。

[dodox86]

C#の勉強中、と言うことのなので様々なやり方があって良いと思いますが、コマンドラインからファイル名を指定した場合、「リストボックスを含んだ画面は表示せず、削除だけしてプログラムを終わる」と言う考え方もできるし、「画面表示をしたときに自動で削除し、リストボックスにも表示する。」と言う考え方もあります。後者を考えられていましたか？　現状のコードではどちらとも取れます。

[dodox86]

追記しました。参考になれば幸いです。

[doro-ro]

遅くなってしまい申し訳ありません。納得できました。１ヶ月で出来てすごいという言葉をいただいてとても嬉しいです。ほとんど調べて移しただけですが。。。引き続き頑張っていきたいと思います。ありがとうございました。

Answer 2

最初は実行ファイルにドラッグ＆ドロップの場合、画面なしかなと思いましたが出るんですね？

戻り値で消したフォルダを返すようにするといいんじゃないでしょうか。

using System;
using System.Collections.Generic;
using System.IO;
using System.Linq;
using System.Windows.Forms;

namespace WindowsFormsApp1
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
            // デザイナで設定できる プロパティウィンドウ－動作
            //this.listBox1.AllowDrop = true;

            string[] files = Environment.GetCommandLineArgs();

            if(files.Length > 1)
            {
                Process processObj = new Process();
                // ↓string[]の仲間 消されたフォルダパス                           ↓いっこ飛ばす
                IEnumerable<string> deleteFolders = processObj.Other_Process(files.Skip(1));
                listBox1.Items.AddRange(deleteFolders.ToArray());
            }
        }

        private void listBox1_DragEnter(object sender, DragEventArgs e)
        {
            // ダメな場合を先に見てreturnすると｛｝が減って見やすい
            if(!e.Data.GetDataPresent(DataFormats.FileDrop))
            {
                e.Effect = DragDropEffects.None;
                return;
            }

            string[] paths = (string[])e.Data.GetData(DataFormats.FileDrop);

            foreach(string path in paths)
            {
                // 選択にフォルダ以外（ファイル）が入っていたら禁止
                if(!Directory.Exists(path))
                {
                    e.Effect = DragDropEffects.None;
                    return;
                }
            }

            e.Effect = DragDropEffects.All;
        }

        private void listBox1_DragDrop(object sender, DragEventArgs e)
        {
            string[] selectFolders = (string[])e.Data.GetData(DataFormats.FileDrop, false);

            Process processObj = new Process();

            // ↓string[]の仲間 消されたフォルダパス
            IEnumerable<string> deleteFolders = processObj.Other_Process(selectFolders);
            listBox1.Items.AddRange(deleteFolders.ToArray());
        }
    }
}

using System.Collections.Generic;
using System.IO;
using System.Linq;

namespace WindowsFormsApp1
{
    public class Process
    {
        //     ↓消したフォルダを戻り値で返すように
        public IEnumerable<string> Other_Process(IEnumerable<string> folders)
        {
            // ↓string[]の仲間 こちらは後から追加できる
            List<string> deleteFolders = new List<string>();

            foreach(string folder in folders)
            {
                string[] subFolders;
                try
                {
                    // アクセスできないフォルダの場合例外
                    subFolders = Directory.GetDirectories(folder);
                }
                catch
                {
                    // アクセスできないフォルダは無視
                    continue;
                }

                foreach(string subFolder in subFolders)
                {
                    // とりあえず数字が入っているフォルダを削除するつもりになる
                    if(subFolder.Any(char.IsDigit))
                    {
                        //DeleteDirectory(subFolder);
                        deleteFolders.Add(subFolder);
                    }
                }
            }

            return deleteFolders;
        }

        private void DeleteDirectory(string dir)
        {
            // 読み取り専用ファイルがあると例外
            // http://dobon.net/vb/dotnet/file/deletedirectory.html
            Directory.Delete(dir, true);
        }
    }
}

using System;
using System.Windows.Forms;

namespace WindowsFormsApp1
{
    static class Program
    {
        [STAThread]
        static void Main()
        {
            Application.EnableVisualStyles();
            Application.SetCompatibleTextRenderingDefault(false);
            Application.Run(new Form1());
        }
    }
}

---

システムファイル等はデフォルトですと削除できず例外となります。
適切にtry catchが必要になります。
テスト中に誤って重要ファイルを消さないように十分注意してください。

1か月でここまで書けるのは素晴らしいですね（参考コードそのままだったとしてもその調査力と理解力が）

Answer 3

Dropだけなら、これで十分かも。

        private void listBox1_DragDrop(object sender, DragEventArgs e)
        {
            string[] fileName =
                (string[])e.Data.GetData(DataFormats.FileDrop, false);
            listBox1.Items.AddRange(fileName);
        }