Функция сложения с переменным числом параметров C++

[lotus1994]

Эта функция почему-то неправильно считает сумму. Выдает результат (-156413698). Не знаю как правильно ее написать. Подскажите

int sum(int count, ...)
{
    int s = 0;

    for(int i = 1; i <= count; i++)
        s += (&count)[i];   
     
    return s;
}

int main()
{
    cout << sum(10, 20, 30, 40) << endl;

    return 0;   
}

[rUs_LAN]

блин, а я даже подумать не мог что так можно. Действительно с + + сложный язык.

Тем не менее полазив по гуглу я понял что первый параметр должен быть количество аргументов. А ваша программа должна выглядеть примерно так

#include <iostream>
#include <stdio.h>
#include <stdarg.h>

int sum(int count, ...)
{
    int s = 0;

	 va_list ap;
	 va_start(ap, count);

    for(int i = 1; i <= count; i++)
        s += va_arg(ap, int);

	 va_end(ap);
     
    return s;
}


int main()
{
	 std::cout << sum(5, 20, 30, 40, 1, 1) << std::endl;
   
}

хотя это скорее С а не С + + ну если не считать iostream

[lotus1994]

Спасибо большое за помощь rUs_LAN. А то я думал-думал. Теперь хоть все стало понятно.

[lotus1994]

Вот так тоже можно. Это даже понятнее получается

int sum(int count, ...)
{
    int s = 0;

    for(int i = 1; i <= count; i++)
        s += (&count)[i];   
     
    return s;
}

int main()
{
    std::cout << sum(5, 20, 30, 40, 1, 1) << std::endl;

    return 0;   
}

[rUs_LAN]

А мне вот по большому счету ничего не понятно.

Как правильно и где это надо использовать

жду комментариев Сыроежки

[Сыроежка]

Правильно конечно использовать стандартные средства, то есть включить заголовок <cstdarg> и применять соответствующие макро va_start, va_arg, va_end и т.д.

Так как предложенное решение

int sum(int count, ...)
{
    int s = 0;

    for(int i = 1; i <= count; i++)
        s += (&count)[i];   
     
    return s;
}

в общем случае не будет работать, если тип первого параметра отличается от типа следующих за ним параметров. Например, если тип первого параметра (count ) int, а последующих, то есть заданных аргументов при вызове функции, short или char.
Также применение нестандартных средств затрудняет чтение кода, так как надо разбираться, а что пытается сделать программист.

[rUs_LAN]

спасибо!!!

[netrino]

Сообщение от Сыроежка

в общем случае не будет работать, если тип первого параметра отличается от типа следующих за ним параметров. Например, если тип первого параметра (count ) int, а последующих, то есть заданных аргументов при вызове функции, short или char.

Вообще-то, как раз в этом случае работать будет, но по чистой случайности, так как int на 32-битных машинах соответствует машинному слову и выравниванию, с которым данные кладутся в стек. Если объявить count, например, как short, то уже работать ничего не должно. Более того, если скомпилировать под Win64, тоже работать не должно. И кстати, на линуксе (или там freebsd?) тоже не работает, не смотря на 32-х битность. Так что, использование stdarg.h не прихоть, а необходимость.

[rUs_LAN]

#include <iostream>
#include <cstdarg>


int sum(short count, ...)
{
    int s = 0;

	 va_list ap;
	 va_start(ap, count);

    for(int i = 1; i <= count; i++)
        s += va_arg(ap, int);

	 va_end(ap);

	
    return s;
}


int main()
{
	 std::cout << sum(6, 20, (unsigned short)30, (long long)40, 1, 10) << std::endl;   
	 std::system("pause");
}

xp 32bit работает

add: ну работает но не совсем работает, мой код скорее всего написан с ошибками, у меня в 1 аргумент (количество параметров) написано 6, хотя в действительности я так понимаю должно быть 5. Если написать 5 последний параметр не воспринимается, это происходит из-за того, что есть тип (long long) подозреваю что из-за того что он занимает 8 байт а не 4. Далее копаться не стал.

[Сыроежка]

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

#include <iostream>
#include <cstdarg>


int sum(short count, ...)
{
    int s = 0;

	 va_list ap;
	 va_start(ap, count);

    for(int i = 1; i <= count; i++)
        s += va_arg(ap, int);

	 va_end(ap);

	
    return s;
}


int main()
{
	 std::cout << sum(6, 20, (unsigned short)30, (long long)40, 1, 10) << std::endl;   
	 std::system("pause");
}

xp 32bit работает

Это скорей всего из-за того, что вы делаете приведение типов в стиле С, которые не влияют на внутреннее представление данных.
Попробуйте заменить вызов функции на следующий

sum(6, 20, 'a', 'b', 40ll, 1, 10ll )

Посмотрим, что будет.

Да, и считайте не сумму, а просто выведите их на печать. Так как в сумме есть особенность. Например, если есть тип long long, то он может быть разбит на два типа int, в котором одна часть будет равна 0, а вторая часть тому значению, которое вы передали. Поэтому при суммировании получится верное значение: сложатся две части одного числа, одна из которых (старшая часть) равна 0.