Skip to content

我的模型类实现#6

Open
shirok1 wants to merge 10 commits into
POPCORNBOOM:mainfrom
shirok1:main
Open

我的模型类实现#6
shirok1 wants to merge 10 commits into
POPCORNBOOM:mainfrom
shirok1:main

Conversation

@shirok1

@shirok1 shirok1 commented Jan 28, 2022

Copy link
Copy Markdown
Contributor

呃呃,看起来我晚了一天

我的分支实现了以下 #5 未实现的功能:

  • 模型类和模板文件的互相转换
  • 将绘制框的功能放到了模型类里
  • 模型类和 DataGridView 的自动绑定,包括颜色,不需要使用 RefreshTable 这样的方法

此外,你的程序中还有以下可能会导致问题的做法:

  • catch(Exception ex) 然后手动收集错误信息。在有多种潜在异常的时候应当自行实现异常类,然后分别 catch。
  • 多次书写重复的 DataGridView 代码。应当建立模型类( changes: Using Object Cardbox to replace data read in datagrid; new i… #5 并没有完全解决这个问题,依旧残留了大量重复的手动指定 Column的代码)。
  • 大量的字符串字面值。应该放到 resx 里。
  • 大量的屏蔽功能通过注释实现。应当直接删掉,需要的时候从版本管理库中调取。

PS 去好好学学面向对象编程吧

@shirok1

shirok1 commented Jan 28, 2022

Copy link
Copy Markdown
Contributor Author

另外冲突已经多到没法合并了,照着描述里自己改吧……

@POPCORNBOOM

Copy link
Copy Markdown
Owner

非常感谢你的PR以及其中详细的问题指出,我会参考你的分支并做出更改!

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

Successfully merging this pull request may close these issues.

2 participants