1

После появления запроса на ввод чисел программа вылет с такой ошибкой:
Segmentation fault (core dumped)

#include "iostream"

using namespace std;

long double raf(int a , ...)
{
    int *p = &a;
    int b = 0;
    for(; a != 0; a++)
    {
        b+= *(++p);
    }
    int c = b/a;
    return c;
}

int main()
{
    cout << "Midle count of numbers 5,3,9,1,7: " << endl;
    raf(5,5,3,9,1,7);
    return 0;
}
5
  • 1
    А в отладчике вы уже смотрели, где именно вылетает? 19 сен 2017 в 20:42
  • 1
    Вы почему-то считаете, что адрес следующего аргумента можно получить из адреса предыдущего при помощи инкремента. Стандарт этого не гарантирует.
    – VladD
    19 сен 2017 в 20:43
  • @ВладимирМартьянов К сожалению компилю на ubuntu с помощью g++ так что возможности глянуть неть
    – TRIGUN117
    19 сен 2017 в 20:44
  • @TRIGUN117: ну так освойте отладчик. Или перейдите на Windows/Visual Studio, если не можете освоить gdb.
    – VladD
    19 сен 2017 в 20:46
  • Боюсь, что даже в машине с передачей аргументов в стеке с соглашением, что первый аргумент задает количество следующих за ним, вот такой for(; a != 0; a++) { b+= *(++p); } (для небольшого положительного a) явно не соответствует задуманному. (наверное, для такой машины надо было бы не увеличивать, а уменьшать a) / (Но в любом случае это не переносимый метод и так писать не стоит. Используйте совет про va_list из ответа)
    – avp
    19 сен 2017 в 22:20

2 ответа 2

7

Доступ к variadic аргументам функций (параметры ...) в С и С++ делается только через va_list и макросы va_start/va_arg/va_end. Других способов не существует.

Поэтому ни о каком int *p = &a; и прочей вашей "адресной арифметике" не может быть и речи. Это какая-то белиберда, непонятно откуда вами выкопанная. Неудивительно, что дело заканчивается Segmentation fault.

1
  • 2
    Как ни смешно (и грустно) но Гугл выкидывает массу ссылок, в которых начинают действовать вот так, как пытается ТС...
    – Harry
    20 сен 2017 в 2:54
5

Если уж хочется использовать нестандартный метод, то все равно делать это нужно правильно:

#include <iostream>

using namespace std;

long double raf(int a , ...)
{
    int *p = &a;
    int cnt = a;
    int b = 0;
    for(; cnt != 0; --cnt)
    {
        b+= *(++p);
    }
    return ((long double)b)/a;
}

int main()
{
    cout << "Midle count of numbers 5,3,9,1,7: " << endl;
    cout << raf(5,5,3,9,1,7) << endl;
    return 0;
}

Посмотрите у себя - цикл ставите бесконечный (пока растущее a не станет нулем, потом делите на него - нулевое... Делите целочисленно, а надеетесь получить long double... Наконец, просто не выводите полученный результат. Об английском языке промолчу :)

Стандартное же решение такое:

#include <iostream>
#include <cstdarg>

using namespace std;

long double raf(int a , ...)
{
    va_list p;
    va_start(p,a);
    int cnt = a;
    int b = 0;
    for(; cnt != 0; --cnt)
    {
        b+= va_arg(p,int);
    }
    va_end(p);
    return ((long double)b)/a;
}

int main()
{
    cout << "Midle count of numbers 5,3,9,1,7: " << endl;
    cout << raf(5,5,3,9,1,7) << endl;
    return 0;
}

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.