我应该在方法本身中调用 set length 方法吗？

Question

我的作业提示状态：

使用以下实例变量创建一个 MP3 类：艺术家、歌曲、专辑、曲目长度（以秒为单位）。提供一个具有四个参数、get/set 方法和一个 toString 的构造函数。方法。 toString 方法应该返回一个带有清晰标记的 MP3 数据的字符串。轨道长度应在 toString 方法中转换为分钟和秒。例如，265 秒将显示为 4:25。注意 MP3 不能有零或负的轨道长度。对无效的曲目长度使用一分钟的默认值。

我对这些说明的最后一句话感到困惑。谁能给我解释一下？

到目前为止我有这个代码：

public class MP3 {

    private String artist;
    private String song;
    private String album;
    private int trackLength;

    public MP3(String artistName, String songName, String albumName, int lengthOfTrack) {
        setArtist(artistName);
        setSong(songName);
        setAlbum(albumName);
        setLength(lengthOfTrack);
    }

    public void setArtist(String artistName) {
        artist = artistName;
    }

    public String getArtist() {
        return artist;
    }

    public void setSong(String songName) {
        song = songName;
    }

    public String getSong() {
        return song;
    }

    public void setAlbum(String albumName) {
        album = albumName;
    }

    public String getAlbum() {
        return album;
    }

    public void setLength(int lengthOfTrack) {
        trackLength = lengthOfTrack;

         if(getLength() <= 0) {
                setLength(60);
            }
    }

    public int getLength() {
        return trackLength;  
    }

    public String toString() {

        return String.format("%s, %s, %s, %d : %d",
            getArtist(), getSong(), getAlbum(),
            getLength() / 60, getLength() - (getLength() / 60) * 60); 
    } 
}

Answer 1

不，在正常情况下会导致递归直到发生StackOverflowException，而不是在您的情况下，因为在下一次调用时您将退出递归。一般来说，如果你不需要做，你应该避免使用递归。

您可以在设置之前简单地修改该值，例如：

public void setLength(int trackLength) {
  if (trackLength <= 0)
    trackLength = 60;

  this.trackLength = trackLength;
} 

Answer 2

这是我的看法：

• 您确实正确地实现了要求（如，它有效）
• 不过，您的实现很复杂，并且使用了不好的做法：在不需要时使用递归、从构造函数调用可覆盖的方法、将字段更改为无效值，然后立即更改
• 忽略在构造函数/设置器中传递的值并将其替换为默认值也是不好的做法：应该抛出异常。但这是你老师要求的，所以你无能为力。

这是我将如何实现它：

public class MP3 {

    private String artist;
    private String song;
    private String album;
    private int trackLength;

    public MP3(String artist, String song, String album, int trackLength) {
        this.artist = artist;
        this.song = song;
        this.album = album;
        this.trackLength = trackLengthOrDefault(trackLength);
    }

    public String getArtist() {
        return artist;
    }

    public void setArtist(String artist) {
        this.artist = artist;
    }

    public String getSong() {
        return song;
    }

    public void setSong(String song) {
        this.song = song;
    }

    public String getAlbum() {
        return album;
    }

    public void setAlbum(String album) {
        this.album = album;
    }

    public int getTrackLength() {
        return trackLength;
    }

    public void setTrackLength(int trackLength) {
        this.trackLength = trackLengthOrDefault(trackLength);
    }

    private int trackLengthOrDefault(int length) {
        return length <= 0 ? 60 : length;
    }

    public String toString() {
        return String.format("%s, %s, %s, %d:%d",
            artist, song, album, trackLength / 60, trackLength % 60);
    }
}

Answer 3

“对无效的曲目长度使用一分钟的默认值”

如果你没有被告知音轨长度，它应该设置为 60。所以

private int trackLength = 60;

那么你所有的setter都需要检查传入的值是否为正。与其他答案没有太大区别，真的