Funkcja jako argument innej funkcji C++

Question

Czy da się zrobić tak, aby funkcja była argumentem innej funkcji? Coś w ten sposób:

void menu(std::string alert)
{
	std::cout<<alert<<std::endl;
	
	//Chcialbym zwalidowac konwersje string -> int
	//Docelowo po prostu walidacja danych wejsciowych i obsluga bledu - moze jakis inny, prostszy pomysl?
	getline(std::cin, pytanie);
	int opcja = stold(pytanie);

	switch (opcja)
	{
	case 1:
		funkcja1();
		break;

	case 2:
		funkcja2();
		break;

	case 3:
		funkcja3();
		break;

	default:
		throw std::logic_error("Brak opcji");
		break;
	}
}

//Sprawdzanie bledow
void sprawdz(mojaFunkcja(std::string mojString))
{
	try
	{
		mojaFunkcja("");
	}
	catch (std::invalid_argument){
		mojaFunkcja("Zly argument");
	}
	catch (std::logic_error e){
		mojaFunkcja(std::string(e.what()));
	}
	catch (std::exception e) {
		mojaFunkcja("Blad: " + static_cast<std::string>( e.what() ));
	}
	catch (...) {
		mojaFunkcja("Nieznany blad");
	}
}

int main(void)
{
	sprawdz(menu(""));
}

 

Comments

Rzuć okiem na wskaźniki do funkcji;

https://cpp0x.pl/kursy/Kurs-C++/Poziom-X/Wskaznik-na-funkcje/249

Może Ci pomoże

---

Super, raczej wszystko działa dokładnie tak jak chciałem. Dzięki.

---

@Mavimix, źle używasz wyjątków. Nie służą do kierowania flow programu. W dodatku, przy ich spełnieniu są bardzo zasobożerne. std::function albo wskaźniki, jak wyżej. 

---

@tkz, Czyli  w jaki sposób mogę jeszcze zwalidować program jak wyżej? Czym jest std::function?

---

@tkz, Napisałem taki program:

#include <iostream>
#include <string>

void funkcja1()
{
    std::cout << "Funkcja JEDEN" << std::endl;
}

void funkcja2()
{
    std::cout << "Funkcja DWA" << std::endl;
}

void funkcja3()
{
    std::cout << "Funkcja TRZY" << std::endl;
}

typedef void(*menuTyp)(std::string);

void menu(std::string alert)
{
    system("cls");
    std::cout << alert << std::endl;

    std::string pytanie;
    getline(std::cin, pytanie);
    std::cin.clear();

    int opcja = stold(pytanie);

    switch (opcja)
    {
    case 1:
        funkcja1();
        break;

    case 2:
        funkcja2();
        break;

    case 3:
        funkcja3();
        break;

    default:
        throw std::logic_error("Brak opcji");
        break;
    }
}

menuTyp wskaznikMenu = menu;

void sprawdz(menuTyp mojaFunkcja, std::string wiadomosc)
{
    try
    {
        mojaFunkcja(wiadomosc);
    }
    catch (std::invalid_argument) {
        sprawdz(mojaFunkcja, "Zly argument");
    }
    catch (std::logic_error e) {
        sprawdz(mojaFunkcja, std::string(e.what()));
    }
    catch (std::exception e) {
        sprawdz(mojaFunkcja, "Blad: " + static_cast<std::string>(e.what()));
    }
    catch (...) {
        sprawdz(mojaFunkcja, "Nieznany blad");
    }
}

int main(void)
{
    sprawdz(menu, "hej");
}

Teraz jest ok? Czy wciąż nie o to chodzi?

---

Zawsze kojarzyłem wyjątki z czymś... wyjątkowym. Podanie błędnej opcji takie nie jest, nie jest również warte odwijania stosu. Co oznacza zwrot „wyjątkowym”? Są to sytuacje, z którymi nic nie możesz zrobić i tak naprawdę nie potrafisz poradzić sobie z wyjątkiem. Zgłaszaj wyjątki tylko w wyjątkowych sytuacjach. Nie nadużywaj ich do zarządzania normalnym przepływem sterowania w programie.

Co do std::function, jest to ładna, wysoko abstrakcyjna, z sporym narzutem, nakładka na wskaźnik do funkcji. https://en.cppreference.com/w/cpp/utility/functional/function Czy powinieneś martwić się wysokim narzutem oraz pewnym idiomem? Nie, na pewno nie w tym przypadku.

Osobiście nie widzę sensu, by sprawdzanie wyjątków wrzucić do osobnej funkcji.

---

@tkz, to jak byś kontrolował poprawność danych w takim przypadku jak tu:

std::string pytanie;
getline(std::cin, pytanie);
int opcja = stold(pytanie);

 

---

Ten stold jest kompletnie niepotrzebny, skoro i tak przypisujesz do inta. Zastanów się, czy naprawdę wybranie złej opcji chcesz karać wyjątkiem. Możesz nie robić nic, albo wypisać,że nie ma takiej opcji.

---

@tkz A co w przypadku gdy zamiast cyfry zostanie wpisana litera? Wtedy całość się chyba wysypie

---

Można to bardzo szybko naprawić. 

if(std::cin>>var)

Nie wiem za bardzo dlaczego chciałeś użyć getline. 

Osobiście rozbiłbym to na dwie funkcje. Jedną, która wczytuj wybór i same menu. 

---

Ok, dzięki za pomoc.

---

@Mavimix, chcesz, aby argumentem funkcji była inna funkcja? Już rozwiązałeś ten problem, czy potrzebujesz nadal pomocy?

---

Już rozwiązałem, sorry zapomniałem kłódkę kliknąć. Już klikam.