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

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

Миниатюры

 

0

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

1

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

0

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

0

#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

сразу 3 ошибки
1) главная- не путай условие равенства с присваиванием. неправильно: правильно: 2)основная- проверочная переменная (кстати, неплохо было бы сделать ее булевой вместо int-а) должна принимать истинное значение перед проверкой каждой записи. неправильно: правильно: 3) ключевая- итератор в условии должен соответствовать итератору в цикле. неправильно: правильно: итогово, вот исправленный кусок кода: P.S.- будь другом, используй теги когда постишь код, а сам код форматируй согласно общепринятым нормам. глаза ж сломать можно.

1

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

0