我最近遇到了MorseString
类的问题。我有两个不同的构造函数可以执行不同的操作,但采用相同的数据类型:
/*
* Constructor that takes the Morse Code as a String as a parameter
*/
public MorseString(String s) {
if(!isValidMorse(s)) {
throw new IllegalArgumentException("s is not a valid Morse Code");
}
// ...
}
和
/*
* Constructor that takes the String as a parameter and converts it to Morse Code
*/
public MorseString(String s) {
// ...
}
我提出了这个解决方案:
public MorseString(String s, ParameterType type) {
if(type == ParameterType.CODE) {
if(!isValidMorse(s)) {
throw new IllegalArgumentException("s is not a valid Morse Code");
}
// Constructor that takes Morse
} else {
// Constructor that takes String
}
}
但它看起来很难看。还有其他解决方案吗?
答案 0 :(得分:9)
是。摆脱构造函数并使用其他方法,例如
1)一个Factory Method,所以你有这样的方法:
class MorseString {
private MorseString(){};
public static MorseString getFromCode(String s) {
// ...
}
public static MorseString getFromString(String s) {
// ...
}
}
// Usage: MorseString.getFromCode(foo);
或
2)一个Builder,所以你有这样的方法
class MorseString {
private MorseString(){};
static class Builder {
Builder initFromCode(String code) {
// ..
return this;
}
Builder initFromString(String str) {
// ..
return this;
}
MorseString build() {
// ..
}
}
}
// Usage: MorseString.Builder.initFromCode(foo).build();
如果你有一个非常复杂的创建逻辑,很多参数,在创建过程中只有一些信息的对象,一些初步验证等等,那么构建器就很好。对于你拥有的情况,工厂方法更轻松使用稍微变化的参数创建对象的多种方法。
答案 1 :(得分:4)
由于其中一个构造函数正在期待现成的摩尔斯电码数据(因此它更像是"构造函数" - 从字面上构建数据中的对象),另一个必须进行一些转换,制作一个名为convert
的静态工厂方法可能更有意义:
/*
* Constructor that takes the Morse Code as a String as a parameter
*/
public MorseString(String s) {
if(!isValidMorse(s)) {
throw new IllegalArgumentException("s is not a valid Morse Code");
}
// ...
}
/*
* Factory method that takes the String as a parameter and converts it to Morse Code
*/
public static MorseString convert(String s) {
// ...
return new MorseString(convertedString);
}
因此,如果您有一个有效的莫尔斯代码字符串,则使用构造函数将其转换为对象。但是,如果您有需要转换的数据,则可以调用静态工厂方法:
MorseString ms = MorseString.convert(myString);