C言語で入力した3つの数字を昇順にソートするコードの改善に関して

Question

###前提・実現したいこと
C言語で入力した3つの数字を昇順にソートするコードを以下のサイトを参考に書いています。
参考サイト

###発生している問題・エラーメッセージ
現状のコードで動くのですが、結果が以下のようになり、以下の点に困っています。
①「After：」の最後の数字の後ろの「,」を表示させないようにするにはどのようにコードを書き換えたら良いのか
②結果が表示された行にすぐ「コマンドプロンプト％」が表示されるので、次の行に表示させるにはどうしたら良いか

input a=15
input b=12
input c=10
Before: 15, 12, 10
After:10,12,15,% 

###該当のソースコード

#include<stdio.h>
int main(){
  int a;
  int b;
  int c;

  printf("input a=");
  scanf(" %d", &a);
  printf("input b=");
  scanf(" %d", &b);
  printf("input c=");
    scanf(" %d", &c);

  int number [3] = {a, b, c};

  int i, j, tmp;
  for(i = 0; i<3; ++i){
    for(j =i+1; j<3; ++j){
      if(number[i]>number[j]){
        tmp = number[i];
        number[i] = number[j];
        number[j] = tmp;
      }
    }

  }

  printf("Before: %d, %d, %d\n", a, b, c);
  printf("After:");
  for(i = 0; i<3; ++i){
    printf("%d," , number[i]);
  }

}

###試したこと
課題に対してアプローチしたことを記載してください

###補足情報(言語/FW/ツール等のバージョンなど)
より詳細な情報

Answer 1

毎回コンマを出力してしまっているので、うまく当てはまるところだけ出すようにする必要があります。あと、最後に改行したいならそれも実装する必要があります。

  printf("After:");
  for(i = 0; i<3; ++i){
    // 数が増えても初回を飛ばすのは変わらないので、ここに入れるほうが管理しやすい
    if(i > 0) printf(",");
    printf("%d" , number[i]);
  }
  printf("\n");

Comments

[退会済みユーザー]

①、②ともにご回答いただきましてありがとうございました。

Answer 2

やり方はいろいろありますが、Beforeがその形式なら、Afterも倣っていいのでは。

printf("Before: %d, %d, %d\n", a, b, c);
printf("After:  %d, %d, %d\n", number[0], number[1], number[2]);

Comments

[退会済みユーザー]

ご回答いただきましてありがとうございました。

Answer 3

afterの部分のみですが、こんな感じでしょうか

  printf("Before: %d, %d, %d\n", a, b, c);
  char p=':';
  printf("After");
  for(i = 0; i<3; ++i){
    printf("%c %d" ,p, number[i]);
    p=',';
  }
  printf("\n");

Comments

[退会済みユーザー]

ご回答いただきましてありがとうございます。
様々な書き方で実装できることがわかりました。

Answer 4

①「After：」の最後の数字の後ろの「,」を表示させないようにするにはどのようにコードを書き換えたら良いのか

それはすべての数字の後ろに「,」が印字されるようにコーディングしているためです。印字したものを後戻りで消すのは面倒なので、「,」を出すべきか否かを判断してやればできます。

  printf("After:");
  for(i = 0; i<3; ++i){
    printf("%d" , number[i]);
    if (i < 2)
      printf(", ");   // 最後の数字でなければカンマを出力
  }

②結果が表示された行にすぐ「コマンドプロンプト％」が表示されるので、次の行に表示させるにはどうしたら良いか

main関数を抜ける前に"\n"を出力すればOKです。

  printf("\n");

Comments

[退会済みユーザー]

ご回答いただきましてありがとうございます。
様々な書き方で実装できることがわかりました。

Answer 5

①表示を行うfor文のループ内で「,」を表示する条件を指定する
for(i = 0; i < 3; ++i){
printf("%d" , number[i]);
if(i < 2) {
putchar(",");
}
}
②ループ処理が終わった後に改行を行う
putchar('\n'); or printf("\n");

他にもきれいな書き方等あると思いますが、そこはご自身で考えてみてください。
ちなみにご存じだとは思いますが、putcharは一文字のみ表示する文です。分からなければググってみてください。

Comments

[退会済みユーザー]

ご回答いただきましてありがとうございます。
様々な書き方で実装できることがわかりました。