0

Я недавно поступил в университет, нам преподают программирование на Java. До этого я писал код только на Python, поэтому множество обыденных вещей для меня в новинку. Недавно на практических занятиях мы получили задание реализовать битву покемонов на основе заданного jar-файла внутрь которого заглядывать нельзя, можно пользоваться только документацией к нему. Я всё сделал, подхожу сдавать. Преподаватель спрашивает: "почему у тебя все классы в одном файле?". Меня озадачил данный вопрос. На мой неопытный взгляд гораздо удобнее написать все классы в одном файле и в нём же их и использовать(я понимаю, что есть задачи, в которых гораздо удобнее разграничить классы, но это не одна из них). На такой ответ мне сказали: "хорошо, я приму работу, если расскажешь, какие могут быть ограничения у такого способа написания кода." Я нашёл причины почему нельзя создать несколько файлов с несколькими классами внутри, но конкретно против способа со всеми классами в одном файле ничего не нашёл.

Так почему нельзя написать все классы в том же файле, где я их использую?

Код прилагаю:

import ru.ifmo.se.pokemon.*;

class Castform extends Pokemon{
    public Castform(){
        super();
        setType(Type.NORMAL);
        setStats(70.0, 70.0, 70.0, 70.0, 70.0, 70.0);
        addMove(new Blizzard());
        addMove(new ScaryFace());
        addMove(new IceBeem());
        addMove(new HydroPump());
    }
    public Castform(String name, int lvl){
        super(name, lvl);
        setType(Type.NORMAL);
        setStats(70.0, 70.0, 70.0, 70.0, 70.0, 70.0);
        addMove(new Blizzard());
        addMove(new ScaryFace());
        addMove(new IceBeem());
        addMove(new HydroPump());
    }
}

class Shelmet extends Pokemon{
    public Shelmet(){
        super();
        setType(Type.BUG);
        setStats(50.0, 40.0, 85.0, 40.0, 65.0, 25.0);
        addMove(new Rest());
        addMove(new DoubleTeam());
        addMove(new RockTomb());
    }
    public Shelmet(String name, int lvl){
        super(name, lvl);
        setType(Type.BUG);
        setStats(50.0, 40.0, 85.0, 40.0, 65.0, 25.0);
        addMove(new Rest());
        addMove(new DoubleTeam());
        addMove(new RockTomb());
    }
} 

class Accelgor extends Shelmet{
    public Accelgor(){
        super();
        setStats(80.0, 70.0, 40.0, 100.0, 60.0, 145.0);
        addMove(new X_Scissor());
    }
    public Accelgor(String name, int lvl){
        super(name, lvl);
        setStats(80.0, 70.0, 40.0, 100.0, 60.0, 145.0);
        addMove(new X_Scissor());
    }
}

class Aron extends Pokemon{
    public Aron(){
        super();
        setType(Type.ROCK, Type.STEEL);
        setStats(50.0, 70.0, 100.0, 40.0, 40.0, 30.0);
        addMove(new Rest());
        addMove(new Stomp());
    }
    public Aron(String name, int lvl){
        super(name, lvl);
        setType(Type.ROCK, Type.STEEL);
        setStats(50.0, 70.0, 100.0, 40.0, 40.0, 30.0);
        addMove(new Rest());
        addMove(new Stomp());
    }
}

class Lairon extends Aron{
    public Lairon(){
        super();
        setStats(60.0, 90.0, 140.0, 50.0, 50.0, 40.0);
        addMove(new Agility());
    }
    public Lairon(String name, int lvl){
        super(name, lvl);
        setStats(60.0, 90.0, 140.0, 50.0, 50.0, 40.0);
        addMove(new Agility());
    }
}

class Aggron extends Lairon{
    public Aggron(){
        super();
        setStats(70.0, 110.0, 180.0, 60.0, 60.0, 50.0);
        addMove(new FireBlast());
    }
    public Aggron(String name, int lvl){
        super(name, lvl);
        setStats(70.0, 110.0, 180.0, 60.0, 60.0, 50.0);
        addMove(new FireBlast());
    }
}




class Blizzard extends SpecialMove{
    public Blizzard(){
        super(Type.ICE, 110, 70);
    }
    protected void applyOppEffects(Pokemon p){
        double ch = Math.random() * 10;
        if(ch >= 0 && ch <= 1){
            Effect.freeze(p);
        }
    }
    protected String describe() {
        return "использует атаку Blizzard";
    }
}

class ScaryFace extends StatusMove{
    public ScaryFace(){
        super(Type.NORMAL, 0, 100);
    }
    protected void applyOppEffects(Pokemon p){
        double ch = Math.random() * 10;
        if(ch >= 0 && ch <= 1){
            p.setMod(Stat.SPEED, -2);
        }
    }
    protected String describe() {
        return "использует атаку Scary Face";
    }
}

class IceBeem extends SpecialMove{     
    public IceBeem(){
        super(Type.ICE, 90, 100);
    }
    protected void applyOppEffects(Pokemon p){
        double ch = Math.random() * 10;
        if(ch >= 0 && ch <= 1){
            Effect.freeze(p);
        }
    }
    protected String describe() {
        return "использует атаку Ice Beem";
    }
}

class HydroPump extends SpecialMove{
    public HydroPump(){
        super(Type.WATER, 110, 80);
    }
    protected String describe() {
        return "использует атаку Hydro Pump";
    }
}

class RockTomb extends PhysicalMove{
    public RockTomb(){
        super(Type.ROCK, 60, 95);
    }
    protected void applyOppEffects(Pokemon p){
        double ch = Math.random() * 10;
        if(ch >= 0 && ch <= 1){
            p.setMod(Stat.SPEED, -1);
        }
    }
    protected String describe() {
        return "использует атаку Rock Tomb";
    }
}

class Rest extends StatusMove{
    public Rest(){
        super(Type.PSYCHIC, 0, 0);
    }
    protected void applySelfEffects(Pokemon p){
        p.setMod(Stat.HP, (int)(p.getHP() - p.getStat(Stat.HP)));
        p.setCondition((new Effect()).condition(Status.SLEEP).turns(2).attack(0));
    }
    protected String describe() {
        return "отдыхает (использует Rest)";
    }
}

class DoubleTeam extends StatusMove {
    public DoubleTeam(){
        super(Type.NORMAL, 0, 0);
    }
    protected void applySelfEffects(Pokemon p){
        p.setMod(Stat.EVASION,(int) p.getStat(Stat.EVASION) + 1);
    }
    protected String describe() {
        return "использует атаку Double Team";
    }
}

class X_Scissor extends PhysicalMove{
    public X_Scissor(){
        super(Type.BUG, 80, 100);
    }
    protected String describe(){
        return "использует атаку X-Scissor";
    }
}

class Stomp extends PhysicalMove{
    public Stomp(){
        super(Type.NORMAL, 65, 100);
    }
    protected void applyOppEffects(Pokemon p){
        double ch = Math.random() * 10;
        if(ch >= 0 && ch <= 3){
            Effect.flinch(p);
        }
    }
    protected String describe(){
        return "использует атаку Stomp";
    }
}

class Agility extends StatusMove{
    public Agility(){
        super(Type.PSYCHIC, 0, 0);
    }
    protected void applySelfEffects(Pokemon p){
        p.setMod(Stat.SPEED,(int) p.getStat(Stat.SPEED) + 2);
    }
    protected String describe(){
        return "ускоряется (использует Agility)";
    }
}

class FireBlast extends SpecialMove{
    public FireBlast(){
        super(Type.FIRE, 110, 85);
    }
    protected void applyOppEffects(Pokemon p){
        double ch = Math.random() * 10;
        if(ch >= 0 && ch <= 1){
            Effect.burn(p);
        }
    }
    protected String describe(){
        return "использует атаку Fire Blast";
    }
}




