c – 编程范例想知道是否需要重写/重构

在一段时间内,我一直在应用程序.由于程序设计只是一个兴趣,所以这个项目已经过时了,但这还不止这一点.我现在正处于一个“每个问题”变得非常难解决的地步.我正在考虑重构代码,但是会导致“完整”重写.

让我解释这个问题,我现在解决了什么.基本上我有数据,我让事情发生在这些数据(我描述的每个程序没有我吗？).会发生什么？

Data -> asks viewer to display -> viewer displays data based on actual data
viewer returns user input -> data -> asks “executor” to execute it -> new data

现在这个工作非常好,我原本以为“我可以用qt或windows来改变命令提示符,或者甚至把外部的(C#),直接调用这个程序”.

然而,随着方案的成长,它变得越来越麻烦了.最重要的是,数据以不同的方式显示,取决于数据是什么,更重要的是它位于何处.所以我回到树上添加到“追踪”父行是什么“,然后一般查看器将搜索最具体的实际小部件.
它使用的列表有[位置;小部件]值,并找到最匹配的位置.

更新新的“数据”时,问题开始 – 我必须通过所有的资源 – 查看器,保护程序等等.更新检查机制给了我很多错误..像“嘿,为什么它显示错误的小部件现在呢？“

现在我可以完全互换这个.而不是树数据结构调用一般的查看器.我将使用OO“内部”树功能.节点将是子节点(当需要新的观察者或节点机制时,形成新的子节点).

这将消除困难的检查机制,其中我检查树中的位置.然而,它可能会打开一个其他的蠕虫病毒.
我想对此有一些意见？我应该保持观众完全分离 – 难以检查数据吗？或者更好的是新的方法,但它结合了数据和执行到单个节点. (所以如果我想从qt改为说cli / C#几乎是不可能的)

最后我应该采取什么方法？还有什么我可以做的吗？为了保持观众分开,但是防止不必做检查,看看应该显示哪个小部件？

编辑,只是为了显示一些“代码”和我的程序如何工作.不知道这是否有好处,因为我已经说过,它已经成为相当多的方法论.

这意味着要合并几个“gamemaker项目”(因为GM：工作室奇怪地缺乏这个功能). Gamemaker项目文件只是一组xml文件. (主XML文件,其中只包含其他xml文件的链接,每个资源的对象,精灵,声音,房间等)的xml文件.然而,有一些“怪癖”使得它不可能用boost属性树或qt来读取：1)属性/子节点的顺序在文件的某些部分是非常重要的.和2)白色空间经常被忽略,但在其他方面,保存它是非常重要的.

就这样说,节点完全一样,还有很多点.像背景一样,< width> 200< / width>房间也可以这样.然而对于用户来说,他谈论的宽度是非常重要的.

无论如何,所以“一般查看器”(AskGUIFn)有以下typedef来处理这个：

typedef int (AskGUIFn::*MemberFn)(const GMProject::pTree& tOut,const GMProject::pTree& tIn,int) const;
    typedef std::vector<std::pair<boost::regex,MemberFn> > DisplaySubMap_Ty;
    typedef std::map<RESOURCE_TYPES,std::pair<DisplaySubMap_Ty,MemberFn> > DisplayMap_Ty;

其中“GMProject :: pTree”是一个树节点,RESOURCE_TYPES是一个常量,用于跟踪目前我在使用什么样的资源(精灵,对象等). “memberFn”将在这里简单地加载一个小部件. (虽然AskGUIFn当然不是唯一的一般查看器,但是只有当其他“自动” – 覆盖,跳过,重命名处理程序失败时,才会打开这个.

现在来显示这些地图是如何初始化的(命名空间“MW”中的所有内容都是一个qt小部件)：

AskGUIFn::DisplayMap_Ty AskGUIFn::DisplayFunctionMap_INIT() {
    DisplayMap_Ty t;
        DisplaySubMap_Ty tmp;

        tmp.push_back(std::pair<boost::regex,AskGUIFn::MemberFn> (boost::regex("^instances "),&AskGUIFn::ExecuteFn<MW::RoomInstanceDialog>));
        tmp.push_back(std::pair<boost::regex,AskGUIFn::MemberFn> (boost::regex("^code $"),&AskGUIFn::ExecuteFn<MW::RoomStringDialog>));
        tmp.push_back(std::pair<boost::regex,AskGUIFn::MemberFn> (boost::regex("^(isometric|persistent|showcolour|enableViews|clearViewBackground) $"),&AskGUIFn::ExecuteFn<MW::ResourceBoolDialog>));
        //etc etc etc
    t[RT_ROOM] = std::pair<DisplaySubMap_Ty,MemberFn> (tmp,&AskGUIFn::ExecuteFn<MW::RoomStdDialog>);

        tmp.clear();
        //repeat above
    t[RT_SPRITE] = std::pair<DisplaySubMap_Ty,MemberFn>(tmp,&AskGUIFn::ExecuteFn<MW::RoomStdDialog>);
    //for each resource type.

然后当树数据结构告诉一般观众希望显示时,观众执行以下功能：

AskGUIFn::MemberFn AskGUIFn::FindFirstMatch() const {
    auto map_loc(DisplayFunctionMap.find(res_type));
    if (map_loc != DisplayFunctionMap.end()) {
        std::string stack(CallStackSerialize());
        for (auto iter(map_loc->second.first.begin()); iter != map_loc->second.first.end(); ++iter) {
            if (boost::regex_search(stack,iter->first)) {
                return iter->second;
            }
        }
        return map_loc->second.second;
    }

    return BackupScreen;
}

这就是问题开始坦白的地方. CallStackSerialize()函数依赖于一个调用栈..但是call_stack存储在一个“处理程序”中.我把它存放在那里,因为一切都从一个处理程序开始.我不太确定我应该在哪里存储这个“call_stack”.介绍另一个跟踪发生了什么的对象？
我试图去路由我存储父节点本身的节点. (防止调用堆栈的需要).然而,这并没有像我所希望的那样：每个节点只有一个包含其子节点的向量.所以使用指针是指向父笔记的问题…
(PS：也许我应该在另一个问题上改革这个)