为什么我的“while”循环没有结束？ 是由于运算符优先级还是其他原因？

Question

Why is my while loop not ending? It is giving the correct result, but it runs in an infinite loop. Is it due to operator precedence?

void equate()
{
    int i = 0, n = 0;
    while (((a[i] != '\0') && (b[i] != '\0')) || (n != 1))
    {
        if (a[i] > b[i])
        {
            std::cout << "\n" << a << " string is greater";
            n = 1;
        }
        else if (a[i] < b[i])
        {
            std::cout << "\n" << b << " string is greater";
            n = 1;
        }
        else
            i++;
    }
    if (n == 0)
    {
        std::cout << "\n" << "Both strings " << a << " and " << b << " are equal";

    }

Answer 1

Within these if statements

   if (a[i] > b[i])
   {
       std::cout << "\n" << a << " string is greater";
       n = 1;
   }
   else if (a[i] < b[i])
   {
       std::cout << "\n" << b << " string is greater";
       n = 1;
   }

the index i is not being incremented. So the first subexpression in the condition of the while loop

((a[i] != '\0') && (b[i] != '\0'))

is always evaluates to true if the control was passed to these if statements. As a result you will have an infinite loop.

At least rewrite the condition like

while ( a[i] != '\0' && b[i] != '\0' && n != 1 )

Though the result of the loop (of the comparison of two strings) can be incorrect in the case when lengths of compared strings are unequal each other.

That is, it is not enough only to check whether n is equal to 0 as you are doing in this statement.

   if (n == 0)
   {
       std::cout << "\n" << "Both strings " << a << " and " << b << " are equal"; 
   }