code-smell

1热度

1回答

我正在写一个函数，在两个不同的服务上调用两个不同的方法。我调用了一个对service1执行操作的方法，并且在调用成功时（没有抛出异常），我调用调用service2的方法。我的代码如下： def action1(param1: String, param2: String): Option[String] = { try{ //Check if service1 contain

4热度

3回答

在switch语句（Java）中重用代码

我想使用switch语句，但是我无法在没有重复代码或使用随附的if语句的情况下构造它。有没有解决的办法？我有5个案例，除了其中一个之外，我想执行某些操作。所以用switch声明，我可以这样做： switch(x) { case A: foo(); break; case B: case C: case D: case E: bar(); brea

0热度

3回答

Python：不应该使用的神奇方法

我创建了一个类，它是一个tuple包装器，元组不支持项目突变。我应该离开__setitem__和__delitem__的实现或实现像例如下面（因此属于Refused Bequest代码的气味）？哪种方法更pythonic？在这种情况下，定制异常是否更好？ def __setitem__(self, key, value): """ :raise: Always. :

4热度

5回答

通过Java中的局部变量在全局声明静态值是否有性能优势？

鉴于这些代码示例：样品1个 public class SomeClass { private static final int onlyUsedByMethodFoo = 1; // many lines of code public static void foo() { final String value = items[onlyUsedByMet

12热度

5回答

如果我想创建多个CRUD方法重载，我该怎么办？

如果我有一个类以某种方式表示映射到我的数据库中的特定表。此类包含约30个属性。我创建了CRUD Methods。并发现自己需要另一个（UPDATE）方法，它应该只更新两个字段。我应该用一个简单的例子以良好的方式做什么？用我的存在的方法，填补了整个对象，并更新所有包括我打算两个字段的字段？（无用功）与另一名 - 创建静态方法（但我要保持我的方法名的Cuz它的表现力）!!并且需要两个参数？

2热度

4回答

堆栈的依赖关系 - 代码嗅觉？

我有一类设置像这样： class ClassToCheck(otherClass class2, nextClass class3) class otherClass(thisClass class4, otherThisClass class5, popularClass class6) class nextClass() （这仅仅是使用类的构造函数签名为例）它是一个代码味道有有许

-1热度

1回答

声呐复制不能按预期工作？

我试图表明有人在这里我怎么好找声纳工具... 后来我写了一个小的Java项目，并定义了许多故意臭方法，那些 2完全相同（复制粘贴+）DO1和DO2 surprisenly，运行后Sonnar镜头，没有重复的错误，也没有警告...... public void do1() { for (int i = 0; i < 10; i++) { if (i != 0) {

0热度

2回答

是一种只调用其他方法，并没有别的代码味道的方法吗？

说我写的包装某些第三方的API和我有这样的方法： public function fetchSomeData() { $url = $this->makeUrl($someArgs); $response = $this->call($url); $this->save($response); return $response; } 这是一个代码气

2热度

6回答

正确的方法来定义长度

所以我想知道什么是最合适的方式来定义长度。现在，我使用的是这样的 Coordinates(4.00 * Length.Millimeter, 5 * Length.Millimeter) 我想知道我是否应该提取到像寻找一种方法来做到这一点太这个 getMillimeter(double value) return value; Coordinates(getMillimeter(4.00),

0热度

1回答

MethodReturnObject：Antipattern？如何重构它？

在我加入的一个项目，最近我们有一个类，看起来像这样： class MethodReturn { public int status = C_OK public string message = "" public string str_value = "" public int int_value = 0 public double dbl_val