how to loop an if statment

There's a few things wrong with your code, unless I've really misunderstood what you're up to (which is possible...).

The first one is that when you write:

void aChoice();

or

void pChoice();

you're prototyping the functions, not calling them. If you really want to call them you need to knock the void off the front.

I'd assume you mean aChoice in the second conditional statement as well. While the code you've got may compile (not sure of the rules for converting function pointers to arbitrary pointers) what it looks like you're doing is comparing the address of the function pChoice with a couple of characters. It might be worth cranking up your compiler warnings to the top level (/W4 /WX on Visual C++) as that might give you some more guidance.

Now, having sorted that out... What it looks like you want to do is loop around while the user hasn't entered what you expected him or her to, otherwise you want to execute one of the functions and bail out of the program. If you don't want to bail out then you don't want to do what I'm going to suggest!!

So how do you loop until the input is what you expect? I'd be tempted to use a while loop (a do/while might actually work better, I'm just a bit adverse to them for no logical reason I can put my finger on). The conditional for the while loop is similar to the two conditions you check later:

char aChoice = ' ';

while( ( aChoice != 'A' ) && ( aChoice != 'a' ) && ( aChoice != 'P' ) && ( aChoice != 'p' ) )
{
    // Code as before in here...
}

Now while this code is okay, it's a bit of a bugger to read. My eyes when crosseyed while reading it. You might want to look at hiding the conditionals in other functions to make it easier to read. Another technique you could look at might be a jump table but that's probably overkill for this sort of problem.

Cheers,

Ash

To do what you want, you need to put your above code into a method, and have the method call itself when want to loop. This is called recursion.

I'd question the need for this practically, you'd be much better off with a do/while or while loop in all probability.

I think that the method which mentioned by previous member is so difficult dear because the code has just one loop with if statement and it's not complex enough to require recursion state...

I think that the loop statement of while or do/while is the suitable case in order to solve this code with all required probabilities.

A set of statements could be executed multiple times either by using recursion or iteration (for. do/while and while loops). Iteration is faster in most cases. Recursion would cause a function to be called again and again. As function calls use up your stack you would eventually end up with a stack overflow if the process continues too long. Iteration handles larger number of repeats better than recursion.