Program prints when i press CANCEL

Question

this is the print method:
public int print(Graphics g, PageFormat pageFormat, int pageIndex) throws PrinterException
     Graphics2D  g2 = (Graphics2D) g;
     g2.setColor(Color.black);
     int fontHeight=g2.getFontMetrics().getHeight();
     int fontDesent=g2.getFontMetrics().getDescent();
     //leave room for page number
     double pageHeight = pageFormat.getImageableHeight()-fontHeight;
     double pageWidth = pageFormat.getImageableWidth();
     double tableWidth = (double) table.getColumnModel().getTotalColumnWidth();
     double scale = 1;
     if (tableWidth >= pageWidth)
          scale =  pageWidth/tableWidth;
     double headerHeightOnPage = table.getTableHeader().getHeight()*scale;
     double tableWidthOnPage = tableWidth*scale;
     double oneRowHeight = (table.getRowHeight() + table.getRowMargin())*scale;
     int numRowsOnAPage = (int)((pageHeight-headerHeightOnPage)/oneRowHeight);
     double pageHeightForTable = oneRowHeight*numRowsOnAPage;
     int totalNumPages = (int)Math.ceil(((double)table.getRowCount())/numRowsOnAPage);
     if(pageIndex>=totalNumPages)
          return NO_SUCH_PAGE;
     g2.translate(pageFormat.getImageableX(),pageFormat.getImageableY());
     g2.drawString("Page: "+(pageIndex+1),
                                           (int)pageWidth/2-35,
                                           (int)(pageHeight+fontHeight-fontDesent));//bottom center
     g2.translate(0f,headerHeightOnPage);
     g2.translate(0f,-pageIndex*pageHeightForTable);
     //If this piece of the table is smaller than the size available,
     //clip to the appropriate bounds.
     if (pageIndex + 1 == totalNumPages)
          int lastRowPrinted = numRowsOnAPage * pageIndex;
          int numRowsLeft = table.getRowCount() - lastRowPrinted;
          g2.setClip(0, (int)(pageHeightForTable * pageIndex),
                                   (int) Math.ceil(tableWidthOnPage),
                                   (int) Math.ceil(oneRowHeight * numRowsLeft));
     //else clip to the entire area available.
     else
          g2.setClip(0, (int)(pageHeightForTable*pageIndex),
                                   (int) Math.ceil(tableWidthOnPage),
                                   (int) Math.ceil(pageHeightForTable));
     g2.scale(scale,scale);
     table.paint(g2);
     g2.scale(1/scale,1/scale);
     g2.translate(0f,pageIndex*pageHeightForTable);
     g2.translate(0f, -headerHeightOnPage);
     g2.setClip(0, 0,(int) Math.ceil(tableWidthOnPage),(int)Math.ceil(headerHeightOnPage));
     g2.scale(scale,scale);
     table.getTableHeader().paint(g2);//paint header at top
     return Printable.PAGE_EXISTS;
}this is the print button:
JButton print = new JButton("Print Results");
print.addActionListener(
     new ActionListener()
          public void actionPerformed(ActionEvent evt)
               PrinterJob pj = PrinterJob.getPrinterJob();
               pj.setPrintable(ScienceOlympiadProgram.this);
               pj.printDialog();
               try
                    pj.print();
               catch (Exception PrintException)
);everything works properly. when i run the program, the print button does its job, prints the data almost the way i want it (that will be fixed later) but the only problem is when i press Cancel on the dialog box, the program prints anyway. seems like an easy fix, but I'm mentally wiped. any suggestions are greatly appreciated!
thank you,
Hristo

Answer 1

That's because you never test the value returned by printDialog().
{color:#0000ff}http://java.sun.com/javase/6/docs/api/java/awt/print/PrinterJob.html#printDialog(){color}
Returns:
    true if the user does not cancel the dialog; false otherwise.
Also, an empty catch block is not advisable, in case of an Exception you won't know what went wrong.if (pj.printDialog ()) {
   try {
      pj.print ();
   } catch (PrinterException ex) {
       ex.printStackTrace ();
}db