Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
Рейтинг 4.89/18: Рейтинг темы: голосов - 18, средняя оценка - 4.89
0 / 0 / 0
Регистрация: 02.11.2018
Сообщений: 17
1

Нарушение доступа для чтения. this было 0xFFFFFFFFFFFFFFFF

09.12.2018, 17:25. Показов 3586. Ответов 3
Метки нет (Все метки)

Здравствуйте, суть проблемы : в программе создается массив объектов данного образца, а при освобождении памяти т.е удалении данного массива происходишь ошибка "Нарушение доступа для чтения. this было 0xFFFFFFFFFFFFFFFF". Копался по форумам все сказали что ошибка из за того что не инициализирую память. Cделал FixErrOne таким образом не помогло. Помогите пожалуйста люди добрые.

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
#define FixErrOne(A) ( strcpy( A , "Fix Err" )); 
 
class company_owner 
{
protected:
    char *legal_address;
    char *company_owner_name;
    int long company_owner_code;
public:
    company_owner()
    {
        legal_address = new char[64];
        FixErrOne(legal_address);
        company_owner_name = new char[32];
        FixErrOne(company_owner_name);
        company_owner_code = 0;
    };
    ~company_owner()
    {
        delete[] legal_address;
        delete[] company_owner_name;
    };
    void operator = (company_owner &Etalon)
    {
        company_owner_code = Etalon.company_owner_code;
        strcpy(company_owner_name, Etalon.company_owner_name);
        strcpy(legal_address, Etalon.legal_address);
    };
};
__________________
Помощь в написании контрольных, курсовых и дипломных работ здесь
0
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
09.12.2018, 17:25
Ответы с готовыми решениями:

Нарушение доступа для чтения. _Pnext было 0x94D3D4
При выводе списка студентов выводит исключение "Нарушение доступа для чтения. _Pnext было...

Вызвано исключение: нарушение доступа для чтения. s было 0x1110112
Товарищи форумчане, нужна ваша помощь! (Для простоты объяснения прикреплю скрин) Насколько я...

Вызвано исключение: нарушение доступа для чтения. this было 0xCDCDCDCD
Есть у меня код со структурой с динамическими массивами внутри. Через клаву я ввожу данные. И...

Ошибка: Вызвано исключение: нарушение доступа для чтения. print было nullptr
Здравствуйте, практикуясь с задание по односвязному списку застрял на непонятной для меня ошибки на...

3
Мозгоправ
1730 / 1024 / 468
Регистрация: 01.10.2018
Сообщений: 2,137
Записей в блоге: 2
09.12.2018, 18:02 2
Не вижу ничего некошерного в вашем коде. Есть несколько шероховатостей.
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
#include <iostream>
#include <array>
 
using namespace std;
 
class company_owner {
protected:
    char *legal_address;
    char *company_owner_name;
    int long company_owner_code;
public:
    company_owner() {
        legal_address = new char[64];
        legal_address[0] = '\0';
        company_owner_name = new char[32];
        company_owner_name[0] = '\0';
        company_owner_code = 0;
    };
    ~company_owner() {
        delete[] legal_address;
        delete[] company_owner_name;
    };
    company_owner& operator = (const company_owner &Etalon) {
        company_owner_code = Etalon.company_owner_code;
        strcpy(company_owner_name, Etalon.company_owner_name);
        strcpy(legal_address, Etalon.legal_address);
        return *this;
    };
};
 
int main() {
    company_owner a, b;
    a = b;
}
Описанную вами ошибку не наблюдаю. Ни в вашем оригинальном коде, ни в модифицированном.

Неплохо было бы ещё сделать, как минимум, конструктор копирования. А как максимум, ещё конструктор копирования перемещением и оператор присваивания с перемещением.

