Skip to content
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

代码问题 #2

Open
Heart-beatsss opened this issue Nov 19, 2022 · 4 comments
Open

代码问题 #2

Heart-beatsss opened this issue Nov 19, 2022 · 4 comments

Comments

@Heart-beatsss
Copy link

文章也许是好文章,但代码是真的烂代码
各种函数乱写乱放,各种定义都没有,可读性较弱

@LemonDrinkTea
Copy link
Collaborator

非常感谢您的建议,我已经对代码进行了初步的整理和修改。如果有哪些具体的地方写的不清楚,还请批评指正。

@Heart-beatsss
Copy link
Author

非常感谢您的建议,我已经对代码进行了初步的整理和修改。如果有哪些具体的地方写的不清楚,还请批评指正。

好的,恭喜文章被录用,那我对代码方面浅显地提几个意见:
1、注释:需要对关键的函数以及变量进行注释,要不然有些变量和函数很难看出来用途是什么;
2、代码规范:函数里面尽量不要去嵌套函数,可读性会下降;删减无用注释掉的代码块;删减无用的函数
3、这里我对代码有个疑问:在aeoe/ungram.py中,函数max_match_no_random和max_match的区别是什么?我看函数代码逻辑是一样的,两个函数都用了random。
以上就是我的意见和疑问,希望能够得到解答,多谢。

@LemonDrinkTea
Copy link
Collaborator

感谢您对我们论文的关注,以及上述提到的三点建议,代码中确实存在很多上述存在的问题,我们将在以后的版本进行更改,对于您上述提到的第三点,因为之前对于这两个函数是有不同的操作,但是最后取消了,所以导致这两个函数体相同,我们也会尽快进行修改的。

@Heart-beatsss
Copy link
Author

感谢您对我们论文的关注,以及上述提到的三点建议,代码中确实存在很多上述存在的问题,我们将在以后的版本进行更改,对于您上述提到的第三点,因为之前对于这两个函数是有不同的操作,但是最后取消了,所以导致这两个函数体相同,我们也会尽快进行修改的。

好的,期待你的后续版本,祝顺利

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants