将过程代码合并到关联数组对象

Consolidating procedural code to an associative array object

本文关键字:关联 数组 对象 合并 过程 代码      更新时间:2023-09-26

我有5个函数检查用户输入的正则表达式…这些正则表达式已经被抽象出来,并按照之前的帖子的建议放在这里。

现在我想抽象出的功能,并把代码到一个关联数组每前一篇文章以及..然而,他们似乎是一些争论,这是否是最佳实践?这样可以吗?

将这些函数放入函数的关联数组中是好的做法吗?之前的帖子没有这样做,我猜他们决定反对。

但是基本上如何将相似的方法正确地组合在一起。

EDIT 1: // consolidated regular expressions
var patterns = {   
  name: /^[a-zA-Z-'s]{1,20}$/, 
  email: /^[a-zA-Z0-9._-]+@[a-zA-Z0-9.-]+'.[a-zA-Z]{1,4}$/,
  pass: /.{6,40}/,
  url:  /^[-'w&:'/'.='?,#+]{1,}$/,
  aml:  /<(.+)_([a-z]){1}>$/
}; 
// checks full name which allows characters and dashes
function check_name(text,id,res) 
  {
  for(var d=0;d<=0;d++)
    {
    if(!patterns["name"].exec(text.value))
      {
      o2(id,res);
      return 0;
      }
    }
    return 1;
  }
// checks for valid email form
function check_email(text,id,res)
  {
  if(!patterns["email"].exec(text.value))
    {
    o2(id,res);
    return 0;
    }
  return 1;
  }
// checks for password length
function check_pass(text,id,res)
  {
  if(!patterns["pass"];.exec(text.value))
    {
    o2(id,res);
    return 0;
    }
  return 1;
  }
// checks for valid url form
function check_url(text,id,res)
  {
  if(!patterns["url"].exec(text.value))
    {
    o2(id,res);
    return 0;
    }
  return 1;
  }
// checks for aml form
function check_aml(text)
  {  
  if(a=patterns["aml"].exec(text))
    {
    if(a[2]=='p')
      {
      return 0;
      }
    else if (a[2]=='f')
      {
      return 1;
      }
    }
  else
    {
    return 2;
    }
  }

我不认为创建一个通用的地方来存储你的模式有任何问题。事实上,这似乎是一个好方法。但是,我认为没有必要让您的单个检查函数知道全局存储。相反,应该让他们简单地了解他们正在检查的模式和他们检查的来源。如果您采用这种方法,您的检查代码可以大大简化

var make_check = function (pattern) {
  return function (text, id, res) {
    if(!pattern.exec(text.value)) {
      o2(id,res);
      return 0;
    }
    return 1;
  };
};
var check_name = make_check(patterns.name);
var check_email = make_check(patterns.email);
var check_pass = make_check(patterns.pass);
var check_url = make_check(patterns.url);

这里我从检查函数中删除了全局存储的概念。由于代码的其余部分非常相似,因此可以将其抽象为一个方法,该方法将传递其寻找匹配的模式。