【问题标题】:Pointers, Class Items and Scope指针、类项和范围
【发布时间】:2013-06-17 13:01:16
【问题描述】:

我似乎找不到答案,但也许我在搜索错误的术语。我没有在热门歌曲中找到我正在寻找的答案。

我有一堆用于菜单系统的派生类。

我有一个 CControl 派生类,它是 CEditBoxCLabel 类的父类。 CLabel 只不过是将文本附加到 SDL_Surface 上,然后将其绑定到纹理以供 openGL 渲染。 CEditBox 将是一个用于显示文本或从用户那里收集文本的字段,例如密码框。显然,CEditBox 可以使用标签来处理框内的文本呈现。 CControl 派生自 CComponent

我不能在CEditBox 中声明CLabel,除非我在标头中包含CLabel,但我认为即使我的所有标头都包含在#ifndef #define class #endif 语法中,我仍然会收到链接器错误,但我是也是菜鸟。相反,我声明了一个 CComponent* 指针,因为它们是从该类派生的。

很好。现在在CEditBox 的构造函数中我有:

#include "CLabel.h" //include in .CPP is fine I reckon.

CEditBox::CEditBox() {
    CLabel Field;      //Create CLabel
    InputType = ALL;   //Not important for my question related to allowed symbols
    Label = &Field;    //CComponent pointer to CLabel

}

当这个构造函数返回时,CLabel 不会超出范围,因此 Feild 会被销毁,现在我的指针指向未定义的内存块吗?什么是合适的方法来做到这一点?有更好的解决方案吗?

谢谢

链接器问题

我不知道问题是否存在,但有些人认为这是一个更重要的问题。那么这里是实际的代码,你们可以告诉我你是否认为它做得不正确。 基类 CMenuObject

#ifndef _CMENUOBJECT_H_
#define _CMENUOBJECT_H_
class CMenuObject {

protected:
    const char* ClassName;
public:
    CMenuObject();
    virtual const char* Object();

};

#endif

下一个类是 CComponent

#ifndef _CCOMPONENT_H_
#define _CCOMPONENT_H_

#include "CMenuObject.h"

class CComponent : public CMenuObject {
protected:
    const char* _Name;
    int _Tag;
    static int _ComponentCount;
    static int _IDCount;

public:
    CComponent();
    virtual const char* Name();
    virtual int Tag();
    virtual void Tag(int t);


};


#endif

然后是 CControl 这些将是用户将与之交互或以某种方式需要控制显示的对象(即计时器不需要用户输入)并且是一个庞然大物。不要介意函数指针的东西,因为我还不知道我在做什么......这是我处理事件的第一个猜测方法。我认为这是有限制的,因为如果函数需要带参数但我可能不需要,我不知道该怎么做,等等......我们现在可以忽略这个细节。

#ifndef _CCONTROL_H_
#define _CCONTROL_H_

#include "CComponent.h"

class CControl : public CComponent {
protected:

    int _X,_Y,_Width,_Height;
    float R,G,B,A;

    void (*OnClk)();
    void (*OnDblClk)();
    void (*OnMOver)();
    void (*OnMHover)();
    void (*OnKDown)();
    void (*OnKUp)();
    void (*OnFcs)();

    bool Visible;

    CComponent* Pappy;


public:
    CControl();

    //Render Control
    virtual void Show();                                            //      Show Component
    virtual void Hide();                                            //      Hide Component
    virtual void OnRender();                                        //      Render Component

    virtual bool IsVisible();                                       //      Get Current Visibility Status

    //Paramater Control
        //Write
    virtual void X(int x);                                          //      Set Component's X coordinate
    virtual void Y(int y);                                          //      Set Component's Y coordinate
    virtual void Width(int w);                                      //      Set Component's Width
    virtual void Height(int h);                                     //      Set Component's Height
        //Read
    virtual int X();                                                //      Get Component's X coordinate
    virtual int Y();                                                //      Get Component's Y coordinate
    virtual int Width();                                            //      Get Component's Width
    virtual int Height();                                           //      Get Component's Height

    //Display Control
    virtual void Color(float r, float g, float b);                  //      Set Color of Component- Multicolored objects, this will be the base or bkg color.  Makes alpha 1.0f.
    virtual void Color(float r, float g, float b, float a);         //      Same as above but allows for input of an alpha value. 

    //Font Control
    virtual void FontName(const char* font);                        //      Name of font to use
    virtual void FontSize(int pt);                                  //      Pt size of font.  Or maybe pixel, no idea.
    virtual void Text(const char* msg);                             //      Text message to render
        //Read
    virtual const char* Text();                                     //      Read Text Message

    //Interactive Control                                           //      These will register call back functions for user events
    virtual void OnClick(void (*func)());                           //      On Single Click
    virtual void OnDoubleClick(void (*func)());                     //      On Double Click
    virtual void OnMouseOver(void (*func)());                       //      On Mouse Over
    virtual void OnMouseHover(void (*func)());                      //      On Mouse Hover
    virtual void OnKeyDown(void (*func)());                         //      On Key Down
    virtual void OnKeyUp(void (*func)());                           //      On Key Up
    virtual void OnFocus(void (*func)());                           //      On Focus 

    //Other
    virtual void Parent(CComponent);                                //      Set Parent
    virtual CComponent* Parent();                                   //      Get Parent
};

#endif

最后是我的 CLabel 和 CEditBox 的最终游戏标题。

#ifndef _CLABEL_H_
#define _CLABEL_H_

#include "CTexture.h"
#include "CFont.h"
#include "CControl.h"


class CLabel : public CControl {
private:

    const char* vText;

    CFont Font;

    CTexture Text_Font;
    SDL_Surface* Surf_Text;

    int X,Y,vWidth,vHeight;

public:
    CLabel();
    CLabel(const char* text);

    virtual void OnRender();
    virtual void OnCleanup();

    virtual void Text(const char* msg);
    virtual const char* Text();

    virtual void FontName(const char* fname);
    virtual void FontSize(int pt);
    virtual void FontColor(float r, float g, float b);
};


#endif

#ifndef _CEDITBOX_H_
#define _CEDITBOX_H_

#include "CControl.h"

class CEditBox : public CControl  {
protected:

    CComponent* Label;
    int InputType;



public:
    CEditBox();
    ~CEditBox();
    virtual void OnRender();
    //virtual void OnCleanup();
    virtual void OnLoop();

    virtual void Text(const char* msg);
    virtual const char* Text();

    virtual void FontColor(float r, float g, float b);

    virtual void OnClick(void (*func)());                           //      On Single Click
    virtual void OnDoubleClick(void (*func)());                     //      On Double Click
    virtual void OnMouseOver(void (*func)());                       //      On Mouse Over
    virtual void OnMouseHover(void (*func)());                      //      On Mouse Hover
    virtual void OnKeyDown(void (*func)());                         //      On Key Down
    virtual void OnKeyUp(void (*func)());                           //      On Key Up
    virtual void OnFocus(void (*func)());                           //      On Focus 


    enum {
        ALL = 0,                //abcdefghijklmnopqrstuvwxyz (and caps) 1234567890!@#$%^&*()_+-=[]{}<>\/|"';:,.?
        ALPHA_NUMERIC,          //abcdefghijklmnopqrstuvwxyz (and caps) 1234567890
        ALPHA,                  //abcdefghijklmnopqrstuvwxyz (and caps)
        NUMERIC,                //1234567890
        PASSWORD,               //abcdefghijklmnopqrstuvwxyz (and caps) 1234567890!@#$%&.     -- Render as *
        IP                      //1234567890 .  Maybe fix feild width and force xxx.xxx.xxx.xxx format.
    }; 
};

#endif

[已解决]

