Подскажите как правильно передавать строки через void*

у меня примерно такой код


Это сообщение отредактировал(а) Burka - 4.9.2011, 13:02

сделайте явное приведение..


Это сообщение отредактировал(а) mes - 4.9.2011, 13:05

черт) видимо промахнулся)) разделом)

переместил

если так
 
тоже не работает, при выводе просто пустая строка

Это сообщение отредактировал(а) Burka - 4.9.2011, 13:16

Второй случай должен работать без приведений:
http://liveworkspace.org/code/acfeaa34ac95...1b7018551e2aa8d

Добавлено через 4 минуты и 44 секунды
вот с выводом 
http://liveworkspace.org/code/59009ef57c8f...0db52d96e8ae756

Добавлено через 6 минут и 38 секунд
а кстати первый случай не компилится, так как там константный указатель.

все с точностью до наоборот.   

у меня выводит что то вроде "ММММММММММММММММММММММММММ" вместо той строки что надо

в liweworkspace не получается провеить) не компилит

я же говорю, не компилит из-за первого варианта.
второй работает без проблем
Вот ваш вариант
http://liveworkspace.org/code/e294e566a355...d4c79d0b8e91b1f
  

Это сообщение отредактировал(а) volatile - 4.9.2011, 13:42

А меняется ли что нибудь при таком раскладе? может содержимое строки удаляется в SetUserData() или что то такое

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

а как быть? что бы не уничтожался)

а зачем в MyClass void * ? с таким подходом у Вас на каждом шагу будут проблемы.. 
 

использовать  new []
но там надо не забыть еще и delete []

по- хорошему надо писать нормальный класс MyClass
с конструктором, деструктором, и оператором присваивания....

Ну вообще это не в моем классе а в классе движка Box2D  void* b2Body::UserData
и мне надо это использовать

Добавлено через 5 минут и 39 секунд

Спасибо, дальше разберусь)

значит Вы должны гарантировать необходимое время жизни объекта.. 
аккуратное использование new и delete в помощь.. 

а у меня наоборот, версия со строковым литералом не работает. говорит, что невозможно преобразовать const void* в void*
а с переменной работает.

могу посоветовать передавать адрес строки: func1(&str)
во-вторых, лучше использовать явное преобразование вместо преобразования в стиле С:

void func1(void *data) {
  //string mystr((char*)data);
  string mystr(static_cast<char*>(data));
  cout << mystr << endl;
}

И по идее, должно работать так:

#include <iostream>
#include <string>

using std::cout;
using std::endl;
using std::string;

void func1(void *data) {
  //string mystr((char*)data);
  string mystr(static_cast<char*>(data));
  cout << mystr << endl;
}

int main() {
  //func1("primer1");
  char str[100] = "primer2";
  func1(&str);
  return 0;
}

в данном случае func1(&str); и func1(str); являются синонимами.
в обоих случаях передается указатель на строку.


а чем лучше-то?
Если вы имеете ввиду хороший стиль, то тут в принципе не может идти речи о стиле..
Это все равно что советовать бомжу, одевать фрак с бабочкой.  

А если писать действительно в хорошем стиле, тут не нужны приведения вовсе (ни в стиле С, ни в стиле С++) 

Добавлено через 42 секунды
зы: да и код пробуйте иногда заключать в теги...