Подправить небольшой код на С++

[MaxBrat]

У меня есть код на С++ который находит подстроку в строке, и мне его нужно переделать так что бы в нём использовалась библиотека STL – find(), find_if(), find_first(), find_end(). Заранее благодарю)).

#include <iostream>
using namespace std;

int Direct_search(const char* text,const char* search_word) {
    int string, substring; 
    int res = -1;
    string = strlen(text);  // присваивание в переменные размер всего текста и заданного слова
    substring = strlen(search_word);
    if (string == 0) {  // проверки на то был ли задан какой-то текст и слово
        cout << "Неверно заданная строка\n";
    }
    else if (substring == 0) {
        cout << "Неверно заданная подстрока\n";
    }
    else {  // метод прямого поиска
        for (int i = 0; i < string - substring + 1; i++) {
            for (int j = 0; j < substring; j++) {
                if (search_word[j] != text[i + j]) { // посимвольная проверка заданного слова с текстом
                    break;
                }
                else if (j == substring - 1) {
                    res = i;
                    break;
                }
            }
        }
    }
    return res;
}

int main() {
    setlocale(LC_ALL, "Russian");
    const char* text = "A graphics processing unit (GPU) is a specialized electronic circuit designed to rapidly manipulate and alter memory to accelerate the creation of images in a frame buffer intended for output to a display device. GPUs are used in embedded systems, mobile phones, personal computers, workstations, and game consoles.";
    const char *search_word = "unit";    // слово которое нужно найти в тексте
    int index = Direct_search(text, search_word);   // вызов функции прямого поиска
    cout << "Индекс первого вхождения заданного слова: " << index;
}

[macomics]

Если не хотите оформлять код правильно, тогда разбирайтесь сами. Вот тут все написано.

И не надо плодить одинаковые темы.

[MaxBrat]

Сообщение от macomics

Если не хотите оформлять код правильно, тогда разбирайтесь сами. Вот тут все написано.

И не надо плодить одинаковые темы.

Вот всё исправил.

[Пётр Седов]

MaxBrat, у класса std::string есть метод поиска подстроки:

#include <iostream>
#include <string>
#include <locale.h>

using namespace std;

int main() {
    setlocale(LC_ALL, "Russian");
    string text = "A graphics processing unit (GPU) is a specialized electronic circuit designed to rapidly manipulate and alter memory to accelerate the creation of images in a frame buffer intended for output to a display device. GPUs are used in embedded systems, mobile phones, personal computers, workstations, and game consoles.";
    const char *search_word = "unit";    // слово, которое нужно найти в тексте
    size_t pos = text.find(search_word); // вызов функции прямого поиска
    if (pos != string::npos) {
        cout << "Индекс первого вхождения заданного слова: " << pos;
    }
    return 0;
}

В новых компиляторах ещё можно использовать класс std::string_view:

#include <iostream>
#include <string_view>
#include <locale.h>

using namespace std;

int main() {
    setlocale(LC_ALL, "Russian");
    string_view text = "A graphics processing unit (GPU) is a specialized electronic circuit designed to rapidly manipulate and alter memory to accelerate the creation of images in a frame buffer intended for output to a display device. GPUs are used in embedded systems, mobile phones, personal computers, workstations, and game consoles.";
    const char *search_word = "unit";    // слово, которое нужно найти в тексте
    size_t pos = text.find(search_word); // вызов функции прямого поиска
    if (pos != string_view::npos) {
        cout << "Индекс первого вхождения заданного слова: " << pos;
    }
    return 0;
}

[macomics]

Сообщение от MaxBrat

Вот всё исправил.

BDA не забудь сказать, что сам исправил. Он пирожком угостит.

Сообщение от Пётр Седов

MaxBrat, у класса std::string есть метод поиска подстроки:

Есть, а он сформулировал, что по заданию надо использовать find() и иже с ним.

Сообщение от MaxBrat

else if (j == substring - 1) {
                    res = i;
                    break;
                }

Вот тут вы прерываете вложенный цикл, а как же внешний? Иначе вы ищите последнее вхождение подстроки в строку.

[Пётр Седов]

Сообщение от macomics

Есть, а он сформулировал, что по заданию надо использовать find() и иже с ним.

Функция std::find здесь не подходит, потому что она ищет один элемент, а нам надо найти подстроку. Если по каким-то причинам не хочется использовать классы std::string/std::string_view, то тогда уж надо использовать функцию std::search:

#include <iostream>
#include <algorithm>
#include <locale.h>
#include <string.h>

using namespace std;

int main() {
    setlocale(LC_ALL, "Russian");
    const char* text = "A graphics processing unit (GPU) is a specialized electronic circuit designed to rapidly manipulate and alter memory to accelerate the creation of images in a frame buffer intended for output to a display device. GPUs are used in embedded systems, mobile phones, personal computers, workstations, and game consoles.";
    const char *search_word = "unit";    // слово, которое нужно найти в тексте
    const char* text_end = text + strlen(text);
    const char* word_in_text = search(text, text_end, search_word, search_word + strlen(search_word)); // вызов функции прямого поиска
    if (word_in_text != text_end) {
        cout << "Индекс первого вхождения заданного слова: " << (word_in_text - text);
    }
    return 0;
}

[MaxBrat]

[QUOTE=Пётр Седов;1853481]Функция std::find здесь не подходит, потому что она ищет один элемент, а нам надо найти подстроку. Если по каким-то причинам не хочется использовать классы std::string/std::string_view, то тогда уж надо использовать функцию std::search:

#include <iostream>
#include <algorithm>
#include <locale.h>
#include <string.h>

using namespace std;

int main() {
    setlocale(LC_ALL, "Russian");
    const char* text = "A graphics processing unit (GPU) is a specialized electronic circuit designed to rapidly manipulate and alter memory to accelerate the creation of images in a frame buffer intended for output to a display device. GPUs are used in embedded systems, mobile phones, personal computers, workstations, and game consoles.";
    const char *search_word = "unit";    // слово, которое нужно найти в тексте
    const char* text_end = text + strlen(text);
    const char* word_in_text = search(text, text_end, search_word, search_word + strlen(search_word)); // вызов функции прямого поиска
    if (word_in_text != text_end) {
        cout << "Индекс первого вхождения заданного слова: " << (word_in_text - text);
    }
    return 0;
}

А нельзя както чтобы програма делала то же самое но при этом было использовано STL – find(), find_if(), find_first(), find_end()?

[Пётр Седов]

MaxBrat, так функция std::search -- тоже часть STL (как и классы std::string/std::string_view). Вместо неё можно использовать функцию std::find_end, но тогда это будет уже поиск последнего вхождения подстроки. Функции std::find/std::find_if/std::find_first_of ищут один элемент, а не подпоследовательность, и они здесь не нужны.

[MaxBrat]

Сообщение от Пётр Седов

MaxBrat, так функция std::search -- тоже часть STL (как и классы std::string/std::string_view). Вместо неё можно использовать функцию std::find_end, но тогда это будет уже поиск последнего вхождения подстроки. Функции std::find/std::find_if/std::find_first_of ищут один элемент, а не подпоследовательность, и они здесь не нужны.

А это как получиться в коде?

[MaxBrat]

Сообщение от Пётр Седов

MaxBrat, так функция std::search -- тоже часть STL (как и классы std::string/std::string_view). Вместо неё можно использовать функцию std::find_end, но тогда это будет уже поиск последнего вхождения подстроки. Функции std::find/std::find_if/std::find_first_of ищут один элемент, а не подпоследовательность, и они здесь не нужны.

Мне как раз нужно чтобы было std::find_end и std::find/std::find_if/std::find_first. А как это поменять чтобы как раз з ними был код, можете показать пример?