Why does my Do While loop terminate?

Question

I am trying to implement a server with multiple commands. Most commands work so far except that I want the server to send a warning to the client when the client writes an argument to the quit command. (IE quit xyz) and lets the user try again rather than exiting the server. Unfortunately the server quits whether the user types quit or quit arguments.

bool done = false;

do
{
    if(strcmp(cmd, "quit") == 0)
    {
        if(strcmp(argument, "") != 0)
            strcpy(replyMsg, "504 Command not implemented for that parameter.\n");
        else
        {
            strcpy(replyMsg,"221 Service closing control connection.\n");
            done = true;
        }
    }
    while(strcmp(cmd, "quit") != 0 && done != true);

Answer 1

I would just use:

...
}
while(!done);

There is no point checking cmd .

Answer 2

Your loop structure seems missing brace before while to be wrong, or it is a typo. Check it first and lets see the correct code.

Answer 3

Remove your test on the cmd in the while condition. Whenever he user chooses 'quit' it evaluates to false, so the whole expression is false due to the conjunction and the loop exits