C言語で３桁ごとにカンマ区切りするプログラム

Question

実現したいこと

「数値Nが入力します。 位の小さい方から 3 けたごとにカンマ区切りで出力してください。
ただし、Nのけた数は 3 の倍数とは限らない」この実現方法がわかりません

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

以下のようなコードを書いてみたのですが、
入力が123456789の場合
期待する出力としては123,456,789で
以下のコードで一応出力としては正しく出るのですが、
自分的にあまり納得のいく綺麗なコードとは言えず、
もっといい実現方法があれば教えていただきたいです。

該当のソースコード

#include <stdio.h>
#include <string.h>

int main(void){
    char n[256];
    scanf("%s",n);
    int len=1;
    for(int i=0;i<strlen(n);i++){
        if(len%3==0){
            if(n[i+1]!=NULL){
                printf("%c,",n[i]);
            }else{
                printf("%c",n[i]);
            }
        }else{
            printf("%c",n[i]);
        }
        len=len+1;
    }
    return 0;
}

Comments

[melian]

※ 質問文を読むと何らかの課題の様なので、参考までに…

そちらの環境が POSIX 2008 に準拠している場合、printf(3) が '(アポストロフィ) フォーマット指示子に対応しているかもしれません。

fprintf - The Open Group Base Specifications Issue 7, 2018 edition IEEE Std 1003.1-2017 (Revision of IEEE Std 1003.1-2008)
https://pubs.opengroup.org/onlinepubs/9699919799/functions/printf.html
> (The <apostrophe>.) The integer portion of the result of a decimal conversion ( %i, %d, %u, %f, %F, %g, or %G ) shall be formatted with thousands' grouping characters. For other conversions the behavior is undefined. The non-monetary grouping character is used. 

#include <stdio.h>
#include <locale.h>

int main(void) {
setlocale(LC_NUMERIC, "");
printf("%'d\n", 123456789);
return 0;
}

[jimbe]

>自分的にあまり納得のいく綺麗なコードとは言えず

個人的な感覚では他人にはどうすれば納得なのか分かりません。
基準を示していただけますか?

Answer 1

位の小さい方から 3 けたごとにカンマ区切りで出力してください。

ひとつの数字を出力した後にカンマが必要となる条件は「未出力の数字があり，かつその個数が 3の倍数」と考えられます。そして，全個数が len で i 番目（最初は 0番目）を出力した後の未出力の個数は r = len - i - 1 になります。

これを基にした記述例を下記に示します。

#include <stdio.h>
#include <string.h>

int main(void) {
    char n[256];
    int len, i, r;

    scanf("%s", n);
    len = strlen(n);

    for (i = 0; i < len; i++) {
        printf("%c", n[i]);
        r = len - i - 1;
        if (r > 0 && r % 3 == 0)
            printf(",");
    }
    printf("\n");

    return 0;
}

Comments

[komori_k]

わかりやすい回答ありがとうございます！

Answer 2

仕様を満たしているかでは無く単にコードを変形させるだけしてみます。方向性としては「同じことは何度も書かない」「短い」です。

目立つのはループ内に 3 つある n[i] を表示する printf 。

        if(len%3==0){
            if(n[i+1]!=NULL){
                printf("%c,",n[i]);
            }else{
                printf("%c",n[i]);
            }
        }else{
            printf("%c",n[i]);
        }

どの if のルートを通っても n[i] を表示するのは変わらず、一箇所後ろに "," があるだけですので if の前でまず n[i] を表示してしまいます。

        printf("%c",n[i]);
        if(len%3==0){
            if(n[i+1]!=NULL){
                printf(",");
            }
        }

すると "," を表示するかの判断の if は一つに出来ます。

        printf("%c",n[i]);
        if(len%3==0 && n[i+1]!=NULL){
            printf(",");
        }

len 変数。

    int len=1;
    for(int i=0;i<strlen(n);i++){
        //処理
        len=len+1;
    }

ループの中でしか使用せず、その値は常に i+1 ですので処理の中の len を (i+1) に変更します。

    for(int i=0;i<strlen(n);i++){
        printf("%c",n[i]);
        if((i+1)%3==0 && n[i+1]!=NULL){
            printf(",");
        }
    }

i+1 が 2 か所あることと for に i++ があるので纏めることも出来ます。

    for(int i=0;i<strlen(n);){
        printf("%c",n[i++]);
        if(i%3==0 && n[i]!=NULL){
            printf(",");
        }
    }

一応文字列の終端は NULL でなくヌル文字 '\0' (のはず)ですので、気にする向きには n[i]!='\0' とかいっそ n[i] だけとか。

---

出題通りの結果を得た上で、 c ライクな感じに。
最初の "," までの桁を表示した後は ","+3桁の繰り返しとします。
※ printf の format に "*" が使えるコンパイラが必要

#include <stdio.h>
#include <string.h>

int main(void) {
    char n[256];
    scanf("%s", n);

    for(char *p=n,w="\3\1\2"[strlen(n)%3];*p;p+=w,w=3){
        printf(",%.*s"+(p==n),w,p);
    }
    printf("\n");

    return 0;
}

Answer 3

位の小さい方から 3 けたごとにカンマ区切りで出力してください

とされているのに，
「位の大きい側から３桁出力する毎にカンマを追加で出力する」という処理では，求められていることと違うことをしていますよね．

例えば 1234 という入力に対する出力結果が 123,4 になってしまいますよね．
（求められている結果とは 1,234 という出力であるハズ．）

---

とりあえず以下のコード（関数）は入力を単にそのまま（先頭から１文字ずつ）出力するだけです．

void Output( const char *Input )
{
    const int len = strlen( Input );
    for( int i=0; i<len; ++i )
    {//i番目の文字を出力する
        putchar( Input[i] );
    }
}

このコードの //i番目の文字を出力する のところを，条件次第では カンマとi番目の文字 を出力する という話に変えてみるとどうでしょうか．

void Output( const char *Input )
{
    const int len = strlen( Input );
    for( int i=0; i<len; ++i )
    {
        //条件次第では先にカンマを出力する
        if(  ???  )
        {   putchar( ',' ); }
        //i番目の文字を出力する
        putchar( Input[i] );
    }
}

あとは ??? の条件を，題意に沿うように考えてやれば良さそうです．

• i というのは先頭側（位の大きい側）から数えている値なわけですが，これを「後ろ側（位の小さい側）から何番目の場所か？」という値にうまく読替えてやれば，題意を満たす方法を考えることができそうですよね．
  例えば，入力の長さが4桁であるとき，i の値は {0,1,2,3} なわけですが，len - i という値を考えれば { 4,3,2,1 } になりますよね．この値を用いればとても考えやすそうに思えます．
• 出力結果の先頭にいきなりカンマが出てくるのはまずいので「先頭文字の場合にはカンマを付けない」みたいな条件が必要かもしれません．

……とか考えてみた結果をコードに反映するならば，以下のようになります．

void Output( const char *Input )
{
    const int len = strlen( Input );
    for( int i=0; i<len; ++i )
    {
        //先頭ではなくて，且つ，後ろ側から数えた位置(len-i)が3の倍数であるならば，
        //先にカンマを出力する
        if(   (i>0)   &&   ( (len-i)%3 == 0 )   )
        {   putchar( ',' ); }

        putchar( Input[i] );
    }
}

※まぁ，この例が

納得のいく綺麗なコード

と言えるのかは否か？　についてはわかりませんが．