Для меня, правда, осталось загадкой как вы засовываете данные в экземпляр этого класса и получаете их обратно.
0
0 / 0 / 0
Регистрация: 02.11.2018
Сообщений: 17
09.12.2018, 18:10  [ТС] 3
Массив объектов через new, ошибка в момент delete массив.

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
class company_owner 
{
protected:
    char *legal_address;
    char *company_owner_name;
    int long company_owner_code;
public:
    company_owner()
    {
        legal_address = new char[64];
        FixErrOne(legal_address);
        company_owner_name = new char[32];
        FixErrOne(company_owner_name);
        company_owner_code = 0;
    };
    ~company_owner()
    {
        delete[] legal_address;
        delete[] company_owner_name;
    };
    void operator = (company_owner &Etalon)
    {
        company_owner_code = Etalon.company_owner_code;
        strcpy(company_owner_name, Etalon.company_owner_name);
        strcpy(legal_address, Etalon.legal_address);
    };
    void set_owner();
    void show_owner();
    void redact_owner();    
    friend void load_data_base();
    friend void record_data_base();
    friend class deal;
    friend class product;
    friend void Show_All(int);
    friend void Edit_All();
    friend void Delit();
    friend void find_owner();
    friend void sort_owner();
};
 
void company_owner::set_owner()
{
    char legal_address[128], company_owner_name[64];
    int long company_owner_code;
    int flag = 0, longs;
    system("cls");
    do {
        flag = 0;
        cout << "\nВведите наименование кампании: ";
        cin.clear();
        cin.getline(company_owner_name, 64);
        longs = strlen(company_owner_name);
        if (longs > 32)
        {
            system("cls");
            cout << "Превышена максимальная длина поля!\a\n";
            flag = 1;
            continue;
        }
        for (int i = 0; i < longs && flag != 1; i++)
        {
            if ((company_owner_name[i] >= 'A' || company_owner_name[i] <= 'z') || (company_owner_name[i] >= 'А' || company_owner_name[i] <= 'я'));
            else flag = 1;
        }
        if (flag == 1) {
            system("cls");
            cout << "Недопустимые символы!\a\n";
            flag = 1;
            continue;
        }
    } while (flag != 0);
    strcpy(this->company_owner_name, company_owner_name);
    do {
        flag = 0;
        cout << "Введите юредический адрес компании: ";
        cin.clear();
        cin.getline(legal_address, 128);
        longs = strlen(legal_address);
        if (longs > 64)
        {
            system("cls");
            cout << "Превышена максимальная длина поля!\a\n";
            flag = 1;
            continue;
        }
        for (int i = 0; i < longs && flag != 1; i++)
        {
            if ((legal_address[i] >= 'A' || legal_address[i] <= 'z') || (legal_address[i] >= 'А' || legal_address[i] <= 'я'));
            else flag = 1;
        }
        if (flag == 1) {
            system("cls");
            cout << "Недопустимые символы!\a\n";
            flag = 1;
            continue;
        }
    } while (flag != 0);
    strcpy(this->legal_address, legal_address);
    do {
        flag = 0;
        do {
            do {
                cout << "Введите код компании: ";
                ignorecin;
            } while (!(cin >> company_owner_code));
            clearcin;
        } while (company_owner_code < 1);
        for (int i = 0; i < count_owner; i++)
            if (Owner_Data[i].company_owner_code == company_owner_code)flag = 1;
        if (flag == 1)
        {
            cerr << "Компания с таким кодом уже существует!\n\a";
            continue;
        }
    } while (flag == 1);
    this->company_owner_code = company_owner_code;
    
    company_owner *Buffer_owner;
    count_owner++;
    
    Buffer_owner = new company_owner[count_owner];
    for (int j = 0; j < count_owner; j++)
        Buffer_owner[j] = Owner_Data[j];
    
    delete[] Owner_Data;
    
    Owner_Data = new company_owner[count_owner + 1];
    for (int j = 0; j < count_owner; j++)
        Owner_Data[j] = Buffer_owner[j];
 
    delete[]Buffer_owner;
}
0
Мозгоправ
1730 / 1024 / 468
Регистрация: 01.10.2018
Сообщений: 2,137
Записей в блоге: 2
10.12.2018, 01:19 4
Извините, JesterHD, но у вы написали ахинею. И проверить что-то в этом нереально. Тем более, что вы дважды скидываете фрагменты кода, в то время, когда ошибка у вас возникает в рантайме. (Только не надо показывать ещё 100500 строк вашего кода. Лично я в этом г (ещё раз извините) разбираться не буду.)

Метод set_owner должен иметь сигнатуру
C++
1
bool set_owner(const char *address, const char *name, long code);
и должен проверять только длину переданных строк адреса и названия. Возвращаемое значение будет показывать были ли присвоены новые значения или что-то пошло не так.

Если вы считаете, что код овнера не должен быть отрицательным, то, опять извините, какого х вы его сделали типом со знаком? Что мешает сделать его unsigned long? Причём на этом вы получите вдвое больше допустимых кодов, чем на типе со знаком.

Если вы не подвязаны под конкретные значения длин строк в 32 и 64 байта, то я бы посоветовал использовать вместо C-строк тип для строк из STL std::string. Вы избавитесь от геморроя с выделением/освобождением памяти под строки, подсчётом длин, учётом завершающего нуля и пр. Кстати здесь вы тоже напахали в своём коде, поэтому ваше изделие может крашится в зависимости от введённых данных.

Проверку введённых строк (названиия и адреса) на допустимость надо вынести в отдельную функцию вне класса. Как я вижу, вы пытаетесь (неправильно) проконтролировать, что бы в названии были только большие и малые русские или латинские символы и длину строки. Максимально допустимую длину строки можно передать в функцию как параметр (а если вы всё-таки будете использовать std::string, то она вам не нужна), а всё остальное для обеих строк одинаково. (Обратите внимание, что я здесь не поднимаю проблем с национальными алфавитами и прочим Юникодом, а также почтовыми индексами, номерами улиц, домов, офисов и цифрами в названии компаний. А также с пробелами, тире, запятыми и прочими лишними знаками.)

Вообще из методов класса company_owner надо убирать всё, что прямо не относится к его данным. В том числе и ввод данных. Данные могут быть введены с клавиатуры, из файла, вычислены в другой части программы, сгенерированы зелёными человечками. Ваш класс предназначен для хранения двух строк и некоего числового кода? Ну вот пусть он этим и занимается. И ничем больше.

И, если вы не переходите на std::string, внесите в свою версию кода те изменения, которые я сделал в вашем коде в предыдущем посте и
Цитата Сообщение от L0M Посмотреть сообщение
сделать, как минимум, конструктор копирования. А как максимум, ещё конструктор копирования перемещением и оператор присваивания с перемещением.
Иначе возможны забавные эффекты.

PS. Директива препроцессора #define - зло. Параметризованный #define - большое зло.

Добавлено через 4 минуты
PPS. И для Owner_Data лучше использовать std::vector.
1
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
10.12.2018, 01:19

Вызвано необработанное исключение: нарушение доступа для чтения. InvMArr было 0x1110113
#include &quot;stdafx.h&quot; #include&lt;iostream&gt; #include&lt;stdlib.h&gt; #include&lt;time.h&gt; #include&lt;Windows.h&gt;...

При реализации метода Хаффмана выскакивает исключение: нарушение доступа для чтения. root было 0xCDCDCDCD
Здравствуйте! Разбираюсь с методом Хаффмана в Visual Studio, язык C++. Выскакивает...

Нарушение доступа для чтения
#include &quot;pch.h&quot; #include &lt;iostream&gt; #include &lt;fstream&gt; #include &lt;string&gt; using namespace std;...

Исключение - нарушение доступа для чтения
Добрый вечер, столкнулся с проблемой нарушения доступа для чтения 0х1110112. Задача - написать...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
4
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2022, CyberForum.ru