プログラマ専用SNS ミクプラ

コード:

#include <stdio.h> 
int main(void)
{
int n, gokei;  
char hinmei[256];  
int tanka[256], suryo[256], kingaku[256];  
int i, count;  

n=0;
suryo=0; 
count=0; 

for(i = 0;i < 256;i++)
{ 
hinmei[i] = 0; 
tanka[i] = 0; 
suryo[i] = 0; 
kingaku[i] = 0; 
} 

printf("入力するデータ数を入力してください＞");
scanf("%d", &n);  
count = n;

while(n != 0)
{ 
printf("品名を入力してください＞"); 
scanf("%c", &hinmei); 
hinmei[i] = hinmei; 

printf("単価を入力してください＞"); 
scanf("%d", &tanka); 
tanka[i] = tanka; 

printf("数量を入力してください＞"); 
scanf("%d", &suryo); 
kingaku[i] = tanka[i] * suryo[i]; 

gokei = kingaku[i] + gokei; 

n--;
i++;
} 

i = 0;
printf("品名　　　　　単価　　　　　数量　　　　　金額\n\n"); 
printf("----------------------------------------------\n\n"); 

while(count != 0) 
{ 
printf("%d　　　　　,%c　　　　　,%d　　　　　,%d", hinmei[i], tanka[i], suryo[i], kingaku[i]);
printf("\n\n"); 
count--; 
i++; 
} 

printf("----------------------------------------------\n\n"); 
printf("合計　　　　　　　　　　　　　　　　　　　%d", gokei); 

return 0; 

}

コード:

#include <stdio.h> 

int main(void)
{
	int n, gokei;  
	char hinmei[256]; // 文字列の場合だと、もう一次元が必要。
//	char hinmei[256][256] = {0}; // ここは入力されるので初期化不要
	int tanka[256], suryo[256], kingaku[256];
//	int tanka[256] = {0}, suryo[256] = {0}, kingaku[256] = {0}; // ここは入力されるので初期化自体が不要
	int i, count;  

	n=0;
	suryo=0; // suryoは配列なので、この代入は不正。
	count=0; 

	for(i = 0;i < 256;i++)	// ここは、省略可能な処理
							// int tanka[256] = {0};で代用可(ないしはmemset関数を使用)
							// また、後で入力されるのでそもそも初期化は不要
	{ 
		hinmei[i] = 0; 
		tanka[i] = 0; 
		suryo[i] = 0; 
		kingaku[i] = 0; 
	} // この時点でiの値は256になってる

	printf("入力するデータ数を入力してください＞");
	scanf("%d", &n);
	count = n;

	// iの値が256のまま、アクセスしており配列外を参照している
	// (さらにそのあとインクリメントしてるのでどんどん配列外を参照することになってる)
	while(n != 0) // for (count = 0; count < n; count++)の方がスマート こうしたらcount = nは不要
	{ 
		printf("品名を入力してください＞"); 
		scanf("%c", &hinmei); // ここは明らかにおかしい
		hinmei[i] = hinmei;   // これも、間違い。

		//scanf("%s", hinmei[count]); で正解 (ただし、項目数が256を超えた時に配列外を参照することになってしまう。)

		printf("単価を入力してください＞"); 
		scanf("%d", &tanka); // 間違い
		tanka[i] = tanka;    // 同様

		//scanf("%d", &tanka[count]); で正解

		printf("数量を入力してください＞"); 
		scanf("%d", &suryo);              // scanf("%d", &suryo[count]):
		kingaku[i] = tanka[i] * suryo[i]; // iをcountに変更

		gokei = kingaku[i] + gokei;       // gokeiが初期化されていない。

		n--; // 不要
		i++; // 不要
	} 

	i = 0;
	printf("品名　　　　　単価　　　　　数量　　　　　金額\n\n"); // 改行しすぎ
	printf("----------------------------------------------\n\n"); // 改行しすぎ

	while(count != 0) // for (count = 0; count < n; count++)の方がスマート
	{
		// iをcountに変更
		printf("%d　　　　　,%c　　　　　,%d　　　　　,%d", hinmei[i], tanka[i], suryo[i], kingaku[i]); // 一個目は%s二つ目は%d(桁数が変動すると表示が乱れる)
		printf("\n\n"); // 改行しすぎ、上のprintfに入れれる。
		count--;        // 不要
		i++;            // 不要
	} 

	printf("----------------------------------------------\n\n"); 
	printf("合計　　　　　　　　　　　　　　　　　　　%d", gokei); // ここも、合計の値の桁が増えると表示が乱れるので美しくない。

	return 0; 
}

// 確かに構造体にした方がスマートになるますね。
// 無駄な初期化が多くて、ほんとに初期化が必要な変数への初期化ができてない。(ケアレスミスかと思いますが・・・)
// 入力の受け方が質問者さんの中でまだ曖昧のように感じます。
// ループにwhileを多用していますが、用途によっては、forの方がスマートにできる。
// (もちろん用途によってはwhileの方がスマートになります。)

コード:

#include <stdio.h> 
 
int main(void)
{
	int n, gokei;  
    char hinmei[256][256] = {0};
    int tanka[256] = {0}, suryo[256] = {0}, kingaku[256] = {0};
    int i, count;  
 
    n=0;
    count=0;
	gokei=0;

   i = 0;
    printf("入力するデータ数を入力してください＞");
    scanf("%d", &n);
    count = n;
 
     for (count = 0; count < n; count++)
    { 
        printf("品名を入力してください＞"); 
        scanf("%s", hinmei[count]);
 
        printf("単価を入力してください＞"); 
        scanf("%d", &tanka[count]);
 
        printf("数量を入力してください＞"); 
        scanf("%d", &suryo[count]);
        kingaku[count] = tanka[count] * suryo[count];
 
        gokei = kingaku[i] + gokei;
 

    } 
 
    i = 0;
    printf("品名　　　　　単価　　　　　数量　　　　　金額\n");
    printf("----------------------------------------------\n"); 
    for (count = 0; count < n; count++)
    {
        printf("%s　　　　　,%d　　　　　,%d　　　　　,%d", hinmei[count], tanka[count], suryo[count], kingaku[count]);
        printf("\n");
    } 
 
    printf("----------------------------------------------\n\n"); 
    printf("合計　　　　　　　　　　　　　　　　　　　%d", gokei);
 
    return 0; 
}
 

コード:

#include <stdio.h>

void main()
{
	char str[11];

	scanf("%10s", str);
	printf("Inputed String Is\n"
		 "123456789A\n"
		 "%-10s\n", str);
}

コード:

#include <stdio.h> 
 
int main(void)
{
	int n, gokei;  
    char hinmei[256][256];
    int tanka[256], suryo[256], kingaku[256];
    int i;  

    printf("入力するデータ数を入力してください＞");
    scanf("%d", &n);
    i = n;
 
     for (i = 0; i < n; i++)
    { 
        printf("品名を入力してください＞"); 
        scanf("%s", hinmei[i]);
 
        printf("単価を入力してください＞"); 
        scanf("%d", &tanka[i]);
 
        printf("数量を入力してください＞"); 
        scanf("%d", &suryo[i]);
        kingaku[i] = tanka[i] * suryo[i];
    } 
 
    i = 0;
    printf("品名　　　　　　　　　　単価　　　　　　数量　　　　　　金額\n");
    printf("------------------------------------------------------------\n"); 
    for (i = 0; i < n; i++)
    {
        printf("%-16s        %-10d        %-10d\n", hinmei[i], tanka[i], suryo[i]);
		printf("%60d\n",kingaku[i]);
    } 
 
    printf("-----------------------------------------------------------\n"); 

   gokei=0;
   for(i=0; i<n; i++)
   { 
	   gokei=gokei+kingaku[i];
   }
   printf("合計%56d\n",gokei);
 
    return 0; 
}
 

コード:

    int n, gokei;  
    char hinmei[256][256];
    int tanka[256], suryo[256], kingaku[256];
    int i;

コード:

#define MAX 256

コード:

	scanf("%255s", str); // これは文字列のみ (256-1の値になってるのは文字列の最後に'\0'が付与されるためです)
//	scanf("%*s", MAX-1, str); 上の定数を使った場合はこのように指定します。
	while (getchar() != '\n'); // もしも制限よりも多くの文字列が入力されたら余りがバッファに残るので、その分を読み取る。

コード:

const int max_number = 256;             //データの最大数
const int max_length_of_name = 255; //文字列最大長
char hinmei[max_number][max_length_of_name+1];