Aizu Oneline JudgeでRunがうまくいかない

総合スコア7703

2019/10/22 15:19

ありがとうございます。実はエラーメッセージの内容が一切わからない状態でここで質問しました・・・いままでは全部グーグル検索で何とかしていたのですがこのエラーメッセージってどこ検索すればいいのかよくわからなくて・・・あと開発環境について、とりあえず「C言語　開発環境」といった感じでグーグル検索をかけるつもりですが、なにかおすすめのサイトなり、開発環境があったら教えてくださると助かります。

2019/10/22 15:37

最近はVisualStudio Code + MinGWが流行りな気がします。 VisualStudio Community(Codeではない)はプロには便利でも初心者にはちょっと使い切れない、という面があるような。学習用C言語開発環境程度のものでいいなら、Code:blocksとか良さそうかな、と私は思いますが、日本語化には難があるようです。メニューのSaveが「保存」じゃなきゃ嫌だ、という方にはおすすめしません。

2019/10/22 16:16

VisualStudio Communityは私も入れてい放置している感じです・・・（使い方がよくわからないという感じで・・・）。こちらでも別のコメントでも本当にありがとうございましたm(__)m

行動規範の内容に同意します

ベストアンサー

問題点1

配列の使い方を間違えています。
例えば、この部分は見るからにおかしいです。

card[4][d] = 1; // int card[4][13] と宣言されているので、card[4]にはアクセスできない

プログラム後半のfor文でも同じようなミスが見受けられます。

n要素の配列にアクセスする際、添え字は 0~n-1 を使います。
つまり初期化済みの int arr[3] に対して次は合法ですが、

C
1for(int i = 0; i < 3; ++i) {
2    printf("%d, %d\n", i, arr[i]);
3}

次は違法です。

C
1for(int i = 1; i <= 3; ++i) {
2    printf("%d, %d\n", i, arr[i]);
3}

後者のコードのような範囲外アクセスは未定義動作を引き起こします。
実行環境に依って動作に差異が出ても何ら不思議ではありません。

問題点2

if (card[i][j] != 1){

ご提示のコードを書く際、一つ思い込みをしているように思います。
『配列の要素は、最初から何らかの数で埋められているのだ』... 残念ながら、それは間違いです。

次のように実際に試してみることができます。

C
1#include <stdio.h>
2
3int main(void) {
4    int arr[10];
5    for(int i = 0; i < 10; ++i) {
6        printf("%d\n", arr[i]);
7    }
8    
9    return 0;
10}

実行結果 Wandbox

実行結果は毎回変わり得ます。それこそ全部1になってもおかしくありません。
そもそも実行できる保証もありませんが。

ともかく、次のように明示的に初期化してやる必要があります。

C
1int card[4][13] = {0};

投稿2019/10/22 14:48

編集2019/10/22 14:57

総合スコア35668

2019/10/22 14:56

ありがとうございました。何とかRunは抜け出したのですが・・・Result Checkで引っかかりました（´・ω・｀）これって別でもう一回質問立て直したほうがいいですか？

2019/10/22 14:58

card[~][13] もアクセスできないのですが、そのあたりも修正しましたか？

2019/10/22 14:59

現時点ではまだ新規の質問は要らないように思います。

2019/10/22 15:10

問題点2の方の話じゃないかと>引っかかる (自動で初期化されないのは配列かどうかじゃなくて(動的な)ローカル変数だからですが。静的な変数は(配列を含め)特に指示しない場合0に初期化されることが保証されます...というのは「とりあえず初期化しておけ」ということで敢えて細かく言わなかったのでしょうか)

2019/10/22 15:25 編集

@thkana さん > 自動で初期化されないのは ~ 敢えて細かく言わなかったのでしょうか静的・動的関わらず、明示的に初期化する習慣を付けるのが特に初心者にとっては有用かと思います。 ...というのは建前で、そこまで考慮していなかったです。配列に限った話だと強調するような意図もありませんでしたが、確かにそのように受け取られかねない記述かもしれません。 thkanaさんのコメントをもってその辺りの穴が補完された、ということにさせて下さい。ご指摘ありがとうございます。

2019/10/22 15:29

```c #include<stdio.h> int main (){ int card[5][14];//修正点 int n,d,i,j; char c; scanf("%d",&n); for (j = 0;j < n; j++){ 省略 } for (i = 1; i <= 4; i++){ 省略 } return 0; } ``` という感じになりましたあとちょっと本題からずれるのですが動的なローカル変数とはmallocを用いて作る配列などという認識で合っていますか？

2019/10/22 15:34

for (i = 1; i <= 4; i++){ がまずいのでは？

2019/10/22 15:37 編集

> 動的なローカル変数とはmallocを用いて作る配列などという認識で合っていますか？ブロック内で普通に宣言した変数だと思って下さい。静的なローカル変数を宣言したい場合static指定子を用います。

2019/10/22 15:48

動的なローカル変数については理解しました。ですが、 >for (i = 1; i <= 4; i++){ がまずいのでは？ card[0]もcard[5]も使ってないないので大丈夫だと思っていたのですが・・・一応この部分をのせます for (i = 1; i <= 4; i++){ for (j = 1; j <= 13; j++){ if (card[i][j] != 1) { if (i == 1){ printf("S %d\n",j); } if (i == 2){ printf("H %d\n",j); } if (i == 3){ printf("C %d\n",j); } if (i == 4){ printf("D %d\n",j); } } } }

2019/10/22 15:50

card[4] にアクセスしている時点でアウトです。 card[0] から card[3] までしか使えません。

2019/10/22 15:57

私の勘違いだとしてら申し訳ないのですが card[5][14]; と宣言しているのに card[0]からcard[3]までしか使えないのですか？

2019/10/22 15:58

int card[5][14];//修正点だそうですからセーフでは。しかし...プログラムとしては「駄目ではない」のですが、card[0][0]...card[0][13]とかcard[1][0], card[2][0]とかが使われず無駄になります。今どきのメモリが有り余ったPC上でならそれくらいは問題ないとも言えますが、配列が大きくなるような場合にはその方針はちょっと考えたほうがいいかも知れません。 C/C++プログラマの間の常識としては、for(i=1;i<=N;i++)というループは「あれれ?」と思われるものと考えて下さい。普通は、for(i=0;i<N;i++)で、card[1][d-1] = 1;とかprintf("S %d\n",j+1);とかして辻褄合わせをするでしょう。 --- 静的というのは「プログラムが始まってから終わるまで存在する」もの、一方「プログラムの途中で出来たり無くなったりする」のが動的です。 mallocで確保した領域は、mallocが実行されるまでは無いので動的ですし、関数の中で宣言する変数も、関数が実行されていない間は存在しないので動的です。キモチとしては、プログラムが始まる時(というか始まる前)は多少手間をかけて初期化してもいいけど、プログラムの実行中は初期化の少しの手間も惜しむ、というところですね。

2019/10/22 16:06

あと配列を宣言するときに０を代入するのを忘れていました・・・ 0を代入することで正解できました！