Не могу найти ошибку в программе на С++

[Jony217]

Доброго времени суток.

#include <list>
#include <iostream>
#include <string>

int main()
{
   list<string> s(10,0);
   for(list<string>::iterator i=s.begin();i!=s.end();++i)
      std::cin >> *i;
   for(list<string>::iterator i=s.begin(), l=s.end();i!=l;++i,--l)
   {
	  string s2=*i;
	  *i=*l;
	  *l=s2;}
   for(list<string>::iterator i=s.begin();i!=s.end();++i)
      std::cout << *i << std::endl;
   system ("PAUSE");
}

жалуется на 7, 8, 10, 12, 14 и 15 строки.
Заранее спасибо.

[_Bers]

int main()
{

using namespace std; //СПРОСИ СЕБЯ, ЧТО ДЕЛАЕТ ЭТА СТРОКА?

list<string> s(10,0);    //СПРОСИ СЕБЯ, ГДЕ ЖИВУТ СТАНДАРТНЫЕ КЛАССЫ И ОБЪЕКТЫ?

for(list<string>::iterator i=s.begin();i!=s.end();++i)

std::cin >> *i; //СПРОСИ СЕБЯ, ЧТО ОЗНАЧАЕТ ТАИНСТВЕННОЕ std:: ?


for(list<string>::iterator i=s.begin(), l=s.end();i!=l;++i,--l)
{
string s2=*i;
*i=*l;
*l=s2;}
for(list<string>::iterator i=s.begin();i!=s.end();++i)
std::cout << *i << std::endl;       
system ("PAUSE");
}

[Jony217]

1)У меня в коде строки "using namespace std;" нет, и зачем Вы используете впоследствии используете "std"?
2)После того, как я написал "using namespace std;" и убрал "std" из кода, при компиляции открывается файл заголовка "stl_list.h" и в нем указывают ошибку, в чем дело?

[_Bers]

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

1)У меня в коде строки "using namespace std;" нет, и зачем Вы используете впоследствии используете "std"?
2)После того, как я написал "using namespace std;" и убрал "std" из кода, при компиляции открывается файл заголовка "stl_list.h" и в нем указывают ошибку, в чем дело?

1. Вопрос не в том, есть оно у вас в коде, или нет.
Вопрос в том, зачем это нужно?

Что означает запись вида "using namespace std" ?
Что означает запись вида "std::" ?

Где и почему их следует применять?

2. Впервые слышу о файле "stl_list.h"
В стандартной библиотеке такого файла нет (у меня по крайней мере).

3. Если компилятор выдал ошибку - показывайте полностью весь текст ошибки. Телепаты в отпуске.

[_Bers]

вот смотри, что я сделал (это твой подправленный код):

#include <list>
#include <iostream>
#include <string>

int main()
{
    using namespace std;

    //list<string> s(10,0);  // : error C2440: приведение типов: невозможно преобразовать 'int' в 'std::string'
    list<string> s(10,"");
    for(list<string>::iterator i=s.begin();i!=s.end();++i)
        std::cin >> *i;
    for(list<string>::iterator i=s.begin(), l=s.end();i!=l;++i,--l)
    {
        string s2=*i;
        *i=*l;
        *l=s2;}
    for(list<string>::iterator i=s.begin();i!=s.end();++i)
        std::cout << *i << std::endl;
    
}

Я не вникал точно, что он у тебя делает. Но в таком вот виде он скомпилировался. Там с клавиатуры что-то вводить надо.
А тебе нужно понять - что было не так? Почему раньше он не хотел компилироваться?

Да, кстати.. ввел последнюю строчку склавиатуры и поймал крэш. Предположительно - не_валидный итератор листовый.
Сейчас выясню точно, из-за чего крэшеццо...



Вот смотри, вот в этой строчке:

 for(list<string>::iterator i=s.begin(), l=s.end();    i!=l;++i,--l)
    {
        string s2=*i;  //создали новый стринг запихав в него значение стринга на который смотрит i
        *i=*l;         //вот здесь ошибка. Крэш в рантайме: Expression: list iterator not deferencable

                         //Объясняю почему:
                         //теперь значение на которое смотрит i нужно заменить значением на которое смотрит l
                         
                        //но в самом начале цикла l смотрит куда??? l=s.end(); На не_валидный_элемент
                        //попытка присвоить *i значение несуществующего элемента и приводит к крышу

        *l=s2;
    }

/ps l=s.end() // здесь end() указывает не на последний элемент контейнера, а на элемент стоящий за последним элементом контейнера.
То есть, он указывает на уже не существующий элемент.

[_Bers]

Вот так все компилируется, и ничего не крэшеться.

int main()
{
    using namespace std;
    
    //закомментировал косяк
    //list<string> s(10,0);  // : error C2440: приведение типов: невозможно преобразовать 'int' в 'std::string'
    
    //исправленный вариант:
    list<string> s(3,""); //первичная инициализация контейнера пустотой
    
    //что бы не вводить в ручную с клавиатуру каждый раз, я решил изменить процедуру ввода исходных данных
   //но ничего не мешает вернуть все, как было
    {
        list<string>::iterator i=s.begin(); 
        *i="111"; ++i;    *i="222"; ++i;    *i="333"; ++i;
    }
    
    //Что бы не лепить непонятную кучу в шапке цикла, я вынес объявления счетчиков цикла наружу, и записал их в отдельные  ПОНЯТНЫЕ предложения
    //Код сразу стал более очевиден, и более понятен.
    list<string>::iterator i=s.begin();    
    list<string>::iterator l= s.end(); --l; //теперь l смотрит на последний элемент, а не на "несуществующий"
    
    //А дальше, все как обычно:
    for( ; i!=l;  ++i,--l) {  string s2=*i;  *i=*l;  *l=s2;  }
    
    for(list<string>::iterator i=s.begin();i!=s.end();++i){ cout << *i << endl; }
}

[Speakon]

using namespace std; //СПРОСИ СЕБЯ, ЧТО ДЕЛАЕТ ЭТА СТРОКА?

_Bers, не путайте человека, нет абсолютно никакой разницы писать std:: ... или using namespace std.
Более того иногда именно из-за написания using namespace std, вместо std:: возникают проблемы в работе программы.

[_Bers]

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

_Bers, не путайте человека, нет абсолютно никакой разницы писать std:: ... или using namespace std.
Более того иногда именно из-за написания using namespace std, вместо std:: возникают проблемы в работе программы.

1. Реально - разница есть.

2. Все ошибки я нашел и исправил. Прочитать о них можно выше.

3. Есть разница, как писать, вот так:

list<string> s(3,"");

или вот так:

std::list<std::string> s(3,"");

Или может быть вы не понимаете смысла того, что это за волшебные буковки такие "std::" , и тупо, как мартышки зазубрили, что они ставятся перед cout и endl ?

[KemanSR]

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

2. Впервые слышу о файле "stl_list.h"
В стандартной библиотеке такого файла нет (у меня по крайней мере).

Думаю что в стандартной библиотеке GNU AVR GCC он есть. Скорее всего програма для МК AVR32 или ARM.

[_Bers]

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

Думаю что в стандартной библиотеке GNU AVR GCC он есть. Скорее всего програма для МК AVR32 или ARM.

Да хоть для чего. Это не суть важно.

Важно только одно: код написанный по стандарту должен делать то, что от него ожидается согласно стандарту.

У парня было три ошибки:
1 .Он пытался заюзать стандартные контейнеры никак не указав пространство имен.
2. Он некорректно инициализировал массив стрингов, пытаясь скормить конструктору стринга тип int вместо char или char*

3. Ошибка в рантайме - попытка доступа к несуществующему элементу контейнера.

Да блин, я все это выше написал, и довольно таки подробно.