Page 1 of 1

Possible TReport problem with ::NeedNewPage()

Posted: Tue Oct 31, 2006 1:03 pm
by Roger Seiler
Antonio,

I had a problem with occasionally getting a blank page printed out between other normal pages. So I took a close look at the way ::NeedNewPage() is implemented in TReport and found that ::nStdLineHeight gets added twice in immediate succession to ::nRow, and this seemed to be the source of my problem. Apparently it should have only been added once.

Here is how it happens...

1. In ::EndLine()...

DEFAULT nHeight := ::nStdLineHeight

::nRow += nHeight

IF ::NeedNewPage()
::EndPage()
ENDIF

2. In the definition of ::NeedNewPage()...

METHOD NeedNewPage INLINE ( ::nRow + ::nStdLineHeight >= ::nBottomRow )

Isn't the addition of ::nStdLineHeight in this method an unnecessary duplication of the addition of this same value immediately prior in ::EndLine() ?

Adding this value a second time sometimes pushes the value of ::nRow to a number greater than ::nBottomRow, even though the bottom of the page has actually not quite yet been reached, thereby forcing an ::EndPage() before it is actually needed.

- Roger

Re: Possible TReport problem with ::NeedNewPage()

Posted: Tue Oct 31, 2006 5:36 pm
by Enrico Maria Giordano
I agree. I always used

Code: Select all

METHOD NeedNewPage() INLINE ( ::nRow >= ::nBottomRow )
in my customized TReport.

EMG

Posted: Wed Nov 01, 2006 8:09 am
by Antonio Linares
Roger,

>
Isn't the addition of ::nStdLineHeight in this method an unnecessary duplication of the addition of this same value immediately prior in ::EndLine() ?
>

Yes, it looks so. Is it working fine now with your fix ?

Thanks for your feedback,

Posted: Thu Nov 02, 2006 2:56 pm
by Roger Seiler
Yes, Enrico's fix works fine. Thanks.

- Roger