Programa con error

[Dasot]

Estoy aprendiendo C++ por mí mismo, y me he tropezado con esta piedra en mi camino. El programa funciona en sí mismo por un simple uso del switch, y una ciclo do... while, que debe reiniciar el programa si escribes "s" y cerrarlo si "n". El programa funciona bien sin el ciclo, así que me gustaria que me ayudaseis con el ciclo.



#include <iostream>
//Este programa tiene que preguntar unos datos y te dice una carta.
bool repetir(bool s, bool n, bool repetir)
{
char respuesta;
bool repetir;
using namespace std;
cout << "¿Quieres realizar más búsquedas?" << endl;
cin >> respuesta;
if(respuesta ==s)
repetir=true;
if(respuesta ==n)
repetir =false;

return repetir;
}

void main()
{
int palo;
int carta;
bool repeticion;


do
{
std::cout << "Introduce el palo" << std::endl << " 1-bastos\n 2-oros\n 3-copas\n 4-espadas\n";
std::cin >> palo;
std::cout << "\nIntroduce el numero de la carta\nDel 1 al 13\n";
std::cin >> carta;
std::cout << "\n";
switch ( carta )
{
case 1 : std::cout << "As de";break;
case 2 : std::cout << "Dos de";break;
case 3 : std::cout << "Tres de";break;
case 4 : std::cout << "Cuatro de";break;
case 5 : std::cout << "Cinco de";break;
case 6 : std::cout << "Seis de";break;
case 7 : std::cout << "Siete de";break;
case 8 : std::cout << "Ocho de";break;
case 9 : std::cout << "Nueve de";break;
case 10 : std::cout << "Diez de";break;
case 11 : std::cout << "Jota de";break;
case 12 : std::cout << "Reina de";break;
case 13 : std::cout << "Rey de";break;
default : std::cout << " Numero no existente";break;
}
switch ( palo )
{
case 1 : std::cout << " bastos.\n\n\n\n\n";break;
case 2 : std::cout << " oros.\n\n\n\n\n";break;
case 3 : std::cout << " copas.\n\n\n\n\n";break;
case 4 : std::cout << " espadas.\n\n\n\n\n";break;
default : std::cout << " Palo no existente\n";break;
}
bool repetir(bool s, bool n, bool repetir);
bool repeticion=repetir;



}
while(repeticion &= true);

}

[Heimish2000]

¿Por que acepta tres parámetros la función repetir? No necesitas ninguno.
La llamada a la función sería repeticion = repetir(); no como lo estás haciendo, ya que estas asignando el valor del parámetro repetir, pero no lo que devuelve la función.

[Gobix]

Opino lo mismo que el usuario de arriba. No necesitas ningún parametro en la función repetir, si te fijas no has usado ninguna de las funciones declaradas dentro de la función.

Estás llamando mal a la función, y usando demasiado el std:: esto lo simplificas escribiendo using namespace std; después de incluir iostream.

RESULTADO:

Código C++:

#include <iostream>
using namespace std;
//Este programa tiene que preguntar unos datos y te dice una carta.
bool repetir()
{
char respuesta;
bool repetir;
cout << "¿Quieres realizar más búsquedas?" << endl;
cin >> respuesta;
if(respuesta =='s')
repetir=true;
if(respuesta =='n')
repetir =false;
 
return repetir;
}
 
int main() //Te recomiendo que declares a main com int siempre
{
int palo;
int carta;
bool repeticion;
 
 
do
{
cout << "Introduce el palo" << endl << " 1-bastos\n 2-oros\n 3-copas\n 4-espadas\n";
cin >> palo;
cout << "\nIntroduce el numero de la carta\nDel 1 al 13\n";
cin >> carta;
cout << "\n";
switch ( carta )
{
case 1 : cout << "As de";break;
case 2 : cout << "Dos de";break;
case 3 : cout << "Tres de";break;
case 4 : cout << "Cuatro de";break;
case 5 : cout << "Cinco de";break;
case 6 : cout << "Seis de";break;
case 7 : cout << "Siete de";break;
case 8 : cout << "Ocho de";break;
case 9 : cout << "Nueve de";break;
case 10 : cout << "Diez de";break;
case 11 : cout << "Jota de";break;
case 12 : cout << "Reina de";break;
case 13 : cout << "Rey de";break;
default : cout << " Numero no existente";break;
}
switch ( palo )
{
case 1 : cout << " bastos.\n\n\n\n\n";break;
case 2 : cout << " oros.\n\n\n\n\n";break;
case 3 : cout << " copas.\n\n\n\n\n";break;
case 4 : cout << " espadas.\n\n\n\n\n";break;
default : cout << " Palo no existente\n";break;
}
repeticion=repetir();
}while(repeticion);
/* Si la función repetir devuelve true no es necesario decir repeticion == true, solo se pasa el valor y ya*/
 
}