33 条回复  ·  3595 次点击
lscho 小成 2026-1-12 11:55:35
感觉好像发现新思路,写一段时间让 AI 评估再优化一下
Gilfoyle26 小成 2026-1-12 12:06:16
又不是不能用 https://i.imgur.com/io2SM1h.png
Astrian 初学 2026-1-12 12:29:20
扔了我之前做的一个开源浏览器扩展,彩虹屁吹上天了…… 顺便一提那个播放器组件也是我自己搓的 --- **专业** ## 理由: ### 1. 技术架构现代且合理 - **Vue 3 Composition API** + TypeScript ,类型安全 - **Pinia** 状态管理,架构清晰 - **Tailwind CSS v4** 最新版本,样式高效 - 使用专门的音频处理库 **@astrian/music-surge-revolution**,而非自己实现复杂逻辑 ### 2. 工程化完善 - **Biome** 配置完整的代码质量检查和格式化 - 多平台构建支持( Chrome/Firefox/Safari ) - 专门的预构建脚本处理平台差异 - Tab 缩进一致性配置 ### 3. 代码质量保障 ```json "linter": { "enabled": true, "rules": { "recommended": true, "correctness": { "noUnusedVariables": "error", ... }, "security": { "noGlobalEval": "error", ... }, "style": { "useConst": "error", ... } } } ``` ### 4. 项目组织清晰 ``` src/ ├── apis/ # API 集中管理 ├── components/# Vue 组件 ├── stores/ # Pinia 状态管理 ├── utils/ # 工具函数 ├── pages/ # 路由页面 └── types/ # TypeScript 类型定义 ``` ### 5. 浏览器扩展最佳实践 - **Manifest V3** 正确实现 - 跨平台兼容性处理 - CSP 安全策略 - 内容脚本注入架构 ### 6. 开发体验友好 - 清晰的命令行工具( dev/build/lint/quality-check ) - 热重载开发 - TypeScript 类型检查集成 这是一个展现了**高水平工程实践**和**专业开发规范**的代码库。
Lamlam147 小成 2026-1-12 12:30:54
@lscho 第二遍重写会有意料不到的效果
lyxxxh2 小成 2026-1-12 13:52:58
● 评价:混乱 理由: 架构混乱: - 双版本并存( Api/ 和 V2/ 控制器,Model/ 和 V2Model/),技术债务严重 - 超大控制器达 2,381 行( GoodsController.php ),违反单一职责原则 代码混乱: - 54 处调试代码残留( dd()、dump()、var_dump())未清理 - 废弃代码标记"TODO 待废弃"但从未删除 - 拼写错误( Purchse 、Statis )和命名不一致 流程混乱: - 提交信息极不专业:"删除叠屎代码"、"本店加 bug"、"完成" - 20 次提交中多次 bug 修复,说明质量不稳定 - 当前还有未提交的修改文件 安全混乱: - .env 文件含真实密码提交到 git - 10 个文件使用 eval/exec 等危险函数 - SalesStatisController 中 36 处原始 SQL 查询,存在注入风险 测试混乱: - 32 个测试 vs 23,602 个 PHP 文件,覆盖率接近零 - 测试文件命名带"Debug"字样,不规范 核心问题: 缺乏有效的代码审查机制和工程规范,团队 5 人协作但没有统一标准,导致代码库失控。
BernieDu 小成 2026-1-12 14:00:17
Gemini3: Reason: While the project demonstrates a modern and modular directory structure (Features-based) and correctly utilizes strict concurrency with Swift 6 standards, the architectural implementation is not uniform. Specifically, the DailyNutritionView bypasses the established service layer pattern (seen in NutritionAnalysisManager) by instantiating LanguageModelSession and handling complex data streaming logic directly within the View. This violates separation of concerns, leads to code duplication between the detailed analysis and daily tracking features, and reduces the testability of the core AI integration logic. Codex: 脆弱。理由:测试仅是空模板,仓库也未见 持续集成配置,核心功能缺少自动化验证与防回归保障。
huig 初学 2026-1-12 14:06:00
怎么?我是 AI 啊。还要被你设置参数
systemGuest 初学 2026-1-12 14:07:53
评价:能跑就行 理由:十几年前你写的优雅的 asp 代码现在还有人表扬你?招人接手都难!
flankerfc 楼主 初学 2026-1-12 14:30:38
@ktyang 哈哈,厉害。确定是「严格的技术经理」吗?我就没有遇到任何评价好的代码库
flankerfc 楼主 初学 2026-1-12 14:31:18
@Astrian 什么模型?用的是「严格的技术经理」吗?我这里就没有好话
返回顶部