C言語 ポインタを使って自然数のすべての位を1の位の数で置き換えたい

Question

早めの回答をしてくださると助かります...!!
現在、高等専門学校でC言語のポインタの範囲を勉強しています。
上のような問題を解いています。プログラムの添削、またはここの部分はこうした方がいいじゃないかというのがあれば教えていただきたいです。よろしくお願いします。

#include <stdio.h>

void convert(int *answer, int org) {
    *answer = 0; 

    //1の位の数字を求める
    int last_digit = org % 10;

    //orgの桁数を求める
    int num_digits = 0;
    int tmp = org;
    while (tmp != 0) {
        num_digits++;
        tmp /= 10;
    }

    //answerに1の位の数字を連結する
    for (int i = 0; i < num_digits; i++) {
        *answer = *answer * 10 + last_digit;
    }
}

int main(void) {
    int org;
    int answer = 0;

    //自然数を入力
    scanf("%d", &org);

    //convert()関数をコール
    convert(&answer, org);

    //結果を表示
    printf("%d\n", answer);

    return 0;
}

Comments

[y_waiwai]

提示しているコードではどういう結果となるんでしょうか。
また、それをどういうふうにしたいという話なんでしょう。

[supercubo]

実行結果は、
（入力）123
（出力）333
というようにできたのですが、コードがこの問題の条件をすべて満たしているのかな..と思ったので質問をしました。条件をすべて満たしていないと０点だぞ！と先生に言われたので..。
自分でチェックはしたつもりですが、良ければ詳しい方にもチェックしていただきたいなと思いました。
意図が分かりにくく、申し訳ないです。

[jimbe]

math.h の log10 は使っちゃいけないんでしょうか。

[supercubo]

使うこと自体はOKだと思いますが、log10 が使えるんですか？
何乗かを求めるやつですよね。使い方が全く想像できないです...

[jimbe]

int num_digits = (int)log10(org)+1;
とか。

[supercubo]

すごい！そうやって桁数が求まるんですね！
高校数学の知識が使えるとは..知識増えました！
ありがとうございます

[fana]

> orgの桁数を求める

こういうのは関数にすると若干すっきりするかも．
まぁ，問題文の指定のされ方的に別の関数を作るのがOKなのか否かよくわかりませんが．
関数化までしないのだとしても，
num_digits の値を求めている部分（line11 ～ line15）を {} で囲むとかすればどうかな．

Answer 1

題意をそのままストレートにコード化した簡明なプログラムで良いと思います。
強いて直すなら、4行目の*answer = 0; は18行目のforの直前に移動します。
あとは変数名のtmpをどうするかですが、「じゃあ、おまえなら何という変数名にするんだ？」と問われるとう唸ってしまいます。あとは17行目のコメントの日本語の「数字を連結する」がやや微妙。それくらいしか言うことがないプログラムかと思います。

whileのループ回数と、forのループ回数は常に同じなので、指摘があるように、1つのループにまとめることは可能です。ただ、そちらの方が良いプログラムかというと、「実行時間を出来るだけ短くせよ」とか「文の数を出来るだけ少なくせよ」というような指示が無い限りは、この問題に対する初学者の回答という意味では、今のままの形の方が良いプログラムだと思います。

あるいはShunlyさんも「トリッキーな」とお書きのようなコードにして、同級生を「すごい」と唸らせたいならもっと短いコードを考えるのも良いかと思いますが、コメントを書かないと「すごい」じゃなくて「なんでこれで求まるの？」と思われるかも知れません。

むしろ、質問文に指摘をしたいです。

　一つの自然数（0を含まない）を

は、正確には「一つの自然数（十進数表記のどの桁にも0を含まない）を」という意味でしょうが、文字通り読むと、「0でない自然数」という意味になります。（横道にそれると https://ja.wikipedia.org/wiki/%E8%87%AA%E7%84%B6%E6%95%B0 をみると分かりますが、「自然数」には「0を含める」と「0を含めない」の2つの流儀があります）
これでは記述が長いし、この問題だと1の位以外はどうでも良いので、ストレートに「1の位が0でない一つの自然数を」で良かったのではないかと思います。
あるいは、00123のようなものも排除したくてこう書いたのなら、「数値」と「数字（または数字文字列、数値文字列、十進数表記）」を区別した日本語記述にしないと駄目です。

最後の

answerは参照渡し

も、一般的なIT用語としての「参照渡し」としては間違った記述です。この引数引き渡し方を「参照渡しというIT用語」だと思ってしまうと、正しい意味での参照渡しのある言語を学ぶ際に困ってしまうかも知れません。

コードがこの問題の条件をすべて満たしているのかな..と思ったので

(a)(b)(c)(d)の条件であれば、main関数の中を書き換えてなければOKでしょう。

Comments

[supercubo]

詳しく書いてくださってありがとうございます。
解決しました。

[otn]

> 強いて直すなら、4行目の*answer = 0; は18行目のforの直前に移動します。
の理由は考えて分かりましたか？

convert関数はコメントの通り3つのパートに別れています。
その前に書いてある *answer = 0; ですが、3つのパートの何処に属するのか、あるいは何処にも属さないのか？を考えると3つ目のパートに属するのでそこに書きます。

[supercubo]

なるほど、*answer = 0; は answer を作っていくパートに属するので、3つ目の部分に書けばいいのですね。
ありがとうございます。

Answer 2

間違ってないし、いいんじゃないですかね？
改良するとするなら、桁数を求める必要はないので、ループは1つにできますね。

Comments

[supercubo]

回答ありがとうございます。
「ループは1つに」の部分がよくわかりません。具体的にどこをどう変えれば良いのでしょうか？

[Shunly]

今桁数を求めるwhileループとanswerを作成するforループがありますよね？これは１つにできます。
学校の課題なので方法は頭を捻ってみてください。
convert関数は少しトリッキーな方法を使って5ステップが最短じゃないかな。

[supercubo]

もっと簡潔にできるんですね...
悩んでみます。

[jimbe]

トリッキーがどんなか分かりませんが {} 省略無しで 4 行なります。

*answer = 0;
for(int last_digit=org%10; org>0; org/=10) {
*answer = *answer * 10 + last_digit;
}

[supercubo]

そんなfor文の書き方があるんですね。まとまって短くなりますね。ありがとうございます！

[melian]

answer は main 関数内で初期化されているので以下の様に書いてもよいかと。

void convert(int *answer, int org) {
for(int last_digit=org%10; org>0; org/=10,*answer=(*answer*10)+last_digit);
}

[jimbe]

まぁ、ポインタ自体とは全然関係無い部分ですのでどこまで弄るか拘るかって感じですね^^;

[Shunly]

確かにforで詰め込んだらさらに減りますね、do{～}while(org/=10);で考えてたのが余計でした、、、