Вызвано исключение: нарушение доступа для записи. 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;
}

Новые блоги и статьи Все статьи Все блоги /
Перемещение выделенных строк ТЧ из одного документа в другой Maks 30.03.2026 Реализация из решения ниже выполнена на примере нетипового документа "ВыдачаОборудованияНаСпецтехнику" с единственной табличной частью "ОборудованиеИКомплектующие" разработанного в конфигурации КА2. . . .	Functional First Web Framework Suave DevAlt 30.03.2026 Sauve. IO Апнулись до NET10. Из зависимостей один пакет, работает одинаково хорошо как в режиме проекта так и в интерактивном режиме. из сложностей - чисто функциональный подход. Решил. . .	Автоматическое создание документа при проведении другого документа Maks 29.03.2026 Реализация из решения ниже выполнена на нетиповых документах, разработанных в конфигурации КА2. Есть нетиповой документ "ЗаявкаНаРемонтСпецтехники" и нетиповой документ "ПланированиеСпецтехники". В. . .	Настройка движения справочника по регистру сведений Maks 29.03.2026 Решение ниже реализовано на примере нетипового справочника "ТарифыМобильнойСвязи" разработанного в конфигурации КА2, с целью учета корпоративной мобильной связи в коммерческом предприятии. . . .
Автозаполнение реквизита при выборе элемента справочника Maks 27.03.2026 Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. При выборе "Спецтехники" (Тип Справочник. Спецтехника), заполняется. . .	Сумматор с применением элементов трёх состояний. Hrethgir 26.03.2026 Тут. https:/ / fips. ru/ EGD/ ab3c85c8-836d-4866-871b-c2f0c5d77fbc Первый документ красиво выглядит, но без схемы. Это конечно не даёт никаких плюсов автору, но тем не менее. . . всё может быть. . .	Автозаполнение реквизитов при создании документа Maks 26.03.2026 Программный код из решения ниже размещается в модуле объекта документа, в процедуре "ПриСозданииНаСервере". Алгоритм проверки заполнения реализован для исключения перезаписи значения реквизита,. . .	Команды формы и диалоговое окно Maks 26.03.2026 1. Команда формы "ЗаполнитьЗапчасти". Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. В качестве источника данных. . .

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

	Вызвано исключение: нарушение доступа для записи. this->max было nullptr. Подскажите в чем проблема,пожалуйста 23.03.2020, 22:31. Показов 2731. Ответов 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