作者adrianshum (Alien)
看板OOAD
标题Re: [其他] 面试题目
时间Sat Nov 29 11:40:49 2014
老实说和 OOAD 扯不上什麽关系吧.
回到问题上，可以大为简化成:
// 後面全部是 psuedo-code
for (i = 1 to 100) {
boolean handled = false
if (isMultipleOfThree(i)) {
print "B" // 你的 constant 老实说极无意义
handled = true;
}
if (isMultipleOfFive(i)) {
printf "F"
handled = true
}
if (!handled) {
print i
}
}
然後加入一些和 OOAD 有点关系的东西, 套用类似 Strategy 的 pattern：
interface ItemHandler { // 用 generics 更佳，这里简单写演示一下概念
// return true if handled, false if not
boolean handleItem(int i);
}
class PrintMultipleOfIntItemHandler implements ItemHandler {
int base;
String output;
public MultipleOfIntItemHandler(int base, String output) {
this.base = base;
this.output = output;
}
boolean handleItem(int i) {
if (i % base == 0) {
print output;
return true
}
return false
}
}
class SimplePrintItemHandler implements ItemHandler {
boolean handleItem(int i) {
print i
return true
}
}
你的 main loop 可以变成
// 这部份其实可以用不同方法达成，比如 config, DI etc
List<ItemHandler> handlers = [new PrintMultipleOfIntItemHandler(3,"B"),
new PrintMultipleOfIntItemHandler(5,"F")];
Handler defaultHandler = new SimplePrintItemHandler();
// 不必改动的 main loop
for (int i = 1 to 100) {
boolean handled = false
foreach (handler in handlers) {
result = handler.handleItem(i);
handled = handled || result;
}
if (!handled) {
defaultHandler.handleItem(i)
}
}
如果每个条件和处理是独立的话，这做法可以让你加入不同的
条件和处理方法时候，不需改动 main loop 的 logic
为了解决额外那个 defaultHandler，可以利用 chain-of-responsibility
pattern。
再进一步，如果你想连 item 的提供方法也可以替换，就弄一个
ItemProvider 的 interface 吧:
class ItemProvider {
// get next item. return null if no more item available
int nextItem();
}
这部份的实作留给你自己练习
--
※ 发信站: 批踢踢实业坊(ptt.cc), 来自: 223.19.45.197
※ 文章网址: http://webptt.com/cn.aspx?n=bbs/OOAD/M.1417232452.A.E2B.html
※ 编辑: adrianshum (223.19.45.197), 11/29/2014 11:49:54
1F：→ GoalBased : THX 11/29 12:03
※ 编辑: adrianshum (223.19.45.197), 11/29/2014 15:29:05
2F：→ andrew43 : 推。在下多年新手，看了有感。 12/20 02:54
3F：推 GoalBased : 再推一次@@ 12/26 17:30
4F：推 liwindgod : 大推 03/05 14:20
5F：推 lin07646 : 题目好像还有一个条件.. 07/18 18:11
6F：推 lin07646 : 对不起 眼拙了 07/18 20:37