C# 从 PictureBox 保存图像

Question

我有这样的代码：

private void Load_Button_Click(object sender, EventArgs e)
    {
        OpenFileDialog dialog = new OpenFileDialog();            
        if (dialog.ShowDialog()==DialogResult.OK){
            MessageBox.Show(dialog.FileName,"My Application", MessageBoxButtons.OK,MessageBoxIcon.Asterisk);
            string s; 
            s=".bmp";
            if (dialog.FileName.Substring(dialog.FileName.LastIndexOf('.')).Equals(s))
            {
                picBox_1.Load(dialog.FileName);
                BitmapFile = new Bitmap(dialog.FileName.ToString());
            }
            else {
                MessageBox.Show("Not a BMP file!");
            }
        }

    }

所以，加载图像。并有一个错误：

private void Save_Button_Click(object sender, EventArgs e)
    {
        SaveFileDialog dialog = new SaveFileDialog();
        try
        {
            if (picBox_1.Image != null)
            {
                if (dialog.ShowDialog() == DialogResult.OK)
                {
                    MessageBox.Show(dialog.FileName, "My Application", MessageBoxButtons.OK, MessageBoxIcon.Asterisk);
                    string s;
                    s = ".bmp";
                    if (dialog.FileName.Substring(dialog.FileName.LastIndexOf('.')).Equals(s))
                    {

                        picBox_1.Image.Save(dialog.FileName.ToString());
                        //BitmapFile.Dispose();
                    }
                    else
                    {
                        MessageBox.Show("Not a BMP file!");
                    }
                }
            }
            else
            {
                MessageBox.Show("My PicBox is empty!");
            }
        }
        catch (Exception) { MessageBox.Show("Cannot save file, error!"); }

    }

这是一般的 GDI 错误。我想，我无法写入文件（可能没有足够的权限）。我该如何改善这个错误？

Answer 1

你吃掉了例外，失去了所有有趣的细节。尝试将您的 catch 块更改为类似的内容，看看发生了什么：

catch (Exception ex)
{
    MessageBox.Show(this, ex.ToString(), "Error Saving Image", MessageBoxIcons.Error);
}

另外，考虑实现一些日志记录（到事件查看器和/或文本文件。这将允许您拥有一个简单的消息框，但所有有趣的细节都放在某个有用的地方，以便在事件发生后获取。

catch (Exception ex)
{
    MessageBox.Show(this, ex.Message, "Error Saving Image", MessageBoxIcon.Error);

    // _logger is a private field on this class in this case.
    _logger.Log(ex, string.Format("Saving image to {0}", dialog.Filename))
}

您可以查看Log4net 以及其他内容以获取实际日志记录，但至少要为write exception detail to the event viewer 编写一个类。

Answer 2

您应该正确捕获异常，而不是使用 MessageBox 来告诉您所引发的确切异常！

你的 catch 块至少应该是这样的：

catch (Exception exc)
{
  MessageBox.Show(exc.Message);
}

我说至少是因为您实际上应该在某处记录异常，使用 NLog 或 Log4Net 之类的日志框架并转储堆栈跟踪和其他详细信息。如果您显示带有静态字符串的消息而不是实际异常的详细信息，您甚至无法判断异常的确切类型。

Answer 3

您应该只捕获您打算处理或从中恢复的特定异常，并记录详细信息。永远不要抓住Exception，因为如果它们发生，您可能会掩盖服务器的更大问题。

意外的异常应该冒出来，以便在发生时能够快速确定原因。

查看这里Best Practices for Handling Exceptions。