Когда запускаю и ввожу параметры, то, в момент когда программа должна выполнять действие, пишет ошибку

@Hellsing616 · Регистрация: 03.12.2016

Недавно начали изучать c++. так как в школе не было программирования (преподу пофиг было) я в универе отстаю ;c Написал программу по заданию (задание будет ниже и код тоже), но выходит ошибка (скрин прикрепил). Помогите новичку разобраться.
ЗАДАНИЕ : Написать программу, запрашивающую информацию о именах, фамилиях, адресах и телефонах абонентов и печатающую эту информацию в виде таблицы (использовать тип – структуру и ограничиться вводом данных о пяти абонентах). По запросу пользователя программа должна выдавать информацию о телефоне абонента по введенной фамилии и имени. В случае отсутствия информации, программа должна сообщать: “Такого абонента нет.” Для задания данных использовать тип struct.
Мои, так сказать, попытки написать код:

C++ (Qt)

#include <iostream>
#include <cstdio>
#include <string>
#include <stdio.h>
#include <fstream>
using namespace std; 
struct book {
char fam[25];
char im[15];
char adress[100];
int phone;} lib[5];
 
int main(int argc, char*argv[])
{
int n;
cout<<"the number of subscribers: ";
cin>>n;
for (int i=1; i<=n; i++)
{cout<<"Enter  subscriber's name "<<endl; cin>>lib[i].im;
cout<<"Enter subscriber's surname"<<endl; cin>>lib[i].fam;
cout<<"Enter the adress without spaces or commas"<<endl; cin>>lib[i].adress;
cout<<"enter a phone number "<<endl; cin>>lib[i].phone;
}
for (int i=1; i<=n; i++)
cout<<lib[i].im<<" "<<lib[i].fam<<" "<<lib[i].adress<<" "<<lib[i].phone<<endl;
char* a,*b;
int i,j,k;
int t=1;
cout<<"Enter a name and a surname of subscriber who want to find  "; cin>>a>>b;
for (i=1; i<=n; i++) {
if (strlen(a)==strlen(lib[i].im)) {
for (j=1; j<=strlen(a); j++)
if (a[j]!=lib[i].im[j]) { t=0; break;}}
if (strlen(b)==strlen(lib[i].fam)) {
for (k=1; k<=strlen(b); k++)
if (b[j]!=lib[i].fam[k]) { t=0; break;}}
if (t=1) {cout<<lib[i].phone;}}
if (t=0) cout<<"not found";
getchar(); getchar();
return 0;
}

скрин ошибки

@lawr · 05.12.2016, 12:56

ну во-первых, нумерация массивов идет с нуля, а последний элемент имеет индекс n-1, где n- длинна массива. потому когда ты пишешь в цикле условие i<=n, цикл выходит за границу массива. переделай все циклы по типу

C++

1	for (i=0; i<n; i++)

во вторых, ты пытаешься применить функцию strlen() к указателю на char, в то время как ее аргументом должна быть си-строка. поменяй 26-ю строку на

C++

1	char a[15], b[25]

@Hellsing616 · 06.12.2016, 16:41 **[ТС]**

Спасибо, заработало

. Но теперь проблема в том, что при поиске он выводит все номера сразу: идешь одного - выводит все номера, что были вписаны. Как можно это исправить ?

@lawr · 06.12.2016, 19:27

Hellsing616, кинь код и скрин консоли.

@Hellsing616 · 06.12.2016, 19:42 **[ТС]**

#include <iostream>
#include <cstdio>
#include <string>
#include <stdio.h>
#include <fstream>
using namespace std;
struct book {
char fam[25];
char im[15];
char adress[100];
int phone;} lib[5];

int main(int argc, char*argv[])
{
int n;
cout<<"the number of subscribers: ";
cin>>n;
for (int i=0; i<n; i++)

{cout<<"Enter subscriber's name "<<endl; cin>>lib[i].im;
cout<<"Enter subscriber's surname"<<endl; cin>>lib[i].fam;
cout<<"Enter the adress without spaces or commas"<<endl; cin>>lib[i].adress;
cout<<"enter a phone number "<<endl; cin>>lib[i].phone;
}
for (int i=0; i<n; i++)

cout<<lib[i].im<<" "<<lib[i].fam<<" "<<lib[i].adress<<" "<<lib[i].phone<<endl;
char a[15], b[25];
int i,j,k;
int t=1;
cout<<"Enter a name and a surname of subscriber who want to find "; cin>>a>>b;
for (i=0; i<n; i++) {
if (strlen(a)==strlen(lib[i].im)) {
for (j=1; j<=strlen(a); j++)
if (a[j]!=lib[i].im[j]) { t=0; break;}}
if (strlen(b)==strlen(lib[i].fam)) {
for (k=1; k<=strlen(b); k++)
if (b[j]!=lib[i].fam[k]) { t=0; break;}}
if (t=1) {cout<<lib[i].phone;}}
if (t=0) cout<<"not found";
getchar(); getchar();
return 0;
}

