避免重复的null检查功能中的参数调用函数

Question

上下文：
getProfilesFromResponse是websocketMessageHandler使用的实用程序函数从WebSocket消息中解析配置文件。

什么是更好，干净的设计？

1. getProfilesFromresponse重复NULL检查吗？
2. 仅通过一次零检查一次可见的性能增益吗？这是一个WebSocket消息回调处理程序，因此由于沉重的WebSocket流，它将被称为50倍。
3. getProfilesFromResponse使用响应？.data wenders> wendment.data吗？

function getProfilesFromResponse(response) {
  // should we add null or undefined checks here as well?  

  if ('profiles' in response.data) { // should I instead do response?.data
    // do remaining data manipulations and return the final data
  }
}

function websocketMessageHandler(response) {
  if (!response) {
    // null checks done here
  }
  ...
  ...
  ...
  const profiles = getProfilesFromResponse(response);
  ...
}

我的意见

1. 不，我们可以为调用getProfilesFromResponse的所有功能制定规则，以自己进行零检查。缺点是我们假设事情永远不会出错，但是我的反论点是，我们确实希望事情崩溃，以便人们实际上确保他们不会用null参数调用功能。
2. 不是真的，这是O（1）操作。对性能的疏忽影响。
3. 更喜欢响应？.data作为其更强大的功能。但是，由于我们已经知道响应不是空的，因此我们可以跳过它。

Answer 1

1-不，您应该避免调用getProfilesFromResponse如果响应是无效的，例如，通过停止websocketMessockegeageHandler execution> excution> execution：

  function websocketMessageHandler(response) {

    if(!response) 
       return; //< avoid further operations
          
    const profiles = getProfilesFromResponse(response);

    // more code here
  }

或bes 2-

  function websocketMessageHandler(response) {

    if(response) {
      const profiles = getProfilesFromResponse(response);
      // more code here
    } else {
      // do something in case response is null or undefined
    }
  }

有：2-始终是避免无用操作的好处。

3-如果正确避免使用null参数调用 getProfiles ，则不必再次检查，您还避免了一个无用的函数调用...