Как при использовании абстрактного класса перезаписать статический метод?

Question

[MishaXXL]

Как реализовать абстрактный класс для синглтона?

Создал абстрактный класс, но при наследовании от него вылетает ошибка, что нельзя перезаписать статический метод
А при смене abstractна static, вылетает ошибка с требованием указать результат

missing return statement public static Sql getInstance() {}

Что в данном случае указать, что нам вернет getInstance или как обойти ошибку со статическим методом?

Или как правильно это делается?
Как правильно указать, что public abstract AbstractSql getInstance(); возвращает класс, который мы использовали с методом getInstance?

public abstract class AbstractSql {
    public abstract AbstractSql getInstance();
    public abstract void setConnection();
    public abstract ArrayList<String> exec(String query);
}

public class Sql extends AbstractSql {
    private static Sql INSTANCE;

    private Sql() {}

    public static Sql getInstance() {
        if(INSTANCE == null) {
            INSTANCE = new Sql();
        }
        return INSTANCE;
    }

    public static void setConnection() { ... }

    public ArrayList<String> exec(String query) { ... }
}

Comments

[сергей кузьмин]

думаю это нызавается this

[MishaXXL]

сергей кузьмин,

non-static variable this cannot be referenced from a static context return this;

[сергей кузьмин]

MishaXXL, а точно - фактори нельзя абстрагировать
не делаейте вбстрактных синглтонов
читаем гугл
Declaring abstract method static
If you declare a method in a class abstract to use it, you must override this method in the subclass. But, overriding is not possible with static methods. Therefore, an abstract method cannot be static.

[Dmitry Roo]

Похоже, вам пора почитать про:
1. SOLID
2. Паттерн "фабрика" (Фабричный метод)
3. Spring

[MishaXXL]

Dmitry Roo, секундочку
Подключение к БД у нас пишется через синглтон, верно?
У нас есть метод обработки запросов handle
Что бы в нем использовать инъекцию зависимости, нам нужен абстрактный класс или интерфейс для ссылки на него по проверке передаваемого аргумента, так же?
Для передачи нам в handle аргумента, который будет соответствовать нашему абстрактному классу

В итоге мы получаем
public void handle(AbstractSql sql) { ... }

Ведь если мы будем использовать, то мы привяжемся к определенному классу, в данном случае Sql
public void handle(Sql sql) { ... }

[mayton2019]

Откуда вообще пошла такая вседозволенность?

И зачем перезаписать статический метод? Какая бизнес задача при этом будет решена?

И как это соотвествует духу ООП? Я даже не говорю о законах ООП. Я просто пытаюсь
понять где в этом месте появляется такая идея.

Answer 1

[Владислав Савостиков]

Во-первых - статические метод нельзя перезаписать. Он принадлежит классу и выгружается в память до запуска программы.
Во-вторых - зачем для Singleton создавать какой-то абстрактный родитель? Советую почитать о вариантах создания "правильного" синглтон здесь https://habr.com/ru/articles/129494/
В твоем случае я бы убрал getInstance из абстрактного класса. То есть - абстрактный класс задает контракт, а конкретная имплиментация Sql - является синглтоном. А какой-нибудь NoSql будет не синглтоном.
Но вообще лучше пересмотреть такую структуру

Answer 2

[Dmitry Roo]

Вам надо вместо абстрактного класс использовать интерфейс, но getInstance() вам в этом интерфейсе не нужен.
В идеале, методы жизненого цикла бина (экземпляра объекта) следует вынести в отдельный класс (класс не должен знать как себя инстанцировать - это не его ответственность).