Skip to content

Conversation

ragnaroks
Copy link

因为开发环境无NF48框架,可执行文件框架临时改为NF472了,可合并后改回去

此次提交为代理输入框增加了对hostname:port形式的解析支持,代码格式可能有较大差异,可在你的环境进行格式化

@HMBSbige HMBSbige added the Enhancement New feature or request label Jul 30, 2021
@HMBSbige
Copy link
Owner

HMBSbige commented Jul 30, 2021

代码质量太差了
IPAddress 和域名居然还用正则...
不过 .NET6.0 出来后本来就要重写

@HMBSbige HMBSbige closed this Jul 30, 2021
@ragnaroks
Copy link
Author

代码质量太差了
IPAddress 和域名居然还用正则...
不过 .NET6.0 出来后本来就要重写

代码质量这块有什么指导?

我以前也是用特征来分辨v4和v6,但是后来发现编译正则远快于字符串操作

@HMBSbige
Copy link
Owner

HMBSbige commented Jul 31, 2021

代码质量太差了
IPAddress 和域名居然还用正则...
不过 .NET6.0 出来后本来就要重写

代码质量这块有什么指导?

我以前也是用特征来分辨v4和v6,但是后来发现编译正则远快于字符串操作

能不用正则就不用正则,速度慢维护性又差
用 TryParse 这种 API 不是很好吗?
那么多 Try Catch 就不说了吧

@ragnaroks
Copy link
Author

原来如此,是我献丑了

HMBSbige added a commit that referenced this pull request Aug 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants