Сравнение полей структур

@kristina-lonsh · Регистрация: 04.03.2012

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

Задача имела формулировку: "Сведения о выдаче экземпляра журнала состоят из названия журнала, номера журнала, фамилии читателя. Написать программу для занесения в массив информации о N таких сведениях. По заданному названию журнала найти выдававшиеся номера и фамилии бравших их читателей. Найти самого активного читателя".

Моя программа почему то не сравнивает поля при поиске(( Не могу причину найти.. Уважаемые знатоки, проверьте, помогите

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
#include<stdlib.h>
#include<iostream>
using namespace std;
 
struct libr
{
char journal[10];
int nomer;
char family[10];
};
 
 
int main()
{int i,N;
struct libr A[3];
struct libr poisk;
cout<<"Vvedite kolichestvo svedeniy--> "; cin>>N;
cout<<"Vvedite nazvanie zhurnala, nomer zhurnala i familiyu chitatelya cherez =PROBEL= "<<endl;
for (i=0;i<N;i++)  
    {cout<<i<<"-ogo chitatelya"<<endl;
    cin>>A[i].journal>>A[i].nomer>>A[i].family;
    }
cout<<endl<<"Vvedite nazvanie iskomogo zhurnala--> "; cin>>poisk.journal;
for (i=0;i<N;i++)  
    {if ((A[i].journal)!=(poisk.journal))cout<<"Совпадений нет!! ";
    else cout<<A[i].family<<endl;
    }
system("PAUSE");
}

@Konstantin_D · 01.04.2012, 15:33

Строки сравнивает функция strcmp(). А вы сравниваете значения указателей.

C++
1
if ((A[i].journal)!=(poisk.journal))

@kristina-lonsh · 01.04.2012, 15:37 **[ТС]**

Мм, спасибо!! Сейчас попробую!

@Konstantin_D · 01.04.2012, 15:48

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
34
35
#include <cstring>
#include <iostream>
using namespace std;
 
const int LEN = 20;
 
struct libr
{
    char journal[LEN];
    int nomer;
    char family[LEN];
};
 
int main()
{
    int i, n;
    char poisk[LEN] ;
    cout << "Vvedite kolichestvo svedeniy: "; 
    cin >> n;
    libr* ArrLib = new libr[n];
    cout << "Vvedite nazvanie zhurnala, nomer zhurnala, familiyu chitatelya: \n";
    for ( i = 0; i < n; ++i)  
    {
        cout << "Chitatel #" << i+1 << ":\n";
        cin >> ArrLib[i].journal >> ArrLib[i].nomer >> ArrLib[i].family;
    }
    cout <<"Vvedite nazvanie iskomogo zhurnala: "; 
    cin >> poisk;
    for ( i = 0; i < n; i++ )  
    {
        if ( strcmp(ArrLib[i].journal, poisk) == 0 )
            cout << "Chitatel: " << ArrLib[i].family << endl;
    }
    delete [] ArrLib;
}

@Konstantin_D · 01.04.2012, 16:12

Так это и есть ваша переделанная

Добавлено через 7 минут
Когда будете сравнивать, обратите внимание на строки 6, 18, 21, 32, 35 в моем варианте. Основные идеи изменений в них.

@kristina-lonsh · 01.04.2012, 16:42 **[ТС]**

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

Так это и есть ваша переделанная

Добавлено через 7 минут
Когда будете сравнивать, обратите внимание на строки 6, 18, 21, 32, 35 в моем варианте. Основные идеи изменений в них.

А можно еще вопросик? Или это будет уже перебор?

Найти самого активного читателя... Немного надо подумать... Но, в любом случае, БоЛьШоЕ спасибо вам!!! А вот мой отредактированный текст программы:

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
#include<stdlib.h>
#include<iostream>
#include<iomanip.h>
using namespace std;
 
struct libr
{
char journal[10];
int nomer;
char family[10];
};
 
 
int main()
{int i,N;
struct libr A[3];
char poisk[10];
cout<<"Vvedite kolichestvo svedeniy--> "; cin>>N;
cout<<"Vvedite nazvanie zhurnala, nomer zhurnala i familiyu chitatelya cherez =PROBEL= "<<endl;
for (i=0;i<N;i++)  
    {cout<<"Chitatelya ¹"<<i+1<<endl;
    cin>>A[i].journal>>A[i].nomer>>A[i].family;
    }
cout<<endl<<"Vvedite nazvanie iskomogo zhurnala--> "; cin>>poisk;
cout<<setw(10)<<"Ôàìèëèÿ"<<setw(10)<<"Íîìåð"<<endl;
for (i=0;i<N;i++)  
    {if (strcmp(A[i].journal,poisk)==0) cout<<setw(10)<<A[i].family<<setw(10)<<A[i].nomer<<endl;
    }
system("PAUSE");
}

Добавлено через 5 минут

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

Так это и есть ваша переделанная

Добавлено через 7 минут
Когда будете сравнивать, обратите внимание на строки 6, 18, 21, 32, 35 в моем варианте. Основные идеи изменений в них.

А в 35 строке, енто зачем енто? какое то освобождение памяти? Я указатели использовать не стала, думаю итак на сдачу пойдет..

@Konstantin_D · 01.04.2012, 16:56

Вам же нужно "Написать программу для занесения в массив информации о N таких сведениях". Вы заранее устанавливаете N = 3.

C++
1
libr A[3];

А что будет, если N введут больше чем 3?
Моя программа в 21-й строке динамически выделяет память для необходимого количества структур и в строке 35 освобождает ее.

@kristina-lonsh · 01.04.2012, 17:04 **[ТС]**

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

Вам же нужно "Написать программу для занесения в массив информации о N таких сведениях". Вы заранее устанавливаете N = 3.

C++
1
libr A[3];

А что будет, если N введут больше чем 3?
Моя программа в 21-й строке динамически выделяет память для необходимого количества структур и в строке 35 освобождает ее.

Ой, ну конечно! Это я экспериментировала на начальных с константой, так и не исправила потом.. Вот так вот правильно для N сведений:

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
#include<stdlib.h>
#include<iostream>
#include<iomanip.h>
using namespace std;
 
struct libr
{
char journal[10];
int nomer;
char family[10];
};
 
 
int main()
{int i,j,N;
char poisk[10];
cout<<"Vvedite kolichestvo svedeniy--> "; cin>>N;
struct libr A[N];
cout<<"Vvedite nazvanie zhurnala, nomer zhurnala i familiyu chitatelya cherez =PROBEL= "<<endl;
for (i=0;i<N;i++)  
    {cout<<"Chitatelya ¹"<<i+1<<endl;
    cin>>A[i].journal>>A[i].nomer>>A[i].family;
    }
cout<<endl<<"Vvedite nazvanie iskomogo zhurnala--> "; cin>>poisk;
cout<<setw(10)<<"Familiya"<<setw(10)<<"Nomer"<<endl;
for (i=0;i<N;i++)  
    {if (strcmp(A[i].journal,poisk)==0) cout<<setw(10)<<A[i].family<<setw(10)<<A[i].nomer<<endl;
    }
 
system("PAUSE");
}

@Konstantin_D · 01.04.2012, 17:26

А у вас это компилируется и строка 19 не вызывает ошибку?

@kristina-lonsh · 01.04.2012, 17:32 **[ТС]**

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

А у вас это компилируется и строка 19 не вызывает ошибку?

когда массив объявляешь до инициализации N, то ошибка выдается, а когда уже N считана и известна - компилируется и работает как часы =)

@Konstantin_D · 01.04.2012, 17:55

Странно. Размер массива должен быть КОНСТАНТОЙ! И НЕ может быть переменной, значение которой устанавливается во время выполнения программы.

Добавлено через 16 минут
Компилятор не может такое пропустить:

C++
1
2
3
    int n;
    cin >> n;
    int ar[n]; // ошибка

У вас единственный выход - использовать динамический массив (операторы new[] и delete[]).

@kristina-lonsh · 01.04.2012, 18:00 **[ТС]**

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

Странно. Размер массива должен быть КОНСТАНТОЙ! И НЕ может быть переменной, значение которой устанавливается во время выполнения программы.

Добавлено через 16 минут
Компилятор не может такое пропустить:

C++
1
2
3
    int n;
    cin >> n;
    int ar[n]; // ошибка

У вас единственный выход - использовать динамический массив (операторы new[] и delete[]).

Не знаю, Константин, Borland C++ блокирует всё, а Dev C++ пропускает многое) У меня как раз Dev C++.. Программа работает без явных ошибок.. А вы не смотрели как бы реализовать поиск самого активного читателя? может ночь сказывается, но что-то идея не приходит... ... ...

