Как возвращать(return) массив из функцию с переменным количеством параметров?

Question

[Sector567]

Вот задание:


В чем моя ошибка?
Язык С (Си)

#include <stdio.h>
#include <stdlib.h>

#define END_PARAM -1

int max(double a, ...)
{
	double *p = &a;	/* указатель на значение параметра */
	double *str = (double*)malloc(30 * sizeof(double));

	while (*p != END_PARAM)
	{
		*str = *p;	/* перезаписываем в массив */

		str++;		/*переходим к следующему индексу в массиве*/
		p++;		/* переходим к следующемк параметру */
	}

	/* возвращаем результат */
	return *str;
}

void main()
{
	printf("result = %d\n\n", max(100.5, 25.5, 30.76, 9.1, 22.45, END_PARAM));

	system("pause");
}

Answer 1

[Денис Загаевский]

Первая ошибка - возвращать нужно не int, a double*.
Вторая - вывод в принтф - опять же выводите инт(%d), это неверно. Формата для вывода массива даблов нет, насколько я знаю, надо выводить в цикле.
Третья - выделение памяти неправильное. Откуда взялось 30 * sizeof(double)? Надо пройти по параметрам, сначала узнать размер, затем скопировать. Копировать можно с помощью memcpy, кстати, а не циклом.
И ходить так по списку параметров кажется неправильно, не знаю.

Comments

[devalone]

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

В C++ точно неправильно, gcc часто генерит передачу параметров через регистры, а не через стек, какой может указатель на регистр? Ну и естественно в стандарте не прописано, что передача должна быть через стек.

[Денис Загаевский]

devalone: потому и написал, что не знаю, это тонкости реализации, которые я как-то начал забывать. Давно не работаю с Си.

Answer 2

[Даниил Демидко]

Ужас. Возвращать результат разыменования double* в int функции это сильно.
Видимо от вас ожидали в общих чертах примерно следующего:

double *const getArray() {
    double *const result = new double[10];
    // Заполняйте массив как вам угодно
    return result;
}

Comments

[Sector567]

ой, точно, по привычки забыл изменить тип.
Все равно немного не понимаю ваше решение.
Я предполагал, что в функцию с переменным числом параметров подаются некоторые значение, после чего их нужно переписать в динамический массив и возвратить результат ретерном. Или я задние не так понял?

[Даниил Демидко]

Sector567: А смысл в вашем возврате одного значения?
Возвращать нужно указатель на массив

Answer 3

[devalone]

Вы гуляете по памяти в надежде, что параметры передаются через стек, но на деле это может быть не так и довольно часто компилятор передаёт их через регистры. Для этого есть специальные средства, гугли va_list