search

xianhc / apevolo-api

.Net 8 、SqlSugar ORM、Vue 2.X、RBAC、前后端分离的开箱则用的企业级中后台权限管理系统
https://www.apevolo.com
Apache License 2.0
809 stars 74 forks source link

问题不少，还有待迭代... #17

Closed dmhai closed 2 years ago

dmhai commented 2 years ago

找轮子ing，被项目介绍吸引到了

直接下载源码，改Appsetting，vs2022直接跑，异常，_lock = null，看了下源码 : public class LogHelper { static UsingLock _lock; public LogHelper() { _lock = new UsingLock(); } ... public static void WriteSqlLog(string filename, string[] dataParas, bool isHeader = true) { ... using (_lock.Write())//出问题的代码，未初始化直接使用 { File.AppendAllText(logFilePath, logContent); } ... } ... }

调用来源：LogHelper.WriteSqlLog(...);

这代码是不是有点不对 ？没有 new LogHelper >> 没初始化 _lock >> _lock.Write() 不就出错了 ?

默认配置理论上来说不少人跑过，没发现有人发现反馈，难道只有我出错了？我打开方式不对？

这问题倒是小问题，可直接跑默认设置都能出错，总感觉有点...

然后代码风格建议注意下，虽然说风格没有标准，但是项目中很多拼接和换行看起来还是怪怪的，比如这种： if (isHeader) { logContent = "--------------------------------\r\n" + DateTime.Now + "|\r\n" + string.Join("\r\n", dataParas) + "\r\n" ; }

感觉这也太随意了点，身为深度强迫症的我表示...

还有，既然支持mssql，考虑下把兼容进一步处理下？不然切数据库还需要手动修正，也是个麻烦事儿，毕竟工程量也不小(这个纯个人建议，非bug，只是觉得如果项目主打易用性，这种小问题处理下更好).

话说回来，写文件好像不用锁? 设置共享模式，就没有文件占用问题？

xianhc commented 2 years ago

@dmhai 已修复异常，现已去除实体字段类型、长度类型。以便支持所有数据库，感谢您反馈问题

  • © Githubissues.
  • Githubissues is a development platform for aggregating issues.