Нарушение прав доступа при чтении

@JIexa01 · Регистрация: 06.12.2015

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

Здравствуйте,реквестирую помощь,я уже волосы на голове рву в попытке решить эту проблему,при чтении из бинарного файла выводит ошибку,строку на которую ругается компилятор закоментировал,скриншот ошибки приложил.

C++

#include <iostream>
#include <conio.h>
#include <string.h>
#include <Windows.h>
#include <fstream>
#include <iomanip>
#include <stdlib.h>
 
using namespace std;
 
struct all
{
    char count[31];
    char mark[31];
    int num;
    all*next;
};
 
struct all1
{
    char count[31];
    char mark[31];
    int num;
    all1*next;
};
 
class qqw
{
private:
    int q, w, e, t, m, k, z, max, k1;
    all *beg1;
    all1 *beg2;
    all1 *beg3;
public:
    //Конструктор по умолчанию
    qqw() { beg1 = NULL;  beg2 = NULL; beg3 = NULL; };
    void input();
    void vibor();
    void inputbin1();
    void output1();
    void outputmark();
    void perech();
    void perechfile();
    void perechout();
    void makeperech();
    void perechsort();
    void outputcountry();
    void perechcountry();
    void add();
    void del();
    void fileout();
    ~qqw() {
        all *p1;
        all1 *p2, *p3;
        while (beg1 != NULL) {
            p1 = beg1;
            beg1 = beg1->next;
            delete p1;
        }
        while (beg2 != NULL) {
            p2 = beg2;
            beg2 = beg2->next;
            delete p2;
        }
        while (beg3 != NULL) {
            p3 = beg3;
            beg3 = beg3->next;
            delete p3;
        }
    };
 
};
 
 
void main() {
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    qqw a;
 
    int c;
    while (1){
        setlocale(LC_ALL, "Russian");
        cout << "1.Ввод данных из файла" << endl;
        cout << "2.Ввод таблицы в бинарный файл" << endl;
        cout << "3.Вывод таблицы из бинарного файла" << endl;
        cout << "4.Ввод таблицы на экран" << endl;
        cout << "0.Выход" << endl;
        cin >> c;
        system("cls");
        switch (c) {
        case 1:a.input(); break;
        case 2:a.vibor(); break;
        case 3:a.inputbin1(); break;
        case 4:a.output1(); break;
        case 0:exit(1);
        }
    }
}
 
void qqw::input()
{
    ifstream fin("Input.txt");
    all *p1;
    while (beg1 != NULL) {
        p1 = beg1;
        beg1 = beg1->next;
        delete p1;
    }
    while (1){
        p1 = new all;
        if (p1 == NULL) { cout << "Нет динамической памяти. \n"; _getch(); exit(1); }
        fin >> p1->count >> p1->mark >> p1->num;
        if (fin.fail()) break;
        p1->next = beg1;
        beg1 = p1;
    }
 
    fin.close();
    cout << "Файл введен. \n";
    _getch();
}
void qqw::vibor()
{
    all *y; //Копия начального элемента
    ofstream fout; //Выходной файловый поток
    fout.open("Output.bin", ios::binary);
    if (fout.fail()) {
        cout << "Файл не создан";
        _getch();
        return;
    }
    y = beg1;
    while (y != NULL) {
        all *t = new all;
        t->num = y->num;
        strcpy(t->count, y->count);
        strcpy(t->mark, y->mark);
        fout.write((char*)&t, sizeof(all));
 
        y = y->next;
    }
    fout.close();
    cout << "Запись в бинарный файл прошла успешно";
    _getch();
}
 
void qqw::inputbin1() {
    all *pend = NULL, *p;
    ifstream fin; //Входной файловый поток
    char file[30]; //Имя файла
    all *t;
    cout << "Введите имя файла: " << endl; cin >> file;
    fin.open(file, ios::binary);
    if (fin.fail()) {
        cout << "Файл не создан";
        _getch();
        return;
    }
    while (1) {
        p = new all;
        if (p == NULL) { cout << "нет памяти"; _getch(); fin.close(); return; }
        [COLOR="SandyBrown"]fin.read((char*)&t, sizeof(all));[/COLOR]                                 //ВОТ ТУТ ОШИБКА
        if (fin.fail()) break;
        strcpy(p->count, t->count);
        strcpy(p->mark, t->mark);
        p->next = NULL;
        if (pend == NULL) beg1 = p; else pend->next = p;
        pend = p;
    }
    fin.close();
    delete p;
    cout << "Файл введен";
    _getch();
}
 
