Зацикливание при сравнении строк

@-Arabella- · Регистрация: 20.10.2013

Студворк — интернет-сервис помощи студентам

Добрый вечер. У меня возникла проблема при сравнивании строк, после сравнивания происходит расчет по циклу [j] и далее по идее должно переходить к циклу по [i], но программа зацикливается на [j] здесь, вечно накапливает k и ничего толкового не считает. Почему зацикливается?

C++ (Qt)
1
2
3
4
5
6
7
8
9
if(strcmp(sell[i].FIO,sell[j].FIO)==0 && sell[j].FIO!="\0")
{
k++;
strcpy(sell[j].FIO,"\0");
st.at=st.at+sell[j].at;
st.ves=st.ves+sell[j].ves;
st.shtuk=st.shtuk+sell[j].shtuk;
st.time=st.time+sell[j].time;
}

Код всей функции

C++ (Qt)
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
34
35
36
37
38
39
40
41
42
43
44
45
void output()
{
seller st;
char filename[]="sellers.dat";
int n;
FILE *f;
clrscr();
f=fopen(filename,"rb+");
if(f)
{
fseek(f,0,SEEK_END);
n=ftell(f)/sizeof(seller);
seller* sell=new seller [n];
fseek(f,0,SEEK_SET);
 
   fread(&sell,sizeof(seller),n,f);
   printf("\n FIO | Weight Things | Time | Service Time Of One Purchase ");
for(int i=0;i<n-1;i++)
{
int k=1;
st.at=sell[i].at;
st.ves=sell[i].ves;
st.shtuk=sell[i].shtuk;
st.time=sell[i].time;
for(int j=i+1;j<n;j++)
{
if(strcmp(sell[i].FIO,sell[j].FIO)==0 && sell[j].FIO!="\0")
{
k++;
strcpy(sell[j].FIO,"\0");
st.at=st.at+sell[j].at;
st.ves=st.ves+sell[j].ves;
st.shtuk=st.shtuk+sell[j].shtuk;
st.time=st.time+sell[j].time;
}
st.at=st.at/k;
}
printf("\n %25s/%2d/%2d/%f/%5.2f",st.FIO,st.ves,st.shtuk,st.time,st.at);
}
}
printf("Press any key");
getch();
fflush(stdin);
fclose(f);
}

Tulosba · 14.05.2014, 22:06

Сообщение от -Arabella-

Почему зацикливается?

С первого взгляда никакого зацикливания быть не должно: i,j растут постоянно, n не меняется в цикле.
Что в исходном файле sellers.dat ?

@-Arabella- · 15.05.2014, 06:01 **[ТС]**

Tulosba,

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

Что в исходном файле sellers.dat ?

Ну по идеи должны быть структуры: фамилия и 4 цифирки, но в него записывается ерунда, вот я и искала ошибку с помощью F7 в функции вывода и после if постоянно возвращается на if, намного больше раз, чем структур

Tulosba · 15.05.2014, 07:45

-Arabella-, чему в 13 строке равно n?
В 16 строке нужно убрать &.

@zss · 15.05.2014, 08:31

Сообщение от -Arabella-

if(strcmp(sell[i].FIO,sell[j].FIO)==0 && sell[j].FIO!="\0")

sell[j].FIO!="\0" - это проверка того, что адрес строки sell[j].FIO не совпадает с адресом
константной строки, не содержащей текста.
Полагаю, что Вы имели ввиду sell[j].FIO[0]!=0 или (что то же самое) strlen(sell[j].FIO)!=0

@-Arabella- · 15.05.2014, 09:25 **[ТС]**

Tulosba, n вычисляется в 12, n - количество структур. А почему в 16 нужно убрать &?
zss, я хотела сделать проверку не пустая(нулевая) ли эта строка, потому что если строки совпадают и чтоб потом опять с этой строкой не сравнивать я ее делаю нулевой(пустой), мне все убрать ковычки и слэш?

Tulosba · 15.05.2014, 10:56

Сообщение от -Arabella-

А почему в 16 нужно убрать &?

Потому что fread принимает адрес начала памяти, куда будет производится чтение. И этот адрес - это sell, полученный в 13 строке. А вовсе не адрес переменной sell, который получается, если написать &sell.
Для проверки что строка не пустая можно использовать такой код:

C++
1
if( *str )

А если обезопасить себя от нулевого указателя, то такой:

C++
1
if( str && *str )

@zss · 15.05.2014, 11:29

Сообщение от -Arabella-

мне все убрать ковычки и слэш?

Да, и сравнивать не адрес, а нулевой элемент массива:

C++
1
if(strcmp(sell[i].FIO,sell[j].FIO)==0 && sell[j].FIO[0]!=0)

@-Arabella- · 16.05.2014, 00:25 **[ТС]**

Tulosba, zss, Спасибо))) Заработало

Новые блоги и статьи Все статьи Все блоги /
Модель микоризы: классовый агентный подход anaschu 02.01.2026 Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .	Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Programma_Boinc 01.01.2026 Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Сочетание глобально распределённой вычислительной мощности и инновационных. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	15.05.2014, 07:45
	-Arabella-, чему в 13 строке равно n? В 16 строке нужно убрать &. 0

@-Arabella- 0 / 0 / 1 Регистрация: 20.10.2013 Сообщений: 233
	15.05.2014, 09:25 [ТС]
	Tulosba, n вычисляется в 12, n - количество структур. А почему в 16 нужно убрать &? zss, я хотела сделать проверку не пустая(нулевая) ли эта строка, потому что если строки совпадают и чтоб потом опять с этой строкой не сравнивать я ее делаю нулевой(пустой), мне все убрать ковычки и слэш? 0

@-Arabella- 0 / 0 / 1 Регистрация: 20.10.2013 Сообщений: 233
	16.05.2014, 00:25 [ТС]
	Tulosba, zss, Спасибо))) Заработало 0