V2EX kriko
 kriko 最近的时间轴更新
kriko

kriko

V2EX 第 498266 号会员,加入于 2020-07-08 17:12:15 +08:00
kriko 最近回复了
基于我对代码库的全面审查,作为一个严格的技术经理,我的评价是:
---
规范
---
理由:
优点 (值得肯定):
1. 清晰的接口抽象 - IGraphicsDevice, IGraphicsContext, IRHICommandList 等接口层与 DX12* 实现层分离干净,符合依赖倒置原则
2. 一致的命名规范 - m_ 前缀成员变量、PascalCase 方法名、I 前缀接口类严格遵守
3. 统一的命名空间宏 - RF_BEGIN_NAMESPACE_GRAPHICS, RF_BEGIN_NAMESPACE_PLATFORM_N(DX12) 等宏的使用一致
4. 正确的错误处理 - ThrowIfFailed 包装所有 HRESULT 调用,无裸 HRESULT 忽略
5. 智能指针使用 - std::shared_ptr, ComPtr 使用恰当,无裸 new/delete
6. RenderGraph 设计成熟 - 依赖图构建、拓扑排序、资源屏障生成、死代码剔除等算法实现正确且有工业级水准
问题 (需要改进):
1. 头文件全局变量 - IGraphicsDevice.h 第 36 行 static std::shared_ptr<IGraphicsDevice> g_graphicsDevice; 在头文件中定义静态变量,会导致每个包含此头文件的翻译单元都有独立实例
2. 部分魔法数字 - ApplySlotConnections() 中 sizeof(*passPtr) + 1024 这种启发式内存范围检测较脆弱
3. 缺乏单元测试 - 虽然 README TODO 已列出,但目前无自动化测试覆盖
4. 文档与代码分离 - README 的中文内容与 AGENTS.md 要求的英文注释不一致
---
2024 年 3 月 29 日
回复了 Yux0 创建的主题 宽带症候群 关于远程、内网穿透的问题请教一下各位
用 Sakura Frp 也行吧,直接图形化操作,也是将 RDP 端口开出来。
我经常这样在公司连家里电脑,还能打游戏
2024 年 3 月 28 日
回复了 Dongxiaohao 创建的主题 生活 想买车,但是又不敢买
先租车,不把所有车型都试过怎么知道哪一款适合自己
其它方面也一样
2024 年 3 月 26 日
回复了 rivercherdeeeeee 创建的主题 问与答 有什么新手入门的搬砖手游吗
元神代肝
没办法,考研吧,本科能做做验证就不错了
希望能攒到钱去老家躺平
给一百块让他活一个月,不给进家门
2024 年 2 月 18 日
回复了 dc2002007 创建的主题 职场话题 新年第一天,你们收到了开工红包没?
收到 300 现金
又中了个奖,一个模型,大概 200 多
有没有支持路径搜索,比如说想从空涡龙配到导弹喵,需要配几次,怎么配
2024 年 2 月 4 日
回复了 jaybing926 创建的主题 职场话题 你们年终奖都发了多少?
发了三千万
关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     1253 人在线   最高记录 6679       Select Language
创意工作者们的社区
World is powered by solitude
VERSION: 3.9.8.5 19ms UTC 16:18 PVG 00:18 LAX 08:18 JFK 11:18
Do have faith in what you're doing.
ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86