0

Всем привет!!! Посмотрите, пожалуйста, нет ли ошибки, проверять долго, а гугл как-то внятного ответа давать не хочет. И нет ли у кого-нибудь ссылочки на мануалчик вроде "Чего нельзя делать в конструкторе класса!"=)

class WorksObject;

struct ThreadData {
    char qwerty[256];
    WorksObject* Obj;
};

class WorksObject {
public:
    WorksObject() : variable(false), ThHandle(NULL)
    {
        td = new ThreadData;

        strcpy(td->qwerty, "Text"); //Никакого Heap corruption не вылезет ?
        strcpy(qwerty1, "Text");    //Тут как ?
        td->Obj = this;             //Будет ли здесь нужный указатель ?
    }

    ~WorksObject()
    {
        if (ThHandle) CloseHandle(ThHandle);
        delete td;
    }

    void ФункцияВызываемаяПотом()
    {
        ThHandle = CreateThread(NULL, 0, WorkThread, td, 0, NULL);
    }

    static DWORD WINAPI WorkThread (LPVOID data)
    {
        ThreadData* ThData = (ThreadData*)data;
        if (ThData->Obj->variable)  //Вот в этом моменте c Obj все ок ???
        //...
    }

    HANDLE ThHandle;
    ThreadData* td;
    char qwerty1[256];
    bool variable;
}

p.s. В вопросах памяти дикие пробелы, исправлюсь в ближайшее время!:)

Спасибо!

10
  • Запутано как-то у Вас все. Если правильно понял, то для каждого потока Вы будете делать свой экземпляр WorksObject. Иначе (IMHO) потоки надо синхорнизировать (если несколько потоков будут работать с одним и тем же экземпляром), а никаких мьютексов или т.п. я не вижу. Тогда td = new ... ни к чему. Вообще, WorkThread() у Вас член класса, поэтому он будет видеть все поля данного экземпляра. Следовательно ему и аргумент td (соответственно и ThData) не нужен. Передайте для проформы NULL и просто обращайтесь у переменным класса по именам variable, qwerty, qwerty1. Остальные не нужны.
    – avp
    26 июн 2012 в 13:31
  • 1. Я это написал для вопроса о конструкторах, а поток использовал для того что бы хоть какой то смысл был в присваивании указателя на себя из конструктора. 2. WorkThread он static, и из него до variable, qwerty и т.д. не достучишься. 3. А если не отдельный экземпляр ? Разве не только при записи синхронизировать надо ?
    – lirik90
    26 июн 2012 в 16:38
  • Да, static я не заметил. Кстати, зачем static ? Пункт 3) не понял, что Вы имеете в виду (особенно "не только при записи синхронизировать" (а когда еще ?)).
    – avp
    26 июн 2012 в 16:53
  • static затем что вы в CreateThread функцию член класса не засуните(ну у меня не получалось еще) 3-ий вопрос в том что в коде только чтение if (ThData->Obj->variable)
    – lirik90
    26 июн 2012 в 17:08
  • @lirik90, со static Вы правы. Нестатический метод вызвать не удастся, т.к. у него есть "скрытый" аргумент - this. Поэтому его сигнатура никак не совпадет с сигнатурой функции, которую требует CreateThread(,,WorkThread...). Но, в WorkThread можно передать this и необходимость в td = new ThreadData (и вообще в структуре ThreadData) отпадает. Получается ... ThHandle = CreateThread(NULL, 0, WorkThread, this, 0, NULL); ... static DWORD WINAPI WorkThread (LPVOID data) { WorksObject *ths = (WorksObject *)data; if (ths->variable) // ну и к другим полям аналогично
    – avp
    26 июн 2012 в 20:02

2 ответа 2

3

strcpy(td->qwerty, "Text"); - здесь должно быть нормально. ThreadData на этот момент уже сконструирован, все будет ок. Но если это с++, то почему не использовать std::string ?

td->Obj = this; вызвать то можно, но только нужно понимать, что на этот момент WorksObject ещё не сконструирован полностью. И если этот указатель использовать для других целей, то может быть плохо:) (другие методы и функции могут не знать, что объект ещё не полностью сконструирован...)

Вот чего нельзя делать в конструкторе (и в деструкторе также) - так это вызывать виртуальные методы. Можно почитать здесь.

8
  • Спасибо, но всетаки про указатель, В момент присваивания ему будет куда указывать, то есть если я потом я по этому указателю к памяти обращусь, все хорошо будет ???
    – lirik90
    26 июн 2012 в 9:08
  • должно быть ок. Если только кто то с другого треда указатель не потрет.
    – KoVadim
    26 июн 2012 в 9:11
  • А если qwerty член WorksObject ???
    – lirik90
    26 июн 2012 в 10:51
  • вопрос не понятен.
    – KoVadim
    26 июн 2012 в 10:55
  • 1
    не используйте чужие слова, если не понимаете сути. Класс не до конца сконструирован? это что то новое:)) не путайте объект и класс. В Вашем коде WorksObject в конструкторе обращается к своему полю. Ему это сделать никто не мешает. Он то ведь знает состояние себя любимого:)
    – KoVadim
    26 июн 2012 в 12:16
0

@lirik90, у меня комментарии кончились. Поэтому последний переношу сюда. В ответ (?)

Про 1) понял, набросок, больше о том как лучше (на мой взгляд) писать не буду.

Про 2) ОК.

Про 3) После того, как Вы добавили ThHandle, стало очевидно, что правильно работать будет только с одним потоком. Для каждого потока, свой экземпляр WorksObject.

Прикиньте, Вы запускаете второй поток. Его HANDLE ThHandle замещает ThHandle предыдущего. В деструкторе сработает CloseHandle только для последнего потока.

Видимо Вы хотите, чтобы несколько потоков разделяли общий объект ? Тогда в нем нужен, например, вектор их HANDLE. Вообще-то от общей логики задачи зависит.

Про доступ к переменным по чтению. По сути надо лочить критические секции, а в них входят, обычно, по проверке переменных. Ну и вообще, где-то ведь ее меняют ? Иначе зачем проверять-то ? Поэтому лучше лочить.

Непосредственно по теме вопроса - что нельзя делать в конструкторе ?

Нельзя вызывать методы, которым нужен полностью сконструированый объект. Понятно, что при вызове конструктора память под все его поля уже отвели, а вот чем она заполнена ? Например, у Вас td до new будет NULL и т.д.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.