Программа не выполняется содержание правильно

Question

у меня возникли проблемы со следующей функцией шахты, где я не смог получить if (sequence[i]!='M' && sequence[i]!='U' && sequence[i]!='I') работы ..

В основном, мои программы работы в этой процедуре -

1. Введите в размере входа
2. Ввод в последовательности символов, где символы могут быть только М, U или я

Например. Если я вхожу в 3 в качестве моего размера ввода, а затем MUUI, он будет печататься как недопустимый ввод и запрашивать новый ввод, пока не будет введено что-то вроде MUI.

Однако, если я вхожу в 3 как мой размер входного, а затем MPO как последовательность символов, программа будет продолжать работать линию if (sequence[i]!='M' && sequence[i]!='U' && sequence[i]!='I'), даже если это не так.

И если бы я попытался переставляя свои строки, в то время как входные символы правильно на этот раз, размер будет некорректным, и программа будет продолжать выполнять ..

Могу ли я делать это не так-то?

int main(){ 
int sz; 
cout<<"Enter in size of input: "; 
cin>>sz; 
char *sequence = new char[sz]; 
cout<<"Enter MU character sequence(M/U/I): "; 
cin>>sequence; 

bool checkMUI =isMUString(sequence, sz); 
while(checkMUI== false){ 
    cout<<"INVALID INPUT. RE-Enter MU character sequence(M/U/I): "; 
    cin>>sequence; 
    checkMUI =isMUString(sequence, sz); 
} 

bool isMUString(char* sequence, int size){ 

    int numberOfChars = 0; 
    while (*sequence != '\0') 
    { 
     numberOfChars++; sequence++; 
    } 

    for(int i=0;i<size;i++){ 
     if (numberOfChars != size){ 
      if (sequence[i]!='M' && sequence[i]!='U' && sequence[i]!='I'){ 
       return false; 
      } 
     } 
    } 
    return true; 

Answer 1

Я думаю, что это то, что вы хотите:

bool isMUString (char* sequence, int size) 
{ 

    int numberOfChars = 0; 
    while (sequence[numberOfChars] != '\0') 
    { 
     numberOfChars++; 
    } 

    if (numberOfChars != size) 
    { 
     return false; 
    } 

    for (int i = 0; i < size; i++) 
    { 
     if (sequence[i] != 'M' && sequence[i] != 'U' && sequence[i] != 'I') 
     { 
      return false; 
     } 
    } 

    return true; 
} 

Answer 2

Это должно работать:

bool isMUString(const char* sequence, int size) { 
    int numberOfChars = strlen(sequence); 
    if(numberOfChars != size) { 
     return false; 
    } 

    for(int i=0;i<size;i++){ 
     if (sequence[i]!='M' && sequence[i]!='U' && sequence[i]!='I'){ 
      return false; 
     } 
    } 
    return true; 
} 

Вот что я изменил:

• Использование StrLen для определения длины строки
• Возьмите if(numberOfChars != size) из корпуса цикла for, потому что его нужно только один раз проверить.

Вот пример, если вы не хотите использовать strlen() (все, что вам нужно). Я делаю копию sequence указателя, так что у вас еще есть указатель на начало строки, когда вы закончите с подсчетом в то время как цикл:

bool isMUString(const char* sequence, int size) { 
    int numberOfChars = 0; 
    const char* ptr = sequence; 
    while (*ptr != '\0') { 
     numberOfChars++; ptr++; 
    } 

    if(numberOfChars != size) { 
     return false; 
    } 

    for(int i=0;i<size;i++){ 
     if (sequence[i]!='M' && sequence[i]!='U' && sequence[i]!='I'){ 
      return false; 
     } 
    } 
    return true; 
}