bad ptr string trong hàm khởi tạo của lớp dẫn xuất

[vipzero]

Nếu mình đặt debug ở hàm dựng của Figure và Reca thì sẽ thấy :
sau khi khởi tạo Figure thì lúc nhảy sang dụng Reca thì tất cả các thuộc tính Attribute::name mà Reca kế thừa từ Figure đều mất hết dữ liệu

Mình có lớp Figure được khai báo và cài đặt hàm khởi tạo như sau :

class Figure{protected:    vector<Attribute*> attribute;//Phục vụ cho việc rút gọn hàm SetAttribute      Color strokeColor;    int strokeWidth;    float strokeOpacity;     Color fillColor;    float fillOpacity;     int translateX;    int translateY;     float rotateAxax;     //Các phương thức chínhpublic:    virtual void Draw(HDC hdc) = 0;//Chỉ có lớp dẫn xuất(lớp con) mới cài đặt và khi lớp cha gọi hàm này thì sẽ được gọi hàm tương ứng trong lớp con    virtual void SetAttribute(char* name,char* value); //Cả lớp con(lớp dẫn xuất) và lớp cha(lớp cơ sở) đều có thể cài đặt                                                        //và khi lớp cha gọi hàm này thì sẽ được gọi hàm tương ứng trong lớp con        void Design(Color strokeColor,float strokeWidth,float strokeOpacity,Color fillColor,float fillOpacity);//Các phần như tô màu,độ sáng public://Hàm khởi tạo    Figure();    virtual ~Figure();}

Figure::Figure(){    strokeColor.SetValue(Color::Black);//màu đen    strokeWidth = 2;//Nên để là 2 thì nhìn thấy rõ hơn    strokeOpacity = 1.0;        fillColor.SetValue(Color::White);//màu trắng    fillOpacity = 1.0;        rotateAxax = 0;     translateX = 0;    translateY = 0;     //attribute trỏ tới đám thuộc tính ở trên và thiết lập name tương ứng cho các thuộc tính    Attribute* tempStrokeColor;    tempStrokeColor = &UpdatedColor(strokeColor);    tempStrokeColor->SetName("stroke");    this->attribute.push_back(tempStrokeColor);     Attribute* tempStrokeWidth;    tempStrokeWidth = &Int(strokeWidth);    tempStrokeWidth->SetName("stroke-width");    this->attribute.push_back(tempStrokeWidth);     Attribute* tempStrokeOpacity;    tempStrokeOpacity = &Float(strokeOpacity);    tempStrokeOpacity->SetName("stroke-opacity");    this->attribute.push_back(tempStrokeOpacity);     Attribute* tempFillColor;    tempFillColor = &UpdatedColor(fillColor);    tempFillColor->SetName("fill");    this->attribute.push_back(tempFillColor);     Attribute* tempFillOpacity;    tempFillOpacity = &Float(fillOpacity);    tempFillOpacity->SetName("fill-opacity");    this->attribute.push_back(tempFillOpacity);     Attribute* tempRotateAxax;    tempRotateAxax = &Float(rotateAxax);    tempRotateAxax->SetName("rotate");    this->attribute.push_back(tempRotateAxax);     Attribute* tempTranslateX;    tempTranslateX = &Int(translateX);    tempTranslateX->SetName("translate-x");    this->attribute.push_back(tempTranslateX);     Attribute* tempTranslateY;    tempTranslateY = &Int(translateY);    tempTranslateY->SetName("translate-y");    this->attribute.push_back(tempTranslateY);}

Lớp Reca kế thừa Figure :

class Reca : public Figure//recangle : hình chữ nhật {private://Các thuộc tính không được truy cập    int x;    int y;    int width;//chiều rộng    int height;//chiều cao public://Các phương thức Thiết kế va vẽ      void Draw(HDC hdc);//đọc hình chữ nhật và vẽ nó trên nền hdcpublic:    ////Thiết lập thuộc tính dựa trên file SVG    //void SetAttribute(char* name,char* value); public://Phương thức lấy giá trị đặc trưng của Rect    int GetX();    int GetY();    int GetWidth();    int GetHeight(); public://Phương thức thay đổi các giá trị đặc trưng của Rect    void SetShape(int x,int y,int width,int height);//thay đổi hình dáng thành Rec khác public://Hàm Khởi tạo    Reca();    Reca(int x,int y,int width,int height);//Thiết lập các thuộc tính còn lại     ~Reca();};

Reca::Reca():Figure(){    x = y = 0;    width = height = 0;     //attribute trỏ tới đám thuộc tính ở trên và thiết lập name tương ứng cho các thuộc tính    Attribute* tempX;    tempX = &Int(x);    tempX->SetName("x");    this->attribute.push_back(tempX);     Attribute* tempY;    tempY = &Int(y);    tempY->SetName("y");    this->attribute.push_back(tempY);     Attribute* tempWidth;    tempWidth = &Int(width);    tempWidth->SetName("width");    this->attribute.push_back(tempWidth);     Attribute* tempHeight;    tempHeight = &Int(height);    tempHeight->SetName("height");    this->attribute.push_back(tempHeight); }

Phần khai báo và cài đặt Attribute :

class Attribute{protected:    string name; public://Khởi tạo và hủy    Attribute();    Attribute(const Attribute&);    virtual ~Attribute();    public://Các hàm chính    virtual void SetValueAttribute(char* value) = 0;    public://Thiết lập thuộc tính    void SetName(string name); public://Lấy giá trị thuộc tính    string GetName();};

Attribute::Attribute(){    this->name = "";}Attribute::Attribute(const Attribute& attr){    this->name = attr.name;}Attribute::~Attribute(){} void Attribute::SetName(string name){    this->name = name;} string Attribute::GetName(){    return this->name;}

Nếu thấy khó nhìn quá thì các bạn có thể debug thử project

[doicanhden]

Nếu mình đặt debug ở hàm dựng của Figure và Reca thì sẽ thấy :
sau khi khởi tạo Figure thì lúc nhảy sang dụng Reca thì tất cả các thuộc tính Attribute::name mà Reca kế thừa từ Figure đều mất hết dữ liệu

Mình có lớp Figure được khai báo và cài đặt hàm khởi tạo như sau :

Figure::Figure(){    strokeColor.SetValue(Color::Black);//màu đen    strokeWidth = 2;//Nên để là 2 thì nhìn thấy rõ hơn    strokeOpacity = 1.0;        fillColor.SetValue(Color::White);//màu trắng    fillOpacity = 1.0;        rotateAxax = 0;     translateX = 0;    translateY = 0;     //attribute trỏ tới đám thuộc tính ở trên và thiết lập name tương ứng cho các thuộc tính    Attribute* tempStrokeColor;    tempStrokeColor = &UpdatedColor(strokeColor);    tempStrokeColor->SetName("stroke");    this->attribute.push_back(tempStrokeColor);     Attribute* tempStrokeWidth;    tempStrokeWidth = &Int(strokeWidth);    tempStrokeWidth->SetName("stroke-width");    this->attribute.push_back(tempStrokeWidth);     Attribute* tempStrokeOpacity;    tempStrokeOpacity = &Float(strokeOpacity);    tempStrokeOpacity->SetName("stroke-opacity");    this->attribute.push_back(tempStrokeOpacity);     Attribute* tempFillColor;    tempFillColor = &UpdatedColor(fillColor);    tempFillColor->SetName("fill");    this->attribute.push_back(tempFillColor);     Attribute* tempFillOpacity;    tempFillOpacity = &Float(fillOpacity);    tempFillOpacity->SetName("fill-opacity");    this->attribute.push_back(tempFillOpacity);     Attribute* tempRotateAxax;    tempRotateAxax = &Float(rotateAxax);    tempRotateAxax->SetName("rotate");    this->attribute.push_back(tempRotateAxax);     Attribute* tempTranslateX;    tempTranslateX = &Int(translateX);    tempTranslateX->SetName("translate-x");    this->attribute.push_back(tempTranslateX);     Attribute* tempTranslateY;    tempTranslateY = &Int(translateY);    tempTranslateY->SetName("translate-y");    this->attribute.push_back(tempTranslateY);}

Tôi đã nhắc nhở bạn xem lại kiếm thức về con trỏ và cấp phát động, tạo đối tượng bằng toán tử new. Có vẻ như bạn bỏ ngoài tai. Vậy thì nên dừng project lại đi. Code sai be bét kiểu này thì có mà làm đến tết.

[vipzero]

Tôi đã nhắc nhở bạn xem lại kiếm thức về con trỏ và cấp phát động, tạo đối tượng bằng toán tử new. Có vẻ như bạn bỏ ngoài tai. Vậy thì nên dừng project lại đi. Code sai be bét kiểu này thì có mà làm đến tết.

ặc mấy cái phần đó đợi đến lúc làm FigureManager đi đã đang quan trọng cái Attribute

Nếu làm theo như cấp phát động :

Attribute* temp = new Int(stroke);

thì nó không đúng với thiết kế vì cái mình cần con trỏ temp sẽ trỏ đến stroke để khi thay đổi *(temp->value) thì sẽ thay đổi stroke

mà nếu làm như thế này

Attribute* temp = new Int;temp = &stroke;

thì như vậy sẽ bị lỗi chưa xóa vùng nhớ Int vừa cấp phát

[doicanhden]

ặc mấy cái phần đó đợi đến lúc làm FigureManager đi đã đang quan trọng cái Attribute

Nếu làm theo như cấp phát động :

Attribute* temp = new Int(stroke);

thì nó không đúng với thiết kế vì cái mình cần con trỏ temp sẽ trỏ đến stroke để khi thay đổi *(temp->value) thì sẽ thay đổi stroke

mà nếu làm như thế này

Attribute* temp = new Int;temp = &stroke;

thì như vậy sẽ bị lỗi chưa xóa vùng nhớ Int vừa cấp phát

Vậy chuyển sang dùng smart pointer của thư viện STL đi. Cụ thể là lớp template std::shared_ptr trong thư viện memory.