n進数を10進数に変換するプログラムがわかりません

整数n(2<=n<=16)とn進数の数字列digitsを与えると10進数に変換して表示するプログラム
関数void base2dec(int n, string digits, int &dec)を使用する

実行例 110101(2)=53
123(8)=83
7ACE(16)=31438
以下はとりあえずつくったみたプログラムです
#include<iostream>
#include<string>
using namespace std;
void base2dec(int n, string digits, int &dec)
{
int i,x=0;
for(i=0; i=digits.size()-1; i++)
{
if(n<10)
{
x=digits[i]-'0';
}
else
{
x=digits[i]-'A'+10;
}
dec=(x*n);
if(cin >> digits[i++])
{
if(n<10)
{
dec=dec+(digits[i++]-'0');
}
else
{
dec=dec+(digits[i++]-'A'+10);
}
}
}
int value = dec;
}

int main()
{
int n,value;
string digits;
cout << "何進数ですか-->";
cin >>n;
cout << "数字列を入力してください-->";
cin >>digits;;
base2dec(n,digits,value);
cout << digits << "(" << n << ")=" << value << endl;
return 0;
}

void関数の中、特に計算方法が間違っていると思うのですが、なにがいけないかわかりません。ちなみにn進数から10進数に直す方法は理解しています。

No.2 ベストアンサー 回答者： Oh-Orange 回答日時： 2007/10/18 06:45

★アドバイス

＞for(i=0; i=digits.size()-1; i++){
　↑
　この行がおかしいです。
　間違い⇒for(i=0; i=digits.size()-1; i++){
　正しい1⇒for(i=0; i<=digits.size()-1; i++){
　正しい2⇒for(i=0; i<digits.size(); i++){
　※回答者 No.1 さんのアドバイス通りです。
＞void関数の中、特に計算方法が間違っていると思うのですが、なにがいけないかわかりません。
　↑
　『dec=(x*n);』ここが違う気がします。
　『dec=(dec*n+x);』とすればいいと思いますけど。
＞ちなみにn進数から10進数に直す方法は理解しています。
　↑
　理解しているのなら『dec=(dec*n+x);』となると思いますが…。

その他:
＞if(cin >> digits[i++]){ … }
　↑
　このifブロック内は必要なのか？
　どういう意味。
＞int value = dec;
　↑
　最後のこの行は意味ないね。
　『value』を参照してないし。
　戻してないし。
　そもそも dec が『参照』ですよ。
　http://next1.msi.sk.shibaura-it.ac.jp/MULTIMEDIA …→『ポインタと参照』
　このリンクを読みましょう。
・以上。あと16進などのA～Fは大文字専用ですね。小文字だと上手く変換できませんよ。

参考URL：http://next1.msi.sk.shibaura-it.ac.jp/MULTIMEDIA …

この回答へのお礼

的確なアドバイスありがとうございます。自分の間違いがよくわかりました。変なミスばっかで申し訳なかったです。参考URLのページもとても参考になりました。ありがとうございました。

お礼日時：2007/10/19 00:14

No.1 回答者： koko_u_ 回答日時： 2007/10/18 03:18

>for(i=0; i=digits.size()-1; i++)

終了条件で代入してどうする？？

i == digits.size()-1 でもモチロン間違ってますが。

それ以降のコードは読んでません。

この回答へのお礼

そりゃそうですよね。よくよく見返したら変なのは当然でした・・・
教えてくれてありがとうございました。

お礼日時：2007/10/19 00:12