何故この数字になるのかわかりません。

Question

C#の勉強を始めたばかりの初心者です。

「九九の計算結果の配列を活用し、1桁の数値(1～9に限定)を2つ入力し、その積を表示するプログラムを作成しなさい。」という問題で、下記のコードが解答例としてありました。

static void Main(string[] args)
{
int[,] kuku = new int[9,9];

for (int i = 0; i < 9; i++)
    for (int j = 0; j < 9; j++)
        kuku[i,j] = (i + 1) * (j + 1);

int p = int.Parse(Console.ReadLine());
int q = int.Parse(Console.ReadLine());

Console.WriteLine(kuku[p-1,q-1]);

}

その中で、一番最後の「Console.WriteLine(kuku[p-1,q-1])」で、何故「[p-1,q-1]」になるのかがわかりません。
わざわざ「-1」にする理由を教えて頂きたく、ご教授頂けたらと思います。

掛け算なので、「P*Q」で良いのではと思ったのですが、そうすると二桁以上の掛け算も出来てしまいます。
1～9までの1桁にする為に「-1」にする理由があると思うのですが、教えて頂けますでしょうか。

また、他のコード例がありましたら教えて頂きたいです。

Comments

[YAmaGNZ]

入力したのがp=1、q=1として１X１の結果がkukuという配列のどこに入っているのか分かりますか？

[fana]

> 「P*Q」で良いのでは

「九九の計算結果の配列を活用し…」という指示を無視してよいのですか？

[otn]

> 掛け算なので、「P*Q」で良いのではと思ったのですが、
「p*q」の書き間違いでしょうけど、積を表示すれば良いだけであれば、別にそれでもいいですよ。
ただし、題意を無視しているので、提出課題であれば、×もしくは減点でしょうけど。

題意の、
> 九九の計算結果の配列を活用し
という日本語の意味を理解できないとすると、なかなかつらいですが、
「問題文のうち理解できない部分は無視する」という方針だと題意に沿った解答は無理です。
「問題文の中に理解できない部分があれば理解できるまで考える」が正しい態度かと思います。

> そうすると二桁以上の掛け算も出来てしまいます。

回答例のコードを見る限り、1~9の数字以外を入力した場合は、どうなってもかまわない
（何が表示されても、エラーでプログラムがこけてもかまわない）
ということかと思います。

[sugaryfig]

返信ありがとうございます。
九九の配列を活用することと、1～9までの人桁の数字のみで掛け算の計算をするコードの作成なので、「p*q」だと二桁でも三桁でも計算が出来てしまうのでどうしたらしいのかと思いまして、質問させて頂きました。

[otn]

> 「p*q」だと二桁でも三桁でも計算が出来てしまうのでどうしたらしいのかと思いまして、
「想定外の入力があった時にどうするか？」は、実用プログラムではもちろん、きちんと対処が必要ですが、
この手の練習問題で、Teratailに質問に出てくる模範解答などを見る限りでは、「想定外の入力があることは全く考慮しなくていい」ようですね。

少なくとも、考慮が必要な場合は、「1桁の数字以外が入力されたら、これこれのエラーメッセージを出して再度入力を求めよ（orエラーで終了しろ）」とか書いてないと、それに応じたプログラムの書きようが無いです。実用プログラムの場合はそのあたりどうするかは設計者が考えます。

書籍の練習問題であれば、書籍の最初の方に、サイトに掲載された問題なら、回答者への注意事項かどこかに、「想定外の入力があることは全く考慮しなくていい」を意味する説明があるべきですね。実際書いてあるのかどうかは知りませんが。

[TN8001]

