下手くそなところを教えてください。

前提・実現したいこと

プログラミング初心者です。やっとのことで三個の値のソートを以下のコードで実現したのですが
下手くそなところ、よくないところ、を教えていただきたいです。よろしくお願いします。

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

なし

該当のソースコード

#include <stdio.h>

void swap(int *a,int *b){

int temp;
temp=*a;
*a=*b;
*b=temp;

}

int main(void){

int a[3];
int i,j;
scanf("%d",&a[0]);
scanf("%d",&a[1]);
scanf("%d",&a[2]);

for(i=0;i<=2;i++){
for(j=0;j<2;j++){
    if(a[j]>a[j+1]){
        swap(&a[j],&a[j+1]);
    }
}
}

printf("%d %d %d\n",a[0],a[1],a[2]);
return 0;

}

試したこと

補足情報（FW/ツールのバージョンなど）

KojiDoi

2019/05/11 07:07

https://teratail.com/help/avoid-asking

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

回答3件

Markdownの使い方が、一番下手くそだと思うのでちゅ。

投稿2019/05/11 08:10

退会済みユーザー

総合スコア0

とりあえず要素が四つのとき、五つのとき... も動作するように改良してみてはいかがでしょうか。
具体的にはこのようなイメージです。

C
1#define NUM_OF_ELEMS 4   
2// ↑ この数値を書き換えるだけで四つのとき、五つのとき... にも対応できるようにする
3
4...
5int main(void) {
6    int elems[NUM_OF_ELEMS];
7
8    ...
9    return 0;
10}

ひとまずこれが完成したら、要素数も入力できるようにすると勉強になるでしょう。

投稿2019/05/11 07:16

LouiS0616

総合スコア35660

akisem

2019/05/11 09:47

勉強してみます！

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

ベストアンサー

ソート処理を関数に切り出して、テストしやすいようにしてみました。
また、for のループ回数が多すぎるようなので変更もしています。

x.c

c
1#include <stdio.h>
2
3void swap(int *a,int *b) {
4  int temp;
5  temp = *a;
6  *a = *b;
7  *b = temp;
8}
9
10void bubble_sort(int *ary, int n) {
11  for(int i = 0; i < n; i++) {
12    for(int j = n -1; j > i; j--) {
13      if (ary[j - 1] > ary[j]) {
14	swap(&ary[j - 1], &ary[j]);
15      }
16    }
17  }
18}
19
20int main(void) {
21  int a[3];
22
23  for(int i = 0; i < 3; i++) {
24    scanf("%d", &a[i]);
25  }
26
27  bubble_sort(a, 3);
28
29  for(int i = 0; i < 3; i++) {
30    printf("%d ", a[i]);
31  }
32  printf("\n");
33
34  // いろいろなパターンでテスト
35  int TESTS[][3] = {
36		    {1, 2, 3},
37		    {1, 3, 2},
38		    {2, 1, 3},
39		    {2, 3, 1},
40		    {3, 2, 1},
41		    {3, 1, 2},
42		    {1, 1, 1},
43		    {1, 2, 1}
44  };
45  for (int i = 0; i < 6; i++) {
46    bubble_sort(TESTS[i], 3);
47    for(int j = 0; j < 3; j++) {
48      printf("%d ", TESTS[i][j]);
49    }
50    printf("\n");
51  }
52  return 0;
53}