数字を並び替えるプログラムを作りたい

Question

実現したいこと

・ユーザーに10個の数字を入力してもらい、それを小さい数字から大きい数字に並び替えて出力をするプログラムを作りたいです
・別のfunctionも足していく予定です
・outputがうまくいかないのでその理由を教えて頂きたいです

発生している問題・エラーメッセージ

error: implicit declaration of function 'smallestToBiggest' is invalid in C99 [-Werror,-Wimplicit-function-declaration]

該当のソースコード

#include <stdio.h>

int main() {

 int num[10], output;

 printf("Please insert 10 numbers:\n");
 for(int i = 0; i < 10; i++) {
  scanf("%d", &num[i]);
 }

 output = smallestToBiggest(num[10]);

 return 0;

}

int smallestToBiggest(int num[10]) {
 for (int i = 0; i < 9; i++) {
  for (int j = i + 1; j < 10; j++) {
   if (num[j] < num[i]) {
    int temp = num[i];
    num[i] = num[j];
    num[j] = temp;
   }
  }
 }
 printf("Sorted numbers smallest to biggest:\n");
 for(int i = 0; i < 10; i++) {
  printf("%d", num[i]);
 }
 printf("\n\n");
 return num[10];
}

Comments

[meg_]

確認ですが課題でしょうか？

[seika]

学校の課題です

[episteme]

C++なら std::sort() でイッパツなんだが、課題ならそうはいかんか...

[seika]

多分習ってる範囲でやらなくちゃいけないんですよね、、、
ChatGPTはそれでやるように指示してきますが笑

Answer 1

1. 関数(smallestToBiggest)を呼び出す前にその関数を定義する必要があります。
2. 関数に渡す変数も見直す必要があるかと思います。

Comments

[seika]

なるほど、、、
順番が関係あると思ってなかったです。
ありがとうございます！

Answer 2

このコードで問題が発生しているのは、「smallestToBiggest」関数が「main」関数の前に宣言されていないためです。C99では、関数を使用する前に宣言されている必要があります。また、関数smallestToBiggestは、整数の配列を返すことができないため、戻り値の型を変更する必要があります。以下の修正を行ってください。

1. 番号リスト関数smallestToBiggestのプロトタイプをmain関数の前に追加します。
2. 番号リスト関数smallestToBiggestの戻り値をvoidに変更します。
3. 番号リスト関数smallestToBiggestで引数として渡される配列をポインタとして受け取ります。

修正後のコードは次のようになります。

#include <stdio.h>

void smallestToBiggest(int *num, int size);

int main() {

 int num[10];

 printf("Please insert 10 numbers:\n");
 for(int i = 0; i < 10; i++) {
  scanf("%d", &num[i]);
 }

 smallestToBiggest(num, 10);

 return 0;

}

void smallestToBiggest(int *num, int size) {
 for (int i = 0; i < size-1; i++) {
  for (int j = i + 1; j < size; j++) {
   if (num[j] < num[i]) {
    int temp = num[i];
    num[i] = num[j];
    num[j] = temp;
   }
  }
 }
 printf("Sorted numbers smallest to biggest:\n");
 for(int i = 0; i < size; i++) {
  printf("%d ", num[i]);
 }
 printf("\n\n");
}

これで、エラーが解消され、プログラムが正常に動作するはずです。

Comments

[seika]

すごくわかりやすく訂正して頂いてありがとうございます！
正常に作動してます。

[seika]

度々すみません。
ポインタ型の変数にした理由を教えていただけますか？

[KenRoda]

ポインタ型の変数にした理由は、関数間で配列を効率的かつ正確に共有するためです。関数に配列を渡すとき、C言語では配列名が実際にはその配列の先頭要素のアドレス（つまりポインタ）を示しているため、ポインタを使用します。

以下の理由から、ポインタを使って配列を関数に渡すことが一般的です。

1. 効率: 配列をポインタとして渡すと、実際のデータのコピーが作成されず、元の配列のメモリアドレスが渡されます。これにより、関数は元の配列のデータを直接操作でき、メモリ使用量や処理時間が節約できます。

2. 反映される変更: ポインタを使用することで、関数内で行われた配列の変更が元の配列にも反映されます。この場合、smallestToBiggest関数でソートされた配列が、main関数内の元の配列にも反映されます。

3. 可変サイズ: 関数にポインタとサイズを渡すことで、配列のサイズが可変になります。これにより、同じ関数を異なるサイズの配列に対して再利用することができます。

このプログラムでは、配列のアドレスをsmallestToBiggest関数に渡すことで、関数内で配列を並べ替え、その変更がmain関数内の元の配列にも反映されるようにしています。これにより、効率的かつ簡潔なコードが実現できます。

[seika]

なるほどmain関数内の元の配列にも反映されるのであれば別の関数でも少ないコードで利用できますね！
まさにそうしたかったので、助かります。

教えて頂きありがとうございます！