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 that you have read and understand our privacy policy and code of conduct.

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