3

我正在尝试打印发票。发票应该可以打印在多页上，但这就是问题所在。我可以完美地在单页上打印发票，但是一旦发票不适合单页，打印作业就会退出第一页。

这是我正在使用的代码。'artikelen' 是文章列表（List）。我已经阅读了几个类似的例子，我很确定我在这里遗漏了一些东西。

（编辑：删除了一些不必要的代码）

public void PrintA4Factuur()
    {
        p = new PrintDocument();
        p.PrintPage +=
            new PrintPageEventHandler(printPage);
        printPreviewDialog.Document = p;
        printPreviewDialog.ShowDialog();
    }

void printPage(object sender1, PrintPageEventArgs e1)
    {
Graphics g = e1.Graphics;
int yPos = 320;
float pageHeight = e1.MarginBounds.Height;
int artikelPosition = 0;
while (yPos + 100 < pageHeight
            && artikelPosition < this.artikelen.Count)
        {
            // Do stuff with articles (printing details in different rectangles

            artikelPosition += 1;
            yPos += 20;
        }

        if (artikelPosition < this.artikelen.Count)
        {
            e1.HasMorePages = true;
            return;
        }
        else
        {
            e1.HasMorePages = false;
        }
}
4

2 回答 2

4

好吧，Lars 指出了在artikelPosition每页开头重置为零的问题，但是这段代码还有一些其他问题。

您应该始终使用e1.MarginBounds 坐标，因为边距可以由用户更改，并且p.DefaultPageSettings不会包括在内。

使用字体度量，例如GetHeight(yourDeviceGraphPort)，不要硬编码行高。

始终使用floats 表示坐标，不要在ints 之间转换。

字体是非托管资源，使用完后必须Dispose使用它们。循环重复创建和处理字体效率低下；在调用之前构造它PrintDocument.Print()并在所有页面打印后处理它。

还有一个 Black SolidBrush 已经定义在System.Drawing.

于 2012-04-19T17:39:38.533 回答
3

我发现你的代码做相反的事情：如果它打印超过一页，它会继续打印到无穷大。

尝试将您的索引位置变量移到PrintPage事件之外，因为将其设置回零只会将其再次设置为开头：

int artikelPosition = 0;

开始打印时重置它：

public void PrintA4Factuur()
{
  artikelPosition = 0

  p = new PrintDocument();
  p.PrintPage += printPage;
  printPreviewDialog.Document = p;
  printPreviewDialog.ShowDialog();
}

然后在您的 PrintPage 例程中将其注释掉：

void printPage(object sender1, PrintPageEventArgs e1)
{
  Graphics g = e1.Graphics;
  int yPos = 320;
  float pageHeight = e1.MarginBounds.Height;

  // int artikelPosition = 0;

  // continue with code
}
于 2012-04-19T17:25:49.480 回答