テーブルの内容を1ページ50行で200ページ印刷したい

Question

###前提・実現したいこと
foreachの読み取り位置を記憶して50行ずつ印刷したい

###発生している問題・エラーメッセージ

同じ50行が200ページ印刷されてしまう

###該当のソースコード

        /// <summary>
        /// 降順印刷ボタンのイベントハンドラ
        /// product_tableを降順に並べ替えて印刷
        /// </summary>
        /// <param name="sender">使用しない</param>
        /// <param name="e">使用しない</param>
        private void buttonDescendingOrderPrint_Click(object sender, EventArgs e)
        {
            string sql = @"
                SELECT *
                FROM product_table
                ORDER BY product_id DESC;";

            using (connection = new SqlConnection(Properties.Settings.Default.ConnectionString))
            {
                try
                {
                    connection.Open();
                }
                catch (Exception ex)
                {
                    MessageBox.Show("データベースを開くのに失敗しました。", "error", MessageBoxButtons.OK, MessageBoxIcon.Error);
                    System.Diagnostics.Debug.WriteLine(ex.ToString());
                }

                List<ProductData> productList = new List<ProductData>();

                SqlCommand command = new SqlCommand(sql, connection);
                using (SqlDataReader reader = command.ExecuteReader())
                {
                    while (reader.Read() == true)
                    {
                        ProductData productData = new ProductData();
                        productData.ProductId     = reader["product_id"].ToString();
                        productData.BasicOrderLot = reader["basic_order_lot"].ToString();
                        productData.ProductCode   = reader["product_code"].ToString();
                        productData.ChassisNumber = reader["chassis_number"].ToString();
                        productData.AssembledYmd  = reader["assembled_ymd"].ToString();
                        productData.AssembledHms  = reader["assembled_hms"].ToString();
                        productData.StructNumber  = reader["struct_number"].ToString();
                        productData.YyyyMm        = reader["yyyymm"].ToString();

                        productList.Add(productData);
                    }
                    reader.Close();
                }

                productDataBindingSource.DataSource = productList;
                PrintOut();
            }
        }

        /// <summary>
        /// PrintOutメソッド
        /// 印刷ダイアログを表示して印刷する
        /// </summary>
        private void PrintOut()
        {
            using (PrintDocument document = new PrintDocument())
            {
                document.PrintPage += new PrintPageEventHandler(PrintPage);
                PrintDialog log = new PrintDialog();
                if (log.ShowDialog() == DialogResult.Cancel)
                {
                    return;
                }
                document.Print();
            }
        }

        /// <summary>
        /// 1ページ印刷するイベントハンドラ
        /// 1ページ分の印刷設定
        /// </summary>
        /// <param name="sender">使用しない</param>
        /// <param name="e">使用する</param>
        private void PrintPage(object sender, PrintPageEventArgs e)
        {
            int line = 0;
            int x, y = 20;
            e.HasMorePages = true;
            Font font = new Font("MS ゴシック", 12);

            foreach (ProductData data in productList)
            {
                x = 30;
                e.Graphics.DrawString(data.ProductId, font, Brushes.Black, x, y);
                x = x + 80;
                e.Graphics.DrawString(data.BasicOrderLot, font, Brushes.Black, x, y);
                x = x + 90;
                e.Graphics.DrawString(data.ProductCode, font, Brushes.Black, x, y);
                x = x + 80;
                e.Graphics.DrawString(data.ChassisNumber, font, Brushes.Black, x, y);
                x = x + 60;
                e.Graphics.DrawString(data.AssembledYmd, font, Brushes.Black, x, y);
                x = x + 150;
                e.Graphics.DrawString(data.AssembledHms, font, Brushes.Black, x, y);
                x = x + 100;
                e.Graphics.DrawString(data.StructNumber, font, Brushes.Black, x, y);
                x = x + 100;
                e.Graphics.DrawString(data.YyyyMm, font, Brushes.Black, x, y);
                y = y + 20;
                line++;

                if (line == 50)
                {
                    break;
                }
            }

            e.Graphics.DrawString(String.Format("{0}ページ", page), new Font("MS ゴシック", 12)
                , Brushes.Black, 365, 1110);
            page++;

            if (MAX_PAGE == page)
            {
                e.HasMorePages = false;
                connection.Close();
            }
        }
    }
}

###補足情報(言語/FW/ツール等のバージョンなど)
5852232 2A11B5632ZABFA157220000825 18:08:57 8D11080200008
このようなデータが1万行あります。

Answer 1

PrintPageメソッドは、１ページ印刷される毎に１回呼び出されます。
なので毎回呼び出されるたびに、次の５０行データに差し替える必要があります。

現在のコードでは、毎回先頭の５０行データをセットしているため、
同じページが何ページにもわたって印刷されます。

追加
pageが1からカウントされる前提になりますが、下記では如何でしょうか？

//変更前
foreach (ProductData data in productList)
//変更後
var ddd = productList.Skip((page - 1) * 50).Take(50);
foreach (ProductData data in ddd)

Comments

[hinatti]

次の50行に差し替えていく方法が分からなくてテラテイルで質問したんですよね…
言葉足らずでした。

[kikukiku]

テストしていませんが、具体的な方法を記載しました

[hinatti]

pageは1から始まる前提なので大丈夫です！
_d_m_itemってdataを格納してるリスト名が来るんですかね？

[kikukiku]

間違えたのですでに修正しています。

[hinatti]

わざわざありがとうございます！
おかげさまで期待していた動作になりました！
本当にありがとうございました。