@Konstantin_D · 01.04.2012, 18:41

C++
1
2
3
4
5
    for (i=0; i<N; ++i)
    {
        if ( !strcmp(A[i].journal, poisk) ) 
            cout << setw(10) << A[i].family << setw(10) << A[i].nomer << endl;
    }

Я бы перебирая этот массив собирал читателей которые попадаются и сколько журналов они прочли, потом выберите самого активного. Тут вам или STL в помощь или опять strcmp() + new + delete (видимо, вы их недавно учили и это задание с их использованием).

@retmas · 01.04.2012, 18:53

Сообщение от kristina-lonsh

Программа работает без явных ошибок..

это потому, что в gcc есть поддержка массивов переменной длины. но не стоит ими злоупотреблять. используйте new/delete.

@kristina-lonsh · 02.04.2012, 02:32 **[ТС]**

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

C++
1
2
3
4
5
    for (i=0; i<N; ++i)
    {
        if ( !strcmp(A[i].journal, poisk) ) 
            cout << setw(10) << A[i].family << setw(10) << A[i].nomer << endl;
    }

Я бы перебирая этот массив собирал читателей которые попадаются и сколько журналов они прочли, потом выберите самого активного. Тут вам или STL в помощь или опять strcmp() + new + delete (видимо, вы их недавно учили и это задание с их использованием).

