0

我有一个命令解析器,每个命令都具有相似的结构(解析、验证、执行、确认/错误),但它们都做不同的事情(打开、关闭、addnote、changestatus)。

我被要求更改错误消息,并且必须分别对所有命令进行相同的更改,因为它们都有不同的参数并从数据库中的不同表中提取。但它们几乎都是一样的。有没有更简单的方法来做到这一点?(而不是开关盒和硬编码所有命令)

我知道还有更多的变化,我宁愿不复制/粘贴,每次为每个命令更改 2 个单词,特别是因为我知道他们想要将命令数量增加一倍......

validate($commands){
 foreach($commands as $command){
  switch($command['name']){
    case 'close':
      validateClose($command);
      break;
    case 'addnote:
      validateAddnote($command);
      break;
  ...
   }
  }
 }

execute($commands){
 foreach($commands as $command){
  switch($command['name']){
    case 'close':
      executeClose($command);
      break;
    case 'addnote:
      executeAddnote($command);
      break;
  ...
   }
  }
 }

validateClose($command){
  // parse here for parameters, 3
  // validate parameters
  $command['valid'] = $validated;
  if(!$validated)
     $message = "Invalid because...";
}

validateAddnote($command){
  // parse here for parameters, 1
  // validate parameters
  $command['valid'] = $validated;
  if(!$validated)
     $message = "Invalid because...";
}

...

executeClose($command){
   $db->status('close', params);
   $message = "closed x with {params}";
}

executeAddnote($command){
   $db->add('note', param);
   $message = "Added {param} as note to x";
}

 ...
4

1 回答 1

0

我决定使用抽象并为每个命令创建一个抽象类和一个子类的命令类。

我放弃了开关并使用变量类名来创建每个命令的实例。

这些更改确实清理了所有内容,使其更具可读性,并使其更加灵活,并且可以轻松地更改单个命令或一次更改所有命令而无需重复。

于 2013-09-12T23:41:30.867 回答