user invalid input

[gabzz29]

hey guys im trying to make the user to loop only with the option of y/n only if ever the user pressed a number or another letter it would say invalid only choose between y/n
ln 123 to 131

#include <iostream> 
using namespace std;

void singlestring(){ //array()
    string name[5];

    cout << "Enter 5 Names: ";
    for (int i = 0; i < 5; i++){
        cin >> name[i];
    }

    for(int g = 0; g < 5; g++){
        cout << name[g];
    }
}

void singleint(){ //array()
    int num [3];

    cout << "Enter three numbers: ";
    for(int i = 0; i < 3; i++){
      cin >> num[i];
    }

    cout << "The Numbers are: ";
    for (int g = 0; g < 3; g++){
        cout << num[g];
    }
}

void concatonate(){
    string name[4];
    cout << "Enter your First Name: ";
    cin >> name[0];
    cout << "Enter your Middle Name: ";
    cin >> name[1];
    cout << "Enter your Surname: ";
    cin >> name[2];

    name[3] = name[0] + " " + name[1] + " " + name[2];

    cout << name[3];
}

void lenght(){
    string word;
    cout << "Enter a word ";
    while(!(cin >> word)){
        cout << "Integers are Invalid: ";
        cin .clear();
        cin.ignore(100 , '\n');
    }

    cout << "Word: \n" << word << endl;
    cout << "contains:" << word.length();
}

void changeletter(){
    char first[100],second;
    cout << "Enter a string: ";
    cin >> first;
    cout << "Change the First letter: ";
    cin >> second;

    first[0] = second;
    cout << first;
}

void array(){
    char choice;
    cout << "A.Single or B.Multi\n";
    cin >> choice;
    while(!(choice == 'a' || choice == 'A') && (choice == 'b' || choice == 'B')){
        cout << "Must choose only the choices above: ";
        cin >> choice;
        cin.clear();
        cin.ignore(100 , '\n');
    }

    if(choice == 'a' || choice == 'A'){
        char chc;
        cout << "A.Integer or B.String: ";
        cin >> chc;
        while(!(chc == 'a' || chc == 'A') && (chc == 'b' || chc == 'B')){
            cout << "Must choose only the choices above: ";
            cin >> chc;
            cin.clear();
            cin.ignore(100,'\n');
        }
        if(chc == 'a' || chc == 'A'){
           singleint();
        }else if(chc == 'b' || chc == 'B'){
           singlestring();
        }
    }
}

int main (){
    cout << "All CT-203\n";
    char choice;
    
    do{
        char opt;
        cout << "A.Concatonate\n";
        cout << "B.Lenghts of a string\n";
        cout << "C.Changing a Letter \n";
        cout << "D.Array\n";
        cin >> opt;

        while(!(opt == 'a' || opt == 'A') && (opt == 'B' || opt == 'B') && (opt == 'c' || opt == 'C') && (opt == 'd' || opt == 'D')){
            cout << "Only enter in the options above: ";
            cin.clear();
            cin.ignore(100, '\n');
            cin >> opt;
        }

        if(opt == 'A' || opt == 'a'){
            concatonate();
            }else if (opt == 'B' || opt == 'b'){
            lenght();
        }

        cout << "\nWould you like to make another transaction? ";
        cin >> choice;
        while(!(choice == 'y' || choice == 'Y') && (choice == 'n' || choice == 'N')){
            cout << "Must be only in the choices above\n Try again: ";
            cin.clear();
            cin.ignore(100, '\n');
            cin >> choice;
        }
        }while((choice == 'y' || choice == 'Y') && (choice == 'n' || choice == 'N'));
        
}
   

[seeplus]

The logic on L125 is not right. Consider:

while (!(choice == 'y' || choice == 'Y'|| choice == 'n' || choice == 'N')) {

What is L131 supposed to do? Do you mean:

} while (choice == 'y' || choice == 'Y');

[gabzz29]

now it works thank you