各桁の和を返す関数

一つの正整数を引数として受け取り、
その整数の各桁の和を返す関数を作成する。
この関数を用い、いくつかの正整数を入力して
その整数の各桁の和を表示する。
入力の終わりは、０または負のデータが現れたことで
判定する。

というC言語の問題なんですが、
整数の各桁の和を返す関数がうまくできません・・。
123が15となってしまいます・・。
とりあえず最初の部分を作ってみました。
初心者なのでミス多いかもしれませんが
どなたか教えていただけませんか？

#include<stdio.h>
#include<conio.h>
int sum(int n)
n=(n/10)+(n%10);

return n;
}
main()
{
int n,sum2;
scanf("%d",&n);
sum2=sum(n);
printf("%d",sum2);
}

No.5 回答者： JaritenCat 回答日時： 2008/06/29 01:01

こんな感じでしょうか。

。
#include <stdio.h>

int sum(int n) {
int sum=0;
if (n<0) return 0;
while (n>0) {
sum += n%10;
n /= 10;
}
return sum;
}

int main(void) {
int n;
while(1) {
scanf("%d",&n);
if (n<=0) break;
printf("%d\n",sum(n));
}
return 0;
}

No.4 回答者： yama5140 回答日時： 2008/06/20 11:32

＞というＣ言語の問題なんですが、

　おそらく、期待される「答え」ではない、と思われますのでソースを投稿します。
　　　　　　　　（ Borland C++ 5.6.4 ）

　こんな方法もあるんだ、と見て頂ければ・・。
-------------------------------------------
★整数を文字列にして、和を求める方法です。

　　123 → '1' + '2' + '3' = 6

１．scanf() は、正常に入力した項目数を返します。
　　先頭にピリオド、途中に英文字等の「入力」で試して下さい。
　　→ 返った項目数を判定することで、想定外入力に対応できます。

　　ちなみに、123+456+789+987+654+321 と入力すると・・・？。

２．下の sprintf() は、"%d" の書式で文字列を作り、かつ文字列の終了を表すヌル文字（ 0x00 ）を付加します。

（３．GetSum() 関数内は、ポインタを使った書き方も可能です。）

《蛇足》
コンパイル時に、メイン関数の return( -1 ); 行に警告等がでましたら、「処理系」を補足して下さい。
------------------------------------------
#include <stdio.h>

int　GetSum( char cStr[] )
{
　int　iTotal = 0, k = 0;

　while( cStr[ k ] ){　// 文字列終端 0x00 の前まで

　　iTotal += ( cStr[ k ] - 0x30 );　// 参考ＵＲＬ

　　k++;
　}
　return( iTotal );
}
int　main( void )
{
　int　n, iSum2;
　char　cStr[16];

　while( 1 == scanf( "%d", &n ) ){

　　sprintf( cStr, "%d", n );　// 整数 → 文字列

　　if( '-' == cStr[0] )　break;　// 負のデータ／終了
　　if( '0' == cStr[0] )　break;　// ０のデータ／終了

　　iSum2 = GetSum( cStr );

　　printf( "%c%s%c %d\n", 0x22, cStr, 0x22, iSum2 );
　}
　return( -1 );　// 0-255
}
注：インデントに全角空白を用いています。タブに一括変換して下さい。
-------------------------------------
おそらく、「期待される『答え』」に近いものは、下のようになるのでは・・。
　
　　for( i = 0; i < 13; i++ ){　// int 型
　
　　　if( 0 == n )　break;
　
　　　iSum2 += ( n % 10 );

　　　n /= 10;　// 下位より切り崩し
　　}
　　↑未確認です。

参考URL：http://e-words.jp/p/r-ascii.html

No.3 回答者： tokichim 回答日時： 2008/06/20 11:15

不慣れなうちは自分がコンピュータになったつもりで、紙の上で試行錯誤してみましょう。

（今回のn/10のところの誤りもそうすればすぐに分かったはずです）

各桁の和を計算するためには各桁の数字を取り出さないとならないわけですが、ではどうすればいいのか？
一部おわかりのようですが、10進数で表された数は各桁ごとに1,10,100,1000...と10倍ずつになっているので、逆に割ってやれば各桁の数が取り出せます。
ただし割っただけだとその上の桁の数もついてきてしまいますから、欲しい桁だけにするために%10することが必要となります。

また、桁数としては1からintで表せる最大の値の桁数まであり得るわけですから、それについても考慮する必要があります。
ループ文について学んでいれば、それを使います。その場合は#2の回答のロジックになります。

No.2 回答者： asuncion 回答日時： 2008/06/20 07:05

１２３の場合：

各桁の和を０で初期化する

１２３を１０で割り、商（１２）とあまり（３）を求める。
あまり（３）を各桁の和に加える。
各桁の和は３になる。

１２を１０で割り、商（１）とあまり（２）を求める。
あまり（２）を各桁の和に加える。
各桁の和は５になる。

１を１０で割り、商（０）とあまり（１）を求める。
あまり（１）を各桁の和に加える。
各桁の和は６になる。

これ以上続けても意味がないので、計算を終える。
最終結果は６となる。

以上のロジックをコード化してください。

No.1 回答者： redfox63 回答日時： 2008/06/20 05:47

n=123の場合 sum関数は

( n / 10 ) + ( n % 10 )
= ( 123 / 10 ) + ( 123 % 10 )
= ( 12 ) + ( 3 )
= 15
となります

ということは 12となる n/10をもっと分解する必要があります
whileループや 再帰呼び出しなどで分解するようにしてみましょう