Javascript：确保参数产生有效的HTML元素（类型检查）

Question

我一直在阅读一些旧代码，其中许多使用了我在网上抓取的代码片段。 我遇到了几个这样的借用函数，它们需要一个元素作为参数，但是可以将该元素作为元素本身或在.getElementById()使用的字符串来传递。 以下两个中哪一个更好，为什么？ 还是有更好的版本？ （更好=可读，标准，安全）

版本1

function doSomethingWithElem( elem /*, param2, ... */ ) {
    elem = (typeof elem === 'string' || elem instanceof String)
         ?  document.getElementById( elem )
         :  elem;

    // more stuff...
}

版本2

function doSomething2( elem /*, param2, ... */ ) {
    elem = document.getElementById( elem ) || elem;

    // more stuff...
}

主要区别在于，一个在调用gEBI之前检查字符串的实例，而另一个依赖gEBI返回undefined 。 他们都不检查确保elem确实是HTMLElement 。

假设有一个定义明确的API（用户知道他应该传递一个字符串或HTML元素），类型检查有多重要？ 是否主要需要类型检查来帮助脚本用户调试自己的代码？ 这两种方法中是否都存在细微的陷阱，例如gEBI的可能的返回值可能会使版本2混乱？

Answer 1

我会选择版本1，因为它更具可读性。 您会看到它处理2种情况，更清楚的是，其中一种是字符串，另一种是直接元素。 不过，我从未见过对instanceof String的检查，因为我在99％的人中都声明过字符串原语。 我对gEBI进行了一些测试，它看起来很健壮，对于任何无效的东西（数组，函数，对象等），它只会返回null 。

要检查elem是否实际上是HTML元素，请参见： JavaScript isDOM-如何检查JavaScript对象是否为DOM对象？