Сбой функции C++ isMember

Question

Я делаю программу, которая проверит существование строки в массиве строк. Я сделал простую функцию, чтобы сделать так:

bool isMemberOfArrayStr(std::string str, std::string array[256]){ 
    for(int i=0;array->length()<10;i++){ 
    if(array[i]==str){ 
     ret=true;//A global variable that will be reset to false after the function call 
    } 
    } 
    return ret; 
} 

Определение функции не вызывает никаких ошибок, но вызов:

if(neighbors[2] == isMemberOfArrayStr(neighbors[2], validTokens)) 
{ 
    std::cout <<"true" 
} 

Вызывает ошибку во время выполнения «неисправность Сегментация: 11». Я не уверен, в чем проблема, и помощь будет приятной.

Answer 1

bool isMemberOfArrayStr(std::string str, std::string array[256]) 

распадается на

bool isMemberOfArrayStr(std::string str, std::string* array) 

Правильный прототип

bool isMemberOfArrayStr(std::string str, std::string (&array)[256]) 

но вы можете использовать std::array или std::vector, которые имеют лучшую (более интуитивный) синтаксис.

Тогда array->length() не 256, а длина первой строки массива.

Ваша функция может быть переписана (C++ 11):

bool isMemberOfArrayStr(const std::string& str, std::string (&array)[256]) { 
    ret = std::find(std::begin(array), std::end(array), str) != std::end(array); 
    return ret; 
} 

C++ 03:

bool isMemberOfArrayStr(const std::string& str, std::string (&array)[256]) { 
    ret = std::find(array, array + 256, str) != array + 256; 
    return ret; 
} 

И я думаю, вы должны удалить ret от этой функции в любом случае.

Answer 2

array->length(); 

Не имеет длины массива. Это длина первой std :: string в массиве. Рассмотрите возможность использования std :: vector of std :: string.

Если заказ не используется в контейнере, лучше выбрать std :: set (multiset).

Answer 3

bool isMemberOfArrayStr(std::string str, std::string array[256]){ 
    for(int i=0;array->length()<10;i++){ 
        ^^^^^^^^^^ 
    if(array[i]==str){ 
     ret=true;//A global variable that will be reset to false after the function call 
    } 
    } 
    return ret; 
} 

Вы сравниваете постоянное значение. Вы выполняете цикл, только если array-> length() < 10, а затем выполняете его бесконечно много раз. Также

array->length() 

- длина первого элемента (если есть!) В массиве.