解析时减少代码重复
Reducing code duplication when parsing
我希望我的代码库中的重复代码尽可能少,因此我一直在寻找减少重复代码的方法。
然而,我对以下情况有点困惑:
假设我有一种方法可以将一个程序的输出解析为一个对象,使其在另一个程序中可读。
我目前的做法是使用正则表达式扫描输入,形成一个新的输出对象。
这会创建一长串 if 语句,它们看起来或多或少相同,只是各处略有不同。有没有一种有意义的方法来减少这里的代码重复,或者这是我必须忍受的?
if ((match = block.match(/bssid=([A-Fa-f0-9:]{17})/))) {
parsed.bssid = match[1].toLowerCase();
}
if ((match = block.match(/freq=([0-9]+)/))) {
parsed.frequency = parseInt(match[1], 10);
}
if ((match = block.match(/mode=([^\s]+)/))) {
parsed.mode = match[1];
}
if ((match = block.match(/key_mgmt=([^\s]+)/))) {
parsed.key_mgmt = match[1].toLowerCase();
}
我猜你想要这样的东西:
var parseConfig = {
bssid: {
expr: /bssid=([A-Fa-f0-9:]{17})/,
handle: match => match[1].toLowerCase()
},
frequency: {
expr: /freq=([0-9]+)/,
handle: match => parseInt(match[1], 10)
},
mode: {
expr: /mode=([^\s]+)/,
handle: match => match[1]
},
key_mgmt: {
expr: /key_mgmt=([^\s]+)/,
handle: match => match[1].toLowerCase()
}
};
function parse(block, cfg) {
var parsed = {};
Object.keys(cfg).forEach(key => {
var item = cfg[key],
match = block.match(item.expr);
parsed[key] = match ? item.handle(match) : null;
});
return parsed;
}
我希望我的代码库中的重复代码尽可能少,因此我一直在寻找减少重复代码的方法。
然而,我对以下情况有点困惑: 假设我有一种方法可以将一个程序的输出解析为一个对象,使其在另一个程序中可读。
我目前的做法是使用正则表达式扫描输入,形成一个新的输出对象。
这会创建一长串 if 语句,它们看起来或多或少相同,只是各处略有不同。有没有一种有意义的方法来减少这里的代码重复,或者这是我必须忍受的?
if ((match = block.match(/bssid=([A-Fa-f0-9:]{17})/))) {
parsed.bssid = match[1].toLowerCase();
}
if ((match = block.match(/freq=([0-9]+)/))) {
parsed.frequency = parseInt(match[1], 10);
}
if ((match = block.match(/mode=([^\s]+)/))) {
parsed.mode = match[1];
}
if ((match = block.match(/key_mgmt=([^\s]+)/))) {
parsed.key_mgmt = match[1].toLowerCase();
}
我猜你想要这样的东西:
var parseConfig = {
bssid: {
expr: /bssid=([A-Fa-f0-9:]{17})/,
handle: match => match[1].toLowerCase()
},
frequency: {
expr: /freq=([0-9]+)/,
handle: match => parseInt(match[1], 10)
},
mode: {
expr: /mode=([^\s]+)/,
handle: match => match[1]
},
key_mgmt: {
expr: /key_mgmt=([^\s]+)/,
handle: match => match[1].toLowerCase()
}
};
function parse(block, cfg) {
var parsed = {};
Object.keys(cfg).forEach(key => {
var item = cfg[key],
match = block.match(item.expr);
parsed[key] = match ? item.handle(match) : null;
});
return parsed;
}