Вычислить среднюю стоимость всех компьютеров и напечатать наименования компьютеров

@Анфисочка · Регистрация: 31.10.2015

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

Есть задача: создать структуру данных про компьютеры которая включает

Список компьютеров:

название
частота
объем оперативной памяти
наличие DVD ROM
стоимость

И вычислить среднюю стоимость всех компьютеров и напечатать наименования компьютеров.

Проблемы таковы:

При добавлении в компонент Мемо1 данных вводимых с компонентов Edit массив не выводит новые введенные данные в следующую строку для нового компьютера, а добавляет при каждом выводе новых данных все предыдущие строки, плюс следующую строку. Короче говоря, строк должно быть пять, и новая строка не должна копировать то, что было введено ранее.
Не высчитывается средняя цена компьютеров. вместо этого она суммируется.

Код

C++

// глобально обявленные структуры
struct comp
{
    AnsiString nazvan;
    float chastot;
    int RAM;
    AnsiString DVD;
    float cost;
};
 
// количество компов
comp p[5];
int n = 0;
 
//  кнопка добавить
 
void __fastcall TForm1::Button1Click(TObject *Sender)
{
p[n].nazvan = Edit1->Text;
p[n].chastot = StrToFloat(Edit2->Text);
p[n].RAM = StrToInt(Edit3->Text);
p[n].DVD = Edit4->Text;
p[n].cost = StrToFloat(Edit5->Text);
n++;
for (int i = 0; i < n; i++){
Memo1->Lines->Add(p[i].nazvan+" "+FloatToStr(p[i].chastot)+"GHz"+" "+IntToStr(p[i].RAM)
+"ГБ"+" "+p[i].DVD+" "+FloatToStr(p[i].cost)+"USD");        
} }
//---------------------------------------------------------------------------
// кнопка решение
void __fastcall TForm1::reshenieClick(TObject *Sender)
{
int Sum = 0;
int kol = 0;
for (int i = 0; i < n; i++)
Sum+=p[i].cost;
kol++;
float SR;
{ SR =(Sum/kol); }
Memo2->Lines->Add("средняя цена="+FloatToStr(SR));
}
//---------------------------------------------------------------------------

То, что выходит.

Вычислить среднюю стоимость всех компьютеров и напечатать наименования компьютеров

Помогите, исправьте пожалуйста, и объясните ошибки.

Сам файл:
Структуры.rar

@Germanant · 06.05.2016, 23:33

Сообщение от Анфисочка

// кнопка решение
void __fastcall TForm1::reshenieClick(TObject *Sender)
{
int Sum = 0;
int kol = 0;
for (int i = 0; i < n; i++)
Sum+=p[i].cost;
kol++;
float SR;
{ SR =(Sum/kol); }
Memo2->Lines->Add("средняя цена="+FloatToStr(SR));
}

C++

void __fastcall TForm1::reshenieClick(TObject *Sender)
{
int Sum = 0;
int kol = 0;
for (int i = 0; i < n; i++)
{
Sum+=p[i].cost;
kol++;
}
float SR;
{ SR =(Sum/kol); }
Memo2->Lines->Add("средняя цена="+FloatToStr(SR));
}

@Sasha · 07.05.2016, 00:08

C++

void __fastcall TForm1::reshenieClick(TObject *Sender)
{
int Sum = 0;
int kol = 0;
for (int i = 0; i < n; i++)
{
Sum+=p[i].cost;
kol++;
}
float SR;
SR = Sum/kol; 
Memo2->Lines->Add("средняя цена="+FloatToStr(SR));

@Анфисочка · 07.05.2016, 00:09 **[ТС]**

Спасибо, теперь осталась только первая проблема:

При добавлении в компонент Мемо1 данных вводимых с компонентов Edit массив не выводит новые введенные данные в следующую строку для нового компьютера, а добавляет при каждом выводе новых данных все предыдущие строки, плюс следующую строку. Короче говоря, строк должно быть пять, и новая строка не должна копировать то, что было введено ранее.

C++

// глобально обявленные структуры
struct comp
{
    AnsiString nazvan;
    float chastot;
    int RAM;
    AnsiString DVD;
    float cost;
};
 
// количество компов
comp p[5];
int n = 0;
 
//  кнопка добавить
 
void __fastcall TForm1::Button1Click(TObject *Sender)
{
p[n].nazvan = Edit1->Text;
p[n].chastot = StrToFloat(Edit2->Text);
p[n].RAM = StrToInt(Edit3->Text);
p[n].DVD = Edit4->Text;
p[n].cost = StrToFloat(Edit5->Text);
n++;
for (int i = 0; i < n; i++){
Memo1->Lines->Add(p[i].nazvan+" "+FloatToStr(p[i].chastot)+"GHz"+" "+IntToStr(p[i].RAM)
+"ГБ"+" "+p[i].DVD+" "+FloatToStr(p[i].cost)+"USD");        
} }

Я это делала по примеру преподавателя на лекции. И где-то тут ошибка...

Найдите, пожалуйста, ее исправьте.

@Sasha · 07.05.2016, 00:21

C++

void __fastcall TForm1::Button1Click(TObject *Sender)
{
p[n].nazvan = Edit1->Text;
p[n].chastot = StrToFloat(Edit2->Text);
p[n].RAM = StrToInt(Edit3->Text);
p[n].DVD = Edit4->Text;
p[n].cost = StrToFloat(Edit5->Text);
 
 
Memo1->Lines->Add(p[n].nazvan+" "+FloatToStr(p[n].chastot)+"GHz"+" "+IntToStr(p[n].RAM)
+"ГБ"+" "+p[n].DVD+" "+FloatToStr(p[n].cost)+"USD");        
 
n++;
 }

@nick42 · 07.05.2016, 00:22

Ну так не нужно цикла по n. Просто перед инкрементом n организуйте то, что в 26-й строчке с n вместо i. В список Memo добавится одна новая строка про очередной комп, а прежние не будут повторяться.

( Синхрон )

@Анфисочка · 07.05.2016, 00:50 **[ТС]**

Всем спасибо за помощь. Вы очень мне помогли.

Вычислить среднюю стоимость всех компьютеров и напечатать наименования компьютеров

Решение

Решение

Решение

@nick42 Практикантроп 4826 / 2718 / 526 Регистрация: 23.09.2011 Сообщений: 5,777
	07.05.2016, 00:22	6
	Сообщение было отмечено Анфисочка как решение Решение Ну так не нужно цикла по n. Просто перед инкрементом n организуйте то, что в 26-й строчке с n вместо i. В список Memo добавится одна новая строка про очередной комп, а прежние не будут повторяться. ( Синхрон ) 1

@Анфисочка 1 / 1 / 1 Регистрация: 31.10.2015 Сообщений: 20
	07.05.2016, 00:50 [ТС]	7
	Всем спасибо за помощь. Вы очень мне помогли. 0