今天,我发现了一个未包含在#ifndef #define #endif 中的 dang 标头。 (它是CTexture,它在CFont 中再次被调用。无论如何,重组也非常有益,因为我已经弄清楚如何使用继承和基类指针,以及派生类如何相互工作。不要提到更多的事情。:)

我为派生类交互采取的路线是使用基类指针,该指针可以通过虚函数访问派生类函数。我使用 new 和 delete 因为那是我喜欢的。对于所有做出贡献的人,谢谢!他们都是很好的答案。

【问题讨论】:

  • 从那时起。就像在数据容器中使用结构和节点一样。好的,那么我需要处理这个以减少内存泄漏吗?喜欢delete [] Label
  • 抱歉,在您回答之前我已经删除了我的评论。但是,我的评论可以作为下面的答案。
  • 是的,在我放弃我的评论和我的评论的答案之后,你的答案就在那里。 +1 ESP。
  • 即使new在这种情况下是正确的,我还是建议您使用包含保护来修复错误。尽可能避免new,以避免出现错误。
  • 我可能有。看看发生了什么,我不是一名程序员(显然),但我上过正规学校,我只是选择了一个不同的专业,并在大学的前两年专注于编程。因此,我以前从未实现过这样的结构。我们更关注数据结构。当我开始时,我从分支开始(制作一个标签和一个编辑框)并尝试将它们链接在一起,我在尝试编译时遇到了各种各样的问题,但是如果我删除了包含并注释掉了这些结构的指针,它编译了。

标签: c++ class pointers inheritance scope


【解决方案1】:

立体的典型方法是:

展示第二种方法:

//////////// CEditBox.hpp header file
#include <memory>
#include <string>

class CLabel; // forward declaration

class CEditBox
{
  public:
    CEditBox(std::string const&);
  private:
    std::unique_ptr<CLabel> _label;
};

前向声明避免了包含 CLabel.hpp。 unique_ptr 管理生命周期 _label,所以我们不必记住 delete 它。

//////////// CLabel.hpp header file

#include <string>
#include "CLabel.hpp"

class CLabel
{
  public:
    CLabel(std::string const& name) 
        : _name(name) 
    {
    }
  private:
    std::string _name;
};

只是一个样本,这里没什么可看的。让我们继续:

///////////// CEditBox.cpp source file

#include "CEditBox.hpp"
#include "CLabel.hpp"

CEditBox::CEditBox(std::string const& name)
    : _label(new CLabel(name)) 
{
}

这就是神奇之处:我们也通过包含 CLabel.hpp 来集成它,并在 初始化器列表中构造它。

///////////// main.cpp source file

#include "CEditBox.hpp"

int main()
{
    CEditBox box("Hello world"); // no need to 'know' CLabel here   
}

布丁的证明在编译中:http://ideone.com/zFrJa8

【讨论】:

  • 是的,我不知道这个 pimpl 到底是什么,而且语法令人困惑。让我问一下:_label 是指向编辑框的指针。好吧,这就是我失去的地方。 CLabel(std::string name) : _name(std::move(name)) {} 吗? CLabel 声明了一个 CLabel 类型的变量,但是 (std::string name) 对它做了什么?那是调用某种构造函数吗?我迷路了。我 认为 正在发生的事情是您将标签的内存移动到 CEditBox 中,但我真的不明白语法。主要是我指出的那一行。
  • @Chemistpp 刚刚收到您的评论。首先,如果 C++ 语法有点多,这几乎不是我的错。毕竟,看起来你真的想学习它?无论如何,我将代码格式化得更友好一些,并删除了std::move 的不相关使用。请注意,我已经命名语法 if constructor initializer list (CLabel() : _name(name)) 所以 google should help you along the way 如果这对你来说是新的。
  • 我希望现在会更清楚。作为记录,我没有展示 pimpl,因为它实际上是您在此处看到的基本技术的“高级”版本。
  • 我现在明白了。 :只是让我们将值 _label 定义为新的 CLabel(name)) ,据我了解, std::unique_ptr LookAtThat 本质上将删除 LookAtThat 超出范围时指向的内存。所以真的,除了头文件中的狡猾定义和自动释放内存之外,第一个答案和这个没有太大区别。
  • 另外,是的,我从不做任何事情只是为了回答或只是为了记住,总是为了理解。我感谢您的解释。
