Обьясните, в чем ошибка.

[beerouser]

Здравствуйте.
Программа расчета периметра десятигранника.
Координаты точек вершин вводятся с клавиатуры в массив.
И цикле вызывается функция расчета длины отрезка и возвращаемые результаты суммируются.
При компилировании в Visual C++ получается, что функция расчета длины
сегмента всегда возвращает 2.
Скажите куда копать (С только осваиваю, если возможно по подробней)

#include <stdio.h>
#include <math.h>
#include <conio.h>


int xy[10][2] ,i = 1,j = 0,count;
float perimetr;

float lenght_of_segment(const int *x2 ,const int *x1 ,const int *y2 ,const int *y1)
{
  float x=sqrt(float((x2-x1)^2+(y2-y1)^2));
  return x;
}

int main()
{
	for( ;j<10;j++,i++)
	{
		printf("\nEnter values of x%i & y%i separeted by commas:\t",i,i);
		scanf("%i%*c%i",&xy[j][0],&xy[j][1]);
	}
	i=1,j=0;
	for(count=0;count<10;count++)
	{
		if(i==0)i=0,j=9;
		perimetr+=lenght_of_segment(&xy [i][0],&xy[j][0],&xy[i][1],&xy[j][1]);
		i++,j++;
	}
	printf("Perimeter = %6.2f",perimetr);
	getch();

	return 0;
}

Зарание благодарю.

[Rififi]

A ^ B

как ты думаешь, что в Си означает эта запись?

[8Observer8]

И ещё, у вас в функции lenght_of_segment параметры - указатели, их нужно разыменовывать.

[beerouser]

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

A ^ B

как ты думаешь, что в Си означает эта запись?

Точно, поразрядное ИЛИ.
Пора в отпуск

[beerouser]

Сообщение от 8Observer8

И ещё, у вас в функции lenght_of_segment параметры - указатели, их нужно разыменовывать.

А вот сдесь не порнял.
Сейчас раскутим man.

[beerouser]

sqrt(float((x2-x1)^2+(y2-y1)^2))
Будет так:
sqrt((pow((float(x2-x1)),2)+pow((float(y2-y1)),2));
Верно?
Как-то уродливо.Целая куча скобок.

[beerouser]

Работает!

[_Bers]

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

sqrt(float((x2-x1)^2+(y2-y1)^2))
Будет так:
sqrt((pow((float(x2-x1)),2)+pow((float(y2-y1)),2));
Верно?
Как-то уродливо.Целая куча скобок.

Что бы улучшить читабельность, можно например, поюзать временные промежуточные переменные.

[8Observer8]

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

А вот сдесь не порнял.
Сейчас раскутим man.

В вашей функции переменные x1 и x2 хранят адреса переменных xy [i][0] и xy [i][0] соответственно. Пройдите в отладчике по шагам и посмотрите. К примеру, посмотрите, что будет в переменной val в следующем коде, при установке точки останова на return.

float lenght_of_segment(const int *x2 ,const int *x1 ,const int *y2 ,const int *y1)
{
    int val= x1;
    return sqrt(pow((float(x2-x1)),2)+pow((float(y2-y1)),2));;
}

Чтобы получить значение по адресу, который храниться в x1 нужно применить операцию разыменования (*).

Вот так работает правильно:

double lenght_of_segment(const int *x2 ,const int *x1 ,const int *y2 ,const int *y1)
{
    double a, b, c;

    a = fabs((double)(*x2-*x1));
    b = fabs((double)(*y2-*y1));
    c = sqrt(pow(a, 2) + pow(b, 2));
    return c;
}

1) Используйте double, вместо float. Пусть double и больше места занимает, но всё же его мат. функции используют, предупреждений от компилятора будет меньше (усечение значений).

2) В данной задачи и подобных лучше использовать структуры, это облегчает программирование.

3) Хотя бы на этапе разработки, заведите файл с координатами, чтобы не мучатся с вводом значений.

[_Bers]

Сообщение от 8Observer8

Вот так работает правильно:

double lenght_of_segment(const int *x2 ,const int *x1 ,const int *y2 ,const int *y1)
{
    double a, b, c;

    a = fabs((double)(*x2-*x1));
    b = fabs((double)(*y2-*y1));
    c = sqrt(pow(a, 2) + pow(b, 2));
    return c;
}

А с какой целью тут вообще указатели передаются? Не проще ли сразу значения:

double lenght_of_segment(const int x2 ,const int x1 ,const int y2 ,const int y1)
{
    double a, b, c;

    a = fabs((double)(x2-x1));
    b = fabs((double)(y2-y1));
    c = sqrt(pow(a, 2) + pow(b, 2));
    return c;
}