告别冗余if-else:LRC歌词分析代码优化策略
针对LRC歌词分析代码中大量if-else语句造成的维护困难问题,本文提出了基于策略模式的优化方案。原代码使用if-else语句逐行判断歌词文件的内容,并分析到changduan对象中,扩展性差,维护成本高。
原代码片段如下:
private static void parselrccontent(changduan changduan, string line, map<string pattern=""> patternmap) throws parselrcexception { if (matcher(line, patternmap, lrcconstants.lrcregenum.title.name())) { changduan.setname(getchangduaninfocontent(line, lrcconstants.lrcregenum.title.getstartindex())); } else if (matcher(line, patternmap, lrcconstants.lrcregenum.origin_title.name())) { changduan.setoriginname(getchangduaninfocontent(line, lrcconstants.lrcregenum.origin_title.getstartindex())); } else if (matcher(line, patternmap, lrcconstants.lrcregenum.jumu.name())) { // ... 省略其他代码 ... } else { throw new parselrclineexception(line); } }
优化方案采用战略模式,将每个if-else分支抽象成独立的分析器。
首先,定义界面IMatchParser:
interface IMatchParser { boolean match(String line, Map<String, Pattern> patternMap); void handle(changduan changduan, String line, Map<String, Pattern> patternMap); }
然后,为每个分析规则创建一个IMatchParser接口类别,例如:
public class TitleParser implements IMatchParser { @Override public boolean match(String line, Map<String, Pattern> patternMap) { return matcher(line, patternMap, lrcconstants.lrcregenum.title.name()); } @Override public void handle(changduan changduan, String line, Map<String, Pattern> patternMap) { changduan.setname(getchangduaninfocontent(line, lrcconstants.lrcregenum.title.getstartindex())); } }
在列表中添加所有分析器:
List<IMatchParser> parserList = new ArrayList<>(); parserList.add(new TitleParser()); parserList.add(new OriginTitleParser()); // ... 添加其它分析器 ...
最后,修改parselrcontent方法:
private static void parseLrcContent(ChangDuan changDuan, String line, Map<String, Pattern> patternMap) throws ParseLrcException { for (IMatchParser parser : parserList) { if (parser.match(line, patternMap)) { parser.handle(changDuan, line, patternMap); break; } } }
通过战略模式,新的分析规则只需创建新的IMatchParser实现类并添加到列表中,无需修改原始代码,显著提高了代码的可维护性和可扩展性。 代码更加清晰,易于理解和维护。
以上是如何优化LRC歌词分析代码中冗余的if-else语句?详情请关注图灵教育的其他相关文章!
