Найти ошибку в задаче, тема массив обьектов

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

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

моя задача: в созданном классе создать массив объектов этого
класса и добавить поля: ID, weight и name. weight – генерируется рандомно в диапазоне от 1 до 100. Метод Run() должен вывести всю информацию о каждом объекте лежащем в инвентаре у которого 30<weight<90. Поля ID и name вводятся вручную. заполнение массива и информации о каждом объекте происходит в том же конструкторе.

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
#include <iostream>
using namespace std;
class Inventory {
public:
double name, id,weight;
int size;
Inventory* mass;
Inventory(int size);
~Inventory();
void run();
 
};
Inventory::Inventory(int size) {
mass = (Inventory*)calloc(size, sizeof(Inventory));
this->size = size;
for (int i = 0; i < size; i++)
 
{
cout << endl << "Введите name предмета" << endl;
cin >> mass[i].name;
cout << endl << "Введите id предмета" << endl;
cin >> mass[i].id;
mass[i].weight = rand() % 101;
 
}
}
Inventory::~Inventory() {
free(mass);
}
void Inventory::run() {
for (int i = 0; i < size; i++) {
if (30 < mass[i].weight < 90)
 
cout << "Имя предмета = " << mass[i].name << endl;
cout << "ID предмета = " << mass[i].id << endl;
cout << "Вес предмета = " << mass[i].weight << endl;
}
 
int main()
{
 
Inventory a(30); <-- почему-то не подсвечивается т.е. нельзя обратиться
a.run();
 
}

@hoggy · 01.04.2020, 21:21

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

почему-то не подсвечивается

закрывающую скобку потерял.

замени:

C++
1
2
cout << "Вес предмета = " << mass[i].weight << endl;
}

на:

C++
1
2
3
cout << "Вес предмета = " << mass[i].weight << endl;
}
}

по хорошему код должен быть с красивыми отступами.
тогда скобки не будут теряться.

так же, обрати внимание на предупреждение компилятора:

Code
1
2
3
source_file.cpp:35:8: warning: comparisons like ‘X<=Y<=Z’ do not have their mathematical meaning [-Wparentheses]
 if (30 < mass[i].weight < 90)
        ^

компилятор предупреждает тебя:
ты написал фигню, и работать эта фигня будет не так, как ты возможно ожидаешь.

возможно ты имел ввиду:

C++
1
 if (30 < mass[i].weight && mass[i].weight < 90)

?

так же обрати внимание на другие предупреждения компилятора:

Code
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
source_file.cpp:3:7: warning: ‘class Inventory’ has pointer data members [-Weffc++]
 class Inventory {
       ^
source_file.cpp:3:7: warning:   but does not override ‘Inventory(const Inventory&)’ [-Weffc++]
source_file.cpp:3:7: warning:   or ‘operator=(const Inventory&)’ [-Weffc++]
source_file.cpp: In constructor ‘Inventory::Inventory(int)’:
source_file.cpp:13:1: warning: ‘Inventory::name’ should be initialized in the member initialization list [-Weffc++]
 Inventory::Inventory(int size) {
 ^
source_file.cpp:13:1: warning: ‘Inventory::id’ should be initialized in the member initialization list [-Weffc++]
source_file.cpp:13:1: warning: ‘Inventory::weight’ should be initialized in the member initialization list [-Weffc++]
source_file.cpp:13:1: warning: ‘Inventory::size’ should be initialized in the member initialization list [-Weffc++]
source_file.cpp:13:1: warning: ‘Inventory::mass’ should be initialized in the member initialization list [-Weffc++]
source_file.cpp:14:50: warning: conversion to ‘size_t {aka long unsigned int}’ from ‘int’ may change the sign of the result [-Wsign-conversion]
 mass = (Inventory*)calloc(size, sizeof(Inventory));
                                                  ^
source_file.cpp:14:50: warning: use of old-style cast [-Wold-style-cast]

с такими предупреждениями не нужно потом удивляться,
если код не заработает.
или заработает но не так, как ожидается.

@FaceProgrammist · 01.04.2020, 21:34 **[ТС]**

hoggy,
я все это исправил и теперь только 1 ошибка - компилятору не хватает размера кучи - что это значит?

Новые блоги и статьи Все статьи Все блоги /
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Нашел на реддите интересную статью под названием «Кто-нибудь знает, где получить бесплатный компьютер или. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД 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 05.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 - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .

@FaceProgrammist 1 / 1 / 0 Регистрация: 22.03.2020 Сообщений: 131
	01.04.2020, 21:34 [ТС]
	hoggy, я все это исправил и теперь только 1 ошибка - компилятору не хватает размера кучи - что это значит? 0