Выводятся две строки подряд при выводе лишь одной - C++

@Infinity20 · Регистрация: 31.01.2013

Баловался со строками. Возник вопрос.
Есть код:

C++

...
char str1[] = "", str2[] = "";
 
cin >> str1;
cout << str1 << endl;
cin >> str2;
cout << str2 << endl;
cout << str1 << endl;
...

Допустим, мы введём в первую строку это: abcde;
Выведет это: abcde.
Вводим данные в str2,например, klmn.
Далее, выведет это:
klmn//cout << str2 << endl;
abcdklmn//cout << str1 << endl;

Почему так работает?cin.clear(); после каждого ввода не помогает.
Ещё заметил, если выделить память динамически, всё работает как надо:

C++

char* str1 = new char[];
char* str2 = new char[];
 
cin >> str1;
cout << str1 << endl;
cin >> str2;
cout << str2 << endl;
cout << str1 << endl;

В конце выведет это:
abcde // str1
klmn // str2
abcde // str1

@SatanaXIII · 20.08.2014, 11:37

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

Почему так работает?

Потому что в конце первого массива не ставится нуль-терминатор.

Добавлено через 2 минуты
Строковые литералы от Evg

@tehnar5 · 20.08.2014, 11:38

Представим память в виде массива. По всей видимости, происходит нечто следующее:
Сначала вы берете и помещаете в память две пустые строки. Это значит, что у вас есть два указателя - str1 и str2, которым нужно выделить только одну ячейку памяти(чтобы записать в нее 0 - символ конца строки). Эти блоки памяти могут быть расположены как угодно и где угодно, например, они могут быть расположены близко друг к другу. В Вашем случае они расположены как-то так:
....x...y...., где х - начало первой строки, а у - начало второй строки. Соответственно, после считывания первой строки память выглядит так: .....abcd0...., первая строка начинается с а, вторая с 0. Вывод будет работать корректно, так как он выводит по одному символу, пока не встретится 0. После считывания второй строки память выглядит так: ..abcdklmn0.., вторая строка так же выведется корректно, но теперь, технически, первая строка стала длиннее, так как ближайший справа 0 стал дальше.

Иными словами, если Вы не выделили память/выделили ее мало, то программа вполне может залезть в чужую память. В лучшем случае, это будет память другого процесса и тогда она вылетит с ошибкой, в худшем - это будет какая-то другая переменная. Как мне кажется, второй вариант тоже не правилен, так как надо указывать размер выделяемой памяти, но, по всей видимости, Вам повезло и память выделилась по-другому, подальше друг от друга. Самый правильный способ это new char[maxlength + 1], +1 нужен для того, чтобы хранить 0 - символ конца строки

@Voivoid · 20.08.2014, 12:11

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

это будет память другого процесса

Только если у тебя windows 95

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

Самый правильный способ это new char[maxlength + 1]

Самый правильный способ это std::string

@tehnar5 · 20.08.2014, 12:30

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

Только если у тебя windows 95

Да ладно, ведь если обратиться к str[10000000], то уж наверняка мы выйдем из памяти данного процесса. Другое дело, что система нам не позволит ничего сделать, но, по сути, выход произошел

@SatanaXIII · 20.08.2014, 12:48

tehnar5, услышит ли кто звук падающего в лесу дерева, если в лесу никого нет?

@Voivoid · 20.08.2014, 12:54

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

Да ладно, ведь если обратиться к str[10000000], то уж наверняка мы выйдем из памяти данного процесса

Ну, это конечно все зависит от ОС, но если ты про windows, то думаю самое время тебе почитать про механизм виртуальной памяти

@Alex5 · 20.08.2014, 15:28

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

Далее, выведет это:
abcdklmn //cout << str1 << endl;

Как строки расположены в памяти - см. рис.

Кликните здесь для просмотра всего текста

C++

    char str1[] = "", str2[] = "";
    cout << "address of str1  " << (void*)&str1[0] << endl;
    cout << "address of str2  " << (void*)&str2[0] << endl;
 
    cin >> str1;
    for( char* p  = &str1[0];  p!= &str1[0]+10 ;  ++p )
    {
        cout << "  " << *p;
    }
    cout << endl;
    //cout << str1 << endl;
    
    cin >> str2;
    for( char* p  = &str1[0];  p!= &str1[0]+10 ;  ++p )
    {
        cout << "  " << *p;
    }
    cout << endl;

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

выведет abcdklmn

В Вашем случае, Infinity20, видимо, адреса str1 и str2 отличаются на 4 байта. (Подробнее, см. сообщение #3 от tehnar5 ).

@SatanaXIII Супер-модератор 5616 / 2651 / 246 Регистрация: 01.11.2011 Сообщений: 6,529 Завершенные тесты: 1
	20.08.2014, 11:37	#2
	Сообщение от Infinity20 Почему так работает? Потому что в конце первого массива не ставится нуль-терминатор. Добавлено через 2 минуты Строковые литералы от Evg 0

@Voivoid 675 / 278 / 12 Регистрация: 31.03.2013 Сообщений: 1,339
	20.08.2014, 12:11	#4
	Сообщение от tehnar5 это будет память другого процесса Только если у тебя windows 95 Сообщение от tehnar5 Самый правильный способ это new char[maxlength + 1] Самый правильный способ это std::string 0

@tehnar5 31 / 31 / 12 Регистрация: 03.05.2011 Сообщений: 84
	20.08.2014, 12:30	#5
	Сообщение от Voivoid Только если у тебя windows 95 Да ладно, ведь если обратиться к str[10000000], то уж наверняка мы выйдем из памяти данного процесса. Другое дело, что система нам не позволит ничего сделать, но, по сути, выход произошел 0

@SatanaXIII Супер-модератор 5616 / 2651 / 246 Регистрация: 01.11.2011 Сообщений: 6,529 Завершенные тесты: 1
	20.08.2014, 12:48	#6
	tehnar5, услышит ли кто звук падающего в лесу дерева, если в лесу никого нет? 0

@Voivoid 675 / 278 / 12 Регистрация: 31.03.2013 Сообщений: 1,339
	20.08.2014, 12:54	#7
	Сообщение от tehnar5 Да ладно, ведь если обратиться к str[10000000], то уж наверняка мы выйдем из памяти данного процесса Ну, это конечно все зависит от ОС, но если ты про windows, то думаю самое время тебе почитать про механизм виртуальной памяти 0

@tehnar5 31 / 31 / 12 Регистрация: 03.05.2011 Сообщений: 84
	20.08.2014, 11:38	#3
	Представим память в виде массива. По всей видимости, происходит нечто следующее: Сначала вы берете и помещаете в память две пустые строки. Это значит, что у вас есть два указателя - str1 и str2, которым нужно выделить только одну ячейку памяти(чтобы записать в нее 0 - символ конца строки). Эти блоки памяти могут быть расположены как угодно и где угодно, например, они могут быть расположены близко друг к другу. В Вашем случае они расположены как-то так: ....x...y...., где х - начало первой строки, а у - начало второй строки. Соответственно, после считывания первой строки память выглядит так: .....abcd0...., первая строка начинается с а, вторая с 0. Вывод будет работать корректно, так как он выводит по одному символу, пока не встретится 0. После считывания второй строки память выглядит так: ..abcdklmn0.., вторая строка так же выведется корректно, но теперь, технически, первая строка стала длиннее, так как ближайший справа 0 стал дальше. Иными словами, если Вы не выделили память/выделили ее мало, то программа вполне может залезть в чужую память. В лучшем случае, это будет память другого процесса и тогда она вылетит с ошибкой, в худшем - это будет какая-то другая переменная. Как мне кажется, второй вариант тоже не правилен, так как надо указывать размер выделяемой памяти, но, по всей видимости, Вам повезло и память выделилась по-другому, подальше друг от друга. Самый правильный способ это new char[maxlength + 1], +1 нужен для того, чтобы хранить 0 - символ конца строки 0

Yandex Объявления	20.08.2014, 15:28

Опции темы

С++ для начинающих

Выводятся две строки подряд при выводе лишь одной - C++