code-smell

    2热度

    3回答

    有很长的变量/函数名(corresponding_OneToMany_task_spec)的一种代码味道吗?建议的简化方法是什么?

    2热度

    1回答

    我有一个控制用户看到网页上的哪个类控制。有几个州。第一组视图由用户对象控制,该对象控制下面的项目。 - 登录 - 未登录 - 喜欢页 - 页面不招人喜欢 然后,还有另外一个标识,如果竞选绑上面的页面是否有效 对象(活动) - 系列活动 - 此役过后 我想知道如果我应该有2类,控制状态 - 一个用于用户和一个广告活动。然后,根据每个类设置的两种状态,根据if和else推导正确的视图。 如果网页喜欢和

    0热度

    1回答

    这个班是否根据“中间人”代码气味计数? 我有一个“人”类。我还有一个班级人员列表,其中有一个Person_list列表,并且具有添加,删除和用Person类别做其他事情的方法。 我从另一个类访问Person_list类。 这是否计数在中间人或任何其他代码的气味? class P{ } public class PL { Vector<P> p; String name;

    0热度

    3回答

    我的代码中至少有10次不同的代码。对我来说似乎有点臭。 public void DisplayTransitInfo(TransitInfo transitInfo) { if (InvokeRequired) EndInvoke(BeginInvoke(new MethodInvoker(() => DisplayTransitInfo(transitInf

    6热度

    5回答

    在我脑海中的某个地方,一个小小的声音告诉我“下面的C#代码有异味”。 private const string STR_ConnectionString = "ConnectionString"; private readonly string upperCaseConnectionString = STR_ConnectionString.ToUpperInvariant(); // a l

    6热度

    1回答

    你经常会看到这样一些代码多余else分支: public void bla() { if (conditionTrue) { // long code here } else { // otherwise do nothing return; } // some more code } else分支显然是下滑向

    1热度

    2回答

    我正在编写一个WCF webapi应用程序,并且需要检查一个电子邮件地址是否被采用。在尝试PUT之前,这需要客户端代码可以执行的查询。 所以,我想要做的是使用HEAD与HTTP状态代码结合使用。我有点不确定如何去做,因为这是一个简单的是/否回答,这是必需的。所以,我用HttpResponseExceptions来返回相关的状态码。 [WebInvoke(Method = "HEAD", UriTe

    0热度

    3回答

    假设你有这样的代码: public Base { abstract void Register(); } public Registrator1: Base { override void Register() { //uses the current state of the object to populate the UI captions

    1热度

    4回答

    在对this other SO post中的答案发表评论时,发现使用Thread.Sleep()方法是一种代码异味。 我很难相信没有使用这种方法并不表示你做错了什么。我可以想象一下我的头顶上有一对夫妇: 以编程方式关闭其他应用程序;除非你想杀死应用程序(有时甚至是这样),你需要等待几秒钟才能正常终止。 .NET 2.0代码:是的,它仍然存在。 Thread.Yield()直到.NET 4才被发明,

    1热度

    1回答

    我正在使用基于Django 1.3的用户注册系统,并提供了一些不寻常的验证要求。我正在使用django注册的当前分支作为我的出发点。 当他注册时,我必须向用户发送电子邮件(对吧?)由django注册处理。 用户必须通过单击具有由django注册处理的散列的链接来激活。 到目前为止,琐碎。 现在,当用户激活他的帐户时,我需要将他的电子邮件域与已知域列表进行比较,并根据这些域向他授予组成员身份。 用户