Счетчик - найти среднее арифметическое

@nesquick · Регистрация: 09.04.2012

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

у меня вот проблема...среднее арифметическое находится неправильно.не пойму в чем дело.может подскажете? вот кусок.

C++

for(int i=0;i<=16;i++) {
  if(strtol(tabl1->Cells[3][i].c_str(),0,10)!=0){
    n++;
    s1+=strtol(tabl1->Cells[1][i].c_str(),0,10);
    s2+=strtol(tabl1->Cells[2][i].c_str(),0,10);
    s3+=strtol(tabl1->Cells[3][i].c_str(),0,10);
  }
  else{}
}
s1=s1/n;
s2=s2/n;
s3=s3/n;
 
n=1;
 
tabl1->Cells[1][16]=FloatToStr(s1);
tabl1->Cells[2][16]=FloatToStr(s2);
tabl1->Cells[3][16]=FloatToStr(s3);
s1=0;s2=0;s3=0;
//___________
for(int i=0;i<=16;i++) {
  if(strtol(tabl1->Cells[3][i].c_str(),0,10)<100 && strtol(tabl1->Cells[3][i].c_str(),0,10)!=0) {
    z1++;
  };
  if(strtol(tabl1->Cells[3][i].c_str(),0,10)<120 && strtol(tabl1->Cells[3][i].c_str(),0,10)>=100) {
    z2++;
  } ;
  if(strtol(tabl1->Cells[3][i].c_str(),0,10)>=120 && strtol(tabl1->Cells[3][i].c_str(),0,10)<140) {
    z3++;
  };
  if(strtol(tabl1->Cells[3][i].c_str(),0,10)>=140) {
    z4++;
  };
}
tabl2->Cells[0][1]=IntToStr(z1);
tabl2->Cells[1][1]=IntToStr(z2);
tabl2->Cells[2][1]=IntToStr(z3);
tabl2->Cells[3][1]=IntToStr(z4);
 
z1=0;z2=0;z3=0;z4=0;

@Avazart · 04.05.2012, 20:16

А какого типа s1,s2,s3 ?
И почему не

C++

1	s1+=StrToFloat(tabl1->Cells[1][i]);

?

@nesquick · 04.05.2012, 21:19 **[ТС]**

s1 s2 s3 типа float

Добавлено через 25 минут
а как сделать счетчик,считающий нулловые значения и отнимающий их от 16?

@Avazart · 04.05.2012, 21:56

а как сделать счетчик,считающий нулловые значения и отнимающий их от 16?

Не пойму !?
16-0 ?

Добавлено через 3 минуты
Насколько я помню strtol и подобные ф-ции не способны различить 0-значение от ошибки преобразования строки, может из-за этого не правильно считает?
Лучше использовать ф-ции :

C++

1 2	StrToFloat(); TryStrToFloat();

@BRcr · 05.05.2012, 06:57

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

счетчик,считающий нулловые значения и отнимающий их от 16?

если речь об указателях, то элементарно... впрочем, о чем бы ни была речь, все равно - элементарно

C++

1
2
3

TObject *apointer = NULL;
int count = 16;
if ( ! apointer ) count--;

@nesquick · 05.05.2012, 16:01 **[ТС]**

благодарю)

@nesquick 0 / 0 / 0 Регистрация: 09.04.2012 Сообщений: 25
	04.05.2012, 21:19 [ТС]	3
	s1 s2 s3 типа float Добавлено через 25 минут а как сделать счетчик,считающий нулловые значения и отнимающий их от 16? 0

@nesquick 0 / 0 / 0 Регистрация: 09.04.2012 Сообщений: 25
	05.05.2012, 16:01 [ТС]	6
	благодарю) 0