void qqw::output1()
{
    all *p;
    cout << "Введённые данные :" << endl;
    cout << "  |---|-------------------------------|------------|----------|" << endl;
    cout << "  | № |            Cтрана             |    Марка   |   Кол-во |" << endl;
    cout << "  |---|-------------------------------|------------|----------|" << endl;
    int i = 1;
    for (p = beg1; p != NULL; p = p->next)
    {
        cout << "  |" << setw(3) << i << "| " << setw(29) << p->count << " | " << setw(10) << p->mark << " |   " << setw(6) << p->num << " |" << endl;
        cout << "  |---|-------------------------------|------------|----------|" << endl;
        i++;
    }
    _getch();
    system("cls");
    return;
}

@DrOffset · 06.12.2015, 14:51

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

fin.read((char*)&t, sizeof(all));

А как это ты так читаешь в невыделенную память? У тебя t - это просто указатель, да к тому же неинициализированный. Так ты еще и адрес от него берешь, затирая функцией read уже часть стека функции.
Пиши уж как-нибудь так тогда:

C++

        if (fin.eof()) break; 
        p = new(std::nothrow) all;
        if (p == NULL) { cout << "нет памяти"; _getch(); fin.close(); return; }
        fin.read((char*)p, sizeof(all));
        //strcpy(p->count, t->count);
        //strcpy(p->mark, t->mark);
        p->next = NULL;

Ну или хочешь если читать сперва в промежуточный объект, ну так и делай его объектом all:

C++

all t;

Тогда взятие его адреса в read будет правомерным. А чтение из него изменится на такое:

C++

1 2	strcpy(p->count, t.count); strcpy(p->mark, t.mark);

@JIexa01 · 06.12.2015, 17:18 **[ТС]**

Спасибо большое,помогло,в этом же коде есть ещё одна точно такая же ошибка,но в другом месте,буду вам очень благодарен за помощь.

На этот раз ошибка в условии(тоже что-то с памятью) для p1
Первый перечень(Который perech) записывается нормально,а вот второй(makeperech) крашит(объявления вроде те же,что и в первом,не понимаю в чем проблема)

C++

void qqw::perech() {
    int  fl;
    all *p1;
    all1 *p2;
    while (beg2 != NULL) {
        p2 = beg2;
        beg2 = beg2->next;
        delete p2;
    }
    for (p1 = beg1; p1 != NULL; p1 = p1->next) {
        fl = 1;
        for (p2 = beg2; p2 != NULL; p2 = p2->next) {
            if (strcmp(p1->mark, p2->mark) == 0) {
                fl = 0;
                p2->num += p1->num;
                break;
            }
        }
        if (fl == 1)
        {
            p2 = new all1;
            if (p2 == NULL) { cout << "Нет динамической памяти. \n"; _getch(); exit(1); }
            strcpy(p2->count, p1->count);
            strcpy(p2->mark, p1->mark);
            p2->num = p1->num;
            p2->next = beg2;
            beg2 = p2;
        }
    }
    cout << "Перечень создан.";
    _getch();
}
void qqw::perechout() {
    all1 *p;
    cout << "  |------------|----------|" << endl;
    cout << "  |    Марка   |   Кол-во |" << endl;
    cout << "  |------------|----------|" << endl;
    for (p = beg2; p != NULL; p = p->next) {
        cout << "  | " << setw(10) << p->mark << " |   " << setw(6) << p->num << " |" << endl;
        cout << "  |------------|----------|" << endl;
    }
    _getch();
    system("cls");
    return;
}
void qqw::makeperech() {
    all1 *p3;//Рабочие переменные
    all1 *p2;
    all *p1;
    int  fl; //флаг
    while (beg3 != NULL) {
        p3 = beg3;
        beg3 = beg3->next;
        delete p3;
    }
    p3 = beg3;
    p2 = beg2;
    p1 = beg1;
    max = p2->num;
    while (p2 != NULL) {
        if (p2->num > max) max = p2->num;
        p2 = p2->next;
    }
    for (p1 = beg1; p1 != NULL; p1 = p1->next){
        for (p2 = beg2; p2 != NULL; p2 = p2->next) {
            [COLOR="Orange"]if ((p2->num == max) && ((strcmp(p2->mark, p1->mark)) == 0)){[/COLOR]  //ВОТ ТУТ
                p3 = new all1;
                if (p3 == NULL) {
                    cout << "Нет памяти." << endl;
                    _getch();
                    return;
                }
                strcpy(p1->count, p3->count);
                p3->next = beg3;
                beg3 = p3;
            }
        }
    }
    cout << "Перечень создан.";
    _getch();
}

	06.12.2015, 17:18