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 · ベストアンサー

★アドバイス
＞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/CPPPUBLISH/node8.html→『ポインタと参照』
　このリンクを読みましょう。
・以上。あと16進などのA～Fは大文字専用ですね。小文字だと上手く変換できませんよ。

参考URL：

http://next1.msi.sk.shibaura-it.ac.jp/MULTIMEDIA/CPPPUBLISH/node8.html

お礼

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

回答No.1

koko_u_

>for(i=0; i=digits.size()-1; i++)
終了条件で代入してどうする？？

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

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

お礼

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