Предупреждение: returning address of local variable or temporary

@JAson13 · Регистрация: 16.05.2012

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

Есть код программы для кодирования в base64 и обратно, не самый лучший код, но все же. И вот выскакивает два предупреждения на 2 return в функциях. Не могу справиться, прошу помощи исправить ошибку или намекнуть как-то и объяснить в двух словах.

Две функции все прекрасно выполняют, но одновременно работать не могу. Т.о. если я запускаю сначала первую функцию, потом вторую, первая возвращает то что нужно, вторая - нет.

Функция - в base64:

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
36
37
38
39
40
41
42
43
char *EncodeToBase(char *inputStr){
 
    int N = strlen(inputStr),
        byte_128 = 128, byte_32 = 32,
        count = 0, j = 0,
        symbCode = 0;
 
    char outputStr[1000] = "";
 
    for (int i = 0; i < N; i++){        
        for (int m = 0; m < 8; m++){
            if (byte_128 & (int)inputStr[i]){
                symbCode += byte_32;    
            }
 
            outputStr[j] = tableBase_64[symbCode];
 
            if (count == 5){                
                count = 0;
                j++;
                symbCode = 0;
                byte_32 = 32;
            }   
            else {
                count++;
                byte_32 >>= 1;
            }
            byte_128 >>= 1;
        }
        byte_128 = 128; 
    }
 
    N = strlen(outputStr);
    int rem = N % 4;
 
    while (rem != 0){
        N++;
        rem = N % 4;
        strcat_s(outputStr, "=");
    }
 
    return outputStr;
}

Функция - из base64:

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
char *DecodeToBase(char *inputStr){
 
    int N = strlen(inputStr),
        byte_128 = 128, byte_32 = 32,
        symbCode = 0, count = 0, j = 0;
 
    char outputStr[1000] = "";
 
    for (int i = 0; i < N; i++){
        if (inputStr[i] == 61){
            break;
        }
        for (int m = 0; m < 6; m++){    
            if (byte_32 & IndexOfElem(tableBase_64, (int)inputStr[i])){
                symbCode += byte_128;
                outputStr[j] = symbCode;
            }
            if (count == 7){
                count = 0;
                j++;
                byte_128 = 128;
                symbCode = 0;
            }
            else{
                count++;
                byte_128 >>= 1;
            }   
            byte_32 >>= 1;
        }
        byte_32 = 32;
    }
 
    return outputStr;
}

Main:

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
int main()
{
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    int value = 0;
 
    for (int i = 0; i < 64; i++){
 
        switch (i){
            case 0:
                value = (int)('A');
                break;
            case 26:
                value = (int)('a');
                break;
            case 52:
                value = (int)('0');
                break;
            case 62:
                value = (int)('+');
                break;
            case 63:
                value = (int)('/');
                break;
            default:
                value++;
        }
 
        tableBase_64[i] = value;    
    }
 
    char inputStr[100] = "";
    int choice = -1;
 
    do{
        printf("\n   -------Base64 Encode/Decode------\n\n   1. Закодировать строку\n   2. Раскодировать строку\n\n   0. Выход\n\n   > ");
        scanf_s("%d", &choice);
 
        switch (choice){
            case 1:
                printf("\n\n   Введите строку: \n   > ");
                scanf_s("%s", inputStr);
                printf("\n   Результат: %s", EncodeToBase(inputStr));
                break;
            case 2:
                printf("\n\n   Введите строку: \n   > ");
                scanf_s("%s", inputStr);
                printf("\n   Результат: %s", DecodeToBase(inputStr));
                break;
            case 0:
                choice = 0;
                break;
            default:
                printf("\n\n   Выберите существующий вариант\n");
        }
    } while (choice != 0);
    return 0;
}

@nmcf · 08.11.2015, 18:56

Нельзя возвращать локальный объект (outputStr). 100 раз объясняли.

@DrOffset · 08.11.2015, 18:59

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

Не могу справиться, прошу помощи исправить ошибку или намекнуть как-то и объяснить в двух словах.

Намек уже дал компилятор:

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

И вот выскакивает два предупреждения на 2 return в функциях.

Нельзя возвращать адреса локальных объектов.
У тебя массив

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

C++
1
char outputStr[1000]

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

@JAson13 · 08.11.2015, 18:59 **[ТС]**

ну тогда как мне результат использовать, совсем сообразить не могу

@nmcf · 08.11.2015, 19:15

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

@JAson13 · 08.11.2015, 21:47 **[ТС]**

Так, я сделал так чтобы функция возвращала string (большое спасибо, надо будет про стринги почитать), но от char'ов решил не избавляться, но появилась другая проблема, а с потоками ввода вывода я вообще не дружу, что печально. Поэтому мне бы понять этот момент:

C++
1
2
3
printf("\n\n   Введите строку: \n   > ");
                scanf_s("%s", inputStr);
                printf("\n   Результат: %s", EncodeToBase(inputStr).c_str());

Дело в том что все это дело прекрасно считывает строку, за исключением одного но! если строка имеет пробелы, то считывать как я понял считывает, но выводит неправильно. Выводит не всей строкой а по отдельности через \n. Например я введу "I love you" а на выходе получу отдельно кодированный "I", "love" "you". если использую что то в стиле scanf("%[^n]s", str) или gets(str) то ввод игнорируется вообще, т.е. у меня даже нет возможности что то ввести

Добавлено через 28 минут
все, спасибо, разобрался))

@nmcf · 08.11.2015, 23:29

Начал string использовать, значит никаких функций и char[] - только потоки.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

@nmcf 7804 / 6568 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	08.11.2015, 18:56
	Нельзя возвращать локальный объект (outputStr). 100 раз объясняли. 0

@JAson13 0 / 0 / 0 Регистрация: 16.05.2012 Сообщений: 36
	08.11.2015, 18:59 [ТС]
	ну тогда как мне результат использовать, совсем сообразить не могу 0

@nmcf 7804 / 6568 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	08.11.2015, 19:15
	Либо создавай его динамически и возвращай указатель, либо (что проще) передавай указатель на буфер в функцию и пусть она его заполняет. 0

@nmcf 7804 / 6568 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	08.11.2015, 23:29
	Начал string использовать, значит никаких функций и char[] - только потоки. 0

Предупреждение: returning address of local variable or temporary

Решение