Почему зацикливается программа на C++?

    while ((maxs = (S.find(max2, i+1)) != -1) && ((i+1)<S.size())) {
        count++;
        i=maxs;
    }

вот этот кусок

(maxs = (S.find(max2, i+1)) != -1)

означает следующее:

1. найти строку max2 в строке S

2. сравнить позицию c -1

3. присвоить maxs результат сравнения (т.е. 0 если false или 1 если true)

значит если в строке присутствует подстрока дальше 1 символа, вы постоянно будете находить эту подстроку (поиск начинается с 1 символа), получать maxs = 1

... и искать и постоянного находить эту подстроку никуда по строке не сдвигаясь

подозреваю, что правильный код должен был бы быть таким:

while (
          ((maxs = S.find(max2, i+1)) != -1) && 
          (i + 1 < S.size()
      )