@lawr · 06.12.2016, 23:30

сразу 3 ошибки
1) главная- не путай условие равенства с присваиванием. неправильно:

C++

1 2	if (t=1) {cout<<lib[i].phone;}} if (t=0) cout<<"not found";

правильно:

C++

1 2	if (t==1) {cout<<lib[i].phone;}} if (t==0) cout<<"not found";

2)основная- проверочная переменная (кстати, неплохо было бы сделать ее булевой вместо int-а) должна принимать истинное значение перед проверкой каждой записи. неправильно:

C++

1
2
3

int t=1;
cout<<"Enter a name and a surname of subscriber who want to find "; cin>>a>>b;
for (i=0; i<n; i++) {...}

правильно:

C++

1
2
3

int t;
cout<<"Enter a name and a surname of subscriber who want to find "; cin>>a>>b;
for (i=0; i<n; i++) {t=1;...}

3) ключевая- итератор в условии должен соответствовать итератору в цикле. неправильно:

C++

1 2	for (k=1; k<=strlen(b); k++) if (b[j]!=lib[i].fam[k]) { t=0; break;}}

правильно:

C++

1 2	for (k=1; k<=strlen(b); k++) if (b[k]!=lib[i].fam[k]) { t=0; break;}}

итогово, вот исправленный кусок кода:

C++

char a[15], b[25];
    int i, j, k;
    bool t;
    cout << "Enter a name and a surname of subscriber who want to find "; cin >> a >> b;
    for (i = 0; i<n; i++) {
        t = true;
        if (strlen(a) == strlen(lib[i].im)) {
            for (j = 0; j < strlen(a); j++)
                if (a[j] != lib[i].im[j]) { t = false; break; }
        }
        if (strlen(b) == strlen(lib[i].fam)) {
            for (k = 0; k < strlen(b); k++)
                if (b[k] != lib[i].fam[k]) { t = false; break; }
        }
        if (t) { cout << lib[i].phone; break; }
    }
    if (!t) cout << "not found";

P.S.- будь другом, используй теги когда постишь код, а сам код форматируй согласно общепринятым нормам. глаза ж сломать можно.

@Hellsing616 · 07.12.2016, 00:10 **[ТС]**

Спасибо большое. Всё работает. В следующий раз всё сделаю как надо, я просто тут новенький

@Hellsing616 0 / 0 / 0 Регистрация: 03.12.2016 Сообщений: 10
	06.12.2016, 16:41 [ТС]	3
	Спасибо, заработало . Но теперь проблема в том, что при поиске он выводит все номера сразу: идешь одного - выводит все номера, что были вписаны. Как можно это исправить ? 0

@lawr 385 / 279 / 478 Регистрация: 09.05.2014 Сообщений: 769
	06.12.2016, 19:27	4
	Hellsing616, кинь код и скрин консоли. 0

@Hellsing616 0 / 0 / 0 Регистрация: 03.12.2016 Сообщений: 10
	06.12.2016, 19:42 [ТС]	5
	#include <iostream> #include <cstdio> #include <string> #include <stdio.h> #include <fstream> using namespace std; struct book { char fam[25]; char im[15]; char adress[100]; int phone;} lib[5]; int main(int argc, char*argv[]) { int n; cout<<"the number of subscribers: "; cin>>n; for (int i=0; i<n; i++) {cout<<"Enter subscriber's name "<<endl; cin>>lib[i].im; cout<<"Enter subscriber's surname"<<endl; cin>>lib[i].fam; cout<<"Enter the adress without spaces or commas"<<endl; cin>>lib[i].adress; cout<<"enter a phone number "<<endl; cin>>lib[i].phone; } for (int i=0; i<n; i++) cout<<lib[i].im<<" "<<lib[i].fam<<" "<<lib[i].adress<<" "<<lib[i].phone<<endl; char a[15], b[25]; int i,j,k; int t=1; cout<<"Enter a name and a surname of subscriber who want to find "; cin>>a>>b; for (i=0; i<n; i++) { if (strlen(a)==strlen(lib[i].im)) { for (j=1; j<=strlen(a); j++) if (a[j]!=lib[i].im[j]) { t=0; break;}} if (strlen(b)==strlen(lib[i].fam)) { for (k=1; k<=strlen(b); k++) if (b[j]!=lib[i].fam[k]) { t=0; break;}} if (t=1) {cout<<lib[i].phone;}} if (t=0) cout<<"not found"; getchar(); getchar(); return 0; } Миниатюры 0

@Hellsing616 0 / 0 / 0 Регистрация: 03.12.2016 Сообщений: 10
	07.12.2016, 00:10 [ТС]	7
	Спасибо большое. Всё работает. В следующий раз всё сделаю как надо, я просто тут новенький 0

Когда запускаю и ввожу параметры, то, в момент когда программа должна выполнять действие, пишет ошибку

Решение