代码重构 —— 化繁为简

本文涉及 KISS原则,DRY原则

1 设计层面

1.1 保持架构的一致性,不要另辟新径

比如渲染场景管理中,最扁平的架构:manager / scener / object 三层。

  • 都是1对N的关系
  • 一个mgr管理一群scener,一个scener下面N个object,

一些新手会把object搞成一个,在object下面搞一层 layer,最终object的含义非常不同了,

让我也"眼前一亮",居然还能这么扩展

1.2 coding的时候,多想一步,不要无脑的直接干

比如类或者结构体,业务A一个struct,业务B一个struct。

好处:物理隔离 没有风险,不会出错,

坏处:代码碰撞,结构体膨胀,

不够简洁,同时触犯了DRY,do not repeat yourself,

2 实战

拿RHI模块来说。见过撇足的设计,耦合了大量的业务类。RHI定义了一层shader,业务模块中又大量的派生,定义出来

cpp 复制代码
XXXShader : public RHI::shader {}
YYYShader : public RHI::shader {}

不止shader,program,renderer都是配套的,无谓的封装导致结构体膨胀,代码大量冗余。一个业务3个类,N个业务 3*N的增长。

如何解决呢?

结构体本质是不同类型的集合,采用反射的形式进行抽象和提炼。

1)函数的形式:

cpp 复制代码
setUniform1f("key", v1f)
setUniform4f("key", v4f)
setUniformMat44("mvp", mat)

2)静态存储的形式

map< std::key, std::any> ,类似lua的metatable,

any在c++17之后采用,自己网上找一个就行。

3 推广

同样的道理,JNI的交互也经常遇到这类问题。

不好的设计,Java中的一个类,为了对应c++的一个类,硬生生的在JNI层又搞了一个类JXXX 类管理反射相关的信息。

cpp 复制代码
class GuideAreaLayerConfig
{
public:
    std::string day_tex;
    std::string night_tex;
    float erase_dis;
}

class JGuideAreaLayerConfig {
public:
    static jobject ToJavaObject(JNIEnv* env, const GuideAreaLayerConfig& layer_config);
    static GuideAreaLayerConfig Parse(JNIEnv* env, jobject obj);
    static void RegisterMe(JNIEnv* env);
public:
  static jclass clazz;
  static jfieldID textureDay;
  static jfieldID textureNight;
  static jfieldID eraseDis;
};
java 复制代码
class GuideAreaLayerConfig extends LayerBaseConfig {
    public GuideAreaLayerConfig() {
    }

    public float eraseDis = 3;
    public String textureDay = "";
    public String textureNight = "";
}

也可以利用反射的形式,采用map进行抽象管理,大幅度压缩代码。JNI的类型setter和getter 自有一套。

相关推荐
兰亭妙微3 小时前
用户体验设计 | 从UX到AX:人工智能如何重构交互范式?
人工智能·重构·ux
SmalBox20 小时前
【渲染流水线】[输出阶段]-[双缓冲机制]以UnityURP为例
unity·渲染
DDC楼宇自控与IBMS集成系统解读1 天前
BA 楼宇自控系统 + AI:重构楼宇设备管理的 “智能决策” 体系
大数据·网络·数据库·人工智能·3d·重构
躺不平的小刘1 天前
通义灵码插件——AI 重构表单开发!半小时搭建可视化拖拽系统,效率碾压传统模式
人工智能·重构
闹小艾1 天前
实体店转型破局之道:新零售社区商城小程序开发重构经营生态
重构·零售
技术与健康2 天前
LLM实践系列:利用LLM重构数据科学流程04 - 智能特征工程
数据库·人工智能·重构
淡海水2 天前
【URP】Unity 插入自定义RenderPass
unity·游戏引擎·渲染·shader·renderpass
SmalBox2 天前
【渲染流水线】[逐片元阶段]-[混合Blend]以UnityURP为例
unity·渲染
技术与健康2 天前
LLM实践系列:利用LLM重构数据科学流程03- LLM驱动的数据探索与清洗
大数据·人工智能·重构