public class Poke {
    public static void main(String[] args){
        Battle b = new Battle();
        Pokemon p1 = new Shelmet("Shelmet", 1);
        Pokemon p2 = new Castform("Castform", 1);
        Pokemon p3 = new Accelgor("Accelgor", 1);
        Pokemon p4 = new Aron("Aron", 1);
        Pokemon p5 = new Lairon("Lairon", 1);
        Pokemon p6 = new Aggron("Aggorn", 1);
        b.addAlly(p1);
        b.addAlly(p2);
        b.addAlly(p3);
        b.addFoe(p4);
        b.addFoe(p5);
        b.addFoe(p6);
        b.go();
    }
}
11
  • 3
    у тебя вещи дома как хранятся? всё в одном мешке? или по полкам раздельно? 9 ноя 2021 в 9:42
  • 3
    Как минимум это читать не удобно. А так: тут нет возможности ограничить область видимости пакетом (package). Если нужно перенести этот пакет куда-л. придётся открывать файл и копировать нужные строки, которые ещё нужно найти, т.к. можно было бы сделать разграничение на 3 пакета, а нужен только один и надо вычленять конкретные классы для переноса. 9 ноя 2021 в 9:45
  • 1
    @стасевич, я понял вашу идею. Просто я видимо тупой, и не могу понять зачем заморачиваться и выстраивать идеальную цепь производства для простой задачи. Вы же не будете строить завод для того, чтобы собрать один стул? 9 ноя 2021 в 10:14
  • 1
    тут как в жизни - хочется ходить голым - на нудистский пляж. хочется писать на джаве (любом другом выбранном языке) - желательно придерживаться устоявшихся правил сообщества. Потом, когда чуточку сил и опыта появится, можно будет и свои идеи предлагать.
    – KoVadim
    9 ноя 2021 в 10:15
  • 2
    Точно затем же, зачем плодить кучу функций, методов и переменных. Технической необходимости нет (кроме некоторых требований языка), но работать проще. 9 ноя 2021 в 10:41

2 ответа 2

5

Дело в контексте. Предполагается, что один класс должен выполнять только одну задачу, причём делать он это должен очень хорошо.

Когда вы смотрите в файл на Java ваш мозг в один момент времени должен держать только маленькую часть контекста всей задачи. Реализуя покемона Aron, вам не требуется думать о Shelmet или о том, как реализуется его SpecialMove.

Кстати в Java в одном файле может быть только 1 публичный класс.

На мой взгляд, самая главная проблема появляется при добавлении контроля версий (возьмём самую популярную - Git). В случае разработки в одном файле у вас всегда будет изменён только 1 файл, коммиты станут не информативными, так как если вы поменяете логику StatusMove, то чтобы это понять, придётся прочитать весь файл, когда в раздельном виде было бы сразу понятно, что поменялась именно эта часть кода. Также будет большая проблема с маржами, если вы добавите в середине файла или поменяете местоположение класса, то скорее всего вам почти всегда придётся решать мёрж конфликты. Ну и pull request на 10к строк тоже читать очень неудобно.

4
  • 2
    Но ведь именно для этого в git при коммите обязательно написание пояснения. Ваш ответ правильный, но меня спросили про ограничения реализации со стороны работы программы, а не про сложность работы. Про публичный класс информация полезна, но я нашёл её ещё до написания вопроса. Спасибо за потраченное время 9 ноя 2021 в 10:19
  • 1
    Тут очень сложно сказать, что именно имел ввиду ваш преподаватель. Возможно у него была мысль, что при компиляции, классы не будут разбиты на отдельные файлы и получится, что при подключении вашей библиотеки, проект потянет за собой кучу ненужного кода. Но Java всё равно разобьёт эти классы на отдельные файлы и в конечный проект пойдут только нужные файлы.
    – Komdosh
    9 ноя 2021 в 10:22
  • 1
    @dark_buckwheat По поводу описания, месседжи в гит не всегда реально олицетворяют намерения разработчиков, более того разработчики не всегда умеют выражать свои мысли, но ещё есть проблема с маржами, сейчас добавлю.
    – Komdosh
    9 ноя 2021 в 10:27
  • спасибо за пояснения. Жаль, ваш комментарий нельзя пометить как ответ на мой вопрос. 9 ноя 2021 в 11:42
5

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

В Java в 1 файле может быть только 1 public класс.
В Джаве принято (на уровне этикета) правило "для каждого класса - отдельный файл".

А то, что вы описываете, реализовано в Kotlin'е. И там это правило "этикета" смягчается до "можно объединять классы с общим смыслом в один файл".

Проблема вашего кода в том, что в нем легко разберетесь только ВЫ.
Разбитие классов на файлы (и объединения этих файлов в папки / package) еще нужно и для того, чтобы другой программист смог разобраться.

Я в покемонах не силен, но у вас уже есть логические разбития:

  • виды покемонов - наследники Pokemon
  • скиллы атаки и передвежения (если я правильно понял) - наследники StatusMove
  • атаки-наследники PhysicalMove
  • ультимативные способности - наследники от SpecialMove

И объединять 4 группы таких разных классов в 1 файл не есть хорошо.

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.