С Новым годом! Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Карта форума Темы раздела Блоги Сообщество Поиск Заказать работу  
 
 
Рейтинг 4.93/15: Рейтинг темы: голосов - 15, средняя оценка - 4.93
0 / 0 / 0
Регистрация: 18.05.2014
Сообщений: 48
1

C++ strtok_s ломается в конце строки

29.10.2015, 14:58. Показов 3122. Ответов 20
Метки нет (Все метки)

Author24 — интернет-сервис помощи студентам
Здравствуйте уважаемые программисты, у меня проблема следующего содержания. Когда отдаю строку из файла, т.е. с \n и \0 он на последнем слове ломается. Нужно читать именно через костыли FILE и f(функция для строк). Общий алгоритм, я должен переписать строки из файла в файл, в которых нет одинаковых слов. Я беру строку, хватаю первое слово, и ищу его в строке, потом второе и т.д. если нашел увеличил счетчик. Компилятор именно Майкрософтовский, потому обязан использовать функции_s включая strtok_s.
И когда он доходит до конца строки, т.е. до последнего слова выкидывает ошибку.

Что не так? Вот код:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
char str[1000];
    char * pch;
    char strBufer[1000];
    //пошли по первому файлу
    FILE *fRead;
    FILE *fWrite;
    fopen_s(&fWrite, "F2.txt", "w");
    fopen_s(&fRead, "F1.txt", "r");
    if (fRead == NULL) perror("Ошибка открытия файла");
    else
    {
        char * content;
        while (fgets(str, 1000, fRead))
        {
            int kol = 0;
            //str[strlen(str)-1] = ' ';
            strcpy_s(strBufer, str); // скопировали строку для записи если сработает
            //cout << strBufer << endl;
            pch = strtok_s(str, " ", &content);
            while (pch != NULL)
            {
                    pch = strtok_s(NULL, " ", &content);
                    if (strstr(str, pch) != NULL) kol++;
                    cout << pch << " ";
            }
            cout << endl;
            if (kol == 0)
                fputs(strBufer, fWrite);
        }
 
        fclose(fRead);
        fclose(fWrite);
    }
0
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
29.10.2015, 14:58
Ответы с готовыми решениями:

В задаче при вводе строки sale все ломается
В стране дефицит чугунных болванок. Их продают на электронных торгах. Покупатели заявляют цену, по...

Строки. Если в конце строки точка отсутствует, то найти слово, заканчивающееся точкой и перенести его в конец строки.
Дана строка, состоящая из слов и содержащая одну точку в конце одного из слов. Если в конце строки...

Файл: Создайте текстовый файл, содержащий в начале каждой строки гласные буквы соответствующей строки файла, а в конце строки - согласные
Создайте текстовый файл, содержащий в начале каждой строки гласные буквы соответствующей строки...

Переписать все строки файла удаляя пробелы в конце каждой строки
Дан текстовый файл F.txt. Переписать все строки файла F.txt в файл G.txt, удаляя все пробелы в...

20
Эксперт PHP
3106 / 2591 / 1219
Регистрация: 14.05.2014
Сообщений: 7,236
Записей в блоге: 1
29.10.2015, 15:07 2
Цитата Сообщение от nxexox Посмотреть сообщение
C++
1
2
pch = strtok_s(NULL, " ", &content);
if (strstr(str, pch) != NULL) kol++;
как функция strstr отнесется к тому, что ей будет передан NULL, вместо указателя на строку
1
0 / 0 / 0
Регистрация: 18.05.2014
Сообщений: 48
29.10.2015, 15:10  [ТС] 3
Хм. не совсем понял. У меня в
C++
1
pch
же лежит слово, т.е. текущее. Разве нет?
0
Эксперт PHP
3106 / 2591 / 1219
Регистрация: 14.05.2014
Сообщений: 7,236
Записей в блоге: 1
29.10.2015, 15:14 4
Лучший ответ Сообщение было отмечено nxexox как решение

