process.standardoutput.ReadToEnd() always empty?

Question

im starting a console application,
but when I redirect the standardoutput I always get nothing!!
when I dont redirect it, and set createnowindow to false,
I see everything correctly in the console, but when I redirect it, standardoutput.ReadToEnd() always returns an emty string.

        Process cproc = new Process();
        cproc.StartInfo.CreateNoWindow = true;
        cproc.StartInfo.FileName = Dest;
        cproc.StartInfo.RedirectStandardOutput = true;
        cproc.StartInfo.WindowStyle = ProcessWindowStyle.Hidden;
        cproc.StartInfo.UseShellExecute = false;
        cproc.EnableRaisingEvents = true;
        cproc.Start();
        cproc.Exited += new EventHandler(cproc_Exited);
        while(!stop)
        {
           result += cproc.StandardOutput.ReadToEnd();
        }

the eventhandler cproc_exited just sets stop to true.
can someone explain why result is always string.Empty?

Answer 1

You have redirection of standard out disabled. Try changing

cproc.StartInfo.RedirectStandardOutput = false;

into

cproc.StartInfo.RedirectStandardOutput = true;

Does the following sample from MSDN work for you?

// Start the child process.
Process p = new Process();
// Redirect the output stream of the child process.
p.StartInfo.UseShellExecute = false;
p.StartInfo.RedirectStandardOutput = true;
p.StartInfo.FileName = "Write500Lines.exe";
p.Start();
// Do not wait for the child process to exit before
// reading to the end of its redirected stream.
// p.WaitForExit();
// Read the output stream first and then wait.
string output = p.StandardOutput.ReadToEnd();
p.WaitForExit();

Answer 2

Why are you looping? Once it's read to the end, it's not going to be able to read any more data, is it?

Are you sure the text is actually being written to StandardOutput rather than StandardError?

(And yes, obviously you want to set RedirectStandardOutput to true rather than false. I assumed that was just a case of you copying the wrong version of your code.)

EDIT: As I've advised in the comments, you should read from standard output and standard error in separate threads. Do not wait until the process has exited - this can end up with a deadlock, where you're waiting for the process to exit, but the process is blocking trying to write to stderr/stdout because you haven't read from the buffer.

Alternatively you can subscribe to the OutputDataReceived and ErrorDataReceived events, to avoid using extra threads.

Answer 3

You would have to use cproc.WaitForExit(); before reading the standardoutput. This should solve your problem & you could avoid your loop as well.

Answer 4

Get rid of the loop and move the call to ReadToEnd to cproc_Exited.