Coding style 在编程中是否有合适的方法进行函数/方法调用

所以，我在编码，我基本上得到了这行代码

返回parseInt（trim（elem.value.substring（1，elem.value.length）））

我在想，这是合适的，还是教授/雇主会因为我允许这样一行代码而责骂我

我觉得，虽然很整洁，但乍一看我很难理解我在做什么。b/c大多数调用都很常见（确保它是int，修剪字符串以使转换工作，并从字符串开头删除不必要的信息）

但是，这样可以吗？或者我应该把它分解并注释每一行，以便将来的程序员更容易理解？ 像

---

我不知道该如何查找，因此如果有任何问题，我深表歉意。

我不在乎教授或雇主会怎么想。除非它真的很长，否则我总是内联这些简单的语句，因为它们更容易理解和清楚

我认为当你把它分解的时候，它看起来更复杂。我讨厌无用的符号和等号。我只在我做了一些非常复杂的事情，需要别人和我以后评论的时候才打破陈词滥调

---

在您的示例中，所有语句都非常简单明了。它们都不需要对普通开发人员进行注释或解释。

第一种格式很好，但它会使运行时异常（如NullPointerException）难以调试（因为堆栈跟踪返回行号，并且在一行上可能有多个内容）。因此，如果NPE可能发生，请用多行写出

---

而且没有必要在每一行的末尾都加上注释。很明显，trim和parseToInt对除了最新的程序员以外的任何人都做了什么，而一个简单的谷歌搜索无论如何都会发现这一点。

只要你能很好地格式化你的代码就行了。e、 g.您的线路可能看起来像：

return parseInt(
         trim(
           elem.value.substring(1, elem.value.length)))

引述：

或者我应该把它分解并注释每一行，以便将来的程序员更容易理解？像

---

这并不容易。过度易变是邪恶的；如果您的值具有不同的类型，那么这将不起作用。（哦，您可能正在使用动态类型语言，但是var

returnInt

的名称会产生误导，因为在某些程序状态下，is确实不是

Int

）。请不要这样写代码。

没有对错之分。任何一种风格都有支持/反对的地方，归根结底都取决于个人偏好或编码团队的偏好。同意Marc B的观点，但要注意，如果你正在做一些不寻常、不寻常或复杂的事情，更详细的风格是合适的。对于简单、常见的用途，单线方法是清晰、简洁和完美的。

return parseInt(
         trim(
           elem.value.substring(1, elem.value.length)))

var returnInt;
returnInt = elem.value.substring(1, elem.value.length); //remove $ symbol
returnInt = trim(returnInt); //trim whitespace
returnInt = parseInt(returnInt); //convert to int
return returnInt;