Могут ли 2 класса совместно использовать одну структуру?

Question

Могут ли 2 класса совместно использовать одну структуру?

0

Предположим, у меня есть 2 класса Foo и Cook, а Book - это структура.

Ранее Book - глобальная переменная, и я хотел бы изменить ее на принадлежность Foo чтобы у меня было много Foo с их собственной Book.

Cook читает Book, но Foo будет время от времени обновлять ценность Book.

Следующий код, который я тестировал, не работает. Я очень новичок в C++, я не понимаю, почему это не работает.

class Foo {
private:
    Book book;
    Cook cook;
public:
    Foo();
    virtual ~Foo();
};

class Cook {
private:
    Book * book;
public:
    Cook();
    virtual ~Cook();
    void setBook(Book * book);
};

Затем внутри конструктора Foo есть: cook.setBook(&book);

Jonathan Mee 11 нояб. 2014, в 20:55

Источник

2

«Не работает» означает, что именно?
tadman 11 нояб. 2014, в 19:45
0

Компилируется, но дает неверный расчет. Поэтому я думаю, что книга, когда к ней обращается Кук, не отражает обновленную ценность.
user591593 11 нояб. 2014, в 19:46
5

То, как вы случайно делитесь указателем на частную собственность другого объекта, на самом деле плохая идея, это может привести к множеству проблем. Что лучше, так это создать отдельную book и передать ее обоим, где ни один из них не владеет, но владельцем управляет что-то вроде std::shared_ptr если вы не можете установить политику строгого владения. Другой подход заключается в том, чтобы использовать Cook в качестве дружественного класса Foo чтобы он мог получать необходимые данные без использования своего собственного указателя.
tadman 11 нояб. 2014, в 19:47
0

Да, я буду это менять, но я просто хотел понять, что происходит.
user591593 11 нояб. 2014, в 19:50
2

Я не могу сказать вам, что происходит, потому что ваш пример явно пропускает ту часть кода, в которой есть проблема. Это выглядит как плохой дизайн, он потерпит неудачу, если не будет использоваться очень специфическим образом, но кроме этого он технически обоснован.
tadman 11 нояб. 2014, в 19:52

Показать ещё 3 комментария

Теги:

c++

1 ответ

Ещё вопросы

«Не работает» означает, что именно?
Компилируется, но дает неверный расчет. Поэтому я думаю, что книга, когда к ней обращается Кук, не отражает обновленную ценность.
То, как вы случайно делитесь указателем на частную собственность другого объекта, на самом деле плохая идея, это может привести к множеству проблем. Что лучше, так это создать отдельную book и передать ее обоим, где ни один из них не владеет, но владельцем управляет что-то вроде std::shared_ptr если вы не можете установить политику строгого владения. Другой подход заключается в том, чтобы использовать Cook в качестве дружественного класса Foo чтобы он мог получать необходимые данные без использования своего собственного указателя.
Да, я буду это менять, но я просто хотел понять, что происходит.
Я не могу сказать вам, что происходит, потому что ваш пример явно пропускает ту часть кода, в которой есть проблема. Это выглядит как плохой дизайн, он потерпит неудачу, если не будет использоваться очень специфическим образом, но кроме этого он технически обоснован.

Jonathan Mee · Accepted Answer · 2014-11-11T17-11-00.000Z

То, что вы делаете, должно работать нормально, хотя я не могу согласиться с тадманом, что вы должны использовать std::shared_ptr.

Если вы видите, что ваши значения не синхронизированы, вероятно, вы перезаписали Foo.book но это изменение не было распространено на Foo.cook.book.

Возможный лучший дизайн:

class Foo {
private:
    Book book;
    Cook cook;
    Book& getBook();
public:
    Foo() : cook(std::bind(getBook,this));
    virtual ~Foo();
};

class Cook {
private:
    std::function<Book&()> _memfunptr;
public:
    Cook(std::function<Book()> memfunptr): _memfunptr(memfunptr);
    virtual ~Cook();
};

Отсюда в любое время, когда вы захотите обратиться к Foo.book внутри Foo.cook вы можете просто использовать Cook._memfunptr для доступа к нему.

В качестве примечания здесь некоторые из вас могут быть связаны с реализацией класса? У вас должна быть точка с запятой в конце каждого class, и вы должны быть конструктором Cook должен быть определен Cook(); не Cook cook; ,