累乗計算をする関数のテンプレート化で、型の異なる引数が二つあるときうまく作動しません。

Question

学校でこのような課題が出され、下記のプログラムを書いたのですが、結果が
4.29497e+09
となってしまい、うまく動作ができません。何が原因なのでしょうか。

#include <iostream>
#include <cstring>
using namespace std;

template< class T1>
T1 power(T1 a, int b)
{
    if (b < 0)
    {
        return 0;
    }
    
    if (b == 0)
    {
        return 1;
    }
    
    
    for (; b >=1; b-- )
    {
        a *=a;
    }
    
    
    return a;
}

int main()
{
    cout << power(4.0,4) << "\n";
    
    return 0;
}

Answer 1

for (; b >=1; b-- )
{
a *=a;
}

ここがもう完全に間違っている．

a *= a; じゃ２回目以降何が起こるのかを考えてみればどうでしょう．
あと，bが1のときどうなの？　ってのもあるし.

Answer 2

T1 power(T1 a, T2 b)
じゃない？

まあ、powerじゃなさそうだけど。

Comments

[PingHermit]

T1,T2と、二つ書くのだめなんかな？
C++はほとんど知らないから
忘れてください(^^;

Answer 3

ぱっと見て問題だと思ったのは、以下の点です

• ~~仕様に戻り値が引数1と同じ型、と書いてあるのに、引数2の型(T2)を返している

（そもそも、仕様に引数2はintって書いてあるので、T2自体不要なのでは）
~~
回答した後に直ってました

• a *=a; だと、べき乗にはならない（掛けた値を更に掛けているため、とんでもない数になる）

処理中の変数の値は、ブレークポイントを仕掛けてデバッグ実行して変数の値を確認したり、変数の値をログ出力して調べればおかしいのはすぐ分かる筈なので、値がおかしいなと思ったら、途中経過の値を順に確認して調べましょう。

Answer 4

得てして、バグはそのくだらない事が意識せずに紛れ込んで発生するものなので、解決方法に対して気付きを得られたのであれば、それは進歩です。重要なのは、同じバグを繰り返さないように知識や経験を蓄積することです。頑張ってください。
（質問者さんのコメントに書くつもりが、間違って回答に書いてしまった、恥ずかしい…）

Answer 5

バッファtempを用意して、その値にaをbの回数分だけかけることによって実現できました。
くだらない質問をしてしまい、申し訳ありません。
回答ありがとうございました。

#include <iostream>
#include <cstring>
using namespace std;

template< class T1>
T1 power(T1 a, int b)
{
T1 temp;
temp = 1;
if (b < 0)
{
return 0;
}

if (b == 0)
{
    return 1;
}


for (; b >=1; b-- )
{
    temp*=a;
}


return temp;

}

int main()
{
cout << power(4,4) << "\n";

return 0;

}