Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.88/8: Рейтинг темы: голосов - 8, средняя оценка - 4.88
0 / 0 / 0
Регистрация: 29.05.2020
Сообщений: 78

Не присваивается значение переменной

13.06.2020, 10:23. Показов 1706. Ответов 19
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
У меня есть объект класса Bag, который хранит в себе объекты типа Product, для чего я писал свой контейнер похожий по функционалу на вектор (в задании нельзя юзать встроенные) В метод setTemp передаётся температура, которая должна установиться в поле temp соответствующего объекта. В функции. всё нормально передаётся и вроде бы присваивается (сто раз проходился отладчиком), но после окончания действия метода, поле принимает прошлое значение. Что не так ?

файл Product.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
#pragma once
#include <iostream>
#include <string>
 
using namespace std;
 
class Product {
public:
    Product(const string& _name, double _mass, double _temp,  double _tempMin, double _tempMax, double _heatCapasity);
    Product();
 
    enum class ProductState { Normal, Overheated, Frostbitten };
 
    void change_status();//метод, который меняет состояние продукта в зависимости от количества переданной энергии
    double getMinTemp() const;
    double getTemp() const;
    double getMaxTemp() const;
    string getName() const;
    ProductState getStatus() const;
    double getMass() const;
    double getHeatCapasity() const;
    void setTemp(double temp); 
 
private:
    string name;
    double mass; // единица измерения: кг.
    double temp; // единица измерения: оС
    double tempMax; // единица измерения: оС
    double tempMin; // единица измерения: оС
    ProductState status;
    double heatCapasity;    //unit: J/(kg*oC)
};
Файл Product.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
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
#include "Product.h"
#include "Bag.h"
 
Product::Product(const string& _name, double _mass, double _temp, double _tempMax, double _tempMin, double _heatCapasity) :
    name{ _name } 
{
    if (_mass > 0)
        mass = _mass;
    else
        throw invalid_argument("Некорректное значение массы!!! ");
 
    if (_temp > -273.15)
        temp = _temp;
    else 
        throw invalid_argument("Значение температуры не может быть меньше абсоютного 0 !!! ");
 
    if (_tempMax < _tempMin)
        throw invalid_argument("Значение максимально допустимой температуры продукта не может быть меньше минимальной !!! "); 
    
    if (_tempMax > -273.15)
        tempMax = _tempMax;
    else
        throw invalid_argument("Значение максимально допустимой температуры не может быть меньше абсоютного 0 !!! ");
 
    if (_tempMin > -273.15)
        tempMin = _tempMin;
    else
        throw invalid_argument("Значение минимально допустимой температуры не может быть меньше абсоютного 0 !!! ");
 
    if (_heatCapasity > 0)
        heatCapasity = _heatCapasity;
    else
        throw invalid_argument("Некорректное значение теплоёмкости продукта!!! ");
    
    if (temp < tempMin)
        status = ProductState::Frostbitten;
    else if (temp > tempMax)
        status = ProductState::Overheated;
    else
        status = ProductState::Normal; 
}
 
Product::Product() : name{ '\0' }, mass{ 0 }, temp{ 0 }, tempMax{ 0 }, tempMin{ 0 }, heatCapasity{ 0 }
    {}
 
void Product::change_status() {
    if (temp > tempMax)
        status = ProductState::Overheated;
    else {
        if (temp < tempMin) status = ProductState::Frostbitten;
        else status = ProductState::Normal;
    }
}
 
double Product::getMaxTemp() const {
    return tempMax;
}
 
double Product::getMinTemp() const {
    return tempMin;
}
 
double Product::getTemp() const {
    return temp;
}
 
Product::ProductState Product::getStatus() const {
    return status;
}
 
string Product::getName() const {
    return name;
}
 
 
 
double Product::getMass() const {
    return mass;
}
 
double Product::getHeatCapasity() const {
    return heatCapasity;
}
 
 
void Product::setTemp(double temp) {
    if (temp > -273.15)
        this -> temp = temp;
    else
        throw invalid_argument("Значение максимально допустимой температуры не может быть меньше абсоютного 0 !!! ");
    if (status == ProductState::Normal) // если продукт в нормальном состоянии, то его состояние может испортиться, если продукт уже испорчен, то своё состояние поменять уже не может
        change_status();
}
Файл Bag.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
#pragma once
#include "Product.h"
#include <iostream>
#include <MyContainer.h>  
using namespace std; 
 
class Bag {
public:
    Product operator[](int index) const;
    void addProduct(const Product& product); //метод добавления нового продукта в корзину
    void delProduct(int delNumber); //метод удаления i продукта
    void getInfoProduct(int infoNumber,string arrPrint[]) const; //метод, возращающий массив с информацией о i продукте
    double getMassBag() const; //метод, возращающий массу сумки
    int getCountProduct() const; //метод, возращающий количество продуктов в сумке
    double getTempBag() const; //метод, возращающий текущую температуру в сумке
    void clearBag(); //метод, который удаляет все продукты в сумке
    void setMaxMassBag(double massBag); //метод установки максимальной массы сумки
    double getMaxMassBag() const; //метод возврата максимальной массы сумки
    void listProducts(string namePrint[]); //метод выводящий список продуктов
    double summHCM(); //метод суммы массы и теплоёмкости всех продуктов
    int getCountBadProduct();//метод, возращающий количество испорченных продуктов в сумке
    void delLastProduct(); //метод, удаляющий последний добавленный продукт в сумку
    int testProductStatus(); //метод, возращающий состояние продукта, который мы хотим протестировать, испортится ли он при помещении его в сумку
 
private:
    myContainer <Product> bag; 
    double maxMassBag;
    int countBadProducts = 0;
    double tempBag = 0; 
};
Файл Bag.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
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
#include "Bag.h"
#include "Product.h"
#include <algorithm>
#include <MyContainer.h>  
 
void Bag::addProduct(const Product& product) {
 
    if (getMassBag() + product.getMass() > getMaxMassBag())
        throw invalid_argument("Превышена максимальная масса, которую выдерживает сумка!");
 
    bag.push_back(product);
    tempBag = tempBag + (product.getHeatCapasity() * product.getMass() * product.getTemp()) / summHCM();
 
    for (int i = 0; i < bag.size(); i++) {
        bag[i].setTemp(tempBag);
    }
}
 
double Bag::summHCM() {
    double sumHCM = 0;
    for (int i = 0; i < bag.size(); i++)
        sumHCM = sumHCM + (bag[i].getHeatCapasity() * bag[i].getMass());
    return sumHCM;
}
 
void Bag::delProduct(int delNumber) {
    if (delNumber < 0 || delNumber >= bag.size())
        throw invalid_argument("Номер продукта не может быть отрицательным или превышать номер последнего продукта в сумке!!! ");
 
    bag.erase(delNumber);
}
 
void Bag::getInfoProduct(int infoNumber, string arrPrint[]) const {
    arrPrint[0] = bag[infoNumber].getName();
    arrPrint[1] = to_string(bag[infoNumber].getMass());
    arrPrint[2] = to_string(bag[infoNumber].getTemp());
    arrPrint[3] = to_string(bag[infoNumber].getMaxTemp());
    arrPrint[4] = to_string(bag[infoNumber].getMinTemp());
    arrPrint[5] = to_string(bag[infoNumber].getHeatCapasity());
 
    Product::ProductState status = bag[infoNumber].getStatus();
    switch (status)
    {
    case Product::ProductState::Normal: arrPrint[6] = "Нормальное";
        break;
    case Product::ProductState::Overheated: arrPrint[6] = "Перегрет";
        break;
    case Product::ProductState::Frostbitten: arrPrint[6] = "Переморожен";
        break;
    }
}
 
int Bag::testProductStatus() {
    int stateProduct;
    Product::ProductState status = bag[bag.size() - 1].getStatus();
    switch (status)
    {
    case Product::ProductState::Normal:stateProduct = 1;
        break;
    case Product::ProductState::Overheated: stateProduct = 2;
        break;
    case Product::ProductState::Frostbitten: stateProduct = 3;
    }
    return stateProduct;
}
 
double Bag::getMassBag() const {
    double massBag = 0;
    for (int i = 0; i < bag.size(); i++)
        massBag += bag[i].getMass();
    return massBag;
}
 
int Bag::getCountProduct() const {
    return bag.size();
}
 
void Bag::setMaxMassBag(double maxMassBag) {
    if (maxMassBag > 0) {
        this->maxMassBag = maxMassBag;
 
    }
    else
        throw invalid_argument("Некорректное значение массы сумки!");
}
 
double Bag::getMaxMassBag() const {
    return maxMassBag;
}
 
void Bag::listProducts(string namePrint[]) {
    for (int i = 0; i < bag.size(); i++)
        namePrint[i] = bag[i].getName();
}
 
double Bag::getTempBag() const {
    return tempBag;
}
 
void Bag::clearBag() {
    bag.clear();
    tempBag = 0;
}
 
int Bag::getCountBadProduct() {
    countBadProducts = 0;
    for (int i = 0; i < bag.size(); i++) {
        if (bag[i].getStatus() != Product::ProductState::Normal)
            countBadProducts++;
    }
    return countBadProducts;
}
 
void Bag::delLastProduct() {
    bag.pop_back();
}

Файл MyContainer.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
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
#pragma once
#include <iostream>
using namespace std;
 
template <typename T>
class myContainer
{
public:
    ~myContainer(); 
    int size() const;
    void push_back(const T& test);
    void pop_back();
    void erase(int delNumber);
    void clear();
    bool empty();
    T operator [] (int index) const;
    
private:
    int size_arr = 0;
    T* arr = NULL;
};
 
template <class  T>
 myContainer<T>::~myContainer() {
     delete[] arr; 
}
 
template <class  T>
int myContainer<T>:: size() const {
    return size_arr;
}
 
template <class  T>
void myContainer<T>:: push_back(const T& ob) {
    T* newArray = new T[size() + 1]; 
    for (int i = 0; i < size_arr; i++)
        newArray[i] = arr[i];
    newArray[size_arr] = ob;
    size_arr++;
    delete[] arr;
    arr = newArray;
} 
 
template <class  T>
void myContainer<T>:: pop_back() {
    size_arr--;
    T* newArray = new T[size()];
    for (int i = 0; i < size_arr; i++)
        newArray[i] = arr[i];
    delete[] arr;
    arr = newArray;
}
 
template <class  T>
void myContainer<T>:: erase(int delNumber) {
    T* newArray = new T[size() -  1];
    int i = 0;
    while (i < delNumber) {
        newArray[i] = arr[i];
        i++;
    }
    while (i < size_arr - 1) {
        newArray[i] = arr[i + 1];
        i++;
    }
    size_arr--;
    delete[] arr;
    arr = newArray;
    
}
 
template <class  T>
void myContainer<T>:: clear() {
    size_arr = 0;
    delete[] arr;
    T* newArray = new T[size()];
    arr = newArray;
}
 
template <class  T>
bool myContainer<T>:: empty() {
     return size_arr == 0;
} 
 
template <class  T>
 T myContainer<T>:: operator[] (int index) const {
    return arr[index];
}
0
Лучшие ответы (1)
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
13.06.2020, 10:23
Ответы с готовыми решениями:

Не присваивается значение переменной
Привет всем форумчанам! :) Собственно само задание в прикрепленных изображениях. Независимо от того, какое бы я значение ни ввел,...

Не присваивается значение переменной в конструкции if-else
В чем может быть проблема? #include &lt;iostream&gt; using namespace std; int main() { int pmCondition = 50; int pmState = 0;...

Переменной float значение присваивается с некоторой погрешностью
Доброго всем времени суток! В С++ Builder 6 при отладке кода не могу понять следующий момент: Задаю переменные float Ro_b, Ro_a; ...

19
"C with Classes"
2022 / 1404 / 523
Регистрация: 16.08.2014
Сообщений: 5,885
Записей в блоге: 1
13.06.2020, 10:39
Цитата Сообщение от Ogurechik Посмотреть сообщение
В метод setTemp передаётся температура, которая должна установиться в поле temp соответствующего объекта.
так и прислал бы только код в котором ошибка проявляется, зачем всю дребедень скидывать, удали все что не относиться к ошибке и пробегись еще раз отладчиком, тогда возможно увидишь в чем проблема.
0
0 / 0 / 0
Регистрация: 29.05.2020
Сообщений: 78
13.06.2020, 10:51  [ТС]
Один класс без другого работать не будет, к тому же я описал, в каком методе траблы возникли. Отладчиком побегать ума хватило самому
0
653 / 466 / 183
Регистрация: 23.04.2019
Сообщений: 1,987
13.06.2020, 10:53
Цитата Сообщение от Ogurechik Посмотреть сообщение
void Product::setTemp(double temp) {
    if (temp > -273.15)
        this -> temp = temp;
на всякий, советую назвать аргумент функции new_temp, а потом попробовать

Добавлено через 38 секунд
возможно компилятор думает что temp > -273.15 - приватный член класса
0
"C with Classes"
2022 / 1404 / 523
Регистрация: 16.08.2014
Сообщений: 5,885
Записей в блоге: 1
13.06.2020, 10:59
Цитата Сообщение от Ogurechik Посмотреть сообщение
Один класс без другого работать не будет
я тебе не про удаление класса говорю, а удалить из класса все методы и поля не нужные, главное что бы ошибка воспроизводилась.

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

Добавлено через 1 минуту
даже то что скролом крутить лишней раз надо, что бы твою ошибку найти, это уже плохо
1
0 / 0 / 0
Регистрация: 29.05.2020
Сообщений: 78
13.06.2020, 11:03  [ТС]
Проблема не в этом, проверял (

Добавлено через 1 минуту
постараюсь переделать

Добавлено через 3 минуты
как то раз выложил сюда вопрос и, как ты сказал, убрал всё не нужное, тут люди сказали, где остальной код. Вот и выложил в этот раз всё, чтоб наверняка
0
"C with Classes"
2022 / 1404 / 523
Регистрация: 16.08.2014
Сообщений: 5,885
Записей в блоге: 1
13.06.2020, 11:06
Цитата Сообщение от Ogurechik Посмотреть сообщение
убрал всё не нужное
значит много убрал
0
653 / 466 / 183
Регистрация: 23.04.2019
Сообщений: 1,987
13.06.2020, 11:09
C++
1
2
3
4
5
6
7
8
int main()
{
    Product inst("name", 90, 30, 120, -120, 2);
    std::cout << inst.getTemp() << std::endl;
    inst.setTemp(40);
    std::cout << inst.getTemp() << std::endl;
 
}
этот код выводит:
30
40
можете ещё раз объяснить, как проявляется проблема?
1
"C with Classes"
2022 / 1404 / 523
Регистрация: 16.08.2014
Сообщений: 5,885
Записей в блоге: 1
13.06.2020, 11:12
Ogurechik, в методе у тебя все нормально, где то в другом месте косяк, я бы давно нашел если бы кода меньше было

Добавлено через 2 минуты
может в классе myContainer косяк
1
0 / 0 / 0
Регистрация: 29.05.2020
Сообщений: 78
13.06.2020, 11:20  [ТС]
Не, класс MyContainer я прописывал после основы с Bag. Bag когда я писал, то использовал std:vector там та же проблема

Добавлено через 47 секунд
Получается метод не при чём. Странно, где же дыра
0
 Аватар для zayats80888
6352 / 3523 / 1428
Регистрация: 07.02.2019
Сообщений: 8,995
13.06.2020, 11:29
Цитата Сообщение от Ogurechik Посмотреть сообщение
C++
1
2
3
4
template <class  T>
 T myContainer<T>:: operator[] (int index) const {
    return arr[index];
}
Ну и в чем смысл оператора возвращающего копию, если ты используешь его для изменения значения в контейнере?
2
"C with Classes"
2022 / 1404 / 523
Регистрация: 16.08.2014
Сообщений: 5,885
Записей в блоге: 1
13.06.2020, 11:31
Ogurechik, скинь как используешь все это добро.
0
0 / 0 / 0
Регистрация: 29.05.2020
Сообщений: 78
13.06.2020, 11:37  [ТС]
Т.е надо передавать по ссылке или как ?

Добавлено через 52 секунды
Ну вообще я учусь основам и в ВУЗе это моя семестровая работа
0
"C with Classes"
2022 / 1404 / 523
Регистрация: 16.08.2014
Сообщений: 5,885
Записей в блоге: 1
13.06.2020, 11:43
Лучший ответ Сообщение было отмечено Ogurechik как решение

Решение

Цитата Сообщение от Ogurechik Посмотреть сообщение
Т.е надо передавать по ссылке или как ?
да, там ссылка нужна

Добавлено через 1 минуту
а вообще два оператора:
C++
1
2
3
4
5
6
7
8
template <class  T>
const & T myContainer<T>:: operator[] (int index) const {
    return arr[index];
}
template <class  T>
 T& myContainer<T>:: operator[] (int index) const {
    return arr[index];
}
1
0 / 0 / 0
Регистрация: 29.05.2020
Сообщений: 78
13.06.2020, 11:47  [ТС]
Можно ли узнать, зачем 2 оператора и в чём отличие константного ?
0
 Аватар для zayats80888
6352 / 3523 / 1428
Регистрация: 07.02.2019
Сообщений: 8,995
13.06.2020, 11:49
_stanislav, вторая перегрузка не должна быть константным методом
1
"C with Classes"
2022 / 1404 / 523
Регистрация: 16.08.2014
Сообщений: 5,885
Записей в блоге: 1
13.06.2020, 11:54
Цитата Сообщение от zayats80888 Посмотреть сообщение
вторая перегрузка не должна быть константным методом
опечатка

Добавлено через 34 секунды
Ogurechik, Эффективное использование C++. 55 верных способов улучшить структуру и код ваших программ - Майерс Скотт Правило 3, можешь читануть.

Добавлено через 20 секунд
если нет книги
1
0 / 0 / 0
Регистрация: 29.05.2020
Сообщений: 78
13.06.2020, 11:57  [ТС]
Теперь всё работает, спасибо за помощь !!!
0
13.06.2020, 11:59

Не по теме:

Цитата Сообщение от _stanislav Посмотреть сообщение
Майерс Скотт Правило 3
C++
1
2
3
4
5
6
7
#include <iostream>
 
int main()
{
    std::cout << const("2 + 2 = ") << const(2) + const(2) << std::endl;
    return const(0);
}

0
13.06.2020, 12:26

Не по теме:

AndryS1, он дает рекомендации, а башка своя должна быть.

0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
13.06.2020, 12:26
Помогаю со студенческими работами здесь

Почему не присваивается значение переменной status типа ProductState (это я enum прописывал)
В конструкторе класса Product в зависимости от температуры в поле статус устанавливается одно из 3 значений, которые определены enum. В...

Почему переменной присваивается произвольное значение("мусор")?
Мой черновой вариант кода. Он очень черновой так что многое не суть важно, но переменной х я присваиваю значение 0. Если после приглашения...

В результате ввода переменной int присваивается символ
Создайте структуру с именем time. Три ее поля, имеющие тип int, будут называться hours, minutes и seconds. Напишите программу, которая...

Не корректно присваивается последняя цифра числа для переменной
Код наподобие этого: #include &lt;iostream&gt; using namespace std; int main() { double integer; int temp, last = 0;

Неправильно присваивается значение массиву
Неправильно присваивается значение массиву Вот код: #include &lt;iostream&gt; using namespace std; int main() { int n,p =...


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

Или воспользуйтесь поиском по форуму:
20
Ответ Создать тему
Новые блоги и статьи
Access
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
Мысли в слух
kumehtar 18.11.2025
Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .
Создание Single Page Application на фреймах
krapotkin 16.11.2025
Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .
Фото: Daniel Greenwood
kumehtar 13.11.2025
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru