程序沒有執行內容正確

Question

我有我的地方，我無法獲得if (sequence[i]!='M' && sequence[i]!='U' && sequence[i]!='I')作品的以下功能麻煩..

基本上，我計劃在此過程中的工作 -

1. 輸入在輸入
2. 輸入在字符序列的大小，其中字符只能是M，U或I

例如。如果我輸入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指針的副本，讓你還有一個指向字符串的開頭，當你與計數做while循環：

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; 
}