ok, попробую!! а динамические массивы мы действительно изучали не так давно!!

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	Установка Emscripten SDK (emsdk) и CMake на Windows для сборки C и C++ приложений в WebAssembly (Wasm) 8Observer8 30.01.2026 Чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. Система контроля версиями Git. . .	Подключение Box2D v3 к SDL3 для Android: физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .
Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .	Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL3_image 8Observer8 27.01.2026 Содержание блога SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .	Влияние грибов на сукцессию anaschu 26.01.2026 Бифуркационные изменения массы гриба происходят тогда, когда мы уменьшаем массу компоста в 10 раз, а скорость прироста биомассы уменьшаем в три раза. Скорость прироста биомассы может уменьшаться за. . .	Воспроизведение звукового файла с помощью SDL3_mixer при касании экрана Android 8Observer8 26.01.2026 Содержание блога SDL3_mixer - это библиотека я для воспроизведения аудио. В отличие от инструкции по добавлению текста код по проигрыванию звука уже содержится в шаблоне примера. Нужно только. . .

@kristina-lonsh 0 / 0 / 1 Регистрация: 04.03.2012 Сообщений: 61
	01.04.2012, 15:37 [ТС]
	Мм, спасибо!! Сейчас попробую! 0

@Konstantin_D 15 / 15 / 6 Регистрация: 21.07.2011 Сообщений: 89
	01.04.2012, 16:12
	Так это и есть ваша переделанная Добавлено через 7 минут Когда будете сравнивать, обратите внимание на строки 6, 18, 21, 32, 35 в моем варианте. Основные идеи изменений в них. 0

@Konstantin_D 15 / 15 / 6 Регистрация: 21.07.2011 Сообщений: 89
	01.04.2012, 17:26
	А у вас это компилируется и строка 19 не вызывает ошибку? 0