Problem in trying to open a file from list box on mouse click

Question

3.00/5 (3 votes)

See more:

C#

namespace DirList1
{
    public partial class Form1 : Form
    {
        const string rootFolder = @"C:\MyDoc";
        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            {
                DirectoryInfo dinfo = new DirectoryInfo(rootFolder);
                FileInfo[] Files = dinfo.GetFiles("*.*");
                foreach (FileInfo file in Files)
                {
                    listBox1.Items.Add(file.Name);
                }
            }
        }
        private void listBox1_MouseDoubleClick(object sender, MouseEventArgs e)
        {

         Process.Start("notepad.exe", rootFolder + @"\" + (sender as ListBox).SelectedItem.ToString());
        }
        private void listBox1_SelectedIndexChanged_1(object sender, EventArgs e)
        {
        }
    }
}

Posted 14-Feb-14 2:55am

izak_il

Updated 14-Feb-14 17:23pm

agent_kruger

v2

Add a Solution

Comments

EduChapow 14-Feb-14 9:04am

your click event is double click. try just a click.

any exception is throw?

izak_il 14-Feb-14 9:24am

I only have text files no error message Just not open files

Member 10434230 14-Feb-14 9:29am

Hi izak,
I checked at my end and it is opening text files properly. Can you send me the full path that you are passing to process.start

Member 10434230 14-Feb-14 9:06am

What error are you getting?

phil.o 14-Feb-14 10:22am

From famous philosopher Yoda: "Don't try ; do it, or not."
:)

BillWoodruff 14-Feb-14 11:07am

Your code should work ! Put a break-point in your code at the line where you invoke Process.Start, and then examine the value in 'rootFolder. Also, there's no need to do the conversion to ListBox: just use:

Process.Start("notepad.exe", rootFolder + @"\" + listBox1.SelectedItem.ToString());

2 solutions

Solution 1

Are you just having notepad compatible files? if you have multiple file types and you want OS to decide which application to use for opening the files, use below code:
Try to use:

Process.Start(rootFolder + @"\" + (sender as ListBox).SelectedItem.ToString());

Posted 14-Feb-14 3:13am

Member 10434230

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

S Houghtelin · Accepted Answer · 2014-02-14T03:44:00

make these changes, then it should work.

C#

// Another string to contain path and file.
string resultFilePath;
...
private void listBox1_SelectedIndexChanged(object sender, EventArgs e)
{
    resultFilePath= rootFolder + @"\" + (sender as ListBox).SelectedItem.ToString();
}

private void button1_Click(object sender, EventArgs e)
{
    Process.Start("notepad.exe", resultFilePath);
}

Also make sure the control events are working. I have seen where someone will cut and paste event code but the event is not connected to the control.

You may also want to filter the file type to those of intrest, for instance if you only want to see text files use the following..

C#

FileInfo[] Files = dinfo.GetFiles("*.txt");