Question

我的代码将包含文件名和二进制数据的对象列表从db传递给创建所有文件的循环。我遇到的问题是下面的代码执行并且似乎正确创建文件（文件名和大小是预期的）但是，大多数文件在打开时“损坏”。文件类型从图像（jpg / png）到Word文档，Powerpoint演示文稿和PDF文件不等。奇怪的是，PDF文件完美运行，其他一切都“腐败”

我的代码在下面（attachment是循环中的对象，此阶段已经创建了路径）

if(Directory.Exists(attachmentPath))
{
    string absolutePath = attachmentPath + "\\importfiles\\" + parentfolders + "\\";

    // no need to check if it exists as it will ignore if it does
    Directory.CreateDirectory(absolutePath);
    absolutePath += filename;
    try
    {
        byte[] byteStream = null;
        object objSave = null;

        objSave = attachment.Image;
        BinaryFormatter tmpBinF = new BinaryFormatter();
        MemoryStream tmpMemStrm = new MemoryStream();
        tmpBinF.Serialize(tmpMemStrm, objSave);
        byteStream = tmpMemStrm.ToArray();

        // Delete the file if it exists.
        if (File.Exists(absolutePath))
        {
            File.Delete(absolutePath);
        }

        // Create the file.
        using (FileStream fs = File.Create(absolutePath))
        {
            fs.Write(byteStream, 0, byteStream.Length);
            fs.Dispose();
        }

    }

    catch (Exception ex)
    {
        Exceptions.Text += ex.ToString();
    }
}

我使用过MSDN提示并遵循this教程，但无法弄清楚为什么会这样。

感谢Amy用我的方法指出问题，如果有人需要，这是我的更新代码，将她的答案考虑在内。我还扩展了它以在数据库中的表上添加日志记录供以后使用。

if (Directory.Exists(attachmentPath))
{
    // build path from the parts
    string absolutePath = attachmentPath + "\\importfiles\\" + parentfolders + "\\";

    // no need to check if it exists as it will ignore if it does
    Directory.CreateDirectory(absolutePath);
    absolutePath += filename;
    byte[] file = attachment.Image;
    try
    {
        // Delete the file if it exists.
        if (File.Exists(absolutePath))
        {
            File.Delete(absolutePath);
        }

        // Create the file.
        using (FileStream fs = File.Create(absolutePath))
        {
            fs.Write(file, 0, file.Length);
        }

        // start logging to the database

        // add the Stored procedure
        string SP = "sp_add_attachment";

        // create the connection & command objects
        MySqlConnection myConnection1 = new MySqlConnection(WPConnectionString);
        MySqlCommand cmd1;
        try
        {
            // open the connection
            myConnection1.Open(); 

            cmd1 = myConnection1.CreateCommand();

            // assign the stored procedure string to the command
            cmd1.CommandText = SP;

            // define the command type
            cmd1.CommandType = CommandType.StoredProcedure;

            // pass the parameters to the Store Procedure
            cmd1.Parameters.AddWithValue("@AttachmentID", attachment.ID);
            cmd1.Parameters["@AttachmentID"].Direction = ParameterDirection.Input;

            cmd1.Parameters.AddWithValue("@subpath", parentfolders);
            cmd1.Parameters["@subpath"].Direction = ParameterDirection.Input;

            cmd1.Parameters.AddWithValue("@filename", filename);
            cmd1.Parameters["@filename"].Direction = ParameterDirection.Input;

            // execute the command 
            int output = cmd1.ExecuteNonQuery();

            // close the connection
            myConnection1.Close();
        }
        catch (Exception ex)
        {
            Exceptions.Text += "MySQL Exception when logging:" + ex.ToString();
        }

    }

    catch (Exception ex)
    {
        Exceptions.Text += ex.ToString();
    }
}

Answer 1

我不认为使用BinaryFormatter是合适的。如果attachment.Image是一个字节数组，只需将其写入文件流即可。完全忘记内存流和二进制格式化程序。

Binary Formatter类用于将.Net类序列化为字节数组。您已经有了一个字节数组，因此不需要该步骤并且是您的问题的根源。仅当使用相同的二进制格式化程序在数据库中创建blob时，才使用二进制格式化程序。但是你存储的是文件，而不是.Net对象，因此在这里没用。

我不确定为什么PDF会在其他文件不加载时加载。您必须使用十六进制编辑器检查文件以查看更改的内容。

从MemoryStream创建的某些文件已损坏

1 个答案: