単純選択ソートでコアダンプエラー

Question

単純選択ノートで、入力された値を昇順に表示するプログラムを作っていますが、実行したところ数値を入力し終えてEnterを押したときにSegmentation fault (コアダンプ)が表示され、プログラムが正常に作動しませんでした。原因が分かれば教えて下さい。ソースコードは以下の通りです。

#include<stdio.h>
#include<stdlib.h>

#define swap(type, x, y)do{type t = x; x=y; y=t;}while(0)

void selection(int a[], int n)
{
	int i,j;

	for(i=0; i<n-1; n++){
		int min= i;
		for(j=i+1; j<n; j++){
			if(a[j]<a[min])
				min=j;
				}
			swap(int, a[i], a[min]);
		}
	




}


int main()
{
	int i,nx;
	int *x;

	puts("単純選択ソート");
	printf("要素数：");
	scanf("%d", &nx);
	x=calloc(nx, sizeof(int));

	for(i=0; i<nx; i++){
		printf("x[%d]", i);
		scanf("%d", &x[i]);

	}
	selection(x,nx);
	puts("昇順にソートしました");
	for(i=0; i<nx; i++)
		printf("x[%d]=%d\n", i, x[i]);

		free(x);


	return 0;
}

Answer 1

void selection(int a[], int n)
{
...
for(i=0; i<n-1; n++){

配列の要素数nが毎周増えていっちゃってますね。

---

nを関数内で書き換える必要は本来無いのですから、const指定して受けてやると良いです。

void selection(int a[], const int n)

こうすると次のようなエラーが出るので、先の凡ミスを未然に防げます。

prog.c:10:22: error: increment of read-only parameter 'n'
   10 |     for(i=0; i<n-1; n++){
      |                      ^~

prog.c:10:22: error: cannot assign to variable 'n' with const-qualified type 'const int'
    for(i=0; i<n-1; n++){
                    ~^

Comments

[Ryuuse]

const で宣言すればコンパイルエラーでミスに気づくことが出来るのですね！
const宣言の重要性を実感しました...
もし質問しなければ、ミスを発見するまでかなりの時間を要するところでした...
回答、本当にありがとうございました！

Answer 2

selection関数内最初のforループのn++が誤りで、正しくはi++ですね。

Comments

[Ryuuse]

単純なミスに気づいていませんでした...
回答ありがとうございます！