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

Globals中FieldType 不支持[]int32 #27

Closed
yingnierxiao opened this issue Feb 7, 2018 · 5 comments
Closed

Globals中FieldType 不支持[]int32 #27

yingnierxiao opened this issue Feb 7, 2018 · 5 comments

Comments

@yingnierxiao
Copy link

yingnierxiao commented Feb 7, 2018

physicalAttack:3-5|mgAttack:3-5 比如字段需要 这种只能拆分成physicalAttackMin, physicalAttackMax 没法直观的用数组表示起来

@Awenllen
Copy link

Awenllen commented Feb 7, 2018

配成两个字段不是更合理吗

@davyxu
Copy link
Owner

davyxu commented Feb 7, 2018

弃用原来的复杂手动解析吧,字段可以单独处理并检查,岂不是更好?
你那种混合信息,连linq都是不兼容的,非常糟糕的设计

@yingnierxiao
Copy link
Author

yingnierxiao commented Feb 7, 2018

嗯 这个需求只是策划觉得合并看起来比较直观,程序角度看肯定是拆分不会有歧义。不过内嵌支持[]int32还是有必要的,比表示区间之类的 还是有这种特殊需求

@davyxu
Copy link
Owner

davyxu commented Feb 7, 2018

我们的策划碰到比你复杂的需求, 我直接用json了,还不是乖乖的用. 你要让他体验到这个工具的强大的地方. 而不是把他需求做个人定制. 你这里一个需求我可以改,一百个人来我都得做需求, 你们之间的需求冲突怎么取舍? 所以这是一个很复杂的问题, 程序强势一点就好, 把坏习惯都摒弃掉吧

@yingnierxiao
Copy link
Author

其实只是如何平衡。无理的要求也不会提。互相配合才能提高大家效率。不存在一定要谁屈服谁。抛开策划的需求。内嵌支持[]int32还是有正常的需求的。

@davyxu davyxu closed this as completed Apr 22, 2018
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

3 participants