Skip to main content
今回の問題に対する言及を追記
ソース リンク

リサイズしたファイと同じサイズに、コピーしたファイルが変わってしまいます。

他の回答にもあるように、ResizeImage メソッドの以下のコードで、FileSystemWatcher.Created が再度発火し、リサイズされた画像が上書きコピーされてしまっているのだと思います。

newImage.Save(filePath); // リサイズした画像を保存

FileSystemWatcher.Created が発火されないようにするためには、ファイルを Stream で開いて変更を反映する必要があります。

これを行うためには、画像ファイルの変更を MemoryStream に保存しておき、その後、対象のファイルを Stream で開いて変更を反映してやります。

private void ResizeImage(string filePath, int width, int height)
{
    using (var memoryStream = new MemoryStream())
    {
        using (var originalImage = Image.FromFile(filePath))
        {
            using (var resizedImage = new Bitmap(width, height))
            {
                using (Graphics g = Graphics.FromImage(resizedImage))
                {
                    g.InterpolationMode = System.Drawing.Drawing2D.InterpolationMode.HighQualityBicubic;
                    g.DrawImage(originalImage, 0, 0, width, height);
                }

                resizedImage.Save(memoryStream, originalImage.RawFormat);
            }                    
        }

        using (var stream = new FileStream(filePath, FileMode.Open, FileAccess.ReadWrite))
        {
            memoryStream.WriteTo(stream);
        }
    }
}

この場合、FileSystemWatcher.Created は発火せず、FileSystemWatcher.Changed が発火されます。

よって、FileSystemWatcher.Created で実装されているコピー操作が再度実行されなくなり、今回の問題は解消するのではないでしょうか?

リサイズしたファイと同じサイズに、コピーしたファイルが変わってしまいます。

他の回答にもあるように、ResizeImage メソッドの以下のコードで、FileSystemWatcher.Created が再度発火し、リサイズされた画像が上書きコピーされてしまっているのだと思います。

newImage.Save(filePath); // リサイズした画像を保存

FileSystemWatcher.Created が発火されないようにするためには、ファイルを Stream で開いて変更を反映する必要があります。

これを行うためには、画像ファイルの変更を MemoryStream に保存しておき、その後、対象のファイルを Stream で開いて変更を反映してやります。

private void ResizeImage(string filePath, int width, int height)
{
    using (var memoryStream = new MemoryStream())
    {
        using (var originalImage = Image.FromFile(filePath))
        {
            using (var resizedImage = new Bitmap(width, height))
            {
                using (Graphics g = Graphics.FromImage(resizedImage))
                {
                    g.InterpolationMode = System.Drawing.Drawing2D.InterpolationMode.HighQualityBicubic;
                    g.DrawImage(originalImage, 0, 0, width, height);
                }

                resizedImage.Save(memoryStream, originalImage.RawFormat);
            }                    
        }

        using (var stream = new FileStream(filePath, FileMode.Open, FileAccess.ReadWrite))
        {
            memoryStream.WriteTo(stream);
        }
    }
}

この場合、FileSystemWatcher.Created は発火せず、FileSystemWatcher.Changed が発火されます。

リサイズしたファイと同じサイズに、コピーしたファイルが変わってしまいます。

他の回答にもあるように、ResizeImage メソッドの以下のコードで、FileSystemWatcher.Created が再度発火し、リサイズされた画像が上書きコピーされてしまっているのだと思います。

newImage.Save(filePath); // リサイズした画像を保存

FileSystemWatcher.Created が発火されないようにするためには、ファイルを Stream で開いて変更を反映する必要があります。

これを行うためには、画像ファイルの変更を MemoryStream に保存しておき、その後、対象のファイルを Stream で開いて変更を反映してやります。

private void ResizeImage(string filePath, int width, int height)
{
    using (var memoryStream = new MemoryStream())
    {
        using (var originalImage = Image.FromFile(filePath))
        {
            using (var resizedImage = new Bitmap(width, height))
            {
                using (Graphics g = Graphics.FromImage(resizedImage))
                {
                    g.InterpolationMode = System.Drawing.Drawing2D.InterpolationMode.HighQualityBicubic;
                    g.DrawImage(originalImage, 0, 0, width, height);
                }

                resizedImage.Save(memoryStream, originalImage.RawFormat);
            }                    
        }

        using (var stream = new FileStream(filePath, FileMode.Open, FileAccess.ReadWrite))
        {
            memoryStream.WriteTo(stream);
        }
    }
}

この場合、FileSystemWatcher.Created は発火せず、FileSystemWatcher.Changed が発火されます。

よって、FileSystemWatcher.Created で実装されているコピー操作が再度実行されなくなり、今回の問題は解消するのではないでしょうか?

ソース リンク

リサイズしたファイと同じサイズに、コピーしたファイルが変わってしまいます。

他の回答にもあるように、ResizeImage メソッドの以下のコードで、FileSystemWatcher.Created が再度発火し、リサイズされた画像が上書きコピーされてしまっているのだと思います。

newImage.Save(filePath); // リサイズした画像を保存

FileSystemWatcher.Created が発火されないようにするためには、ファイルを Stream で開いて変更を反映する必要があります。

これを行うためには、画像ファイルの変更を MemoryStream に保存しておき、その後、対象のファイルを Stream で開いて変更を反映してやります。

private void ResizeImage(string filePath, int width, int height)
{
    using (var memoryStream = new MemoryStream())
    {
        using (var originalImage = Image.FromFile(filePath))
        {
            using (var resizedImage = new Bitmap(width, height))
            {
                using (Graphics g = Graphics.FromImage(resizedImage))
                {
                    g.InterpolationMode = System.Drawing.Drawing2D.InterpolationMode.HighQualityBicubic;
                    g.DrawImage(originalImage, 0, 0, width, height);
                }

                resizedImage.Save(memoryStream, originalImage.RawFormat);
            }                    
        }

        using (var stream = new FileStream(filePath, FileMode.Open, FileAccess.ReadWrite))
        {
            memoryStream.WriteTo(stream);
        }
    }
}

この場合、FileSystemWatcher.Created は発火せず、FileSystemWatcher.Changed が発火されます。