【解决方案2】:

你的想法是正确的。正确的方法是动态分配这个对象,即

Label = new CLabel;

别忘了在析构函数中释放内存:

delete Label;

【讨论】:

  • 请不要在 C++ 中教新/删除。尤其是对“初学者”。这很容易出错。
  • 如果您在谈论诸如 C 和 C++ 之类的语言,首先应该学习内存管理。
  • 我有 2 年的 C、C++ 和汇编经验。我非常理解记忆。如果你看,我在他的评论中问他,甚至在他提到我应该删除记忆之前。到目前为止,这是最简单的方法,最容易理解,也是我问题的准确答案。
  • 我最薄弱的领域是语法知识和大程序开发(结构)。我几乎没有接触过阅读代码,尤其是在过去的几年里。我只有 2 个学期的 C++,而且都是基于控制台的算法和数据结构(不是编程,而是更像是创建节点并将它们与指针和算法链接起来以进行排序、搜索和导航),所以继承对我来说是全新的。
  • @AndrejsCainikovs 好吧,我完全同意。仅:内存管理!= new/delete!所以是的,尽早教授内存管理(值/引用语义)。但是只在接近尾声时教new(至少在容器和算法之后)。
【解决方案3】:

如果我正确理解您的问题,您有两个类,它们的成员变量与彼此的类有关?

例如:

// A.h
#ifndef A_H
#define A_H

#include "B.h"

class A {
public:
    ...
private:
    B* pB;
};
#endif // A_H

还有:

// B.h
#ifndef B_H
#define B_H

#include "A.h"

class B {
public:
    ...
private:
    A* pA;
};

#endif // B_H

将这些编译在一起会导致某种形式的链接器错误?如果是这种情况,您可以通过前向声明类来规避这种情况,因此您可以在 class Aclass A; 的声明上方简单地将 class B; 写在 class B 声明之上,而不是包括 A.hB.h ,然后在您的 cpp 文件中包含标头。所以A.h 看起来像:

// A.h
#ifndef A_H
#define A_H

class B;
class A {
public:
    ...
private:
    B* pB;
};
#endif // A_H

【讨论】:

  • 是的!我听说您可以转发声明类,只要在您尝试访问其成员之前为转发声明的类运行实现即可。出于某种原因,我认为这似乎有点奇怪,而且我知道必须有一种方法可以做到这一点,因为我已经看到了使用基类指针的示例。
【解决方案4】:

CLabel Field; 应该是CEditBox 的成员。真正的问题是您提到“链接器错误”(或其他任何问题)的问题。这就是你应该解决的问题。

【讨论】:

  • 是的,我认为它来自结构不佳的系统。我只是想完成这最后一件事,以检查并确保它现在已正确链接。我还听说我可以创建一个“Implement.h”文件并基本上以正确的顺序实现该文件中的所有内容,这样所有其他文件都可以只包含那个头文件。不确定这是否是明智的选择。
  • 好吧,提供一些关于这个问题的信息,包括真正的错误和部分来源,这里的人会尽力帮助你。
  • 我会添加它,但我已经将我的文件混为一谈了,以至于无法再次收到错误消息。而且已经完全重组了,我什至不知道问题存在了。
猜你喜欢
  • 1970-01-01
  • 2011-07-22
  • 2016-11-15
  • 1970-01-01
  • 1970-01-01
  • 2018-10-07
  • 1970-01-01
  • 2011-02-26
  • 2021-11-25
相关资源
最近更新 更多