С: Почему выводит не то? Где ошибка?

Question

[TONoRNaE]

Задание: дано натуральное число N. Сформировать массив, элементами которого есть цифры числа N.
Я на выводе получаю такое: number= 532
5 ╠ ╠

#include <stdio.h>
#include <stdlib.h>
#include <math.h>

char* CreateArrByNumber(int number, int* len)
{
    *len = (int)log10((double)number) + 1;
    int mas[10];
    int* arr = mas;
    int i;
    for (i = *len - 1; i >= 0; --i)
    {
        *(arr + i) = '0' + (number % 10);
        number /= 10;
    }

    return arr;
}

int main()
{
    int i;
    int len;
    int number;
    printf("number= ");
    scanf_s("%i", &number);
    if (number == 0)
    {
        printf("Input 0, error!");
    }
    else {
        char* arr = CreateArrByNumber(number, &len);

        for (i = 0; i < len; ++i)
            printf("    %c", arr[i]);

        return 0;
    }
}

Comments

[Александр Ананьев]

Массив mas нужно создавать динамически

[TONoRNaE]

Александр Ананьев, нельзя динамически, нужен статический

[Rsa97]

TONoRNaE, Статический массив надо создавать не в функции, а на верхнем уровне. Иначе он создаётся в стеке и после завершения работы функции может быть затёрт.
Ну и сама функция у вас переусложнена и некорректно работает при number = 0.

[Antony]

Rsa97, не просто может, а гарантированно будет затерт при вызове других функций (у ТС там прямо следом идет вызов printf). Для меня загадка почему пятерка у него не затирается.

[TONoRNaE]

Rsa97, мне предложили как-то так, но почему-то на VS 2019 видит только первую цифру.

#include <stdio.h>
#include <stdlib.h>
#include <math.h>

char* CreateArrByNumber(int number, int* len)
{
    *len = (int)log10((double)number) + 1;
    char mas[10];
    char* arr = mas;
    int i;
    for (i = *len - 1; i >= 0; --i)
    {
        *(arr + i) = '0' + (number % 10);
        number /= 10;
    }

    return arr;
}

int main()
{
    int i;
    int len;
    int number;
    printf("number= ");
    scanf_s("%i", &number);
    if (number == 0)
    {
        printf("Input 0, error!");
    }
    else {
        char* arr = CreateArrByNumber(number, &len);

        for (i = 0; i < len; ++i)
            printf("    %c", arr[i]);

        return 0;
    }
}

В онлайн компиляторе вывод норм:

number= 532
5 3 2

[Rsa97]

Antony, Не факт, стек может не дорасти до массива. Но вероятность очень высокая.

[Rsa97]

TONoRNaE, И чем эта функция отличается от той, что в вопросе?

[TONoRNaE]

Rsa97, измененным типом данных. с int на char, но теперь проблема в том, что VS 2019 не видит остальные цифры.

[Rsa97]

TONoRNaE, В данном случае что int, что char, разницы никакой. Вам уже написали - нельзя статически выделенный в функции массив передавать наружу.

[Antony]

TONoRNaE, Вам выше Rsa97 ответил что вы создаете массив на стеке. (все локальные переменные в функциях хранятся в стеке), он затирается при выходе из функции (точнее указатель стека будет уменьшен и при последующем вызове другой функции ваши данные затрутся).

Соответственно решения, на выбор:
1. Объявить mas в main и передавать его в CreateArrByNumber
2. Сделать mas глобальной переменной (худшее решение)
3. Хранить все в куче (heap) (лучшее решение)

ну и к стати:

char mas[10];
    char* arr = mas;

Зачем еще один указатель на массив, если вы его не изменяете?

Вы вполне можете вместо *(arr + i) писать mas[i] (ровно как и arr[i])

[TONoRNaE]

Antony, объявил mas в main, теперь в функции char* arr = mas; не видит mass, а если я переношу и это то, не видит дальше

for (i = *len - 1; i >= 0; --i)
    {
        *(arr + i) = '0' + (number % 10);
        number /= 10;
    }
  

    return arr;

[Antony]

TONoRNaE, ну так вы в функцию-то его передайте, как вы с числом и длинной делаете

Answer 1

[Rsa97]

int CreateArrByNumber(int number, char *arr) {
  int len;
  if (number == 0) {
    arr[0] = '0';
    arr[1] = 0;
    return 1;
  }
  len = 0;
  while (number > 0 && len < 9) {
    arr[len++] = '0' + (number % 10);
    number /= 10;
  }
  arr[len] = 0;
  return len;
}

int main(void) {
  char arr[10];
  int len, number;
  // Здесь ввод
  len = CreateArrByNumber(number, arr);
  // Здесь вывод в обратном порядке
  return 0;
}