告别冗余的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()); // ... 添加其他解析器 ...
最后,修改parselrccontent方法:
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实现类并添加到列表中,无需修改原有代码,显著提高了代码的可维护性和扩展性。 代码更加清晰,易于理解和维护。
© 版权声明
文章版权归作者所有,未经允许请勿转载。
THE END