Странное поведение кода

Question

Я видел, что часть моего кода не работает должным образом. Он не входит в инструкцию if и дает ошибку, пока это необходимо. Я попытался изучить код и нашел что-то очень странное. Когда я добавляю cout << i в цикле for, чтобы проверить, он начинает работать хорошо. Может ли кто-нибудь объяснить, в чем проблема? Примечание. airports - это vector объектов Аэропорта. Airport::getName() возвращает свое имя string.

string name = "smth"; 
//this is the loop with unexpected behaviour 
for (int i = 0; i < airports.size(); i++) 
{ 
    //when the following line is taken outside the comment, it works well: 
    //cout << i; 
    if (isEqualNoCaseSense(name, airports[i].getName())) 
    { 
     cout << "Could not create airport " << name << ". It already exists." << endl; 
     return; 
    } 
} 

Функция I используется для сравнения строк без учета регистра:

bool isEqualNoCaseSense(string str1, string str2) 
{ 
if (str1.size() != str2.size()) 
    return false; 
char *lowerStr1 = new char[str1.size()]; 
char *lowerStr2 = new char[str2.size()]; 
for (int i = 0; i < str1.size(); i++) 
{ 
    lowerStr1[i] = tolower(str1[i]); 
    lowerStr2[i] = tolower(str2[i]); 
} 
if (strcmp(lowerStr1, lowerStr2) == 0) 
{ 
    delete[] lowerStr1; 
    delete[] lowerStr2; 
    return true; 
} 
else 
{ 
    delete[] lowerStr1; 
    delete[] lowerStr2; 
    return false; 
} 
} 

EDIT: Коррекция функции сравнения фиксирует ее, но я до сих пор удивляюсь, почему cout << i установил ее

Answer 1

Проблема заключается в том, что вы не имеете прекращено ваши строки, но если вы просто написать функцию толковой не будет работать правильно:

bool isEqualNoCaseSense(const string& str1, const string& str2) 
{  
    if (str1.size() != str2.size()) 
     return false; 
    for (int i = 0; i < str1.size(); ++i) 
     if (tolower((unsigned char)str1[i]) != tolower((unsigned char)str2[i])) 
      return false; 
    return true; 
} 

нет ненужное выделение, и петля только через строку до первого несовпадения.

EDIT: исправляющие функцию сравнения фиксирует ее, но я до сих пор удивляюсь, почему cout << i установил ее

Поскольку программа имела неопределенное поведение, так что все может случиться. Добавление случайного, несвязанного кода может изменить шаблоны распределения памяти для кода или заставить компилятор оптимизировать его несколько иначе. Вы сойдете с ума, пытаясь рассуждать о некоторых типах неопределенного поведения. Просто избегайте этого.

Answer 2

strcmp ОЖИДАЕТ Строки с нулевым завершением. Поэтому вам нужно зарезервировать еще один символ для массивов lowerStr1, lowerStr2 и установить последний символ каждого массива в NULL. В противном случае создайте неопределенный бахавир для своей программы.

Но вы могли бы обойти все эти вопросы, если вы используете функцию strncmp проходящее str1.size() в качестве третьего аргумента, так как вы уже проверить, если две строки имеют одинаковый размер:

if (strncmp(lowerStr1, lowerStr2, str1.size()) == 0)