Решение

Да и к тому же вы пропускаете первое слово, выделенное из строки, можету лучше так
C++
1
2
3
if (strstr(str, pch) != NULL) kol++;
    cout << pch << " ";
pch = strtok_s(NULL, " ", &content);
Добавлено через 4 минуты
nxexox,
C++
1
2
3
4
pch = strtok_s(str, " ", &content);
while (pch != NULL)
{
        pch = strtok_s(NULL, " ", &content);
у вас сначала производится "изъятие" совпадения, затем проверка на вхождения, а уже после всего этого проверка на NULL. А проверка на NULL должна предварять любые действия с указателем pch.
1
0 / 0 / 0
Регистрация: 18.05.2014
Сообщений: 48
29.10.2015, 15:21  [ТС] 5
Я совсем запутался. Спасибо, вот так заработало.
Да и к тому же вы пропускаете первое слово, выделенное из строки, можету лучше так
C++
1
2
3
if (strstr(str, pch) != NULL) kol++;
    cout << pch << " ";
pch = strtok_s(NULL, " ", &content);
Но только в pch он хранит теперь строку. Т.е. нифига не ищет и файл F2 на выходе пустой. Я не совсем понимаю видимо как эта система работает. Разъясните пожалуйста
0
7803 / 6567 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
29.10.2015, 15:52 6
Лучший ответ Сообщение было отмечено nxexox как решение

Решение

strtok() изменяет строку, вставляя '\0', поэтому искать там что-то бесполезно. Искать в резервной strBufer можно, но надо учитывать, что подстрока там как минимум 1 раз встречается, а strstr() ищет первое вхождение. К тому же слово твоё может быть не само по себе, а в составе сложного слова, это strstr() не учитывает.
По-правильному тебе нужно разделить строку на слова и сохранить их в массив, после чего искать повторы по массиву.
1
0 / 0 / 0
Регистрация: 18.05.2014
Сообщений: 48
29.10.2015, 16:02  [ТС] 7
Цитата Сообщение от nmcf Посмотреть сообщение
strtok() изменяет строку, вставляя '\0', поэтому искать там что-то бесполезно. Искать в резервной strBufer можно, но надо учитывать, что подстрока там как минимум 1 раз встречается, а strstr() ищет первое вхождение. К тому же слово твоё может быть не само по себе, а в составе сложного слова, это strstr() не учитывает.
По-правильному тебе нужно разделить строку на слова и сохранить их в массив, после чего искать повторы по массиву.
А как это сделать? Можно пример кода?
0
Эксперт PHP
3106 / 2591 / 1219
Регистрация: 14.05.2014
Сообщений: 7,236
Записей в блоге: 1
29.10.2015, 16:04 8
nxexox, вы это
C++
1
2
3
4
5
6
while (pch != NULL)
{
    pch = strtok_s(NULL, " ", &content);
    if (strstr(str, pch) != NULL) kol++;
    cout << pch << " ";
}
заменили на это
C++
1
2
3
4
5
6
while (pch != NULL)
{
    if (strstr(str, pch) != NULL) kol++;
    cout << pch << " ";
    pch = strtok_s(NULL, " ", &content);
}
???
1
0 / 0 / 0
Регистрация: 18.05.2014
Сообщений: 48
29.10.2015, 16:07  [ТС] 9
Цитата Сообщение от Kerry_Jr Посмотреть сообщение
nxexox, вы это
C++
1
2
3
4
5
6
while (pch != NULL)
{
 pch = strtok_s(NULL, " ", &content);
 if (strstr(str, pch) != NULL) kol++;
 cout << pch << " ";
}
заменили на это
C++
1
2
3
4
5
6
while (pch != NULL)
{
 if (strstr(str, pch) != NULL) kol++;
 cout << pch << " ";
 pch = strtok_s(NULL, " ", &content);
}
???
Заменил. И теперь он не ищет слова. Но не ломается.
0
7803 / 6567 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
29.10.2015, 16:21 10
Цитата Сообщение от nxexox Посмотреть сообщение
А как это сделать?
Ну берёшь за основу свой цикл с strtok(), только вместо strstr() записываешь слово в массив.
1
0 / 0 / 0
Регистрация: 18.05.2014
Сообщений: 48
29.10.2015, 16:55  [ТС] 11
А, и просто от начала массива по убывающей сравнивать с нижними элементами массива.
ок. Попробую сделать. Спасибо!)

Добавлено через 30 минут
Швыряет ошибку при работе, и аварийно завершается.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
bool povtorSlov(char stroka[1000], int const kolSl = 15)
{
    char * pch;
    char *content;
    char slova[15][50];
    pch = strtok_s(stroka, " ", &content);
    //char BUF[kolSlov(str)]
    int i = 0;
    while (pch != NULL)
    {
        pch = strtok_s(NULL, " ", &content);
        //slova[i] = pch;
        strcpy_s(slova[i], pch);
        cout << slova[i] << " | ";
        i++;
    }
    cout << endl;
    bool flag = false;
    for (int j = 0; j < kolSl; j++)
    {
        for (int q = j + 1; q < kolSl; q++)
        {
            if (slova[j] == slova[q]) flag = true;
        }
    }
    return flag;
}
Что в этот раз не так, помогите пожалуйста
0
7803 / 6567 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
29.10.2015, 17:07 12
Лучший ответ Сообщение было отмечено nxexox как решение

Решение

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
bool povtorSlov(char stroka[1000])
{
    char * pch;
    char *content;
    char slova[200][1000];
 
    int i = 0;
    pch = strtok_s(stroka, " ", &content);
    while (pch != NULL)
    {
        strcpy_s(slova[i], pch);
        cout << slova[i] << " | ";
        ++i;
 
        pch = strtok_s(NULL, " ", &content);
    }
    cout << endl;
 
    for (int j = 0; j < i; j++)
    {
        int count = 0;
        for (int q = j + 1; q < i; q++)
            if (strcmp(slova[j], slova[q]) == 0) ++count;
        if (count > 1) return false;
    }
    return true;
}
Ты снова не туда strtok() ставишь - в цикле всегда должна быть в конце.
Сравнение древних строк - функцией.
1
0 / 0 / 0
Регистрация: 18.05.2014
Сообщений: 48
29.10.2015, 22:02  [ТС] 13
спасибо!)) моя невнимательность

Добавлено через 3 часа 31 минуту
Тема пока не закрыта! Не могу понять, он почему то копирует все строки. Без разбора. Хотя в pch нормально кладутся слова.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
bool povtorSlov(char stroka[1000]) // проверяет есть ли в строке одинаковые слова
{
    char * pch; // для каждого слова
    char *content; // для функции strtok_s, указатель на остаток строки, когда разделять будем
    char slova[200][1000]; // предполагаем что слов может быть не больше 200. Создаем массив, где каждая строка это слово
 
    int i = 0;// ставим счетчик, что бы запомнить сколько слов мы нашли
    pch = strtok_s(stroka, " ", &content);//взяли первое слово и положили в pch
    while (pch != NULL)// пока есть слова в строке будем делать
    {
        strcpy_s(slova[i], pch);// копируем в наш массив слов, слово
        //cout << slova[i] << " | ";
        ++i;//увеличиваем счетчик кол-ва слов
        cout << pch << " |\n";
        pch = strtok_s(NULL, " ", &content);//берем следующее слово из строки
    }
    cout << endl;
 
    //после того как сформировали массив, мы начинаем искать повторяющиеся слова в нем
    for (int j = 0; j < i; j++)//идем от 0 до кол-ва слов, которое запомнили, до i
    {
        int count = 0;// счетчик повторений
        for (int q = j + 1; q < i; q++)// запускаем цикл от текущего слова до конца массива слов
            // в цикле проверяем, сравниваем текущее слово, со всеми следующими
            //и если нашли, то увеличиваем счетчик повторений
            if (strcmp(slova[j], slova[q]) == 0) ++count; 
        if (count > 1) return false; // если не нашли ни одного повторения, то возвращаем false
    }
    //если повторения нашлись, то мы вернем true
    return true;
}
0
7803 / 6567 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
29.10.2015, 22:19 14
Ты неправильно интерпретируешь результат: false означает, что повторы есть.
Поменяй тогда false и true местами.
0
0 / 0 / 0
Регистрация: 18.05.2014
Сообщений: 48
29.10.2015, 22:40  [ТС] 15
Я пробовал. но увы, результат правильно интерпретирую. Проверял на таких входных:
mama lupila papu
papa lox
maam toge balbes
mama papka bla
pup pup aga

он либо копирует все, либо при замене true и false не копирует ничего
0
Эксперт PHP
3106 / 2591 / 1219
Регистрация: 14.05.2014
Сообщений: 7,236
Записей в блоге: 1
29.10.2015, 22:42 16
Цитата Сообщение от nxexox Посмотреть сообщение
C++
1
char slova[200][1000]; // предполагаем что слов может быть не больше 200. Создаем массив, где каждая строка это слово
Не больше 200 слов и каждое слово может состоять из 1000 символов? Не многовато ли?

Добавлено через 1 минуту
C++
1
2
3
4
5
6
7
8
9
10
11
for (int j = 0; j < i; j++)//идем от 0 до кол-ва слов, которое запомнили, до i
{
    for (int q = j + 1; q < i; q++)// запускаем цикл от текущего слова до конца массива слов
        // в цикле проверяем, сравниваем текущее слово, со всеми следующими
        //и если нашли, то увеличиваем счетчик повторений
        if (strcmp(slova[j], slova[q]) == 0) return true; // зачем увеличивать счетчик?
        // Если есть хоть один повтор,
        // то строка не годна для записи в файл, а значит не нужно мучить каждое слово на поиск повторов.
}
 
return false;
0
0 / 0 / 0
Регистрация: 18.05.2014
Сообщений: 48
29.10.2015, 22:43  [ТС] 17
Kerry_Jr, дак а какая разница, то ли я 10 поставлю то ли 200, т.е. алгоритм в обоих случаях должен же работать
0
Эксперт PHP
3106 / 2591 / 1219
Регистрация: 14.05.2014
Сообщений: 7,236
Записей в блоге: 1
29.10.2015, 22:46 18
nxexox, про количество символов в слове я сказал так, для экономии памяти.
0
0 / 0 / 0
Регистрация: 18.05.2014
Сообщений: 48
29.10.2015, 22:58  [ТС] 19
Kerry_Jr, А. Ну это я понимаю. Мне вопрос с работой надо решить. Он почему то неправильно сравнивает слова в массиве. Т.е. когда сравнивает два слова, у него в любом случае они не равны. strstr не выдает нужный результат
0
Эксперт PHP
3106 / 2591 / 1219
Регистрация: 14.05.2014
Сообщений: 7,236
Записей в блоге: 1
29.10.2015, 23:43 20
Цитата Сообщение от nxexox Посмотреть сообщение
он либо копирует все, либо при замене true и false не копирует ничего
а как функцией пользуетесь?
0
29.10.2015, 23:43
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
29.10.2015, 23:43
Помогаю со студенческими работами здесь

Вставить в начало строки ее порядковый номер, в конце строки - ее длину
Заданный текстовый файл F. Переписать компоненты файла F в файл H. При этом вставить в начало...

Автоматический перенос на 2 строки вниз в конце каждой строки документа
Доброго времени суток. Работаю с архивом фотографий, есть вакуумная директория с N-ым кол-вом...

