C# 配列の値を定数として指定すると正しく動作

Question

C# 配列の値を定数として指定すると正しく動作します。

drv1.Row.ItemArray[1]　→　1250
drv1.Row.ItemArray[2]　→ 　180

グラフを描くプログラムです。
配列のまま(drv1.Row.ItemArray[1])では、グラフ（GraphMen）を押すと、

「型 'System.InvalidCastException' のハンドルされていない例外が Graph.exe で発生しました

追加情報: 指定されたキャストは有効ではありません。」
と出て、止まります。
配列のまま、正しく動作させるためにはどうしたら良いでしょうか。
よろしくお願いします。

環境は
Visual Studio 2012
Windows Forms アプリ
.NET Framework 3.5 Service Pack 1
で作っています

エラーとなった際に値を調べると、
drv1.Row.ItemArray[1]=1250
drv1.Row.ItemArray[2]=180 となっています。
2007年から2010年までのデータを読み込んでグラフを書くのですが、
一番最初の2007年のデータを読み込むところでエラーが出てしまいます。

drv1のデータ元である profitTable が

Year	Sales	Gain
2007	1250	180
2008	1380	195
2009	1450	201
2010	1600	253

となっています。

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Text;
using System.Windows.Forms;

namespace Graph
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }
        private void Form1_Load(object sender, EventArgs e)
        {
            // TODO: このコード行はデータを 'profitDataSet.ProfitTable' テーブルに読み込みます。必要に応じて移動、または削除をしてください。
            this.profitTableTableAdapter.Fill(this.profitDataSet.ProfitTable);
            GraphSales.Checked = true;
        }

        private void GraphSales_Click(object sender, EventArgs e)
        {
            GraphGain.Checked = GraphSales.Checked;
            GraphSales.Checked = !GraphSales.Checked;
            Refresh();
        }

        private void GraphGain_Click(object sender, EventArgs e)
        {
            GraphSales.Checked = GraphGain.Checked;
            GraphGain.Checked = !GraphGain.Checked;
            Refresh();
        }

        private void GraphGain_Paint(object sender, PaintEventArgs e)
        {
            printGraph(e.Graphics);
        }
        private void printGraph(Graphics g)
        {
            //PenとFontを作る
            Pen pn = new Pen(Color.Black);
            Font fnt = new Font("Arial" , 10);
            //グラフの軸を描く
            g.DrawLine(pn, 50, 50, 50, 250);
            g.DrawLine(pn, 50, 250, 310, 250);
            //　グラフのメモリを描く
            //縦軸
            for (int i=1; i<11; i++)
            {
                g.DrawLine(pn, 50, 30+i*20, 54, 30+i*20);
                string s;
                if (GraphSales.Checked) //売上の目盛り
                {
                    s=String.Format("{0}", 200*i);
                }
                else 
                {
                    s=String.Format("{0}", 30*i);
                }
                g.DrawString(s, fnt, new SolidBrush(Color.Black),
                    new Point(12,242-i*20));
            }
            //横軸
            for (int i=0 ; i<5; i++)
            {
                g.DrawLine(pn, i*50+50, 250, i*50+50, 246);
                string s= String.Format("{0}", 2007+i);
                g.DrawString(s, fnt, new SolidBrush(Color.Black), new Point(i*50+30, 258));
            }
            //レコード数
            int nRec = profitTableBindingSource.Count;
            //レコードの数-1だけ線を引く
            for (int i=0; i<nRec-1; i++)
            {
                //現在のレコードのビューを取得する
                DataRowView drv1=profitTableBindingSource[i] as DataRowView;
                //売上と利益を取得する
                int sales1 = (int)(drv1.Row.ItemArray[1]);★★★エラー箇所★★★
                int gain1 = (int)(drv1.Row.ItemArray[2]);
                //次のレコードのビューを取得する
                DataRowView drv2 = profitTableBindingSource[i+1] as DataRowView;
                //売上と利益を取得する
                int sales2 = (int)(drv2.Row.ItemArray[1]);
                int gain2 = (int)(drv2.Row.ItemArray[2]);
                int x1 = 50 + 50*i;
                int x2 = 100 + 50*i;
                int y1, y2;
                if (GraphSales.Checked) // 売上をグラフ化する
                {
                    y1 = 250 - sales1 / 10;
                    y2 = 250 - sales2 / 10;
                }
                else //利益をグラフ化する
                {
                    y1 = 250 - (gain1 * 2) /3;
                    y2 = 250 - (gain2 * 2) /3;
                }
                g.DrawLine(pn, x1, y1, x2, y2);
            }
        }

        private void FilePrint_Click(object sender, EventArgs e)
        {
            if (MessageBox.Show("グラフを印刷します", "印刷の確認", MessageBoxButtons.OKCancel) == DialogResult.OK)
            {
                printDocument1.Print();
            }
            //PrintDocumentにグラフを描く

        }

        private void printDocument1_PrintPage(object sender, System.Drawing.Printing.PrintPageEventArgs e)
        {
            printGraph(e.Graphics);
        }

        private void FileExit_Click(object sender, EventArgs e)
        {
            this.Close();
        }
    }
}

Comments

[YAmaGNZ]

どこでエラーとなるのですか？
またエラーとなる場合どのような値の時なのでしょうか？
デバッガで実行すれば確認できると思いますがそういったことはやったのですか？

[YAmaGNZ]

そもそもdrv1.Row.ItemArray[1]とやらの型は何なのですか？

[退会済みユーザー]

何を何で作っているかぐらいは書けませんか？　（例: Visual Studio 2022 で Windows Forms アプリをターゲットフレームワーク .NET Framework 4.8 で作っています）

[退会済みユーザー]

YAmaGNZ さんの質問に答えませんか？

[nkym771]

回答しました。

[退会済みユーザー]

> そもそもdrv1.Row.ItemArray[1]とやらの型は何なのですか？

[t-n]

drv1.Row.ItemArray[1] → int へのキャストが不可能なのであればintに変換可能な数値型ではないということです。

まずは型を確認してみてはいかがでしょうか。
System.Console.WriteLine(drv1.Row.ItemArray[1].GetType());

この情報があると回答がしやすくなると思います。

例えば文字列型であれば
int.Parse(drv1.Row.ItemArray[1])
で変換可能です。

[nkym771]

ありがとうございます。
こちらで分かる情報は全部書いて困っていました。
＞System.Console.WriteLine(drv1.Row.ItemArray[1].GetType());
System.Decimal
となりました。

[t-n]

ここまでわかればあとは調べればすぐ答えが見つかると思います。
参考：
https://learn.microsoft.com/ja-jp/dotnet/api/system.decimal.toint32?view=netframework-3.5#system-decimal-toint32(system-decimal)

[退会済みユーザー]

> ここまでわかればあとは調べればすぐ答えが見つかると思います。

何を言ってるのですかね？　ホントに drv1.Row.ItemArray[1] が decimal 型なら (int)(drv1.Row.ItemArray[1]) は有効でしょ。

[退会済みユーザー]

> drv1.Row.ItemArray[1]

drv1 は DataRowView 型ですよね。とすると、DataRowView.Row プロパティは DataRow を返す。DataRow.ItemArray プロパティは Object[] を返す。ItemArray[1] は Object[] のインデックスが 1 の要素すなわち Object を返すということです。

で、その Object というのは decimal をボックス化したものではないのですか？　であれば、それを int にキャストすることはできませんよ。

[t-n]

> ホントに drv1.Row.ItemArray[1] が decimal 型なら (int)(drv1.Row.ItemArray[1]) は有効でしょ。
ごめんなさい、おっしゃる通りです。
一度decimalに戻してからintにする必要がありそうですね。

[退会済みユーザー]

> 一度decimalに戻してからintにする必要がありそうですね。

戻すというよりボックス化解除でしょう。

[退会済みユーザー]

質問者さん、無言ですが、回答したのでそれに対するフィードバックを返してください。役に立った/立たなかったぐらいはすぐに返せるのでは？ 役に立たなかったならどこがダメかを書くとより期待に近い回答が出てくるかも。とにかく無言は NG です。

[退会済みユーザー]

質問者さん、またまた無言ですが、あなたが立てたスレッドなのだから放置しないで、クローズするまできちんとフォローしてください。

Answer 1

drv1.Row.ItemArray[1]

drv1 は DataRowView 型ですよね。とすると、DataRowView.Row プロパティは DataRow を返す。DataRow.ItemArray プロパティは Object[] を返す。ItemArray[1] は Object[] のインデックスが 1 の要素すなわち Object を返すということです。

で、その Object というのは decimal をボックス化したものではないのですか？　であれば、それを int にキャストすることはできませんよ。

その Object（decimal をボックス化した object）をボックス化解除してから int にキャストしてください。

ボックス化とボックス化解除 (C# プログラミング ガイド)
https://learn.microsoft.com/ja-jp/dotnet/csharp/programming-guide/types/boxing-and-unboxing

Comments

[nkym771]

ご回答ありがとうございます。
＞ Object というのは decimal をボックス化したものではないのですか？
そうだと思います。
プログラムを一時停止して変数にカーソルを合わせるとobject[] と表示されました。
int sales1 = decimal.ToInt32((decimal)(drv1.Row.ItemArray[1]));
に変更することでエラーが出なくなりました。

[退会済みユーザー]

> C# 配列の値を定数として指定すると正しく動作します。
> drv1.Row.ItemArray[1]　→　1250
> drv1.Row.ItemArray[2]　→ 　180

という話だったはずです。

> （グラフは描画されません）

「正しく動作します」と言うのは何だったのですか？

[退会済みユーザー]

質問の本題とは外れますが・・・

.NET Framework の Windows Forms アプリなら Chart を使ってみませんか？　サンプルが入手できます。自分の目的に会ったグラフも見つかるのではないでしょうか？

Windows Forms 用 Chart Samples
http://surferonwww.info/BlogEngine/post/2021/11/30/chart-samples-for-windows-forms-application.aspx

[nkym771]

ありがとうございます。
>Windows Forms 用 Chart Samples
参考にします。