-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
代码规范+新语法糖 #4
Comments
你好,目前还是以开发功能和稳定性为优先,代码风格和规范方面没有很严格控制,我自身也非专业的C#程序员。给一些mod作者使用的时候,发现mod 作者的环境里不一定有装最新的.net sdk,为了降低使用成本,所以把一些最新的语法改回了兼容旧版的写法。欢迎提交pr。 |
一般来说就算不是最新的.net sdk应该也是.net6左右(? |
@lxymahatma 可以的 |
代码框架与与语言版本无关,升级sdk样式然后改个参数就行了 |
高语言版本得你下了新的sdk才能编译,比如c# 12的collection expression 必须有.net8在电脑上才可以编译,项目框架可以是net6 |
没具体试过,装个vs就能开发 |
合理,但是没必要,能不麻烦别人就不麻烦 |
符合代码规范和新版本虽然很好(可以自己安装类似python的pylint/codestye或者js的eslint来自己约束?然后加上precommit就行?),但是我感觉目前最迫切的需求可能是;
|
这个pr就是先解决代码规范,开发的事情我估计除了控制台以外都得c++解决,而且估计作者忙着研究怎么弄patch都没看github( |
我不会C#,想开发mod可咋办,我还得学学C#啊。 |
或者你可以用C++和uess开发mod,C#开发其实挺简单的,最重要的是游科的代码,我觉得是比较难读的,很别扭 |
C++和uess开发mod,这个我可以考虑,我会c++,uess是指这个吗?https://github.com/UE4SS-RE/RE-UE4SS |
@lxymahatma hook支持已经发布了,可以继续这个PR了,目前CSharpModExample是源码依赖CSharpModBase的,是否能在MR里改进一下。让mod作者在低版本.net sdk环境下也能直接编译CSharpModExample |
因为我的想法就是让mod作者引用编译的dll而不是这个project( 如果可以的话我就按照这个来做 |
@czastack 以及就是,我因为之前用的是MelonLoader,他们的mod dll就直接丢到mods文件夹里,然后专门新建一个UserLib文件夹用来放mod的依赖项目什么的,所以比较好奇为啥要创建一个文件夹然后把mod丢进去才能加载这件事情( 以及有很多乱七八糟的可能可以修改的小地方,但是因为是breaking change所以需要讨论 |
模板好啊, 像我这种业余C#选手 .csproj写的稀里糊涂的 |
但是作者没说打算咋样所以目前就没在弄这个( |
Q群发你邮箱了,可以加群方便讨论 |
ok 已加现在等通过 |
因为使用了
C#12
以及开启了nullable
,可以使用很多新的语法糖比如 using statements 可以不用括号围起来而是用;
,在方法结束会自动释放,前面写了类型后面new的时候不用加类型名称,collection expression (可以增加性能)等等代码规范方面,不清楚是不是因为比较赶着做功能,很多都不太统一
private field
命名规范一般为下划线开头如 _index,方便与传入的参数做区分Common
类应该命名为CommonDirs
希望可以在项目添加 .editorconfig 或者告知代码规范,如果允许的话我可以根据以上内容等做一个pr
The text was updated successfully, but these errors were encountered: