Простой вопрос STL

[AnKor94]

Пришлось мне написать одну программу на этом языке звёздочек и амперсандов.. Возникла проблема, вынес её в отдельный код - повторилась, ура.

Задача - положить N строк из строки ch (она изменяется в процессе) в вектор, а потом положить их по одной в ch и что-то с ними делать.

В моём коде получается, что я кладу в массив не строки, а указатели на них, следовательно, при изменении ch меняется весь вектор. И при вводе
--> qwer
qwerty
qw
вывод будет таким
<-- qw
qw
qw

Как изменить код? И сделайте на сколько можно проще, пожалуйста.

Вот код:

   #include <iostream>
   #include <vector>
   #include <cstring>

   int main()
   {
      std::vector <char*> vec;
      char ch [1024];

      std::cin >> ch;
      vec.push_back(ch);
      std::cin >> ch;
      vec.push_back(ch);
      std::cin >> ch;
      vec.push_back(ch);

      std::vector <char*>::iterator i;
      for (i=vec.begin(); i<vec.end(); i++)
      {
           unsigned int j=0;
           for (j=0; ((j<strlen(*i))&&(j<1024)); j++)
                ch[j] = (*i)[j];
           ch[j] = '\0';
           std::cout << ch  << std::endl;
      }
      return 0;
   }

[Abstraction]

Как изменить код?

Не класть указатели, очевидно. В данном случае спасёт простая замена char* на std::string, который и разрулит все проблемы с копированием.

[AnKor94]

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

Не класть указатели, очевидно. В данном случае спасёт простая замена char* на std::string, который и разрулит все проблемы с копированием.

Ну да, так я уже попробовал, но.. Чтобы сохранить массив символов нужно преобразовать его в стринг, а чтобы достать - обратно? Какой-то троллейбус из булки хлеба.

[Abstraction]

Чтобы сохранить массив символов нужно преобразовать его в стринг, а чтобы достать - обратно?

Зачем - обратно? Чем Вас std::string не устраивает?
Нет, можете делать всё то, что делает за Вас std::string, самостоятельно - выделять новую память, копировать в неё строку и класть указатель на неё в вектор; потом при уничтожении вектора не забыть её освободить. Просто std::string - это класс, специально предназначенный, чтобы заниматься всем этим за Вас.

[AnKor94]

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

Зачем - обратно? Чем Вас std::string не устраивает?

В программе использую библиотеку muParser, которая принимает на вход си-строку. Данные беру с клавиатуры pocketbook'a, библиотека libInkView, которая возвращает тоже си-строку. Так что использовать std::string бессмысленно.

[Abstraction]

Так что использовать std::string бессмысленно.

Неверный вывод. std::string - это просто обёртка над char*, реализующая прозрачное для программиста копирование, ну, и некоторые другие полезные вещи. Поэтому преобразовывать получаемый char* в std::string и передавать дальше по вызовам результат вызова std::string::c_str() - лучший метод, который я могу предложить, в плане экономии нервов и минимизации ошибок.

Разумеется, вольному - воля:

   #include <iostream>
   #include <vector>
   #include <cstring>

   int main()
   {
      std::vector <char*> vec;
      char ch [1024];
      char* temp;

      std::cin >> ch;
      temp = new char[1024];
      strncpy(temp, ch, 1023);
      vec.push_back(temp);
      std::cin >> ch;
      temp = new char[1024];
      strncpy(temp, ch, 1023);
      vec.push_back(temp);
      std::cin >> ch;
      temp = new char[1024];
      strncpy(temp, ch, 1023);
      vec.push_back(temp);

      std::vector <char*>::iterator i;
      for (i=vec.begin(); i<vec.end(); i++)
      {
           unsigned int j=0;
           for (j=0; ((j<strlen(*i))&&(j<1024)); j++)
                ch[j] = (*i)[j];
           ch[j] = '\0';
           std::cout << ch  << std::endl;
      }
      for (i=vec.begin(); i<vec.end(); i++)
      {
            delete[] *i;
      }
      return 0;
   }

[AnKor94]

Спасибо, всё понял. Ух, от с++ аж руки трясутся.

offtop: вспомнился Керниган, : «Си — инструмент, острый, как бритва: с его помощью можно создать и элегантную программу, и кровавое месиво».