0

Нужна функция, которая создавала бы массив из цифр четырехзначного числа. Однако при попытке вывести массив через return компилятор ругается: "Тип возвращаемого значения не соответствует типу функции".

int NumSeparator(int xn) {
    int xz[4]{};
    xz[0] = xn / 1000;
    xz[1] = xn / 100 % 10;
    xz[2] = xn / 10 % 10;
    xz[3] = xn % 10;
    return xz;
}

Если подставить указатель, то выдает ошибку: "возвращение адреса локальной или временной переменной: xz"

int *NumSeparator(int xn) {
    int xz[4]{};
    xz[0] = xn / 1000;
    xz[1] = xn / 100 % 10;
    xz[2] = xn / 10 % 10;
    xz[3] = xn % 10;
    return xz;

int main() {
int x = 1234;
x1 = *NumSeparator(x);
cout << x1;

Вывод консоли после компиляции и выполнения кода}

3
  • 2
    Простой способ вернуть массив (его копию) из функции, это заключить массив в структуру и возвращать уже ее
    – avp
    1 ноя 2022 в 23:19
  • 2
    Возвращайте вектор, это проще всего.
    – Harry
    2 ноя 2022 в 4:14
  • Используйте вместо массива std::array<int, 4>. Его же и возвращайте.
    – wololo
    5 ноя 2022 в 13:18

2 ответа 2

0

Чтобы решить Вашу задачу Вам нужно сделать вот так:

    #include <iostream>

    int* NumSeparator(int, size_t);


    int main(int argc, char **argv)
    {
        const size_t N = 4;
        int x = 1234;

        int* result = NumSeparator(x, N);

        for (size_t i = 0; i < N; ++i)
        {
            std::cout << result[i] << std::endl;
        }

        delete []result;

        return 0;
    }


    int* NumSeparator(int xn, size_t N)
    {
        int *xz = new int[N];
        xz[0] = xn / 1000;
        xz[1] = xn / 100 % 10;
        xz[2] = xn / 10 % 10;
        xz[3] = xn % 10;
        return xz;
    }

Но создавать массивы в функциях это плохая практика в языке С++. Это может привести к утечке памяти. Лучше создавать массив отдельно а потом передавать его адрес в функцию и уже в функции проводить над ним разные операции. Так не придется возвращать массив из функции. Соответственно минимизируется риск утечки памяти.

1
  • "Соответственно минимизируется риск утечки памяти" — в фукции, в которую вы передаёте массив, но не там, где он создался. Лучше использовать std::vector или std::unique_ptr для этих целей.
    – tocic
    2 ноя 2022 в 4:03
0

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

int *NumSeparator(int xn) {
    int *xz = (int*)malloc(sizeof(int) * 4);
    xz[0] = xn / 1000;
    xz[1] = xn / 100 % 10;
    xz[2] = xn / 10 % 10;
    xz[3] = xn % 10;
    return xz;
}

Нужно выделить пямять для четырех значений, она может быть очищена только вызовом free или по завершении программы.

3
  • 1
    Хорошо, я попробовал использовать выделение памяти, но он выдает ошибку E0144: значение типа "void*" нельзя использовать для инициализации сущности типа "int*".
    – dikobrxz
    1 ноя 2022 в 23:19
  • @dikobrxz, у вас какой-то строгий компилятор. Попробуйте привести к типу int.
    – Olegator36
    2 ноя 2022 в 6:59
  • 2
    @Olegator36 не строгий, а соответствующий стандарту. См. stackoverflow.com/q/605845/6944845 и stackoverflow.com/q/15118990/6944845.
    – tocic
    2 ноя 2022 в 10:47

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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