Question

请C#/.net高手来看看这个关于百度知道团队的简单算法题，控制台的！代码全部再图片里！ 能找出漏洞最好！





Answer 1

你是要让我们纠错还是解决问题？我也加入了这个团，问这问题是为了考验大家？...太低级趣味了吧,第一，基本常识问题，接口的成员是方法体，你给他写的是字段，你要使用的不能是接口，当变量太多我们就构造他，这么几个变量直接写在class里面就好了；第二IEnumerable在哪呢，我怎么没看到？第三，我从来没见识过DateTime(2014，12，1)还可以这么用，100%报错；至于什么用户啊团的总财富值啊那都跟你上面代码没关系，是写在你分配财富方法里的

追问

这个问题其实是广告，只有关注c#的回答者才会看到这题，而且水平不低的才能看懂，所以我现在主要想招人进团队来，你没看到我那财富值奖励算法是很鼓舞人心的吗？！

这个问题其实是广告，只有关注c#的回答者才会看到这题，而且水平不低的才能看懂，所以我现在主要想招人进团队来，你没看到我那财富值奖励的计算公式是很有吸引力的吗？！

追答

天，我已经泥足深陷了......

追问

何处此言啊。。。好歹我们团队现在已经位居C#第一，谁不服来比。

追答

开玩笑的，你要拉人，发动团里面没人帮忙拉几个就好了，我们看重实践能力，对财富无所谓。语言是互通的，无论是C#还是java或者其他的语言或者编译环境，没有绝对的王道，况且这玩意是老外技术顶尖。我们不能画地为牢、固步自封，真正的高手他未必理会这些枝毛祘节的问题，团里面如果有高手可以留个联系方式，我正在自己接项目，根据个人需要，不限VS平台，完整的项目包括WEB、智能设备、PC，有兴趣可以共享一下资源

追问

拿你可以先去团队贴吧发表一点意见试试，团队就是要多交流才有意义啊。

追答

可以 QQ472544436

Answer 2

漏洞如下：
1、没有对Money的取值做控制，比如是否负数，Money不够分配等；
2、没有对用户名是否已重复分配财富值做控制，比如恶意入团等；
3、在分配财富值前先判断当前团的财富是否足够分配，若不够分配则回滚事务等。

追问

好专业！那你愿意来做一下测试么？

追答

另外，你的接口写错了，接口不允许有字段，应该改为属性。

追问

这个太对了，我的错！以前很少定义接口内非方法成员。

Answer 3

呵呵，干劲儿挺足啊，精神支持一下。

追问

你没回答问题啊。

Answer 4

我觉得你应该把要达到的效果先用文字描述一下，然后再对比下代码是否达到效果，最后再考虑特殊情况

追问

说的极是，但是要的就是会看代码的人，即便是没文档。

Answer 5

AddMember没有考虑到user为null的情况

追问

太对了！还有其他bug。。。