0

提示コードの#コンバート関数の///内部のnew String(file[i])で以下の例外が発生してしまいます。これはなぜなのでしょうか?Index was out of range.という例外が出る理由がわかりません。

セマフォを使って三つにスレッドを制限しています。
また同じパスのものがRun()関数の中を通ったりと全体に的に挙動がおかしいのですが原因がわかりません。

例外
例外がスローされました: 'System.ArgumentOutOfRangeException' (System.Private.CoreLib.dll の中)
例外がスローされました: 'System.ArgumentOutOfRangeException' (System.Private.CoreLib.dll の中)
型 'System.ArgumentOutOfRangeException' の例外が System.Private.CoreLib.dll で発生しましたが、ユーザー コード内ではハンドルされませんでした
Index was out of range. Must be non-negative and less than the size of the collection.
試したこと、確認したこと

file.Countで5個のファイルパスがあります。
fileリスト変数の中身を確認して中のパスを確認しました、同じもはないです。
for文も回っていますがなぜか5ループ目が実行されています。
fileの変数の中身を数を確認しまししたが問題はありませんでした。
セマフォを使った非同期処理を行っています。
stringは参照型なのでnewしてみまいしたが同じ挙動でした。(提示コード)

using System.Diagnostics;
using System.Drawing;
using System.IO;
using System.Runtime.CompilerServices;
using System.Runtime.ConstrainedExecution;
using System.Threading;
using System.Windows.Forms;
using ImageMagick;

namespace HEIC_SimpleConverter
{
    public partial class Form : System.Windows.Forms.Form
    {
        int threadNum = 3;
        int num = 0;
        string extension = ".jpeg";
        SemaphoreSlim semaphore;
        List<string> file = new List<string>();
        float per = 0;
        string folderPath = "";
        List<Task> task = new List<Task>();

        public Form()
        {
            InitializeComponent();


            semaphore = new SemaphoreSlim(threadNum);
        }

        
        /*##########################################################################
        # D&D されたとき
        ############################################################################*/
        private void Form_DragDrop(object sender, DragEventArgs e)
        {
            string[] str = (string[])e.Data.GetData(DataFormats.FileDrop, false);
            //Debug.WriteLine(str[0]);

            for (int i = 0; i < str.Length; i++)
            {
                file.Add(str[i]);
                Debug.WriteLine(str[i]);
                //listBox.Items.Add(Path.GetFileName(str[i]));
                listBox.Items.Add(str[i]);
            }
        }


        /*##########################################################################
        # ファイルがウインドウに乗った時
        ############################################################################*/
        private void Form_DragEnter(object sender, DragEventArgs e)
        {
            if (e.Data.GetDataPresent(DataFormats.FileDrop))
            {
                string[] str = (string[])e.Data.GetData(DataFormats.FileDrop, false);

                bool flag = false;
                for(int i = 0; i < str.Length; i++)
                {
                    string ext = Path.GetExtension(str[i]);
                    if ( (ext != ".HEIC") && (ext != "HEIF") )
                    {
                        flag = true;
                        break;
                    }
                }

                if(flag == true)
                {
                    e.Effect = DragDropEffects.None;
                }
                else
                {
                    e.Effect = DragDropEffects.All;
                }
            }
            else
            {
                e.Effect = DragDropEffects.None;
            }

        }

        /*##########################################################################
        # コンバートボタン クリック
        ############################################################################*/
        private void button_Click(object sender, EventArgs e)
        {
            folderBrowserDialog.ShowDialog();                   //保存場所表示
            folderPath = folderBrowserDialog.SelectedPath;      //保存場所を格納

            Convert();      //保存ディレクトリ指定
        }


        /*##########################################################################
        # コンバート
        ############################################################################*/
        private async void Convert()
        {
            progressBar.Value = 0;
            per = 100.0f / (float)file.Count;

            int t = 0;
            semaphore.Release(threadNum);
/////////////////////////////////////////////////////////////////////////////////////////
            for (int i = 0; i < file.Count; i++)
            {
                Debug.WriteLine("file       " + file[i]);
                task.Add(Task.Run(() => Run(new String(file[i]))));

            }
//////////////////////////////////////////////////////////////////////////////////////////
            button.Enabled = false;
            await Task.WhenAll(task.ToArray());

         //   semaphore.Release(threadNum);

            button.Enabled = true;

            //リセット
            progressBar.Value = 0;
            file.Clear();
            listBox.Items.Clear();
        }

        /*##########################################################################
        # 非同期関数
        ############################################################################*/
        private void Run(string path)
        {
            

            string filePath = folderPath + "/" + Path.GetFileName(Path.ChangeExtension(path, extension));
            semaphore.Wait();
            ImageMagick.MagickImage img = new ImageMagick.MagickImage(path);
            img.Write(filePath);

            Debug.WriteLine(filePath);

            img.Dispose();

            this.Invoke(() => 
            { 
                progressBar.Value += (int)per; 
                //listBox.Items.Remove(path);
            });


                
            this.Invoke(() => { num++; Debug.WriteLine(num); });

            semaphore.Release();
        }






        private void Form_Load(object sender, EventArgs e)
        {

        }

        

    }
}

2 件の回答 2

2

質問ですが確かに治りましたが、どういった理由なのかもう少し詳細に聞いてもいいですか?

for (int i = 0; i < file.Count; i++)
{
    Debug.WriteLine("file       " + file[i]);
    task.Add(Task.Run(() => Run(new String(file[i]))));
}

こちらですが、ラムダ式の中で、外側の変数iを参照しています。ラムダ式は構築された際の変数iの値が記憶されるのではなく、ラムダ式が実行された際に改めて変数iの値を読みます。

forループは早々に進めるため、iはきっとfile.Countに到達しforループを抜け出ていることでしょう。結果、ラムダ式を実行するタイミングでは file[i]file[file.Count] 相当であり、ArgumentOutOfRangeExceptionが発生するのも当然と言えます。


別の記述方法として

foreach (var f in file)
{
    Debug.WriteLine("file       " + f);
    task.Add(Task.Run(() => Run(f)));
}

とか、デバッグ表示やタスク化不要なら

Parallel.ForEach(file, f => Run(f));

とか。

1

Task.Run() の中は、別スレッドなので i を参照してはいけません。
動作したときに、別の値に置き換わっていることがあります。

for (int i = 0; i < file.Count; i++) {
    var s = file[i];
    task.Add(Task.Run(() => Run(s)));
}

これで良いと思います。

2
  • ありがとうございます。質問ですが確かに治りましたが、どういった理由なのかもう少し詳細に聞いてもいいですか? Commented 2022年11月6日 8:57
  • 他に言いようがないです。 メインスレッドが for ループを抜けてから Run(File[i]) が実行されると ArgumentOutOfRangeException が起きます。 提示したコードで問題ないのは、メインスレッドで file[i] を解決しているからです。
    – KOZ
    Commented 2022年11月6日 11:15

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。