доброе время суток!
помогите пожалуйста разобраться. пытаюсь сделать хранилище для будущих сигнатур библиотечных функций (вообще это пока просто первые 32 байта любого ехе). хранилище двухуровневое - на первом в векторе лежат по 12 байт от каждой сигнатуры, на втором - остальные 20 байт, которые будут листьями от элементов первого уровня. получается список из деревьев, состоящих из корня и нескольких листьев.
ну это примитивная оптимизация, в плане что многие сигнатуры начинаются одинаково.
а проблема вот в чем - в метод добавления сигнатуры подаются все 32 байта, но на первый уровень должны попасть только 12. написала метод для откусывания части строки, все вроде нормально, но после выполнения пуш_бэка строка превращается в нечто непотребное.. сначала список был реализован вручную с указателями и всем таким, думала проблема в них. переписала с вектором - то же самое.. без отделения куска строки все норм..




Это сообщение отредактировал(а) notnat - 26.2.2011, 00:53

осмысленный конструктор копирования у SignatureBase есть?

мм, я думала о нем, но что-то меня остановило. так что нет

Вы возвращаете указатель на стековый объект, который уже уничтожен.

Это сообщение отредактировал(а) volatile - 26.2.2011, 01:14

спасибо, почти получилось) 

Это сообщение отредактировал(а) notnat - 26.2.2011, 01:38

Напишите еще такие функции:
  
SignatureBase ( const SignatureBase & sb );
const SignatureBase & operator = ( const SignatureBase & sb );
это конструктор и оператор=

Рекомендую использовать не char* а, std::string
Вы ведь все равно STL используете.

зачем вы пишете this-> в мемберах?. (Это не ошибка просто лишнее).

спасибо большое!
да, я знаю, что можно не писать 

На конструкцию *tmpData++; должен был совершенно справедливо изругаться компилятор - вы модифицируете const объект (и заодно разрушаете входной параметр)

гораздо короче, проще и безопасней пользоваться std::copy или strncpy.
еще проще std::string (data, 24);

Добавлено через 2 минуты и 7 секунд

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

Это сообщение отредактировал(а) baldina - 28.2.2011, 17:42

http://liveworkspace.org/code/ea5543d95298...0a8d992235e1578

спасибо большое, baldina, за такие подробности! попробуем

Честно говоря, странно слышать такое..
Имхо, вы ошибаетесь, *tmpData++ интерпретируется как  *(tmpData++),
модифицируется указатель, а константный объект на который он указывает не модифицируется.
И от него берется значение, которое не используется.
Да, разыменование там лишнее, только и всего. Никакой ошибки, "де юре" нет.




Это сообщение отредактировал(а) volatile - 28.2.2011, 23:31

Во избежание от придирок, полный и точный аналог выражения *tmpData++

что сути сказанного мной выше не меняет.

Вы правы, тут я слегка погорячился. Посыпаю голову пеплом   

xvr, да с каждым бывает, респект