Questions tagged «bad-code»

我有以下代码： public void moveCameraTo(Location location){ moveCameraTo(location.getLatitude(), location.getLongitude()); } public void moveCameraTo(double latitude, double longitude){ LatLng latLng = new LatLng(latitude, longitude); moveCameraTo(latLng); } public void moveCameraTo(LatLng latLng){ GoogleMap googleMap = getGoogleMap(); cameraUpdate = CameraUpdateFactory.newLatLngZoom(latLng, INITIAL_MAP_ZOOM_LEVEL); googleMap.moveCamera(cameraUpdate); } 我认为通过这种方式，例如，我消除了了解LatLng另一门课中的内容的责任。 而且，您无需在调用函数之前准备数据。 你怎么看？ 这种方法有名称吗？真的是不好的做法吗？

10 design  design-patterns  clean-code  code-smell  bad-code 

我一直被教导，在某种if情况下产生副作用是不好的。我的意思是; if (conditionThenHandle()) { // do effectively nothing } ...而不是 if (condition()) { handle(); } ...我知道，我的同事很高兴，因为我不这样做，我们每个星期五的17:00回家，每个人都有一个快乐的周末。 现在，ECMAScript5推出类似的方法every()，并some()到Array了，我发现它们非常有用。他们是清洁比for (;;;)的，再给你一个范围，并且使元素由变量访问。 但是，在验证输入时，我经常发现自己在条件中使用every/ some来验证输入，然后再次在正文中使用every/ 将输入转换为可用模型。some if (input.every(function (that) { return typeof that === "number"; })) { input.every(function (that) { // Model.findById(that); etc } } else { return; } ...当我想做的是 if (!input.every(function (that) { var …

9 javascript  bad-code  side-effect