printdocument adds blank page

You wrote that you have used sources from the web and I guess it shows. We all do that but it is really important to be rather suspicious about their quality. And of course combining two sources most likely will not work at all..

I'll first go over a couple of issues with your code and then show you an example of how to print items from a DB on several pages..

Please do yourself a favor and chose the names of your variables with care! Yes, that takes a little longer but it will pay, believe me. (And if you can't come up with anything convincing, step back! This is usually an indication that you don't understand the problem well enough..)

Let's look at some: There is currentY, which is pretty good. And there are itemperpage, totalnumber and amount, all of which are terrible! They either are totally unclear or even misleading!

Let's set up a few ones with better names:

// layout variables
int itemsPerPage = 40;  // how many items fit on one page
int itemHeight = 25;    // height of one item in the chosen unit

// print job variables
int totalNumber = -1;   // total number of records/items coming from the dbms
int itemsToPrint = 75;  // total number of items we want to print

// print progress variables
int itemsPrinted = 0;   // number of items printed so far
int pagesPrinted = 0;   // number of pages printed

Another issue is the unit you chose, or to be precise the lack of choosing one. The default GraphicsUnit is Display which for printing amounts to 1/100 inch. If you like that I suggest to write it in code to document it. This unit will be used during the whole layout process so it should be visible in the code! I personally prefer Millimeters. There are several other units..pick yours but document it!

One more thing you should look into are String.Format() and string.PadLeft(). They are so much nicer than the loop you have in the tostring() method!

Now let's look at your specific problems:

The one with an empty page at the end of the job will go away once we use enough variables with names one can actually understand without learning them..

The other problem is the way you set up the DB-reading: You need to set up the reader where you start the printing and not in the PrintPage loop. Note: These actually are two loops: one is visible as the while (...reader.Read() ) and the other, outer loop is implict: The whole PrintPage event itself is the loop body.

First let's look at the range() code. I have turned it into a function that returns the maximum number of records. I fetch the count in a first step. Then I set up the DataReader for the real reading, which will happen in the PrintPage event: I enumerate the fields so I have control over their order and don't pull in stuff I don't need.

Note that I am using MySql but the classes are pretty much the same, just with a MySQl-prefix. I don't include the connect code here. And of course I read from a database of my own..

MySqlConnection DBC = new MySqlConnection("");
MySqlCommand CMD = null;
MySqlDataReader DR = null;

int range ()
{
    CMD = new MySqlCommand("select count(0) from test.artists ", DBC);
    var count = CMD.ExecuteScalar();
    int counter = Convert.ToInt32(count);
    CMD = new MySqlCommand("select artist_ID, artist, genres from test.artists ", DBC);
    DR = CMD.ExecuteReader();
    return (int)counter;
}

Now let's look at the print command, using the variables from above:

private void cb_print_Click(object sender, EventArgs e)
{
    totalNumber = range();
    try
    {
        if (DBC.State == ConnectionState.Open)
        {
            pagesPrinted = 0;
            printPreviewDlg.Document = printDocument1;
            printPreviewDlg.ShowDialog();
        }
    } catch (Exception q) { MessageBox.Show("" + q); }
    DR.Dispose();
}

Note that the Reader is created in the range call!

Finally the PrintPage event:

private void printDocument1_PrintPage(object sender, 
                            System.Drawing.Printing.PrintPageEventArgs e)
{
    // my page unit
    e.Graphics.PageUnit = GraphicsUnit.Millimeter;

    // starting a new page
    int itemsOnPage = 0;
    pagesPrinted++;

    float currentY = 12;
    e.Graphics.DrawString(String.Format(
               "HEADER -  printing {0} items of {1}    - Page {2}",
               itemsToPrint, totalNumber, pagesPrinted), 
               Font, Brushes.Black, 1, currentY);  

    currentY += 30;  // header height

    try
    {
        // page is not full and we want to print more items
        while (itemsOnPage < itemsPerPage && itemsPrinted < itemsToPrint - 1
               && DR.Read())
        {
            string row = String.Format("{0,5:000} Artist: {1,20}  ({2})   ",
                                         DR[0], DR[1], DR[2]);

            e.Graphics.DrawString(row, DefaultFont, Brushes.Black, 50, currentY);  
            // Console.WriteLine("" + row);

            currentY += itemHeight; 
            itemsPrinted++; 
            itemsOnPage++;

            // we want to print more items but now the page is full
            if ( itemsPrinted < itemsToPrint && itemsOnPage >= itemsPerPage)
            {
                itemsOnPage = 0;        
                e.HasMorePages = true;         
                return;                 
            }
            else 
            {
                // this will only be used after all data are printed
                e.HasMorePages = false; 
            }
        }
    } catch (Exception ef)
    {
        MessageBox.Show("" + ef);
    }
}