Память не может быть "written"

@FlamedoFF · Регистрация: 17.08.2008

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

Помогите найти ошибку в следующем коде:
На этапе вхождения в функцию "vvod_vseh", а именно при вводе первой фамилии - выскакивает ошибка памяти. "Память не может быть written."
Как исправить?

C++

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <windows.h>
struct abonent
{
    char familiya[20];
    int year, nomer[7], kolvo;
};
void vvod_vseh(abonent*, int);
void vivod_vseh(abonent*, int);
void sort(abonent*, int);
void vvod(abonent*, int, int);
void vivod(abonent*);
void vivod_s_opred_goda(abonent*, int, int);
 
void main(void)
{
    int god, n, k;
    abonent *p=0;
    printf("Vvedite kol abonentov: \n");
    scanf_s("%d",&n);
    if(n<=0) 
    {
        printf("Nevernoe kolichestvo!");
        return;
    }
 
    vvod_vseh(p,n);
 
    while(1)
    {
        puts("BBeduTe: 1-BblBOD, 2-COPTuPOBKA,");
        puts("0-Exit. \n");
        scanf_s("%d",&k);
        if(!k)
            break;
        switch(k)
        {
        case 1:    vivod(p);
            break;
        case 2:    sort(p,0);
            break;
        }
    printf("Vvedite god: \n");
    scanf_s("%d",&god);
 
    vivod_s_opred_goda(p, n, god);
    }
}
void vvod(abonent*p)
{
    printf("Lastname:\n");
    scanf_s("%s",p->familiya);
    printf("Year:\n");
    scanf_s("%d",&p->year);
    printf("Number:\n");
    scanf_s("&s",p->nomer);
}
 
void vvod_vseh(abonent*p, int n)
{
    for(int i=1;i<n;i++)
        vvod(&p[i]);
}
 
void vivod(abonent*p)
{
    printf("%-20s%-20s%-15d\n", p->familiya, p->year, p->nomer);
}
 
void vivod_s_opred_goda(abonent*p, int n, int god)
{
    int t;
    for(int i=0;i<n;i++)
    {
        t=p[i].year;
        if(t>=god)
        {
            vivod_vseh(&p[i],n);
        }
    }
}
void vivod_vseh(abonent*p, int n)
{
        printf("%-20s%-20s%-15d\n", p->familiya, p->nomer, p->year);
}
void sort(abonent*p, int n)
{
    int i,j,k;
    abonent buf;
    for(i=0;i<n;i++)
    k=i;
    for(j=i+1;j<n;j++)
    {
        if(_strcmpi(p[k].familiya,p[j].familiya)>0)
            k=j;
        buf=p[i];
        p[i]=p[k];
        p[k]=buf;
    }
}

@EnzoMatrix · 29.04.2009, 20:19

C++

1	vvod_vseh(p,n);

вызов замени на

C++

1	vvod_vseh(&p,n);

@Ultrator · 29.04.2009, 20:26

"Память не может быть written."

Праильна... указатель p инициализирован нулём, а vvod() пытается чё-то присваивать (54) - а куда - в ту область памяти, где вектор прерываний (он начинается как раз с нулевого адреса).
Прежде чем с объектом (структурой) работать, надо ж его создать...

@FlamedoFF · 29.04.2009, 20:27 **[ТС]**

А это я его не создал?

C++

struct abonent
{
    char familiya[20];
    int year, nomer[7], kolvo;
};

@Ultrator · 29.04.2009, 20:31

Представляете ? - нет !!!

@FlamedoFF · 29.04.2009, 20:34 **[ТС]**

И если

C++

1	abonent *p=0;

заменить на

C++

1	abonent *p;

ошибка не устраняется.
Так же как и первый совет: Если вызов заменить на

C++

1	vvod_vseh(&p,n)

То появляется ошибка:

error C2664: vvod_vseh: невозможно преобразовать параметр 1 из 'abonent **' в 'abonent *'

Добавлено через 2 минуты 53 секунды

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

Представляете ? - нет !!!

По моему ты ошибаешься. Я объявил структуру. Все функции работают, кроме этой.

@FlamedoFF · 29.04.2009, 20:38 **[ТС]**

Ладно, раз никто не знает придется разбираться самому.

@XuTPbIu_MuHTAu · 29.04.2009, 21:15

Объявление структуры - это фактически объявление типа.
abonent *p=0; - указатель,т.е переменная размером 4 байта(на 32 разрядной платформе).
Обращение к структуре по такому указателю будет некорректным - память,на которую он указывает,
может быть занята кем и чем угодно.Объявлять следует так:
abonent p;
а потом делать vvod_vseh(&p...ну тут что там еще было);
К стати,ультратор дело говорит,хоть и не очень вежливо. Вам же стоит почитать основы языка.

@FlamedoFF · 29.04.2009, 21:16 **[ТС]**

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

Объявление структуры - это фактически объявление типа.
abonent *p=0; - указатель,т.е переменная размером 4 байта(на 32 разрядной платформе).
Обращение к структуре по такому указателю будет некорректным - память,на которую он указывает,
может быть занята кем и чем угодно.Объявлять следует так:
abonent p;
а потом делать vvod_vseh(&p...ну тут что там еще было);
К стати,ультратор дело говорит,хоть и не очень вежливо. Вам же стоит почитать основы языка.

Спасибо. Теперь понял. А основы я и так читаю. 2 курс института.

@XuTPbIu_MuHTAu · 29.04.2009, 21:25

Стоп.Зачеркнуть выше сказанное.Я тут посмотрел на программу. Видимо, имеется ввиду массив абонентов. Тогда объявлять нужно все же указатель. Но перед vvod_vseh вызвать new,чтобы выделить память нужного размера.

Код

abonent *p=0;
......тут вводим n
p = new abonent[n];
....дальше как было.

Добавлено через 7 минут 11 секунд

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

(он начинается как раз с нулевого адреса).

Разве мы с вами работаем в ДОСе? В защищенном режиме вектор прерываний не обязан там находиться.

@FlamedoFF · 29.04.2009, 21:50 **[ТС]**

Спасибо. Сделал. Теперь ошибка памяти в vivod'е.

Добавлено через 23 минуты 10 секунд
Не хочет считывать номер. Когда дело доходит до ввода номера но это просто игнорирует и сразу требует ввести год. Хоть ты тресни. Я уже всё менял - бесполезно. Отсюда и ошибка в выводе - поле с пустыми значениями номера. Помогите!

@Mecid · 29.04.2009, 22:46

слушай у тя стоко ошибок в коде
если ты выводешь инт,то надо использовать "%d" а не "%s"
потом вот ты объявил указатель
struct type* p=NULL;
то надо выделить под него память,

@FlamedoFF · 29.04.2009, 22:48 **[ТС]**

Блин. Идиотизм, нельзя отредактировать первый пост.
Вот новый код. Не хочет считывать nomer:

C++

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
struct abonent
{
    char familiya[20];
    int year[4];
    int nomer;
};
void vvod_vseh(abonent*, int);
void vivod_vseh(abonent*, int);
void sort(abonent*, int);
void vvod(abonent*, int, int);
void vivod(abonent*);
void vivod_s_opred_goda(abonent*, int, int);
void main(void)
{
    int god, n, k;
    abonent *p;
    printf("Vvedite kol abonentov: \n");
    scanf("%d",&n);
    p = new abonent[n];
    if(n<=0)
    {
        printf("Nevernoe kolichestvo!");
        return;
    }
 
    vvod_vseh(p,n);
    while(1)
    {
        puts("BBeduTe: 1-BblBOD, 2-COPTuPOBKA,");
        puts("0-Exit. \n");
        scanf("%d",&k);
        if(!k)
            break;
        switch(k)
        {
        case 1:    vivod(p);
            break;
        case 2:    sort(p,0);
            break;
        }
    printf("Vvedite god: \n");
    scanf("%d",&god);
 
    vivod_s_opred_goda(p, n, god);
    }
}
void vvod(abonent*p)
{
    printf("Lastname:\n");
    scanf("%s",p->familiya);
 
    printf("Number:\n");
    scanf("&d",&p->nomer);
 
    printf("Year:\n");
    scanf("%d",&p->year[4]);
}
void vvod_vseh(abonent*p,int n)
{
    for(int i=0;i<n;i++)
        vvod(&p[i]);
}
void vivod(abonent*p)
{
    printf("qpAMuJIu9l: %s, rOD: %d, NoMEP%d\n", p->familiya, p->year[4], p->nomer);
}
void vivod_s_opred_goda(abonent*p, int n, int god)
{
    int t;
    for(int i=0;i<n;i++)
    {
        t=p[i].year[4];
        if(t>=god)
        {
            vivod_vseh(&p[i],n);
        }
    }
}
void vivod_vseh(abonent*p, int n)
{
    printf("Lastname: %s, Phone: %d, Year of installation: %d\n", p->familiya, p->nomer, p->year[4]);
}
void sort(abonent*p, int n)
{
    int i,j,k;
    abonent buf;
    for(i=0;i<n;i++)
    k=i;
    for(j=i+1;j<n;j++)
    {
        if(strcmpi(p[k].familiya,p[j].familiya)>0)
            k=j;
        buf=p[i];
        p[i]=p[k];
        p[k]=buf;
    }
}

@Mecid · 29.04.2009, 22:51

и почему ты используешь сомнительные функции,юзай обычный scanf

@FlamedoFF · 29.04.2009, 22:53 **[ТС]**

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

и почему ты используешь сомнительные функции,юзай обычный scanf

Да я бы с радостью, но таково задание.

@Mecid · 29.04.2009, 23:01

ясно,так вот номер не вводится потому,что функция скан должна принимать указатель

@FlamedoFF · 29.04.2009, 23:05 **[ТС]**

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

ясно,так вот номер не вводится потому,что функция скан должна принимать указатель

Вообще-то она его и принимает. 52-59 строки. Там и год и фамилия принимается оператором scanf. А номер приниматься не хочет. Это просто чудеса...

@Mecid · 29.04.2009, 23:08

сорри описался

Добавлено через 30 секунд
не знаю у меня принимается

@Gravity · 29.04.2009, 23:08

C

1	56 scanf("&d",&p->nomer);

Да, с чего бы это, может быть дело в спецификаторе?)

@NickA · 29.04.2009, 23:10

C++

void vvod(abonent*p)
{
    printf("Lastname:\n");
    scanf("%s",p->familiya);
 
    printf("Number:\n");
    scanf("&d",&p->nomer);
 
    printf("Year:\n");
    scanf("%d",&p->year[4]);
}

C++

1	scanf("&d",&p->nomer);

А & вместо % это нормально?

@Mecid 683 / 232 / 16 Регистрация: 15.10.2007 Сообщений: 1,247
	29.04.2009, 22:46	12
	слушай у тя стоко ошибок в коде если ты выводешь инт,то надо использовать "%d" а не "%s" потом вот ты объявил указатель struct type* p=NULL; то надо выделить под него память, 0

@Mecid 683 / 232 / 16 Регистрация: 15.10.2007 Сообщений: 1,247
	29.04.2009, 22:51	14
	и почему ты используешь сомнительные функции,юзай обычный scanf 0

@Mecid 683 / 232 / 16 Регистрация: 15.10.2007 Сообщений: 1,247
	29.04.2009, 23:08	18
	сорри описался Добавлено через 30 секунд не знаю у меня принимается 0

@Ultrator 14 / 10 / 1 Регистрация: 28.04.2009 Сообщений: 219
	29.04.2009, 20:26	3
	"Память не может быть written." Праильна... указатель p инициализирован нулём, а vvod() пытается чё-то присваивать (54) - а куда - в ту область памяти, где вектор прерываний (он начинается как раз с нулевого адреса). Прежде чем с объектом (структурой) работать, надо ж его создать... 0

@Ultrator 14 / 10 / 1 Регистрация: 28.04.2009 Сообщений: 219
	29.04.2009, 20:31	5
	Представляете ? - нет !!! 0

@FlamedoFF 0 / 0 / 0 Регистрация: 17.08.2008 Сообщений: 14
	29.04.2009, 20:38 [ТС]	7
	Ладно, раз никто не знает придется разбираться самому. 0

@XuTPbIu_MuHTAu 2255 / 770 / 25 Регистрация: 27.05.2008 Сообщений: 1,496
	29.04.2009, 21:15	8
	Объявление структуры - это фактически объявление типа. abonent *p=0; - указатель,т.е переменная размером 4 байта(на 32 разрядной платформе). Обращение к структуре по такому указателю будет некорректным - память,на которую он указывает, может быть занята кем и чем угодно.Объявлять следует так: abonent p; а потом делать vvod_vseh(&p...ну тут что там еще было); К стати,ультратор дело говорит,хоть и не очень вежливо. Вам же стоит почитать основы языка. 0

@FlamedoFF 0 / 0 / 0 Регистрация: 17.08.2008 Сообщений: 14
	29.04.2009, 21:16 [ТС]	9
	Сообщение от XuTPbIu_MuHTAu Объявление структуры - это фактически объявление типа. abonent *p=0; - указатель,т.е переменная размером 4 байта(на 32 разрядной платформе). Обращение к структуре по такому указателю будет некорректным - память,на которую он указывает, может быть занята кем и чем угодно.Объявлять следует так: abonent p; а потом делать vvod_vseh(&p...ну тут что там еще было); К стати,ультратор дело говорит,хоть и не очень вежливо. Вам же стоит почитать основы языка. Спасибо. Теперь понял. А основы я и так читаю. 2 курс института. 0

@XuTPbIu_MuHTAu 2255 / 770 / 25 Регистрация: 27.05.2008 Сообщений: 1,496
	29.04.2009, 21:25	10
	Стоп.Зачеркнуть выше сказанное.Я тут посмотрел на программу. Видимо, имеется ввиду массив абонентов. Тогда объявлять нужно все же указатель. Но перед vvod_vseh вызвать new,чтобы выделить память нужного размера. Код abonent p=0; ......тут вводим n p = new abonent[n]; ....дальше как было. Добавлено через 7 минут 11 секунд Сообщение от Ultrator* (он начинается как раз с нулевого адреса). Разве мы с вами работаем в ДОСе? В защищенном режиме вектор прерываний не обязан там находиться. 0

@FlamedoFF 0 / 0 / 0 Регистрация: 17.08.2008 Сообщений: 14
	29.04.2009, 21:50 [ТС]	11
	Спасибо. Сделал. Теперь ошибка памяти в vivod'е. Добавлено через 23 минуты 10 секунд Не хочет считывать номер. Когда дело доходит до ввода номера но это просто игнорирует и сразу требует ввести год. Хоть ты тресни. Я уже всё менял - бесполезно. Отсюда и ошибка в выводе - поле с пустыми значениями номера. Помогите! 0

@FlamedoFF 0 / 0 / 0 Регистрация: 17.08.2008 Сообщений: 14
	29.04.2009, 22:53 [ТС]	15
	Сообщение от Mecid и почему ты используешь сомнительные функции,юзай обычный scanf Да я бы с радостью, но таково задание. 0

@Mecid 683 / 232 / 16 Регистрация: 15.10.2007 Сообщений: 1,247
	29.04.2009, 23:01	16
	ясно,так вот номер не вводится потому,что функция скан должна принимать указатель 0

@FlamedoFF 0 / 0 / 0 Регистрация: 17.08.2008 Сообщений: 14
	29.04.2009, 23:05 [ТС]	17
	Сообщение от Mecid ясно,так вот номер не вводится потому,что функция скан должна принимать указатель Вообще-то она его и принимает. 52-59 строки. Там и год и фамилия принимается оператором scanf. А номер приниматься не хочет. Это просто чудеса... 0