Вставить в начало строки ее порядковый номер, в конце строки - ее длину
Заданный текстовый файл F. Переписать компоненты файла F в файл H. При этом вставить в начало...

В конце заданной строки дописать символы,доведя длину строки до 25
Строки вводятся с клавиатуры. В конце заданной строки дописать символы,доведя длину строки до 25.


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
20
Ответ Создать тему
Новые блоги и статьи
Что такое CQRS и как это реализовать на C# с MediatR
InfoMaster 15.01.2025
Концепция CQRS и её роль в современной разработке В современном мире разработки программного обеспечения архитектурные паттерны играют ключевую роль в создании масштабируемых и поддерживаемых. . .
Как настроить CI/CD с Azure DevOps
InfoMaster 15.01.2025
CI/ CD, или непрерывная интеграция и непрерывное развертывание, представляет собой современный подход к разработке программного обеспечения, который позволяет автоматизировать и оптимизировать процесс. . .
Как настроить CI/CD с помощью Jenkins
InfoMaster 15.01.2025
Введение в CI/ CD и Jenkins В современной разработке программного обеспечения непрерывная интеграция (CI) и непрерывная доставка (CD) стали неотъемлемыми элементами процесса создания качественных. . .
Как написать микросервис на Go/Golang с Kafka и GitHub CI/CD
InfoMaster 14.01.2025
Определение микросервиса, преимущества использования Go/ Golang Микросервис – это архитектурный подход к разработке программного обеспечения, при котором приложение состоит из небольших, независимо. . .
Как написать микросервис с нуля на C# с RabbitMQ, CQRS и CI/CD
InfoMaster 14.01.2025
В современном мире разработки программного обеспечения микросервисная архитектура стала стандартом де-факто для создания масштабируемых и гибких приложений. Этот архитектурный подход предполагает. . .
Как создать интернет-магазин на PHP и JavaScript
InfoMaster 14.01.2025
В современном мире электронная коммерция стала неотъемлемой частью бизнеса. Создание собственного интернет-магазина открывает широкие возможности для предпринимателей, позволяя достичь большей. . .
Как написать Тетрис на Ассемблере
InfoMaster 14.01.2025
Тетрис – одна из самых узнаваемых и популярных компьютерных игр, созданная в 1984 году советским программистом Алексеем Пажитновым. За прошедшие десятилетия она завоевала симпатии миллионы людей по. . .
Как создать игру "Танчики" на Unity3d и C#
InfoMaster 14.01.2025
Разработка игр – это увлекательный процесс, сочетающий в себе творчество и технические навыки. В этой статье мы рассмотрим создание классической игры "Танчики" с использованием Unity3D и языка. . .
Организую платный онлайн микро-курс по доработке Android-клиента Telegram
_Ivana 14.01.2025
Официальная версия и распространенные форки не полностью устраивают? Сделай свою кастомную версию клиента! 4 занятия по 2 часа (2 недели пн, ср 19:00-21:00 по Москве). Первое вводное занятие. . .
Как создать приложение для фитнеса для iOS/iPhone на Kotlin
InfoMaster 14.01.2025
Создание собственного фитнес-приложения — это не только захватывающий, но и полезный процесс, ведь оно может стать вашим верным помощником на пути к здоровому и активному образу жизни. В современных. . .
Как создать приложение магазина для iOS/iPhone на Swift
InfoMaster 14.01.2025
Введение в разработку iOS-приложений Разработка приложений для iPhone и других устройств на базе iOS открывает огромные возможности для создания инновационных мобильных решений. В данной статье мы. . .
Это работает. Скорость асинхронной логики велика. Вопрос видимо останется в стабильности. Плата - огонь!
Hrethgir 13.01.2025
По прошлому проекту в Logisim Evolution https:/ / www. cyberforum. ru/ blogs/ 223907/ blog8781. html прилагаю файл архива проекта в Gowin Eda. Восьмибитный счётчик из сумматора+ генератор сигнала. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru