Проверка правильности переменной перед вызовом суперструктора

Question

Итак, я пишу код, который включает в себя расширение класса, который я ранее написал, в котором файлы создаются именовались с использованием конструктора, который принимает имя и размер типа длинный. В этом исходном классе я проверил внутри конструктора, что введенное имя файла содержит один «.». но не требует определенного расширения в файле. Для этого нового класса, который я пишу, мне требуется расширение имени «.mp3». Однако мой компилятор не любит проверку перед супер-конструктором.

Это мой текущий код:

public class Song extends DigitalMedia{ 

private String artist; 
private String album; 
private String name; 
private long size; 

public Song(String aName, long aSize, String aArtist, String aAlbum){ 
    super(aName, aSize); 
    setArtist(aArtist); 
    setAlbum(aAlbum); 
} 

Есть ли способ, чтобы проверить, что «aName» содержит».mp3" , прежде чем создать этот конструктор?

Answer 1

Я не могу сказать, является ли это лучший способ разработать программу, но вы можете вызвать метод валидатора внутри одного из super аргументов:

public Song(String aName, long aSize, String aArtist, String aAlbum){ 
    super(validateName(aName), aSize); 
    setArtist(aArtist); 
    setAlbum(aAlbum); 
} 

private static String validateName(String name) { 
    if (whatever) { 
     throw new Whatever(); 
    } 
    return name; 
} 

Answer 2

Альтернативным решением было бы применять свои правила с помощью встроенная проверка типов.

Вы можете создать MediaFormat:

interface MediaFormat { } 

MusicFormat, который реализует MediaFormat, что позволяет определить, какие музыкальные форматы:

enum MusicFormat implements MediaFormat { 
    MP3("mp3"); 

    private final String format; 

    MusicFormat(String format) { 
     this.format = format; 
    } 

    @Override 
    public String toString() { 
     return format; 
    } 
} 

DigitalMedia затем может быть составлен из MediaFormat:

class DigitalMedia { 
    private final MediaFormat format; 
    private final String name; 

    public DigitalMedia(String name, MediaFormat format) { 
     this.name = name; 
     this.format = format; 
    } 
} 

Song может согласиться с MusicFormat:

class Song { 
    public Song(String name, MusicFormat format) { 
     super(name, format); 
    } 
} 

Это заставит пользователь использовать все, что указано в MusicFormat, избегая все эти неприятные проверки. Затем вы можете открыть метод String, который возвращает name + "." + format

Answer 3

С точки зрения наследования подкласс не должен быть более ограничительным, чем суперкласс.

Но если вы хотите ограничить создание субкласса, вы можете сделать конструктор private и предоставить фабричный метод, который сначала выполняет валидацию.

public class Song extends DigitalMedia { 

    private String artist; 
    private String album; 
    private String name; 
    private long size; 

    private Song(String aName, long aSize, String aArtist, String aAlbum) { 
     super(aName, aSize); 
     setArtist(aArtist); 
     setAlbum(aAlbum); 
    } 

    public static Song makeSong(String aName, long aSize, String aArtist, String aAlbum) { 
     //... validation code 
     return new Song(aName, aSize, aArtist, aAlbum); 
    } 
    ... 
} 

Вместо ограничения самого типа вы используете инкапсуляцию для принудительного применения инварианта.

Answer 4

Выполнение кода, достигающего constructor, подразумевает, что объект находится в прямом эфире и теперь готов к инициализации его states (поля).

Объект класса A.java также может быть назван объектом класса Super A.java. Перед тем как класс A.java инициализирует состояние, его смысл в том, что объекты наследуют свойства и функции суперкласса. После того, как суперкласс выполнит инициализацию, класс A.java получает возможность выполнить инициализацию.

Конструктор суперкласса неявно вызывается, если нет конструктора параметров, присутствующего в суперклассе else, который вам нужно вызвать явно для любого из конструкторов конструктора суперкласса.

Что вы хотите сделать, если состояние не работает в constructor? У вас есть возможность выбросить исключение, но все же объект создан, вы можете проверить его, переопределив метод finalize() и проверив объект this. Вы можете захотеть повлиять на сборщик мусора, вызвав System.gc() для выполнения кода, достигнув метода finalize() раньше.

Предлагаемое решение Перед вызовом конструктора необходимо проверить параметры конструктора. Если вы хотите инкапсулировать его в свой класс, то вы можете выбрать добавочный статический метод, не являющийся приватным (вы можете назвать его как getInstance()), создавая и возвращая объект класса Song. В таком случае вы можете иметь свой конструктор как приватный. Обратите внимание, что это сделает ваш класс нерасширяемым, это просто выбор дизайна.