Printing problem

[eric]

I have a text file that I would like to print. I load the contents of
this text file into a string, call it strData. I then pass strData
into my print method, which creates it as a StreamReader. The print
method creates an instance of the PrintDocument_P rintPage method, which
I got from the following website:

http://msdn.microsoft.com/library/de...tPageTopic.asp

My problem is that when the document actually prints out, instead of
printing two pages, it will only print one page, with multiple lines of
text printed on top of each other. Can anyone see where my error is?

[Bruce Wood]

Could you post your code, including the method you copied from the page
to which you linked?

There are several possible sources for the problem you described. It
all depends upon the code.

[eric]

public void PrintData(strin g userInput)
{
streamToPrint = new StringReader(us erInput);

printFont = new Font("Times New Roman", 10);

PrintDocument.P rintPage += new
PrintPageEventH andler(PrintDoc ument_PrintPage );

PrintDocument.P rint();

streamToPrint.C lose();
}
private void PrintDocument_P rintPage(object sender,
System.Drawing. Printing.PrintP ageEventArgs e)
{
float linesPerPage = 0;
float yPos = 0;
int count = 0;
float leftMargin = e.MarginBounds. Left;
float topMargin = e.MarginBounds. Top;
string line = null;

linesPerPage = e.MarginBounds. Height /
printFont.GetHe ight(e.Graphics ) ;

while (count<linesPer Page && ((line=streamTo Print.ReadLine( ))!=null))
{
yPos = topMargin + (count * printFont.GetHe ight(e.Graphics ));
e.Graphics.Draw String (line, printFont, Brushes.Black, leftMargin,
yPos, new StringFormat()) ;
count++;
}

if (line!=null)
e.HasMorePages = true;
else
e.HasMorePages = false;
}

[Bruce Wood]

I tested your code (with one modification) and it works fine for me.

I think it depends upon how you get the user input into the userInput
string. Can you post the code that does that? I simply initialized it
as follows:

string userInput = "The quick brown fox\r\njumped over the
lazy\r\ndog.";

and I got three lines of output.

Well, I got three lines of output only after I corrected something in
the code you posted, which didn't compile. The part where you say:

PrintDocument.P rintPage += new
PrintPageEventH andler(PrintDoc ument_PrintPage );
PrintDocument.P rint();

doesn't compile, because PrintDocument is the class name, not an
instance, and the PrintPage event and the Print method are instance
members, not static members, so I changed this to:

PrintDocument pd = new PrintDocument() ;
pd.PrintPage += new
PrintPageEventH andler(PrintDoc ument_PrintPage );
pd.Print();

but that was to get it to compile at all. Once I did that, it produced
correct output.

So, my conclusion is that somehow when you load the text into the
userInput string, the line termination characters are being eliminated.
So, please post the code that loads that information into the string.

[eric]

I have the following method load the text file.

private void LoadDataFile(st ring filePath)
{
using(StreamRea der sr = new StreamReader(fi lePath))
{
string line;

while ((line = sr.ReadLine()) != null)
{
if((line!=@"\n" || line!=@" \n") && fileContents!=" ")
{
fileContents = string.Concat(f ileContents, '\n', line);
}
else
{
fileContents = string.Concat(f ileContents, line);
}
}
}
}

Suppose that the text file loaded has 85 lines. Then I don't have a
problem printing the first 56 lines, but lines 57-85 then reprint over
lines 1-28. Could it be due to the fact that I don't have the \r
character returns in my string?

[Bruce Wood]

> I don't have a problem printing the first 56 lines, but lines 57-85 then reprint over lines 1-28.

Ahh. You didn't say that before. :-)

I tried my little test again with many more lines and I got two pages
of output. I also tried with with just \n instead of \r\n and I still
got two pages of output.

The only change I made was the one I stated in my previous post.

I do notice, however, that there is something wrong with your test:

if((line!=@"\n" || line!=@" \n") && fileContents!=" ")

This has the following problems. First, since @"\n" and @" \n" are not
equal, the test (line != @"\n" || line != " \n") will always be true.
It can be false only when line equals BOTH @"\n" AND @" \n", which is
impossible. So, effectively the test becomes

if (fileContents != "")

In addition, I think that you don't want @"\n", but rather "\n". The
former compares against a backslash and a normal character n. The
latter against a newline character.

Finally, what you probably want to ask is whether the line is blank. I
would prefer to do this rather than what you've done:

bool isBlankLine = true;
int i = 0;
while (i < line.Length && isBlankLine)
{
isBlankLine = Char.IsWhiteSpa ce(line[i]);
i += 1;
}

then I think that what you want to do is this (I may be wrong):

if (!isBlankLine)
{
if (fileContents.L ength > 0)
{
fileContents = string.Concat(f ileContents, "\n", line);
}
else
{
fileContents = line;
}
}

Or you could do it better using a System.Text.Str ingBuilder:

System.Text.Str ingBuilder contents = new System.Text.Str ingBuilder();
....
if (!isBlankLine)
{
if (fileContents.L ength > 0)
{
contents.Append ("\n");
}
contents.Append (line);
}

Anyway, I don't see how any of this would produce your problem, since
your code works for me, even with these difficulties.

[eric]

Thanks for the help. I'm not for sure, but I think that my problem was
I wasn't creating a new instance of the PrintDocument class, I was just
using the PrintDocument that I added to my main form. Also, thanks for
correcting my if statement. I think I initially meant it to be an &&
instead of an ||. Also, I'll look into using the StringBuilder, but am
a little unclear as to why it's better, any particular reasons?

[eric]

Also, a question about your StringBuilder code. If I replaced

if (!isBlankLine)
{
if (fileContents.L ength > 0)
{
fileContents = string.Concat(f ileContents, "\n", line);
}
else
{
fileContents = line;
}

}

with

if (!isBlankLine)
{
if (fileContents.L ength > 0)
{
contents.Append ("\n");
}
contents.Append (line);

}

wouldn't it always add a "\n" because the fileContents string never
gets appended to, or added onto?

[Bruce Wood]

Yes it would. I should have written:

if (!isBlankLine)
{
if (contents.Lengt h > 0)
{
contents.Append ("\n");
}
contents.Append (line);
}

instead. As for why StringBuilder, I'm a little rushed right now, so
I'll cheat and let Jon Skeet explain: :-)

http://www.yoda.arachsys.com/csharp/stringbuilder.html