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

@William_ · Регистрация: 08.05.2019

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

Задание из учебника Лафоре ООП:

Модифицируйте класс Person из программы PERSORT этой главы так, чтобы он включал в себя не только имя человека, но и
сведения о его зарплате в виде поля salary типа float. Вам будет необходимо изменить методы setName() и printName() на
setData() и printData(), включив в них возможность ввода и вывода значения salary, как это можно сделать с именем.
Вам также понадобится метод getSalary(). Используя указатели, напишите функцию salsort, которая сортирует указатели
массива persPtr по значениям зарплаты. Попробуйте вместить всю сортировку в функцию salsort(), не вызывая других функций,
как это сделано в программе PERSORT. При этом не забывайте, что операция -> имеет больший приоритет, чем операция *.

Я решил немного ~~выпендриться~~ разнообразить код и разбить его на несколько файлов:
main.cpp:

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 "Header.h"
#include "Header2.h"
using namespace std;
 
const short int MAX_LEN = 3;
 
 
int main() {
 
    int i = 0;
    Person **pers = new Person *[MAX_LEN];
    for (int j = MAX_LEN; j; --j) { pers[j] = new Person(); }
 
    while (i != MAX_LEN) {
        pers[i]->SetData();
        ++i;
    }
 
    for (int j = 0; j < MAX_LEN; --j) {
        std::cout << "Unsorted:\n";
        pers[i]->PrintData();
    }
    
    // Sorting
    bubble_sort(pers);
 
    for (int j = 0; j < MAX_LEN; --j) {
        std::cout << "Sorted:\n";
        pers[i]->PrintData();
    }
    system("pause");
    return 0;
}

Файлы-заголовки:
Header.h :

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
#pragma once
 
// 449
// 470
 
#define CONST_SIZE 40
 
 
class Person {
private:
    char* name;
    float* salary;
public:
    Person();
    ~Person();
 
    void SetData();
    void PrintData();
    char* getName();
    float getSalary();
};
 
Person::Person() {
    name = new char[CONST_SIZE];
    salary = new float;
    //std::cout << "Created: " << name << " " << *salary << "\n";
}
 
Person::~Person() { delete[] name; delete salary; }
 
void Person::SetData() {
    std::cout << "Cin >> name >> salary: ";
    std::cin >> name >> *salary;
    system("pause");
//  std::cin >> name >> *salary;
}
 
void Person::PrintData() {
    std::cout << "Name: " << name << "\nSalary: " << salary << "\n";
}
 
char* Person::getName() { return name; }
float Person::getSalary() { return *salary; }

Header2.h :

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
#pragma once
 
extern const short int MAX_LEN;
 
void bubble_sort(Person **ptr) {
    int i, j;
    Person *tempptr;
    for (i = 0; i < MAX_LEN - 1; ++i) {
        for (j = i + 1; j < MAX_LEN; ++j) {
            if ((*(ptr + i))->getSalary() > (*(ptr + j))->getSalary()) {
                tempptr = *(ptr + i);
                *(ptr + i) = *(ptr + j);
                *(ptr + j) = tempptr;
            }
        }
    }
}

Ошибка находится в файле Header.h. Там ошибка адресации в строке 33. Очень странное, на мой взгляд, поведение, ведь я выделил в конструкторе память для хранения float* salary и char* name, а ошибка все равно есть. Прошу помощи, не могу ни как разобраться.

@zayats80888 · 08.05.2019, 17:45

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

Person **pers = new Person *[MAX_LEN]

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

for (int j = MAX_LEN; j; --j) { pers[j] = new Person(); }

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

pers[i]->SetData()

дальше не смотрел

@William_ · 08.05.2019, 17:55 **[ТС]**

Да, я тоже только что несколько изменил main. Теперь лучше. Однако нарушение доступа для чтения происходит, когда
срабатывает метод PrintData(). Хотя до этого все данные вводятся корректно.

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
#include <iostream>
#include "Header.h"
#include "Header2.h"
using namespace std;
 
const short int MAX_LEN = 3;
 
 
int main() {
 
    int i = 0;
    Person **pers = new Person *[MAX_LEN];
    for (int j = MAX_LEN-1; j+1; --j) { pers[j] = new Person();}
 
    while (i != MAX_LEN) {
        pers[i]->SetData();
        ++i;
    }
    std::cout << "Unsorted:\n";
    for (int j = 0; j < MAX_LEN; ++j) {
        pers[i]->PrintData();
    }
    
    // Sorting
    bubble_sort(pers);
 
    for (int j = 0; j < MAX_LEN; ++j) {
        std::cout << "Sorted:\n";
        pers[i]->PrintData();
    }
    system("pause");
    return 0;
}

@zayats80888 · 08.05.2019, 18:03

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

Хотя до этого все данные вводятся корректно.

не корректно

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

std::cin >> name

cin.getline(name, CONST_SIZE);

Добавлено через 46 секунд

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

float* salary;

нафига?
И где деструктор?

@William_ · 08.05.2019, 18:14 **[ТС]**

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

cin.getline(name, CONST_SIZE);

Хорошо, пусть так:

C++
1
2
3
std::cout << "Cin >> name >> salary: ";
std::cin.getline(name, CONST_SIZE);
std::cin >> *salary;

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

float* salary;
нафига?

Хотелось использовать побольше указателей.

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

И где деструктор?

Вот же (строка 29, Header.h):

C++
1
Person::~Person() { delete[] name; delete salary; }

@zayats80888 · 08.05.2019, 18:24

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

Вот же (строка 29, Header.h)

не заметил

Добавлено через 5 минут

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

C++
1
2
for (int j = 0; j < MAX_LEN; ++j) {
* * * * pers[i]->PrintData();

ничего не замечаете?

@William_ · 08.05.2019, 18:28 **[ТС]**

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

for (int j = 0; j < MAX_LEN; ++j) {
* * * * pers[i]->PrintData();
ничего не замечаете?

Недоумеваю. Наверное, я не могу найти какую-то очевидную ошибку?

@zayats80888 · 08.05.2019, 18:32

j или i?

@William_ · 08.05.2019, 18:35 **[ТС]**

Ааааааааааааа.........
Спасибо!

А можно еще вопрос насчет std::cin.getline();
В этом коде что-то не так. После того, как ввожу второй раз данные сразу выводит неотсортированный массив pers.

C++
1
2
3
std::cout << "Cin >> name >> salary: ";
std::cin.getline(name, CONST_SIZE);
std::cin >> *salary;

@zayats80888 · 08.05.2019, 18:57

William_,

C++
1
2
3
    std::cin.getline(name, CONST_SIZE); //вводим имя жмем enter
    cin >> *salary; //вводим число
    cin.get();

Новые блоги и статьи Все статьи Все блоги /
Управление камерой с помощью скрипта 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 позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .

@zayats80888 6352 / 3523 / 1428 Регистрация: 07.02.2019 Сообщений: 8,995
	08.05.2019, 18:32
	Сообщение было отмечено William_ как решение Решение `j` или `i`? 0

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

Решение