疑問が解消したなら、質問を「解決済」にしてください。
[teratail【テラテイル】｜ヘルプ](https://teratail.com/help#resolve-question)

解消していないなら、その点を質問に追記してください。
「ほかのパターンももっと見てみたい」ということだと、配列を使う縛りだとあまり余地がない気が^^;

[TN8001]

> 「ほかのパターンももっと見てみたい」ということだと、配列を使う縛りだとあまり余地がない気が^^;

配列がint[,]でなくてもいいなら、int[][]やint[]を使う手もあるか。
まああまり面白くもならないので特に例示はしませんが。

Answer 1

その中で、一番最後の「Console.WriteLine(kuku[p-1,q-1])」で、何故「[p-1,q-1]」になるのかがわかりません。
わざわざ「-1」にする理由を教えて頂きたく、ご教授頂けたらと思います。

y_waiwaiさんの回答の通り、インデックスがゼロ始まりのためですね。
forループ内ではわざわざ「+1」していますよね？

index	0	1	2	3	4	5	6	7	8
0	1	2	3	4	5	6	7	8	9
1	2	4	6	8	10	12	14	16	18
2	3	6	9	12	15	18	21	24	27
3	4	8	12	16	20	24	28	32	36
4	5	10	15	20	25	30	35	40	45
5	6	12	18	24	30	36	42	48	54
6	7	14	21	28	35	42	49	56	63
7	8	16	24	32	40	48	56	64	72
8	9	18	27	36	45	54	63	72	81

1～9までの1桁にする為に「-1」にする理由があると思うのですが、教えて頂けますでしょうか。

いいえ。「1桁にする為」ではありません。
もうすでに分かっておられるとは思いますが、「インデックスがゼロ始まりのため」です。

この問題はおそらく配列の練習で、「インデックスはゼロ始まり」というのを確認させるものなのでしょう。
二桁の数字（や１未満の数値）を入力すると例外が出て止まります（やってみればすぐわかりますが...
IndexOutOfRangeException クラス (System) | Microsoft Learn

数字以外の入力や小数・大きすぎる数値等、ほかの例外も出ます。
FormatException クラス (System) | Microsoft Learn
OverflowException クラス (System) | Microsoft Learn

また、他のコード例がありましたら教えて頂きたいです。

「+1」・「-1」するのが面倒＆わかりにくいので、ちょっと無駄が出ますが配列サイズを大きくするのは割とありますね。

int[,] kuku = new int[10, 10]; // 0～9までの10個*10個

for (int i = 1; i <= 9; i++) // int i = 0; としてもいいが（０の段？）どうせ０なので省略
    for (int j = 1; j <= 9; j++) // 同上
        kuku[i, j] = i * j;

int p = int.Parse(Console.ReadLine()); // 0も通ってしまうが...
int q = int.Parse(Console.ReadLine());

Console.WriteLine(kuku[p, q]);

index	0	1	2	3	4	5	6	7	8	9
0	0	0	0	0	0	0	0	0	0	0
1	0	1	2	3	4	5	6	7	8	9
2	0	2	4	6	8	10	12	14	16	18
3	0	3	6	9	12	15	18	21	24	27
4	0	4	8	12	16	20	24	28	32	36
5	0	5	10	15	20	25	30	35	40	45
6	0	6	12	18	24	30	36	42	48	54
7	0	7	14	21	28	35	42	49	56	63
8	0	8	16	24	32	40	48	56	64	72
9	0	9	18	27	36	45	54	63	72	81

---

提出はもちろん普段の利用にも全く勧めませんが、１始まりの配列も作れるには作れます。

int[,] kuku = (int[,])Array.CreateInstance(typeof(int), [9, 9], [1, 1]);

for (int i = 1; i <= 9; i++) // int i = 0; とするとエラー。ほんとに１始まりになっている
    for (int j = 1; j <= 9; j++) // 同上
        kuku[i, j] = i * j;

int p = int.Parse(Console.ReadLine());
int q = int.Parse(Console.ReadLine());

Console.WriteLine(kuku[p, q]);

Array.CreateInstance メソッド (System) | Microsoft Learn

index	1	2	3	4	5	6	7	8	9
1	1	2	3	4	5	6	7	8	9
2	2	4	6	8	10	12	14	16	18
3	3	6	9	12	15	18	21	24	27
4	4	8	12	16	20	24	28	32	36
5	5	10	15	20	25	30	35	40	45
6	6	12	18	24	30	36	42	48	54
7	7	14	21	28	35	42	49	56	63
8	8	16	24	32	40	48	56	64	72
9	9	18	27	36	45	54	63	72	81

Comments

[sugaryfig]

丁寧な説明、ありがとうございます。
教えて頂いたコードを試して、更に理解を深めたいと思います。
ありがとうございました。

Answer 2

    kuku[i,j] = (i + 1) * (j + 1);

こうやっているから、ですね
kuku[0,0] には、１＊１の結果がはいってます

Comments

[sugaryfig]

回答ありがとうございます。
なるほどです。
ご説明ありがとうございます!

Answer 3

配列番号が0から8まで(Length-1)なので、仮に入力が9だと境界外となってエラーです

入力に対処したいだけなら三項演算子を使うといいでしょう

for (int i = 0; i < 9; i++)
    for (int j = 0; j < 9; j++)
        kuku[i,j] = (i + 1) * (j + 1);

int p = int.Parse(Console.ReadLine());
int q = int.Parse(Console.ReadLine());

Console.WriteLine(kuku[p>0&&p<=9?p-1:0,q>0&&q<=9?q-1:0]);

これで想定外の数値の入力に対してインデックスの指定が0に置き換えられ、1*xまたはx*1の形の値が返されます
数値以外の文字列の検知にはTryParseを利用する手段もあります
0の代わりにthrow newを入れてもいいでしょう
これらは例外処理の一端ですが、学習機会はもう少し先になるはずです

Comments

[sugaryfig]

返信頂きありがとうございます。
引き続き勉強をし、理解を深めていきたいと思います。