NetworkStreams

[Ron]

Hi all,

I created a POP3 client (with the help of some examples of the net:).
I'm using a StreamReader to read from the NetworkStream as follows:
while (sTmp != ".")
{
sMsg += sTmp + "\r\n";
sTmp = strmReader.ReadLine();
}

The problem is that this is really slow and uses a lot of CPU time.

Is there a faster way to do this??

Thanx,
Ron

[Justin Rogers]

Your string concatenation is going to kill you. You'd be better off using
a collection or array based storage for holding each individual string and
then joining them later. I do approximately the same operations for
newsgroups, downloading at over 1mbit per second while reconstructing
and decoding on a pretty lame machine. Never seen more than 10-15%
CPU utilization for these processes.
--
Justin Rogers
DigiTec Web Consultants, LLC.
Blog: http://weblogs.asp.net/justin_rogers

"Ron" <rv*******@hotmail.com> wrote in message
news:eJ**************@TK2MSFTNGP12.phx.gbl...

Hi all,

I created a POP3 client (with the help of some examples of the net:).
I'm using a StreamReader to read from the NetworkStream as follows:
while (sTmp != ".")
{
sMsg += sTmp + "\r\n";
sTmp = strmReader.ReadLine();
}

The problem is that this is really slow and uses a lot of CPU time.

Is there a faster way to do this??

Thanx,
Ron

[Daniel O'Connell [C# MVP]]

"Ron" <rv*******@hotmail.com> wrote in message
news:eJ**************@TK2MSFTNGP12.phx.gbl...

Hi all,

I created a POP3 client (with the help of some examples of the net:).
I'm using a StreamReader to read from the NetworkStream as follows:
while (sTmp != ".")
{
sMsg += sTmp + "\r\n";
sTmp = strmReader.ReadLine();
}

The problem is that this is really slow and uses a lot of CPU time.

Is there a faster way to do this??

Where is your slow down occuring? Is it in building sMsg or is it in the
call to strmReader.ReadLine()?

If its in building sMsg, you may want to look at System.Text.StringBuilder,
if its in strmReader.ReadLine(), you may get better performance by copying
large blocks into a MemoryStream and doing your parsing from there.

[Ron]

It was indeed the string concatenation. I am using the StringBuilder now,
and it has decreased the cpu usage dramatically.

Thanks.
Ron
"Daniel O'Connell [C# MVP]" <[e-mail]> wrote in
message news:#z**************@TK2MSFTNGP15.phx.gbl...

"Ron" <rv*******@hotmail.com> wrote in message
news:eJ**************@TK2MSFTNGP12.phx.gbl...

Hi all,

I created a POP3 client (with the help of some examples of the net:).
I'm using a StreamReader to read from the NetworkStream as follows:
while (sTmp != ".")
{
sMsg += sTmp + "\r\n";
sTmp = strmReader.ReadLine();
}

The problem is that this is really slow and uses a lot of CPU time.

Is there a faster way to do this??
Where is your slow down occuring? Is it in building sMsg or is it in the
call to strmReader.ReadLine()?

If its in building sMsg, you may want to look at

System.Text.StringBuilder, if its in strmReader.ReadLine(), you may get better performance by copying
large blocks into a MemoryStream and doing your parsing from there.

[Daniel O'Connell [C# MVP]]

"Ron" <rv*******@hotmail.com> wrote in message
news:%2****************@TK2MSFTNGP12.phx.gbl...

It was indeed the string concatenation. I am using the StringBuilder now,
and it has decreased the cpu usage dramatically.

I suspected that it was, but it was possible that you were experiancing come
obscure bug caused by something with your socket settup, thats why I
suggested both, ;).

Depending on your purposes, Justin's suggestion may be better. If you are
decoding attachments or multipart mime messages it almost certainly is, but
if you are just sucking out plain text messages StringBuilder will work
fine. Thanks.
Ron
"Daniel O'Connell [C# MVP]" <[e-mail]> wrote in
message news:#z**************@TK2MSFTNGP15.phx.gbl...

"Ron" <rv*******@hotmail.com> wrote in message
news:eJ**************@TK2MSFTNGP12.phx.gbl...

> Hi all,
>
> I created a POP3 client (with the help of some examples of the net:).
> I'm using a StreamReader to read from the NetworkStream as follows:
> while (sTmp != ".")
> {
> sMsg += sTmp + "\r\n";
> sTmp = strmReader.ReadLine();
> }
>
> The problem is that this is really slow and uses a lot of CPU time.
>
> Is there a faster way to do this??

Where is your slow down occuring? Is it in building sMsg or is it in the
call to strmReader.ReadLine()?

If its in building sMsg, you may want to look at

System.Text.StringBuilder,

if its in strmReader.ReadLine(), you may get better performance by
copying
large blocks into a MemoryStream and doing your parsing from there.