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

@JesterHD · Регистрация: 02.11.2018

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

Здравствуйте, суть проблемы : в программе создается массив объектов данного образца, а при освобождении памяти т.е удалении данного массива происходишь ошибка "Нарушение доступа для чтения. 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);
    };
};

@L0M · 09.12.2018, 18:02

Не вижу ничего некошерного в вашем коде. Есть несколько шероховатостей.

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;
}

Описанную вами ошибку не наблюдаю. Ни в вашем оригинальном коде, ни в модифицированном.

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

Для меня, правда, осталось загадкой как вы засовываете данные в экземпляр этого класса и получаете их обратно.

@JesterHD · 09.12.2018, 18:10 **[ТС]**

Массив объектов через 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;
}

@L0M · 10.12.2018, 01:19

Извините, 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.

Новые блоги и статьи Все статьи Все блоги /
Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .
Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .