2変数関数の一方の変数の和を取り1変数関数にする方法

Question

###前提・実現したいこと
xの関数 f(x)=Σ1/(n+x)
を計算し、main関数の中でf(1),f(2)など計算できるようにしたいです。（Σはnについて1から１０までの和）

###発生している問題・エラーメッセージ

コンパイルエラーがでます。forループのあたりがまずい気がします

###該当のソースコード

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

double f(double x){
    double f1(x,int n){
        return 1./(n+x);
    }
    
    for(int i=0;i<11;i++){
        n=i;
        f1(x,n)+=f1(x,n);
    }
    return f1(x,n);
}

int main(){
    f(1);
    return 0;
}

###試したこと
課題に対してアプローチしたことを記載してください

###補足情報(言語/FW/ツール等のバージョンなど)
より詳細な情報

Comments

[WoodenHamlet]

「nの和を上手くとれていないと思います。」じゃなくて、どんなエラーが出ているのか、どんな入力でどんな出力だから「nの和を上手くとれていない」と思うのかをかけ…きましょう。

Answer 1

内側の関数が差し替え可能な版。

#include <iostream>
#include <functional>

using namespace std;

double f(double x, function<double(double,int)> fn) {
  double result = 0.0;
  for(int i = 0; i < 11; ++i){
    result += fn(x, i);
  }
  return result;
}

int main(){
  auto f1 = [](double x, int n) { return 1.0/(n + x); };
  std::cout << f(1, f1);
}

Comments

[sukiyaki]

ありがとうございます。<functional>は使ったことがないので少し調べてみます

Answer 2

こんにちは。

どぞ。できるだけ少しの修正で済ませましたので、ご自身のプログラムと見比べてみて下さい。

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

double f1(double x,int n){
    return 1./(n+x);
}

double f(double x){

    double ret=0;
    for(int i=0;i<11;i++){
        int n=i;
        ret+=f1(x,n);
    }
    return ret;
}

int main(){
    std::cout << f(1) << "\n";
    return 0;
}

Comments

[sukiyaki]

単純なことだったのですね
ありがとうございます解決しました

Answer 3

そもそもコンパイルに失敗してません?
C/C++ は関数の中に関数書けませんよ?