1

ここにコードへのリンクがあり、私もそれを以下に投稿しました。

 #include<math.h>
void pentagon(int n)
 {
    int k,p[10],a[10],b[10];
    if(n<0)
      p[n]=0;
    if(n==0)
      p[n]=1;
    for(k=1;k<n;k++)
     {
        a[k]=((3*pow(k,2))-k)/2;
        b[k]=((3*pow(k,2))+k)/2;
     }
    for(k=1;k<n;k++)
     {
       p[n]=pow(-1,k-1)(pentagon(n-a[k])+pentagon(n-b[k]));
     }
   cout<<p[n];
 }
 int main()
 {
   pentagon(4);
   return(0);
 }

次のエラーが表示されます:
In function 'void pentagon(int)': Line 11: error: call of overloaded 'pow(int&, int)' is ambiguous compilation fails due to -Wfatal-errors

4

4 に答える 4

2

(11,12行目)の第二引数を に2置き換えます。2.0pow

参照: http://www.cplusplus.com/reference/clibrary/cmath/pow/

于 2011-03-11T19:18:14.030 に答える
1

すべてのエラー (および警告) を収集して修正すると、次のコード ( codepad ) になります。何が変わったのかコメントしました。

#include <math.h>
#include <iostream> // Some compilers will complain if missing

using namespace std; // Some compilers will complain if missing

// returns int instead of being void
int pentagon(int n)
 {
    int k,p[10],a[10],b[10];
    // recursion end - we want to jump out here right away
    if(n<0) return 0;
    if(n==0) return 1;
    for(k=1;k<n;k++)
     {
        a[k]=((3*(int)pow(k,2.0))-k)/2; // pow needs double as second argument
        b[k]=((3*(int)pow(k,2.0))+k)/2; //   and returns double
     }
    for(k=1;k<n;k++)
     {
       // pow casting and double as second argument, multiplication with pentagon
       p[n]=(int)pow(-1,k-1.0)*(pentagon(n-a[k])+pentagon(n-b[k]));
     }
   cout<<p[n]<<endl; // cleaner output
   return p[n]; // return the calculated value
 }
 int main()
 {
   pentagon(4);
   return(0);
 }

しかし、出力が次のようになるため、基になるアルゴリズムはまだ間違っていると思います。

-1084535312
-1084535311
1074838088
0
3
4
0
于 2011-03-11T19:29:53.593 に答える
0

私が見つけたいくつかのエラーまたは改善点を次に示します。

iostream を追加し、名前空間 std を使用:

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

pow(k,2)k*k に変更:

a[k]=((3*(k*k))-k)/2;
b[k]=((3*(k * k))+k)/2;

p[n]割り当てに乗算記号を追加します。

p[n] = pow(-1.0,k-1) * (pentagon(n-a[k]) + pentagon(n-b[k]));

メソッドは、上記pentagonのステートメントで使用するために値を返す必要があります。

于 2011-03-11T19:45:11.327 に答える
0

方程式の合計部分がありません。http://en.wikipedia.org/wiki/Pentagonal_number_theoremを参照してください。

関数pentagonは一度に 1 つの項のみを計算します。すべての用語を合計するコードはありません。

于 2011-03-11T20:09:07.930 に答える