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

@woolfskinny · Регистрация: 18.11.2018

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

Здравствуйте! Есть задача: найти среднее гармоническое числового ряда двумя способами: через передачу количества параметров как параметра функции и через использование признака конца списка параметров. Спасибо пользователю SmallEvil за помощь в решении: Среднее гармоническое элементов числового ряда. Функции с переменным числом параметров. Решение верное, но сделано при помощи va_list, а нужно использовать указатели, я нашел подобный метод в интернете:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
unsigned summ(unsigned char num, unsigned first, ...) {
    unsigned sum = 0;
    unsigned testsum = 0;
    unsigned *p = &first;
    while (num--) {
        testsum += *p;
        if (testsum >= sum) {
            sum = testsum;
        } else {
            exit(UNSIGNED_OVERFLOW);
        }
        p++;
    }
    return sum;
}
 
void main() {
    int sum = summ(5, 1u, 2u, 3u, 4u, 5u);
    printf("summ = %u\n", sum);
    sum = summ(7, 0u, 27u, 0u, 4u, 5u, 60u, 33u);
    printf("summ = %u\n", sum);
    getch();
}

Источник: https://learnc.info/c/vararg_functions.html

Реализовав задачу подобным методом, у меня выходил неккоректный результат, после чего решил реализовать простую функцию, которая бы выводила аргументы функции, чтобы посмотреть, как работает функция:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
#include <iostream>
#include <conio.h>
#include <cmath>
 
using namespace std;
 
double Args(int n, int first, ...)
{
    int* args = &first;
    for (int i = n; i > 0; i--)
    {
        cout << *args << endl;
        args++;
    }
}
 
int main()
{
    setlocale (0, "Russian");
    
    cout << Args(4, 1, 2, 3, 4);
    
    _getch();
    return 0;
}

По идее она должна выводить: 1234, но у меня же она выводит такой результат: 10200. С чем может быть связана эта проблема? Вроде как делаю по шаблону, который должен работать корректно, но выходит ахинея.

@woolfskinny · 17.02.2021, 17:46 **[ТС]**

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

согласишься пройти процедуры?

Интересный вопрос, вероятнее всего нет. Но тут уж кому как, кто-то и на вакцинацию от COVID-19 ходит

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

"вроде как - жену, а меня - как следует" (ц) Любовница.

Само собой я не уверен стопроцентно в качественной работе этой программы, просто проверил работу на простейших примерах, и результат выводится верный, следовательно вроде как работает, а всё остальное - лирика, запросил преподаватель указатели - вот ему через указатели. А так я бы их, конечно, не использовал, а пользовался va_list.

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

Кстати, если бы эта тема не болталась в С++ форуме (по какой-то необъяснимой причине), а по праву находилась в форуме по С

А почему она должна находиться в разделе по C? Дисциплина называется программирование на C++, задание с курса, вот я в разделе C++ и создал.

Сообщение от Kuzia domovenok

double summ(const int* data)//0-терминированный массив

Здравствуйте, насколько я понял, вы используете массив в качестве параметра. Меня этот способ заинтересовал и я попытался реализовать через массив, вот, что у меня вышло:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
double HarmonicMean2(double* arr)
{
    double summary = 0;
    int cnt = 0;
    double val = 0;
    cout << "\n\n";
    
    for (int i = 0; ; i++)
    {
        val = *arr;
        if (val == -1)
        {
            break;
        }
        else
        {
            val = *arr;
            summary += 1.0 / *arr;
            cnt++;
            cout << cnt << ". " << "Сумма: " << summary << endl;
            *(arr++);
        }
    }
    return cnt / summary;
}
 
int main()
{
    setlocale (0, "Russian");
    double array[] = {1, 1, 1, 1, 22, 33, 44, -1};
    cout << "\n\nСреднее гармоническое (второй способ): " << HarmonicMean2(array);
    
    _getch();
    return 0;
}

Только не могу понять, подходит ли такая реализация под условие?

Новичок · 17.02.2021, 20:05

Неплохо б уточнить условие. Действительно ли нужна функция с переменным числом параметров(для таких задач по-моему смысла никакого, разве что потренироваться) ? Или все-таки хватит массива(в функцию передается указатель). Если функция с переменным кол-во параметров, то можно ли вместо ... и va_args использовать variadic templates ?

@TheCalligrapher · 17.02.2021, 20:10

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

вот, что у меня вышло:

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

C++
1
double HarmonicMean2(double* arr)

C++
1
double HarmonicMean2(const double* arr)

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

C++
1
2
3
    double summary = 0;
    int cnt = 0;
    double val = 0;

Объявления переменных зачем-то сгруппированы в начале функции. Зачем здесь переменная val?

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

C++
1
for (int i = 0; ; i++)

Переменная i не играет в коде никакой роли. Зачем было ее заводить?

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

C++
1
*(arr++);

Бессмысленное применение оператора *.

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

C++
1
2
3
4
5
6
7
8
9
        val = *arr;
        if (val == -1)
        {
            break;
        }
        else
        {
            val = *arr;
            ...

Зачем повторять присваивание val = *arr? В ветке else значение val не используется вообще, а идет работа напрямую с *arr? Почему? Может переменная val вообще не нужна?

@woolfskinny · 17.02.2021, 20:51 **[ТС]**

Сообщение от Новичок

разве что потренироваться)

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

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

1
double HarmonicMean2(const double* arr)

Почему нужно использовать именно const, это обязательно?

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

Зачем здесь переменная val?

Да, действительно переменная val лишняя, можно было сделать просто вот так, насколько я понимаю:

C++
1
2
3
4
5
if (*arr == -1)
        {
            break;
        }
        else{...}

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

Объявления переменных зачем-то сгруппированы в начале функции.

Не понял, а как иначе?

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

Переменная i не играет в коде никакой роли. Зачем было ее заводить?

Чтобы был цикл в ходе которого считывался следующий элемент массива. Но сейчас понял, что можно было сделать подобным образом:

C++
1
2
3
4
5
6
7
    while (*arr != -1)
    {
        summary += 1.0 / *arr;
        cout << cnt << ". " << "Сумма: " << summary << endl;
        cnt++;
        arr++;
    }

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

Зачем повторять присваивание val = *arr? В ветке else значение val не используется вообще, а идет работа напрямую с *arr?

Она лишняя, не знаю даже, к чему вводил её.

---
Исходя из всего этого написал так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
double HarmonicMean2(const double* arr)
{
    double summary = 0;
    int cnt = 0;
    cout << "\n\n";
    while (*arr != -1)
    {
        summary += 1.0 / *arr;
        cout << cnt << ". " << "Сумма: " << summary << endl;
        cnt++;
        arr++;
    }
    return cnt / summary;
}

Новичок · 17.02.2021, 21:25

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

cout << cnt << ". " << "Сумма: " << summary << endl;

Все лишние выводы из функции желательно убрать, у функции одна ответственность, посчитать среднее гармоническое и вернуть, все остальное она делать не должна. А summary лучше заменить на какой-то sum, потому что summary это не сумма(название переменной как бы сбивает с толку, хоть тут и кода несколько строчек, но лучше привыкать хорошо называть переменные).

@woolfskinny · 17.02.2021, 21:39 **[ТС]**

Сообщение от Новичок

Все лишние выводы из функции желательно убрать

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

Сообщение от Новичок

потому что summary это не сумма

Сообщение от Новичок

что summary это не сумма

М-да, действительно ведь.. Я почему-то был убежден в обратном всё это время

Сообщение от Новичок

но лучше привыкать хорошо называть переменные

Да, хорошо, спасибо, буду стараться так делать.

@TheCalligrapher · 17.02.2021, 22:15

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

Почему нужно использовать именно const, это обязательно?

Ну, допустим, завтра кто-то захочет использовать вашу функцию с массивом const double array[] = { .... } и не сможет это сделать.

Если ваша функция не модифицирует данные в массиве, то лучше ссылаться на эти данные с const.

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

Не понял, а как иначе?

Ну как же? Переменную i ведь вы объявили не в начале функции, а локально - в цикле for. Точно так же и переменную val можно объявить локально - внутри цикла for, а не в начале функции

C++
1
double val = *arr;

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

Чтобы был цикл в ходе которого считывался следующий элемент массива.

Для того, чтобы просто "был цикл" можно просто написать for (;;). Для этого не нужно заводить никаких фиктивных переменных.

Добавлено через 6 минут

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

Исходя из всего этого написал так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
double HarmonicMean2(const double* arr)
{
    double summary = 0;
    int cnt = 0;
    cout << "\n\n";
    while (*arr != -1)
    {
        summary += 1.0 / *arr;
        cout << cnt << ". " << "Сумма: " << summary << endl;
        cnt++;
        arr++;
    }
 
    return cnt / summary;
}

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

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
double HarmonicMean2(const double* arr)
{
  double summary = 0;
  int cnt = 0;
  cout << "\n\n";
 
  const double* element = arr;
  while (*element != -1)
  {
    summary += 1.0 / *element;
    cout << cnt << ". " << "Сумма: " << summary << endl;
    cnt++;
    element++;
  }
 
  return cnt / summary;
}

А теперь можно заметить, что переменная cnt является необязательной

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
double HarmonicMean2(const double* arr)
{
  double summary = 0;
  cout << "\n\n";
 
  const double* element = arr;
  while (*element != -1)
  {
    summary += 1.0 / *element;
    cout << element - arr << ". " << "Сумма: " << summary << endl;
    element++;
  }
  return (element - arr) / summary;
}

и избегать цикла for тоже совсем не обязательно

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
double HarmonicMean2(const double* arr)
{
  double summary = 0;
  cout << "\n\n";
 
  const double* element = arr;
  for (; *element != -1; ++element)
  {
    summary += 1.0 / *element;
    cout << element - arr << ". " << "Сумма: " << summary << endl;
  }
 
  return (element - arr) / summary;
}

Новые блоги и статьи Все статьи Все блоги /
Access VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .	Фото: Daniel Greenwood kumehtar 13.11.2025

Новичок 1682 / 1098 / 489 Регистрация: 17.07.2012 Сообщений: 5,360
	17.02.2021, 20:05
	Неплохо б уточнить условие. Действительно ли нужна функция с переменным числом параметров(для таких задач по-моему смысла никакого, разве что потренироваться) ? Или все-таки хватит массива(в функцию передается указатель). Если функция с переменным кол-во параметров, то можно ли вместо ... и va_args использовать variadic templates ? 0