Форум программистов, компьютерный форум, киберфорум
Наши страницы
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
SpaceTime
0 / 0 / 0
Регистрация: 26.10.2015
Сообщений: 3
#1

Ошибка ввода символов строки, при количестве символов кратных 8

15.05.2016, 07:17. Просмотров 201. Ответов 5
Метки нет (Все метки)

Здравствуйте, уважаемые форумчане!

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

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

Этим примером хотелось освоить процесс вода строки с ограничением по максимальному размеру и символу и одновременно посмотреть работу с памятью в случае, когда она динамически выделяется внутри функции и указатель передается в main. Я предполагал, что увижу утечку памяти при многократном вызове функции т.к. каждый раз там будет выделяться память, но не очищаться т.к. указатель на данные нужно передать в main.

Проблема возникла уже при вводе данных в указатель Result внутри функции. Если вводилась строка с длинной кратной 8 символам, т.е. 8, 16, 24 и т.д. То к концу строки цеплялись произвольные символы (см. миниатюру вывода).
Если число символов не было кратно 8 - все вводилось и выводилось корректно. Я смотрел состояние переменных отладчиком Qt и он мне показывал, что при заполнении указателя данными в хвост всегда записывался какой-то мусор из 8 символов, но при не кратном 8 числе введенных символов строки эти мусорные символы не выводились, а при кратном - выводились. Почему так происходит?

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
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
#include <iostream>
#include <stdlib.h>
#include <string.h>
#include <malloc.h>
 
using namespace std;
 
//Объявляем прототипы
char* get_one_string(int, char); //Ввод строки текста с отсечкой по заданному символу
void put_text(char *);  //Вывод текста
 
//Объявляем глобальные переменные
int max_string_size = 3000;
char def_stop_char = '\n';
char menu;
 
int main()
{
    /*
    Работа со строками и динамической памятью
    */
 
    char text[max_string_size];
    char* _buf;
 
    cout << "Start" << endl;
    cout << "____________________________________________" << endl << endl;
 
 
    start:
    _buf = get_one_string(max_string_size, def_stop_char);
    put_text(_buf);
    cout << "_buf=" << cout << &_buf << endl;
 
 
    cout << endl << "Insert q for exit:";
    while( 1 )
    {
        cin >> menu;
        if( menu == 'q')
        { break; }
        else
        {
            cin.clear();
            while(cin.get() != '\n');
            goto start;
        }
    }
 
 
    free(_buf);
 
    return 1;
 
}
 
 
char* get_one_string(int max_size, char stop_char)
{
    //Ввод строки размером не более max_size до появления символа, заданного в stop_char
    char temp[max_size];
    int c_count = 0;
    char *Result;
 
    cout << endl << "Please, Insert string: ";
 
 
    //Заполняем строку текста
    while( (temp[c_count] = getchar()) != stop_char && c_count < max_size )
    { c_count++; }
 
    if( Result != NULL )
    {
        cout << "&Result = " << &Result << endl;
        Result = NULL;
    }
 
    //Выделяем память
    Result = (char*) calloc(c_count, sizeof(*Result));
 
    if(Result == NULL)
    { return Result; }
 
    cout << "strlen = " << strlen(Result) << endl;
 
    for(int i=0; i<c_count; i++)
    {
        Result[i] = temp[i];
        //cout << "i=" << i << " " << Result[i] << endl;
    }
    cout << "strlen = " << strlen(Result) << endl;
 
    return Result;
}
 
void put_text(char* text)
{
    //Вывод строки
    cout << endl << endl << "put_text length = " << strlen(text) << endl;
 
    for(int i=0; i<strlen(text); i++)
    {
        cout << text[i];
    }
    cout << endl;
 
}

Есть еще два более мелких вопроса по коду.
1. Почему адрес указателя _buf в main выглядит так странно? (см. миниатюру). Он как будто составлен из двух адресов, но с разной длиной байт.

2. Поскольку внутри функции указатель создается каждый раз заново я ожидал увидеть разные адреса для него. Но это не так (см. миниатюру.) Предположим, что из кучи память всегда выделялась с этого адреса т.к. пример простой и других потребителей памяти из кучи - нет. Тогда все-же что происходит при передаче указателя из функции во внешний указатель _buf? Он ссылается в конце-концов на адрес памяти выделенной в функции или ему выделяется его собственная память нужного размера и данные из указателя функции копируются туда, а сам указатель функции уничтожается как локальный. И есть ли способ правильно очистить память при ее динамической генераци в функции и передачи в main? Это я уже смотрел по форуму и предлагалось два подхода:
1. никогда так не делать в функции
2. Очищать в main указатель принимающий значение из указателя функции (в моем случае - _buf). В этом случае не будет утечки?
0
Миниатюры
Ошибка ввода символов строки, при количестве символов кратных 8  
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
15.05.2016, 07:17
Ответы с готовыми решениями:

Ошибка при чтении символов строки
static char strNAME = {&quot;D\\0000.f&quot;}; // &lt;&lt;-- ... void...

Ошибка при чтении символов строки
Здравствуйте, уважаемые форумчане! Решил вспомнить программирование и написать...

Ошибка при чтении символов строки
Здравствуйте, уважаемые программисты! Решил написать программу, которая бы...

Ошибка при чтении символов строки
Ошибка при чтении key_text при вызове метода object_yes_no(), почему? ...

Ошибка при чтении символов строки
Как я понял, что ошибка из-за функции ChooseAnswer. При использовании return....

5
zss
Модератор
Эксперт С++
7068 / 6608 / 4184
Регистрация: 18.12.2011
Сообщений: 17,412
Завершенные тесты: 1
15.05.2016, 08:10 #2
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
//Выделяем память
    Result = (char*) calloc(c_count, sizeof(*Result));
 
    if(Result == NULL)
    { return NULL; }
 
    //cout << "strlen = " << strlen(Result) << endl; // здесь нельзя вызывать strlen - в Result сидит мусор
    for(int i=0; i<c_count; i++)
    {
        Result[i] = temp[i];
        //cout << "i=" << i << " " << Result[i] << endl;
    }
    Result[c_count]=0; // Обязательно надо дописать в конец терминальный нуль
    cout << "strlen = " << strlen(Result) << endl;
 
    return Result;
1
SpaceTime
0 / 0 / 0
Регистрация: 26.10.2015
Сообщений: 3
15.05.2016, 08:19  [ТС] #3
Точно! Спасибо. Строка должна заканчиваться нулем, а я ее не завершаю.

Но почему все-же при количестве вводимых символов не кратных 8 эта ошибка не проявляется в выводе? По идее мусор должен показываться всегда. В отладчике Qt ея го наблюдал в значении Result всегда, как и должно было бы быть без завершающего нуля, но на выводе в консоль наблюдал при кратности количества вводимых символов = 8.
0
zss
Модератор
Эксперт С++
7068 / 6608 / 4184
Регистрация: 18.12.2011
Сообщений: 17,412
Завершенные тесты: 1
15.05.2016, 08:45 #4
Случайное совпадение - в этом месте были нули в каждом 8 байте.
1
SpaceTime
0 / 0 / 0
Регистрация: 26.10.2015
Сообщений: 3
15.05.2016, 09:01  [ТС] #5
Согласен. Припоминаю теперь, что и при 3-х символах бывал мусор, но редко. Но при кратности длины введенной строки - 8 мусор 100% выдавался. Этот вопрос - закрыт. Спасибо.

А по второй части - не разъясните?

1. Что за адрес памяти у _buf выводится?
2. Что неправильно в моих рассуждениях о работе указателей при присваивании указателя с динамически выделенной памятью из функции указателю в main? Тут можно ссылку на литературу или пример на форуме. Я на форуме нашел только рекомендацию - "Очищать в main указатель принимающий значение из указателя функции" (что и делаю), но хотелось бы поточнее понять механизм, что в этом случае происходит. Если локальный указатель с динамически выделенной памятью в функции уничтожается (как и должен), то каким механизмом данные попадают в _buf? А если не уничтожается - то как правильно очищать память в этом случае?
0
zss
Модератор
Эксперт С++
7068 / 6608 / 4184
Регистрация: 18.12.2011
Сообщений: 17,412
Завершенные тесты: 1
15.05.2016, 10:24 #6
1. &_buf - это адрес ячейки, в которую записывается адрес выделенной памяти
Ответ на второй следует из первого. Ячейка, в которую записывают выделенный адрес не меняется (ее имя _buf, ее адрес &_buf).
0
15.05.2016, 10:24
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
15.05.2016, 10:24

Ошибка при чтении символов строки
Помогите с кодом #include&lt;iostream&gt; #include&lt;conio.h&gt; using namespace...

Ошибка при чтении символов строки, работа с классом
Делаю лабораторную на тему Класс студентов. Реализовал несколько функций ввода...

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


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
6
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2018, vBulletin Solutions, Inc.
Рейтинг@Mail.ru