coding-style - 在编程中是否有适当的方法进行函数/方法调用

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

返回parseInt(trim(elem.value.substring(1，elem.value.length)))

我当时在想，这是否合适，或者教授/雇主会因为允许这样的代码行而抨击我。

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

但是，可以吗？还是我应该细分并注释每一行，以便将来的程序员更容易理解？
喜欢...

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;

我不确定该如何搜索，因此对是否有此内容表示歉意。

最佳答案

第一种格式很好，除了它可以使运行时异常(如NullPointerExceptions)难以调试(因为堆栈跟踪返回一个行号，并且可能在一行上包含多种内容)之外，第一种格式是可以的。因此，如果可能发生NPE，请多行写出。

而且，不必在行尾注释每一行。很明显，trim和parseToInt对除了最新手的程序员以外的任何人都起作用，而简单的Google搜索无论如何都会发现这一点。