funkcja jako parametr funkcji

Question

Witam, nie biorąc pod uwagę obecnego stanu kodu, ponieważ były w nim liczne modyfikacje mam problem z wywołaniem funkcji, której jednym z parametrów jest inna funkcja.

#include <iostream>

double add (double x, double y);
double calculate (double p, double m, double * (fun));

int main()
{
        double l, k;
        for(int i=0; i<3; i++)
        {
                std::cout << "Podaj wartosc 1: ";
                std::cin >> l;
                std::cout << "Podaj wartosc 2: ";
                std::cout << k;

                double p = calculate(l, k, add);
        }
        return 0;
}

double add (double x, double y)
{
        return x + y;
}

double calculate (double p, double m, double * (fun))
{
        double q = add (p, m);

        return q;
}

 

Comments

std::cout << k;

Tu raczej powinno być `cin`.

Answer 1

Krótko i na temat:

Podajesz błędny typ dla przekazywanej funkcji. Zamiast

double * (fun)

w deklaracji i definicji funkcji calculate powinno być

double (*fun)(double, double)

Jeżeli masz Symfonię C++ Grębosza, to jest tam dość fajnie napisane jak tworzyć definicje odpowiednich wskaźników na funkcje. Jeśli nie, to przyjrzyj się powyższemu przykładowi i spróbuj na jego podstawie to przećwiczyć tworząc wskaźniki służące do "pokazywania" na funkcje innego typu - wtedy załapiesz w mig.

---

double calculate (double p, double m, double (*fun)(double, double))
{
        double q = add (p, m); 
 
        return q;
}

Tutaj zawsze wywołujesz funkcję add, a chcesz wywołać funkcję przekazywaną przez wskaźnik, a więc poprawnie powinno być:

double q = fun(p, m); 

Pozdrawiam! 

Answer 2

Witam,

wydaje mi się że źle zadeklarowałeś wskaźnik do funkcji. Powinno być

double calculate (double p, double m, double * (fun));
//zamień na 
double calculate (double p, double m, double (*fun)(int, int));

double (*fun)() -> to wskaźnik na funkcje co ma pustą listę argumentów a zwraca double

double (*fun)(int, int) -> to wskaźnik na funkcje co ma dwa argumenty int i  zwraca double

Answer 3

Skoro piszesz w C++, a nie w C, używaj klasy std::function w taki sposób:
http://coliru.stacked-crooked.com/a/c67308d6f84e98bc
Obiekt taki, przyjmuje zwykłą funkcję i functor(lambda).

Comments

Skoro piszesz w C++, a nie w C, używaj klasy std::function w taki sposób:

To nie jest tak, że "jeśli C++ to std::function jest obiektywnie lepszym zamiennikiem wskaźnika"; są sytuacje gdy potrzebujesz std::function, ale w tym przypadku tak nie jest.

Można użyć `using`, z którym deklarowanie wskaźników na funkcje jest dość czytelne:

using Fun = double(double, double);

double calculate (double p, double m, Fun *fun);

 

---

Jak już robić alias, to dobrze też byłoby pozbyć się *:

using Fun = double (*)(double, double);
 
double calculate (double p, double m, Fun fn) { return fn(p, m); }

 

 

 

---

zawsze można zrobić po prostu tak:
 

template<typename T>
void func(T func)

 

---

Jak już robić alias, to dobrze też byłoby pozbyć się *:

Zaletą rozwiązania podrzuconego przez adrian17 jest jego przejrzystość i prostota. Nie musisz pamiętać, jak dokładnie rozlokować nawiasy.

Answer 4

Dziękuję Wam wszystkim za pomoc i za ukazanie mi wszystkich możliwości. Moje umiejętności jeszcze nie pozwalają w pełni na wykorzystanie ich wszystkich jednak zrobię wszystko aby się tego nauczyć.