在运行时发生此错误,我不确定是什么导致它 - 代码对我来说是正确的。
#include <iostream>
#include <string>
using namespace std;
struct Room {
int d_noSeat;
bool d_hasProjector;
Room() = default;
Room(const Room& r);
};
class Event {
Room* d_room;
std::string d_name;
public:
Event();
Event(const Event& e);
~Event();
void set(Room r, const std::string& name);
void print();
};
Event::Event() : d_room(0), d_name("") {};
void Event::print() {
std::cout << "Event: " << d_name;
if (d_room != 0) {
std::cout << " in size " << d_room->d_noSeat;
if (d_room->d_hasProjector)
std::cout << " with";
else
std::cout << " without";
std::cout << " projector";
}
std::cout << std::endl;
return;
}
void printEvent(Event e) {
e.print();
return;
}
void Event::set(Room r, const std::string& name) {
d_room = &r;
d_name = name;
}
// Room shallow copy constructor
Room::Room(const Room& r) :
d_noSeat(r.d_noSeat),
d_hasProjector(r.d_hasProjector)
{ }
// Event deep copy constructor
Event::Event(const Event& e) :
d_name(e.d_name),
d_room(new Room(*e.d_room))
{ }
// Event destructor
Event::~Event()
{
delete[] d_room;
}
int main() {
const int noLect = 5;
Room r;
Event lectures[noLect];
for (int i = 0; i < noLect; ++i) {
r.d_noSeat = i + 1;
r.d_hasProjector != r.d_hasProjector;
lectures[i].set(r, "CSI2372");
lectures[i].print();
}
std::cout << "-------------------" << std::endl;
for (int i = 0; i < noLect; ++i) {
printEvent(lectures[i]);
}
return 0;
}
错误显然发生在第52行(print()函数的第一行)。除此之外,印刷文本显示非常大且通常为负数的数字。造成这种情况的原因是什么?
答案 0 :(得分:9)
void Event::set(Room r, const std::string& name)
{
d_room = &r;
// ^
d_name = name;
}
您正在引用临时对象:Room r
按值传递,该值在范围的末尾被销毁:}
。
相反,您必须重新分配成员指针:
d_room = new Room(r);
因为您正在用C ++类编写C风格的代码。
在C ++中我们倾向于:
避免使用裸指针,更喜欢智能指针:
class Event
{
std::shared_ptr<Room> d_room;
...
Event::~Event() { /* no need to delete */ }
使用构造函数重载(而不是使用set
- 构造后的类似函数):
Event(Room& r, const std::string& name):
d_room(new Room(r)),
d_name(name)
{}
通过引用传递:
void set(Room& r, const std::string& name);
避免使用原始数组,而是使用STL工具:
std::vector<Event> lectures;
// or
std::array<Event, 5> lectures;
r.d_hasProjector != r.d_hasProjector; // checks if r.d_hasProject is not itself
你可能想要
r.d_hasProjector = !r.d_hasProjector;
完整代码: link
另外,这里有关于高级C ++内容的必读链接,我相信这些内容对您非常有用:http://www.parashift.com/c++-faq/
修改:我忘记了您的问题:
除此之外,印刷文本显示非常大且通常为负数的数字。造成这种情况的原因是什么?
这些数字都是垃圾。未明确初始化的变量根本没有初始化。内存已分配但保留以前程序的旧信息。它可以包含任何东西。当你从未初始化的变量中读取时,你会得到这个垃圾。你有一个指向被破坏对象的指针。所以指针实际上是未初始化的。
答案 1 :(得分:1)
你的问题在这里:
void Event::set(Room r, const std::string& name) {
d_room = &r;
d_name = name;
}
&r
获取一个对象的地址,该对象的生命周期在函数返回时结束,当您稍后尝试访问它时会导致未定义的行为。
如果要使用指针,则需要动态分配它们:
void Event::set(Room* r, const std::string& name) {
d_room = r;
d_name = name;
}
// ...
for (int i = 0; i < noLect; ++i) {
Room* r = new Room;
r->d_noSeat = i + 1;
r->d_hasProjector != r.d_hasProjector;
lectures[i].set(r, "CSI2372");
lectures[i].print();
}
// ...
但看起来你不需要指针,你应该能够
Room d_room;
在Event
班。