неверный счет с использованием strstr

Question

Im пытается написать функцию, которая возвращает индекс первого появления подстроки внутри строки.

как поиск небоскреба для «СМ» вернется 2.

Похоже, strstr делают свою работу, потому что после того, как время цикла она выводит правильный первый символ подстроки, но оно не считая правильно и возвращение правильного внутр.

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

Я все еще довольно новичок в указателях и передаю их в качестве аргументов, поэтому там может быть проблема.

все, что угодно!

int findSubstring(char *s, char substring[]) 
{ 
    int j = 1; 
    char* strPtr = NULL; 

    while (strPtr == NULL) 
    { 
     strPtr = strstr(s, substring); 
     if (strPtr != NULL) 
      break; 
     j++; 
    } 

    cout << *strPtr << endl; 
    cout << "The substring begins at element: " << j << endl; 
    return j; 
} 

Answer 1

Как насчет использования только указательной арифметики?

int findSubstring(char *s, char substring[]) 
{ 
    char* strPtr = strstr(s, substring); 
    if (strPtr != NULL) 
    { 
     return (int)(strPtr - s); 
    } 
    else 
    { 
     return -1; //no match 
    } 
} 

Answer 2

Вы, кажется, более усложняет задачу, так как вы используете C++ вы должны использовать std::string::find.

std::string s = "skyscraper"; 
std::size_t pos = s.find("ysc"); 
if(pos != std::string::npos) 
    std::cout << "ysc found at " << pos << "\n"; 
else 
    std::cout << "ysc not found" << "\n"; 

Answer 3

Fix ваш, как, обратитесь this example:

int findSubstring(char *s, char substring[]) 
{ 
    char* pos = strstr(s, substring); 
    if(pos) 
    return (int)(pos - s); 
    else -1; 
} 

И вы используете C++, поэтому, почему бы не std::string?

int findSubstring(const std::string& s, const std::string& substring) 
{ 

std::size_t j =s.find(substring); 
return (j!=std::string::npos) ? j :-1; 
}