首页
学习
活动
专区
圈层
工具
发布
首页
学习
活动
专区
圈层
工具
MCP广场
社区首页 >问答首页 >迷你js工具库

迷你js工具库
EN

Code Review用户
提问于 2022-07-04 11:15:16
回答 1查看 49关注 0票数 2

我是一个新手js“开发人员”,为了实践,我创建了一个很小的(8种方法) JavaScript库。我需要你给我反馈“我是应该继续扩展我的库还是切换到其他东西”,以及如何提高代码质量?

完整代码:

代码语言:javascript
运行
复制
function isObject(val) {
      if (val === null) { return false;}
      return ( (typeof val === 'function') || (typeof val === 'object') );
  }
  
function isDate(value) {
      switch (typeof value) {
          case 'number':
              return true;
          case 'string':
              return !isNaN(Date.parse(value));
          case 'object':
              if (value instanceof Date) {
                  return !isNaN(value.getTime());
              }
          default:
              return false;
      }
  }
  
function isEmptyOrOnlySpacesString(str) {
      return (!str || /^\s*$/.test(str));
  }
  
function getTimeZone() {
      return Intl.DateTimeFormat().resolvedOptions().timeZone;
  }
  
function doesObjectHaveEmptyProperties(obj) {
    let emptyKeys = [];
          for(let key in obj) {
          if(obj[key] === "") {
             emptyKeys.push(key + " is empty.\n");
          }
            
      }
    return emptyKeys.join('') + ' others keys are filled';
  }

function isSorted (arr){
  return arr.every((a,b,c) => !b || c[b-1] <= a);
}

function shuffleArray (arr){
  let len = arr.length;
  while (len) {
    const i = Math.floor(Math.random() * len--);
    [arr[len], arr[i]] = [arr[i], arr[len]];
  }
  return arr;
}

function generateRandomIntArr(len, upTo){
  return Array.from({ length: len }, () => Math.floor(Math.random() * upTo));
}

你也可以访问我的github回购

提前感谢

EN

回答 1

Code Review用户

回答已采纳

发布于 2022-07-04 12:54:48

一次简短的回顾;

  • 您可能希望签出isObject库,实现有isObject(值){ var type = type of value,返回值!= null &( == 'object‘而型==’函式‘);}
    • 它确实完整地说明了value,这是更好的可读性
    • 它不重复类型的值,但是将其存储在临时值中(虽然这里的const是这样的)
    • 它有一个出口语句,看起来更干净。

  • isDate为37返回true,感觉是错误的。我也会试着解析数字。此外,函数名有一点,它不检查值是否是日期,而是检查它是否可以转换为日期。
  • 我更喜欢list而不是arr,它的整体读起来更好
  • arr.every((a,b,c),->,那些变量名不好。every并不那么常见。如果你坚持用一个字母变量来表示v,i,l/a (值,索引,列表/数组)
  • 我希望shuffleArray会创建一个新的洗牌列表,而不是洗牌。
  • let emptyKeys可能是const
票数 0
EN
页面原文内容由Code Review提供。腾讯云小微IT领域专用引擎提供翻译支持
原文链接:

https://codereview.stackexchange.com/questions/277872

复制
相关文章

相似问题

领券
问题归档专栏文章快讯文章归档关键词归档开发者手册归档开发者手册 Section 归档