java - 构造函数调用必须是使用 this 的构造函数中的第一个语句

Question

这个问题已经有答案了:

Why do this() and super() have to be the first statement in a constructor? (22 个回答)

已关闭 5 年前。

为什么我不能使用将输入引用到第一个构造函数的 switch case 语句来创建第二个构造函数？它显示错误“构造函数调用必须是使用 this 的构造函数中的第一个语句”。所以看来我必须为第二个构造函数中的每个 case 语句重新输入第一个构造函数的赋值。

public class Card {
        public static final String CLUBS = "Clubs";
        public static final String DIAMONDS = "Diamonds";
        public static final String HEARTS = "Hearts";
        public static final String SPADES = "Spades";
        public static final int ACE = 1;
        public static final int JACK = 11;
        public static final int QUEEN = 12;
        public static final int KING = 13;

        public Card(int rank, String suit) {
            this.rank = rank;
            this.suit = suit;
        }

        public Card(String rank, String suit) {
            if (!isCorrectSuit(suit)) throw new IllegalArgumentException("incorrect suit");
            switch(rank) {
            case ACE: this(1, suit);
            case JACK: this(11, suit);
            case QUEEN: this(12, suit);
            case KING : this(13, suit);
            default: throw new IllegalArgumentException("incorrect rank");
            }
        }



        private boolean isCorrectSuit(String suit) {
            return (suit.equals(CLUBS) || suit.equals(DIAMONDS) || suit.equals(HEARTS) || suit.equals(SPADES));
        }

        private boolean isCorrectRank(int rank) {
            return rank == 1 || rank == 11 || rank == 12 || rank == 13;
        }

        private int rank;
        private String suit;    

    }

Answer 1

您可能正在寻找一个静态工厂方法，其中 this(...) 可以替换为 new Card(...):

class Card {
    ...

    private Card(int rank, String suit) {
        this.rank = rank;
        this.suit = suit;
    }

    public static Card of(String rank, String suit) {
        if (!isCorrectSuit(suit)) {
            throw new IllegalArgumentException("incorrect suit");
        }

        final Card card;
        switch (Integer.valueOf(rank)) {
            case ACE:
                card = new Card(1, suit);
                break;
            case JACK:
                card = new Card(11, suit);
                break;
            case QUEEN:
                card = new Card(12, suit);
                break;
            case KING:
                card = new Card(13, suit);
                break;
            default: {
                throw new IllegalArgumentException("incorrect rank");
            }
        }
        return card;
    }
}

这四个实例可以预定义，无需重新创建。

<小时/>

我想知道为什么您没有对 rank 值应用相同的方法:

public Card(int rank, String suit) {
    if (!isCorrectSuit(suit)) {
        throw new IllegalArgumentException("incorrect suit");
    }

    if (!isCorrectRank(rank)) {
        throw new IllegalArgumentException("incorrect rank");
    }

    this.rank = rank;
    this.suit = suit;
}

public Card(String rank, String suit) {
    this(Integer.valueOf(rank), suit);
}

请注意，您有 2 个公共(public)构造函数，其中只有一个具有某种验证功能。我可以创建一张新卡(2, "Unknown")并且不会出现任何异常。

其他可供考虑的选项可能是:

• 编写枚举而不是原始值
• 编写值集以用 Set#contains 替换 isCorrectX 方法