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

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

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

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

C++

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++

#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++

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++

*(arr++);

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

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

C++

        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++

if (*arr == -1)
        {
            break;
        }
        else{...}

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

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

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

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

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

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

C++

    while (*arr != -1)
    {
        summary += 1.0 / *arr;
        cout << cnt << ". " << "Сумма: " << summary << endl;
        cnt++;
        arr++;
    }

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

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

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

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

C++

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++

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++

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++

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++

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;
}

@woolfskinny 126 / 83 / 44 Регистрация: 18.11.2018 Сообщений: 299
	17.02.2021, 21:39 [ТС]	26
	Сообщение от Новичок Все лишние выводы из функции желательно убрать Вообще они особо и не нужны в коде, я сделал эти выводы для того, чтобы отследить, как работает программа и показать преподавателю потом это для наглядности. Сообщение от Новичок потому что summary это не сумма Сообщение от Новичок что summary это не сумма М-да, действительно ведь.. Я почему-то был убежден в обратном всё это время Сообщение от Новичок но лучше привыкать хорошо называть переменные Да, хорошо, спасибо, буду стараться так делать. 0

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

Новичок 1642 / 1091 / 487 Регистрация: 17.07.2012 Сообщений: 5,345
	17.02.2021, 21:25	25
	Сообщение от woolfskinny cout << cnt << ". " << "Сумма: " << summary << endl; Все лишние выводы из функции желательно убрать, у функции одна ответственность, посчитать среднее гармоническое и вернуть, все остальное она делать не должна. А summary лучше заменить на какой-то sum, потому что summary это не сумма(название переменной как бы сбивает с толку, хоть тут и кода несколько строчек, но лучше привыкать хорошо называть переменные). 0