How can I open a file when it is already opened by other user in system?

Question

PD4ML pd4ml = new PD4ML();
pd4ml.enableTableBreaks(true);
pd4ml.PageInsets = new System.Drawing.Rectangle(5, 5, 5, 5);
pd4ml.PageSize = PD4Constants.getSizeByName("LETTER");
Byte[] byteArray = Encoding.ASCII.GetBytes(content);
MemoryStream stream = new MemoryStream(byteArray);

FinalPath = FinalPath + @"\" + VersionID;
        if (!Directory.Exists(FinalPath))
            Directory.CreateDirectory(FinalPath);

string FileName = FinalPath +FileName+ ".pdf";

pd4ml.render(stream,new System.IO.FileStream(FileName, System.IO.FileMode.CreateNew));        
stream.Flush();
stream.Close();
stream.Dispose();

//In another method I'm opening this file
File stream fs = File.Open(path, FileMode.Open, FileAccess.Read);`

I'm generating PDF using pd4ml.render() method. When I create this file using render method it is opened somewhere in system internally. That's why when I tried to open it mannualy using Filestream fs=new Filestream(path,FileMode.Open,FileAccess.Read)

It throws and exception of file is being used by another process. Please guide me what to do.

I've already used FileShare.ReadWrite attribute and File.OpenRead(path) with my code but it doesn't work for me.

try using (var fs = new FileStream(path, FileMode.Open, FileAccess.Read, FileShare.ReadWrite)) — Marcus Höglund, May 26 '16 at 06:09
but to close it I've to open it first and when I tried to open it it gives me exception. And I've used FileShare.ReadWrite but it doesn't work for me. — Khanjan Bhatt, May 26 '16 at 06:14
Yes I'm creating a new file using pd4ml.render(stream, new System.IO.FileStream(FileName, System.IO.FileMode.CreateNew)). And after this creation file is opened somewhere in system internally that I can't detect. — Khanjan Bhatt, May 26 '16 at 06:22

S M · Answer 1 · 2016-05-26T06:30:04.970

0

Your problem is that File.Create will open a stream allowing you to do what you like to the file refer : http://msdn.microsoft.com/en-us/library/d62kzs03.aspx

Therefore, technically, it is in use already.

Just remove the File.Create altogether. The StreamWriter will handle creating the file if it doesn't exist.

When using streams it is good practice to do

using (Stream s = new Stream())
{
} // Stream closes here
If you also create the output stream, make sure to close it.

refer http://www.codeproject.com/Questions/1097511/Can-not-opening-pdfs-generated-using-pd-ml-using-C

edited May 26 '16 at 06:30

answered May 26 '16 at 06:20

S M

3,133
5
30
59

I'm creating file using pd4ml object. – Khanjan Bhatt May 26 '16 at 06:24

score 0 · Accepted Answer · answered May 26 '16 at 07:33

You're leaking a stream object that you should be disposing. Specifically, the one passed as the second parameter here:

pd4ml.render(stream,new System.IO.FileStream(FileName, System.IO.FileMode.CreateNew));

Rather than creating the new stream as part of that method call, you should place it in another variable and Dispose it (preferably making use of using statements for it and for stream, rather than manually).

using(var stream2 = new System.IO.FileStream(FileName, System.IO.FileMode.CreateNew))
{
  pd4ml.render(stream,stream2);
}

How can I open a file when it is already opened by other user in system?

2 Answers2