回答編集履歴

見直しキャンペーン中

2023/07/21 08:22

投稿

TN8001

スコア9326

test CHANGED Viewed

@@ -1,309 +1,149 @@
 最初は実行ファイルにドラッグ＆ドロップの場合、画面なしかなと思いましたが出るんですね？
 戻り値で消したフォルダを返すようにするといいんじゃないでしょうか。
-```C#
-// Form1.cs
+```cs:Form1.cs
 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());
+        }
+    }
+}
+```
+```cs:Process.cs
+using System.Collections.Generic;
+using System.IO;
+using System.Linq;
+namespace WindowsFormsApp1
+{
+    public class Process
+    {
+        //     ↓消したフォルダを戻り値で返すように
+        public IEnumerable<string> Other_Process(IEnumerable<string> folders)
+        {
-            // ↓string[]の仲間 消されたフォルダパス
+            // ↓string[]の仲間 こちらは後から追加できる
+            List<string> deleteFolders = new List<string>();
+            foreach(string folder in folders)
+            {
+                string[] subFolders;
+                try
+                {
+                    // アクセスできないフォルダの場合例外
-            IEnumerable<string> deleteFolders = processObj.Other_Process(selectFolders);
+                    subFolders = Directory.GetDirectories(folder);
+                }
+                catch
+                {
+                    // アクセスできないフォルダは無視
+                    continue;
+                }
+                foreach(string subFolder in subFolders)
+                {
+                    // とりあえず数字が入っているフォルダを削除するつもりになる
+                    if(subFolder.Any(char.IsDigit))
+                    {
+                        //DeleteDirectory(subFolder);
-            listBox1.Items.AddRange(deleteFolders.ToArray());
+                        deleteFolders.Add(subFolder);
+                    }
+                }
+            }
+            return deleteFolders;
         }
+        private void DeleteDirectory(string dir)
+        {
+            // 読み取り専用ファイルがあると例外
+            // http://dobon.net/vb/dotnet/file/deletedirectory.html
+            Directory.Delete(dir, true);
+        }
     }
 }
 ```
-```C#
-// Process.cs
+```cs:Program.cs
-using System.Collections.Generic;
-using System.IO;
+using System;
-using System.Linq;
+using System.Windows.Forms;
 namespace WindowsFormsApp1
 {
-    public class Process
+    static class Program
     {
+        [STAThread]
-        //     ↓消したフォルダを戻り値で返すように
+        static void Main()
-        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)
+            Application.EnableVisualStyles();
-                {
-                    // とりあえず数字が入っているフォルダを削除するつもりになる
-                    if(subFolder.Any(char.IsDigit))
-                    {
-                        //DeleteDirectory(subFolder);
+            Application.SetCompatibleTextRenderingDefault(false);
-                        deleteFolders.Add(subFolder);
+            Application.Run(new Form1());
-                    }
-                }
-            }
-            return deleteFolders;
         }
-        private void DeleteDirectory(string dir)
-        {
-            // 読み取り専用ファイルがあると例外
-            // http://dobon.net/vb/dotnet/file/deletedirectory.html
-            Directory.Delete(dir, true);
-        }
     }
 }
 ```
-```C#
-// Program.cs
-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か月でここまで書けるのは素晴らしいですね（参考コードそのままだったとしてもその調査力と理解力が）