初めまして。
私は今C言語のプログラミングを勉強しています。
2次元配列へ文字列の標準入力を行い、それをプリントさせるという問題を考えています。(配列の行の要素はMAXに達しなくても空行が入力されたら
終了させたい。)
下のようなプログラムを書きましたが、空行を入力してプリントする際に
文字が崩れてしまって上手く表示されませんでした。
初歩的で恐縮ですが、どのように改善したら良いか、
教えて頂けたら嬉しいです。
#include <stdio.h>
#include <ctype.h>
#define MAX 10
int main(){
char array[MAX][MAX];
int i;
for(i = 0; i < 10; i++){
fgets(array[i],sizeof(char)*MAX,stdin);
if(array[i][0] == '\n')
break;
}
for(i = 0; i < 10; i++){
printf("%s", array[i]);
}
return 0;
}
No.4ベストアンサー
- 回答日時:
★回答者 No.1 です。
・もう少し詳しいアドバイスをしてみます。
サンプル:
#include <stdio.h>
#include <ctype.h>
// 定数
#define MAX 10
// 便利マクロ関数(要素数を求める)
#define ArrayOf(x) (sizeof(x)/sizeof((x)[0]))
int main( void )
{
char array[ MAX ][ MAX ]; ←できれば array[ MAX ][ LEN ] と分けたほうがいいかも。
int i, max; ←入力した件数を max に入れいるため。
for ( max = 0 ; max < ArrayOf(array) ; max++ ){ ←ArrayOf() マクロで MAX を求める
if ( fgets(array[max],sizeof(array[0]),stdin) == NULL ){ ←戻り値をチェックすべき!
break;
}
if ( array[max][0] == '\n' ){ ←空行で抜ける
break;
}
}
for ( i = 0 ; i < max ; i++ ){ ←ここで入力された max までを表示
printf( "%s", array[i] );
}
return 0;
}
解説:
・最初のアドバイスで array を初期化すると入力されなかった部分は NULL 文字が入っています。
printf() 文では NULL 文字列は全く表示されませんが、入力された数を変数 max などに持ち、
表示するときに i カウンタと比較します。そして、入力されなかった部分は表示しない方が
バグとならずに正しい記述になります。array を初期化しただけでも上手く動作しますがね。
・あと fgets() 関数の戻り値もチェックした方が良い。
まぁ、いろいろと試して見て下さい。
それでは。
No.3
- 回答日時:
>for(i = 0; i < 10; i++){
がバグってる
これだと常に10行入力することが前提となる。
正しくは、取り込んだ行数までで出力をとめないといけない。
int j;
for(j = 0; j < i; j++){
あと、fgetsはNULL止めするのでバッファを初期化する必要は無いが、
初期化することが望ましい。
No.2
- 回答日時:
#include <stdio.h>
#include <ctype.h>
#define MAX 10
int main(){
char array[MAX][MAX];
int i, j;
for(i = 0; i < 10; i++){
fgets(array[i],sizeof(char)*MAX,stdin);
if(array[i][0] == '\n')
break;
}
for(j = 0; j < i; j++){
printf("%s", array[j]);
}
return 0;
}
No.1
- 回答日時:
★文字が崩れるというよりは『ゴミ』が表示されませんか。
・array 配列を最初に初期化して置けばよいんですが…。
つまり、
char array[MAX][MAX] = { 0 };
という1行に宣言部を書き直します。
これでコンパイルすれば多分、上手く表示されると思います。
お試しあれ。
その他:
・fgets() 関数のバッファ容量は sizeof(array[0]) とすれば取得できます。
改良前⇒fgets(array[i],sizeof(char)*MAX,stdin);
改良後⇒fgets(array[i],sizeof(array[0]),stdin);
とします。
どちらも同じですが二次配列の sizeof では array[0] とすれば求められます。
これは、char array[10][20]; の場合
sizeof(array)⇒200…10×20
sizeof(array[0])⇒20
という数が求まります。
・以上。参考に。
早速のお返事ありがとうございます。
なるほど、初期化してなかったからなのですね。
char array[MAX][MAX] = { 0 };
の書き方で一括して初期化されるのですね。
その他の方も参考になります。
勉強になりました。
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
似たような質問が見つかりました
- C言語・C++・C# c言語の問題です 課題1 (二分探索木とセット) 大きさ size の配列 array を考える。す 2 2023/01/10 21:08
- C言語・C++・C# C言語のエラーについて 2 2022/07/11 13:56
- C言語・C++・C# c言語 プログラムのエラー 1 2023/02/11 20:31
- C言語・C++・C# プログラミングの授業の課題です 1 2023/01/17 22:15
- C言語・C++・C# C言語 プログラミング 4 2022/05/22 11:53
- C言語・C++・C# c言語の問題の説明、各所ごとに 5 2023/07/26 11:03
- Java javaでのプログラム(配列)について質問です. 2 2022/10/14 22:27
- C言語・C++・C# このプログラミング誰か教えてくれませんか 1 2022/06/02 15:27
- C言語・C++・C# プログラミングを教えて欲しいです。 配列aは、int a[9]={7,6,12,8,3,5,10,9 4 2022/12/19 23:27
- C言語・C++・C# c言語配列の結合についてです。 なぜうまくいかないのでしょうか。 #include <stdio.h 4 2022/05/30 22:42
関連するカテゴリからQ&Aを探す
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
PHPについて教えて下さい。
-
C言語で全角文字の扱いについて
-
PHPのセレクトボックスの初期値...
-
C言語の配列をPush(追加)する...
-
C言語 最大値と最小値を求めて...
-
2次元配列のソート
-
MYSQLとPHPによって取得する多...
-
PHPによる並べ替え(ソーティン...
-
C言語の2次元配列における行・...
-
PHPに関する質問です
-
どのような関数名を付けるべき...
-
配列から値を取得する方法。
-
読み(あ行~わ行)ごとに分け...
-
【PHP】配列に出てくるこの...
-
配列について
-
配列を返す関数から、一つの要...
-
pythonのnumpyでの繰り返しでの...
-
テキストエリアに入力した複数...
-
templateを使ったXOR swapのバ...
-
配列内の値を分解して再格納
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
プログラミングのPythonのnoteb...
-
スカラーのベクトル微分
-
特定の文からメールアドレスの...
-
CArrayの要素としてCStringArra...
-
C言語 最大値と最小値を求めて...
-
行列
-
読み(あ行~わ行)ごとに分け...
-
配列の要素(value)に、変数を...
-
C言語の配列をPush(追加)する...
-
STLのvectorで作った配列をメン...
-
php で1から100までの素数の表...
-
【PHP】配列のキー名の修正は可...
-
Perlで重複行を削除したい
-
fgetc関数について
-
CArrayのソート
-
ファイルの書き込みについて教...
-
配列の添え字が小数だとどうなる?
-
PHPのmin関数、「1」以上の数値...
-
forとかで連番の変数を一気に格...
-
delphi 2次元配列がわかりません
おすすめ情報