Расчет среднего значения в колонке ListView с учетом значения в ComboBox

@uran924 · Регистрация: 20.05.2016

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

Помогите решить такую задачу.Пользователь заполняет таблицу ListView данными о моторах, включающими тип мотора, получаемый от ComboBox, и его КПД, получаемый от TEdit.Необходимо рассчитать средний КПД по типу мотора.
Неудачная попытка реализации:

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
//---------------------------------------------------------------------------
 
#include <vcl.h>
#pragma hdrstop
#include <stdio.h>
#include "Unit1.h"
#include <string.h>
 
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma resource "*.dfm"
TForm1 *Form1;
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{
}
//---------------------------------------------------------------------------
 
         typedef struct
        {
        char mot[40]; //тип мотора
        char m[40];
        float t;
        float v;
        float r;
        float k; //кпд мотора
        } A;
        A dat[200];
int emot=0;
//---------------------------------------------------------------------------
 __fastcall TForm1::func()
{
         ListView1->Items->Clear();
        for (int i=0; i<emot; i++)
 {
 
 TListItem * ListIt= ListView1->Items->Add();
 ListIt->Caption= i+1 ;
 ListIt->SubItems->Add(dat[i].mot);
 ListIt->SubItems->Add(dat[i].m) ;
 ListIt->SubItems->Add(dat[i].t) ;
 ListIt->SubItems->Add(dat[i].v) ;
 ListIt->SubItems->Add(dat[i].r) ;
 ListIt->SubItems->Add(dat[i].k) ;
 }
}
 
//---------------------------------------------------------------------------
void __fastcall TForm1::vvodClick(TObject *Sender)
{
{
 if (model->Text.IsEmpty(),
 tok->Text.IsEmpty(),
 volt->Text.IsEmpty(),
 rev->Text.IsEmpty(),
 kpd->Text.IsEmpty())
 {
 ShowMessage("Введены не все данные");
 return;
 }
 {
        sscanf(MotorType->Text.c_str(), "%s" , &dat[emot].mot);
        sscanf(model->Text.c_str(), "%s" , &dat[emot].m);
        sscanf(tok->Text.c_str(), "%g" , &dat[emot].t);
        sscanf(volt->Text.c_str(), "%g" , &dat[emot].v);
        sscanf(rev->Text.c_str(), "%g" , &dat[emot].r);
        sscanf(kpd->Text.c_str(), "%g" , &dat[emot].k);
        emot++   ;
           func();
 
 }
}
}
//---------------------------------------------------------------------------
 
 
void __fastcall TForm1::midClick(TObject *Sender)
{
char mot[40];
if(MotorType->Text!="")sscanf(MotorType->Text.c_str(),"%s",&dat[emot].mot);
else
  {
  ShowMessage("Вы не указали тип мотора");
  return;
  }
 
float kpd=0;
int n;
for(int i=0; i<emot ;i++)
{
 if(!strcmp(mot,dat[emot].mot))
 {
  kpd+=dat[i].k;
  n++;
 }
}
float mid=kpd/n  ;
kpds->Text = FloatToStr(mid);
}
//---------------------------------------------------------------------------

@nmcf · 20.05.2016, 21:46

Для чего переходить на char, а потом снова преобразовывать при добавлении? Используй String.
Не ясно, что делает func(). Выбрось её и добавляй сразу во vvodClick()

@uran924 · 20.05.2016, 22:12 **[ТС]**

char использую т.к. тип мотора представляется символьно. Конкретно здесь использую его чтобы учесть тип мотора при расчете.
func() используется еще для кнопки загрузки данных. Эту кнопку тут не показываю т.к. для расчетов она не важна.
Можете показать пример: как тут использовать String?

@nmcf · 21.05.2016, 08:33

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
        struct A
        {
        String mot; //тип мотора
        String m;
        float t;
        float v;
        float r;
        float k; //кпд мотора
        } dat[200];
 
int emot=0;
 
void __fastcall TForm1::vvodClick(TObject *Sender)
{
 if (model->Text.IsEmpty() ||
 tok->Text.IsEmpty() ||
 volt->Text.IsEmpty() ||
 rev->Text.IsEmpty() ||
 kpd->Text.IsEmpty())
 {
 ShowMessage("Введены не все данные");
 return;
 }
        dat[emot].mot = MotorType->Text;
        dat[emot].m = model->Text;
        dat[emot].t = StrToFloat(tok->Text);
        dat[emot].v = StrToFloat(volt->Text);
        dat[emot].r = StrToFloat(rev->Text);
        dat[emot].k = StrToFloat(kpd->Text);
        ++emot;
}
 
void __fastcall TForm1::midClick(TObject *Sender)
{
String mot;
if(!MotorType->Text.IsEmpty()) mot = MotorType->Text;
else
  {
  ShowMessage("Вы не указали тип мотора");
  return;
  }
 
float kpd=0;
int n;
for(int i=0; i<emot ;i++)
 if(mot == dat[i].mot))
 {
  kpd += dat[i].k;
  n++;
 }
 
float mid=kpd/n;
kpds->Text = FloatToStr(mid);
}

А тип мотора при вводе и при поиске среднего вводится через одно и то же поле MotorType?

@uran924 · 21.05.2016, 10:10 **[ТС]**

Да, при вводе и поиске тип мотора задается в MotorType.
Попробовал сделать как вы предложили, теперь выдает ошибки:
[C++ Error] Unit1.cpp(150): E2188 Expression syntax в этой строчке

C++
1
 if(mot == dat[i].mot))

[C++ Error] Unit1.cpp(157): E2034 Cannot convert 'TButton *' to 'long double'
[C++ Error] Unit1.cpp(157): E2342 Type mismatch in parameter 'Value' (wanted 'long double', got 'TButton *') в этой строчке

C++
1
kpds->Text = FloatToStr(mid);

@nmcf · 21.05.2016, 11:00

Скобка лишняя. Сам бы исправил.

C++
1
if (mot == dat[i].mot)

Добавлено через 34 секунды
kpds - это строка ввода?

@uran924 · 21.05.2016, 11:03 **[ТС]**

Каюсь, недосмотрел.
kpds - это компонент TEdit.

@nmcf · 21.05.2016, 11:08

Ну, теперь работает?

@uran924 · 21.05.2016, 11:14 **[ТС]**

Все время считает 0.

@nmcf · 21.05.2016, 12:30

Проще всего посмотреть отладчиком. Поставь точку останова на цикл суммирования и в пошаговом режиме смотри за содержимым переменных и за проверкой условия.

@uran924 · 22.05.2016, 01:07 **[ТС]**

Попробовал сделать так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
void __fastcall TForm1::midClick(TObject *Sender)
{
char mot[40];
sscanf(MotorType->Text.c_str(), "%s", &mot);
int kpd=0;
int n=0;
int mid;
for(int i=0; i<emot ;i++)
{
 if(strcmp(dat[i].mot,mot)==0);
 {
  kpd+=dat[i].k;
  n++;
 }
}
mid=kpd/n  ;
kpds->Text = mid;
}

Рассчитывает среднее по всем данным.
Пробовал как вы предложили:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
void __fastcall TForm1::midClick(TObject *Sender)
{
String mot;
int kpd=0;
int n=0;
int mid;
for(int i=0; i<emot ;i++)
{
 if(mot == dat[i].mot);
 {
  kpd+=dat[i].k;
  n++;
 }
}
mid=kpd/n  ;
kpds->Text = mid;
}

Аналогичный результат.
Очевидно надо по другому условие или функцию strcmp задать. Но как, придумать не получается.
Среди вариантов было:

C++
1
 if(strcmp(ListView1->Items->Item[i]->SubItems->Strings[0].c_str(), MotorType->Text.c_str()) != 0);

Учитывать MotorType никак не хочет.

@nick42 · 22.05.2016, 01:25

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

if(mot == dat[i].mot);

У вас же String mot не определено, как можно его с чем-то сравнивать?

@nmcf · 22.05.2016, 08:42

Так куда делась строка с присвоением mot и что показал отладчик?

@uran924 · 22.05.2016, 12:26 **[ТС]**

А как String mot определить?
Отладчик пишет сначала mot:{"S<Ш<ГиВя\x01"} потом mot:{NULL}

@nmcf · 22.05.2016, 14:01

C++
1
2
3
4
5
6
7
String mot;
if(!(MotorType->Text.IsEmpty())) mot = MotorType->Text;
else
  {
  ShowMessage("Вы не указали тип мотора");
  return;
  }

@uran924 · 22.05.2016, 14:56 **[ТС]**

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
void __fastcall TForm1::midClick(TObject *Sender)
{
String mot;
if(!(MotorType->Text.IsEmpty()))
mot= MotorType->Text;
else
  {
  ShowMessage("Âû íå óêàçàëè òèï ìîòîðà");
  return;
  }
int kpd=0;
int n=0;
int mid;
for(int i=0; i<emot ;i++)
{
 if(mot==dat[i].mot);
 {
  kpd+=dat[i].k;
  n++;
 }
}
mid=kpd/n  ;
kpds->Text = mid;
}

По отладчику mot получает значение MotorType , но строчка:

C++
1
 if(mot==dat[i].mot);

ни на что не влияет.

@nmcf · 22.05.2016, 16:21

У тебя там нет пробелов лишних в начале и в конце названий?
А в dat[i].mot названия есть? Цикл перебирает их?

@uran924 · 22.05.2016, 16:58 **[ТС]**

В MotorType варианты такие:
постоянного_тока
переменного_тока
специальные
прочие

dat[i].mot получает одно из вышеперечисленных текстовых значений, но если тексты mot и dat[i].mot не совпадают
программа все равно продолжает обрабатывать эти строки:

C++
1
2
kpd+=dat[i].k;
  n++;

вероятно надо чтобы после

C++
1
 if(mot==dat[i].mot);

при несовпадении программа возвращалась в начало цикла и обрабатывала следующую строку таблицы.

@nmcf · 22.05.2016, 17:01

Точку с запятой убери после if.

@uran924 · 22.05.2016, 17:20 **[ТС]**

Заработало!
Спасибо вам огромное!

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .

@nmcf 7804 / 6568 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	20.05.2016, 21:46
	Для чего переходить на char, а потом снова преобразовывать при добавлении? Используй String. Не ясно, что делает func(). Выбрось её и добавляй сразу во vvodClick() 0

@uran924 0 / 0 / 0 Регистрация: 20.05.2016 Сообщений: 10
	20.05.2016, 22:12 [ТС]
	char использую т.к. тип мотора представляется символьно. Конкретно здесь использую его чтобы учесть тип мотора при расчете. func() используется еще для кнопки загрузки данных. Эту кнопку тут не показываю т.к. для расчетов она не важна. Можете показать пример: как тут использовать String? 0

@uran924 0 / 0 / 0 Регистрация: 20.05.2016 Сообщений: 10
	21.05.2016, 11:03 [ТС]
	Каюсь, недосмотрел. kpds - это компонент TEdit. 0

@nmcf 7804 / 6568 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	21.05.2016, 11:08
	Ну, теперь работает? 0

@uran924 0 / 0 / 0 Регистрация: 20.05.2016 Сообщений: 10
	21.05.2016, 11:14 [ТС]
	Все время считает 0. 0

@nmcf 7804 / 6568 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	21.05.2016, 12:30
	Проще всего посмотреть отладчиком. Поставь точку останова на цикл суммирования и в пошаговом режиме смотри за содержимым переменных и за проверкой условия. 0

@nmcf 7804 / 6568 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	22.05.2016, 08:42
	Так куда делась строка с присвоением mot и что показал отладчик? 0

@uran924 0 / 0 / 0 Регистрация: 20.05.2016 Сообщений: 10
	22.05.2016, 12:26 [ТС]
	А как String mot определить? Отладчик пишет сначала mot:{"S<Ш<ГиВя\x01"} потом mot:{NULL} 0

@nmcf 7804 / 6568 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	22.05.2016, 16:21
	У тебя там нет пробелов лишних в начале и в конце названий? А в dat[i].mot названия есть? Цикл перебирает их? 0

@nmcf 7804 / 6568 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	22.05.2016, 17:01
	Сообщение было отмечено uran924 как решение Решение Точку с запятой убери после if. 0

@uran924 0 / 0 / 0 Регистрация: 20.05.2016 Сообщений: 10
	22.05.2016, 17:20 [ТС]
	Заработало! Спасибо вам огромное! 0

Расчет среднего значения в колонке ListView с учетом значения в ComboBox

Решение