Вызвано исключение: нарушение доступа для записи. this->max было nullptr. Подскажите в чем проблема,пожалуйста

@FaceProgrammist · Регистрация: 22.03.2020

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

Нужно вводить в конструктор 3 зарплаты,и чтобы выводилось значение (максимальная зп - минимальная зп).
Только пытаюсь разобраться в ооп,поэтому прошу обьяснить нормально!
#include <iostream>
using namespace std;
class Salary {
public:
int *Salary1;
int *Salary2;
int *Salary3;
int *max;
int *min;

Salary(int a, int s, int d);
~Salary();
void run();

};
Salary::Salary(int a, int s, int d) {
Salary1 = new int(a);
Salary2 = new int(s);
Salary3 = new int(d);
}

void Salary::run() {
if (*Salary1 >= *Salary2) {
*max = *Salary1;
}
else { *max = *Salary2; }
if (*Salary3 >= *max) {
*Salary3 = *max;
}

if (*Salary1 <= *Salary2) {
*min = *Salary1;
}
else {
*min = *Salary2;
}
if (*Salary3 <= *min) {
*Salary3 = *min;
}

cout << "Разница между максимальной и минимальной - " << max - min << endl;
}
Salary::~Salary() {
delete(Salary1, Salary2, Salary3);
}

int main()
{
setlocale(LC_ALL, "Russian");
Salary a(100, 500, 1000);
a.run();
}

@DrOffset · 23.03.2020, 22:35

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

Только пытаюсь разобраться в ооп,поэтому прошу обьяснить нормально!

ООП тут не при чем. Ошибки ваши с ООП связаны.

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

int *max;
int *min;

Указатели объявили, но на что они у вас указывают? Правильно, ни на что.

На самом деле они (указатели) вам и не нужны тут совсем. Храните min и max просто по значению.

@FaceProgrammist · 23.03.2020, 22:41 **[ТС]**

да,спасибо ,помогло,но новая проблема - выводится число 400 ,то есть 500 - 100 ,но максимальая 1000,почиму не из 1000 отнимается?

@DrOffset · 23.03.2020, 23:31

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

почиму не из 1000 отнимается?

Честно говоря, потому что у вас со всех сторон переусложненный код написан.
*) Выделение памяти через new, которое не нужно.
*) функция run, которая делает не свое дело (такое, например, как вывод в консоль)
*) min и max, которые по-хорошему вообще не должны быть членами класса.
*) сам алгоритм сомнительный, зачем-то меняет данные класса, нарушая инвариант.
В общем, все тут плохо, неудивительно, что вы быстро теряете нить и делаете ошибки. Попробуйте не усложнять.

Кликните здесь для просмотра всего текста

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
#include <iostream>
 
using namespace std;
class Salary {
public:
    int Salary1;
    int Salary2;
    int Salary3;
 
    Salary(int a, int s, int d);
 
    int run();
};
 
Salary::Salary(int a, int s, int d) {
    Salary1 = a;
    Salary2 = s;
    Salary3 = d;
}
 
int Salary::run() {
    int min = std::min(Salary1, std::min(Salary2, Salary3));
    int max = std::max(Salary1, std::max(Salary2, Salary3));
    
    return max - min;
}
 
int main()
{
    setlocale(LC_ALL, "Russian");
    Salary a(100, 500, 1000);
    int d = a.run();
    
    cout << "Разница между максимальной и минимальной - " << d << endl;
}

Если не хотите использовать библиотечные функции min\max, можете написать свои - это очень просто:

C++
1
2
3
4
5
6
7
8
int imin(int a, int b)
{
    return (b < a) ? b : a;
}
int imax(int a, int b)
{
    return (b > a) ? b : a;
}

Новые блоги и статьи Все статьи Все блоги /
Загрузка PNG-файла с альфа-каналом с помощью библиотеки SDL3_image на Android 8Observer8 27.01.2026 Содержание блога SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .	влияние грибов на сукцессию anaschu 26.01.2026 Бифуркационные изменения массы гриба происходят тогда, когда мы уменьшаем массу компоста в 10 раз, а скорость прироста биомассы уменьшаем в три раза. Скорость прироста биомассы может уменьшаться за. . .	Воспроизведение звукового файла с помощью SDL3_mixer при касании экрана Android 8Observer8 26.01.2026 Содержание блога SDL3_mixer - это библиотека я для воспроизведения аудио. В отличие от инструкции по добавлению текста код по проигрыванию звука уже содержится в шаблоне примера. Нужно только. . .	Установка Android SDK, NDK, JDK, CMake и т.д. 8Observer8 25.01.2026 Содержание блога Перейдите по ссылке: https:/ / developer. android. com/ studio и в самом низу страницы кликните по архиву "commandlinetools-win-xxxxxx_latest. zip" Извлеките архив и вы увидите. . .
Вывод текста со шрифтом TTF на Android с помощью библиотеки SDL3_ttf 8Observer8 25.01.2026 Содержание блога Если у вас не установлены Android SDK, NDK, JDK, и т. д. то сделайте это по следующей инструкции: Установка Android SDK, NDK, JDK, CMake и т. д. Сборка примера Скачайте. . .	Использование SDL3-callbacks вместо функции main() на Android, Desktop и WebAssembly 8Observer8 24.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	моя боль iceja 24.01.2026 Выложила интерполяцию кубическими сплайнами www. iceja. net REST сервисы временно не работают, только через Web. Написала за 56 рабочих часов этот сайт с нуля. При помощи perplexity. ai PRO , при. . .	Модель сукцессии микоризы anaschu 24.01.2026 Решили писать научную статью с неким РОманом

@FaceProgrammist 1 / 1 / 0 Регистрация: 22.03.2020 Сообщений: 131

	Вызвано исключение: нарушение доступа для записи. this->max было nullptr. Подскажите в чем проблема,пожалуйста 23.03.2020, 22:31. Показов 2699. Ответов 3 Метки нет (Все метки) Нужно вводить в конструктор 3 зарплаты,и чтобы выводилось значение (максимальная зп - минимальная зп). Только пытаюсь разобраться в ооп,поэтому прошу обьяснить нормально! #include <iostream> using namespace std; class Salary { public: int Salary1; int Salary2; int Salary3; int max; int min; Salary(int a, int s, int d); ~Salary(); void run(); }; Salary::Salary(int a, int s, int d) { Salary1 = new int(a); Salary2 = new int(s); Salary3 = new int(d); } void Salary::run() { if (Salary1 >= Salary2) { max = Salary1; } else { max = Salary2; } if (Salary3 >= max) { Salary3 = max; } if (Salary1 <= Salary2) { min = Salary1; } else { min = Salary2; } if (Salary3 <= min) { Salary3 = *min; } cout << "Разница между максимальной и минимальной - " << max - min << endl; } Salary::~Salary() { delete(Salary1, Salary2, Salary3); } int main() { setlocale(LC_ALL, "Russian"); Salary a(100, 500, 1000); a.run(); } 0

@FaceProgrammist 1 / 1 / 0 Регистрация: 22.03.2020 Сообщений: 131
	23.03.2020, 22:41 [ТС]
	да,спасибо ,помогло,но новая проблема - выводится число 400 ,то есть 500 - 100 ,но максимальая 1000,почиму не из 1000 отнимается? 0