使用itextsharp合并文件夹中的pdf文件

Question

I'm trying to use codes below to merge the pdf files in a folder and output into a new file but apparently the generated file seems corrupted.

public Boolean MergeForm(String destinationFile, String sourceFolder)
    {
        try
        {
            using (MemoryStream stream = new MemoryStream())
            using (Document doc = new Document())
            using (PdfCopy pdf = new PdfCopy(doc, stream))
            {
                doc.Open();

                PdfReader reader = null;
                PdfImportedPage page = null;

                foreach (var file in Directory.GetFiles(sourceFolder))
                {
                    reader = new PdfReader(file);
                    for (int i = 0; i < reader.NumberOfPages; i++)
                    {
                        page = pdf.GetImportedPage(reader, i + 1);
                        pdf.AddPage(page);
                    }

                    pdf.FreeReader(reader);
                    reader.Close();
                }
                using (FileStream streamX = new FileStream(destinationFile, FileMode.Create))
                {
                    stream.WriteTo(streamX);
                }
            }
            return true;
        }
        catch (Exception)
        {
            return false;
        }
    }

Can anyone spot on where's the problem? Thank you.

Answer 1

Can anyone spot on where's the problem?

Your main problem is that you use the contents of the MemoryStream before the Document and PdfCopy have finished creating the PDF (during the Dispose at the end of the using block). Thus, you save an incomplete PDF file as a result.

Doing it like this instead should work:

    using (MemoryStream stream = new MemoryStream())
    {
        using (Document doc = new Document())
        {
            PdfCopy pdf = new PdfCopy(doc, stream);
            pdf.CloseStream = false;
            doc.Open();

            PdfReader reader = null;
            PdfImportedPage page = null;

            foreach (var file in Directory.GetFiles(sourceFolder))
            {
                reader = new PdfReader(file);
                for (int i = 0; i < reader.NumberOfPages; i++)
                {
                    page = pdf.GetImportedPage(reader, i + 1);
                    pdf.AddPage(page);
                }

                pdf.FreeReader(reader);
                reader.Close();
            }
        }
        using (FileStream streamX = new FileStream(destinationFile, FileMode.Create))
        {
            stream.WriteTo(streamX);
        }
    }

BTW, you also see here that I did not put PdfCopy into a using block. This is because the Document implicitly closes the PDFCopy when it is disposed. First disposing the PdfCopy and then the Document (which tries to close the PdfCopy again), therefore, is not necessary and can result in hiding exceptions thrown from within the block by other exceptions occurring in this closing circus.

Furthermore I needed to add the pdf.CloseStream = false , otherwise the memory stream would have been closed when the PdfCopy is closed.

---

That been said,

1. Of course you should also use AddDocument instead of iterating over the document pages yourself as already explained by @Bruno.
2. Your memory footprint would decrease if you immediately wrote to the file stream instead of the memory stream.