closing the open file

[basti42]

hello!
i could i force to close the file when it is opened?

tnx bunch

[mikelynch]

if you were using a StreamReader , it would be something like

sr.Close();

[drattansingh]

e.g.

StreamWriter sw = File.AppendText(Path);
sw.WriteLine(txtInput.Text);
sw.Close();

[basti42]

here is the code:
try
{
StreamReader sr = new StreamReader(txtPath.Text);
string lineHeader = sr.ReadLine(); //disregard the heading
string lineContents = sr.ReadLine(); //read the contents until EOF

while (lineContents != null)
{
string[] strValues = lineContents.Split(',');
listBox1.Items.Add(strValues[0] + " - " + strValues[1]);
lineContents = sr.ReadLine();
}
}
catch (Exception ex)
{
//MessageBox.Show(ex.Message,"Error",MessageBoxButtons.OK,MessageBoxIcon.Error);
}

if the above Try found that the file is opened, how could close it on Catch?
or close the file and then read the contents?

[ComputerJy]

EDIT: You should catch System.IO Exceptions before the general exception. And you shouldn't show ex.Message in a message box, you should do something to handle the error and show a meaningful message to the user

        StreamReader sr = new StreamReader(txtPath.Text);
        try
        {
            string lineHeader = sr.ReadLine(); //disregard the heading
            string lineContents = sr.ReadLine(); //read the contents until EOF

            while ( lineContents != null )
            {
                string[] strValues = lineContents.Split(',');
                listBox1.Items.Add(strValues[0] + " - " + strValues[1]);
                lineContents = sr.ReadLine();
            }
        }
        catch ( Exception ex )
        {
            //MessageBox.Show(ex.Message,"Error",MessageBoxButtons.OK,MessageBoxIcon.Error);
        }
        finally
        {
            sr.Close();
        }

[jmcilhinney]

You should be making use of 'using' blocks so there's no need for a 'finally' block:

using (StreamReader sr = new StreamReader(txtPath.Text))
{
    try
    {
    
    }
    catch
    {
 
    }
}

The StreamReader will be disposed and the file closed at the last closing brace no matter what happens inside the 'using' block. You should do this with all short-lived objects that require disposing.

[VBGuy]

Maybe the original poster is saying that when they try to open the file, it errors because it is open not because he (she) opened it and didn't close it but because some other user/process has the file open.

[ComputerJy]

You should be making use of 'using' blocks so there's no need for a 'finally' block:

using (StreamReader sr = new StreamReader(txtPath.Text))
{
    try
    {
    
    }
    catch
    {
 
    }
}

The StreamReader will be disposed and the file closed at the last closing brace no matter what happens inside the 'using' block. You should do this with all short-lived objects that require disposing.

I'm not sure. but is the using keyword available in all versions of C#?

[jmcilhinney]

I'm not sure. but is the using keyword available in all versions of C#?

It may not be supported before 2005 but if someone's not going to bother specifying their version then I'll assume they're using the latest.

[basti42]

tnx for sharing ur ideas...
to let this discussion end, when i found out the file is open
i will close it because it keeps showing the message "already use by the other process".

tnx to all of you