Статические переменные и методы

Question

Я побежал через класс, который был создан как это:

public class MyClass { 

    private static boolean started = false; 

    private MyClass(){ 
    } 

    public static void doSomething(){ 
    if(started){ 
     return; 
    } 
    started = true; 
    //code below that is only supposed to run 
    //run if not started 
    } 
} 

Мое понимание со статическими методами является то, что вы не должны использовать переменные класса в них, если они не являются постоянными, а не изменить , Вместо этого вы должны использовать параметры. Мой вопрос: почему это не нарушается, когда вызывается несколько раз, выполняя MyClass.doSomething(). Мне кажется, что это не должно работать, но делает. Он будет проходить только после выполнения инструкции if.

Так может кто-нибудь объяснить мне, почему это не сломается?

Answer 1

Метод doSomething() и переменная started являются статическими, поэтому имеется только одна копия переменной и она доступна из doSomething(). В первый раз вызывается doSomething(), started является ложным, поэтому он устанавливает started в true, а затем делает ... ну, что-то. Второй и последующий раз, когда он называется, started - это правда, поэтому он возвращается, ничего не делая.

Answer 2

В статическом методе вы можете вызвать или получить доступ к статическим членам в одном классе.

Отказ от сценариев с несколькими потоками, Первый вызов doSomething сделает логическую статическую переменную равной true, поэтому второй вызов будет выполнять код блока if, который просто просто выйдет из метода.

Answer 3

Приведенный выше код полностью работает (если он не работает в многопоточной среде). Почему вы думаете, что это должно сломаться?

Answer 4

Мое понимание со статическими методами является то, что вы не должны использовать переменные класса в них, если они не являются постоянными и не меняются

я думаю только статические члены могут быть доступны. Это не обязательно быть постоянным!

Мой вопрос: почему это не нарушается при вызове несколько раз, выполняя MyClass.doSomething(). Мне кажется, что это не должно работать, но делает. Он будет проходить только после заявления if

В соответствии с существующей логикой. Только первый вызов запускает //code to be run часть

Answer 5

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

Что должно случиться:

1. Первый вызов сделан. Класс инициализирован, запущен false.
2. doЧто-то называется. Ошибка if и код обходит его. start установлен в true и выполняется другой код.
3. doSomething называется снова. Пропускание и выполнение останавливаются.

Следует отметить, что здесь нет синхронизации, поэтому, если doSomething() вызывается в отдельных потоках, невероятно близко друг к другу, каждый поток может считываться как false, обходить оператор if и выполнять работу, т. Е. есть состояние гонки.

Answer 6

Это не очень приятный код. Обычно проекты должны использовать экземпляры объектов, в которых состояние изменяется, но с ним нет ничего незаконного.

Мое понимание статическими методами заключается в том, что вы не должны использовать переменные класса в них, если они не являются постоянными, и не меняются.

Вы, кажется, экстраполированы из руководства по дизайну на язык. Прочтите одно из многих обучающих программ Java, доступных в Интернете, относительно того, что на самом деле разрешено на этом языке. Вы используете без статических полей без статических полей, но это приводит к процедурному, а не объектно-ориентированному коду.

Вместо этого вы должны использовать параметры.

Это трудно понять, как бы быть использован started параметр - если вызывающий абонент знал, что был запущен процесс, почему они называют метод?

Answer 7

Вы статический метод разговариваете со статической переменной класса, поэтому все должно быть хорошо. Вы могли бы подумать об этом как об общем глобальном коде и глобальной переменной, а именно в пространстве имен этого класса.

Если вы пытались получить доступ к нестатический переменной-члена:

private int foo = 0; 

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

started is false - initial state. 
MyClass.doSomething() - statered is now true 
MyClass.doSomething() - started is STILL true 

MyClass foo = new MyClass(); 
foo.started -> it's STILL true, because it's static 
foo.doSomething() - not sure you can do this in Java, but if you can, it's be STILL TRUE! 

В приведенном выше коде есть проблемы с безопасностью потоков, но, кроме того, он работает как сконструированный.

Answer 8

Код не является потокобезопасным. Самый простой способ сделать этот код потокобезопасным будет делать что-то вроде

public class MyClass { 

    private static AtomicBoolean started = new AtomicBoolean(false); 

    private MyClass(){ 
    } 

    public static void doSomething(){ 
    boolean oldValue = started.getAndSet(true); 
    if (oldValue) 
     return; 
    } 

    //code below that is only supposed to run 
    //run if not started 
    } 
} 

Это должно быть поточно как синхронизирован AtomicBoolean getAndSet.

Понятно, что это не проблема, если вы не используете потоки (обратите внимание, что webapp может использовать довольно много потоков, обрабатывающих различные запросы, не зная об этом).

Answer 9

Просто помните правило большого пальца, что «Статические переменные класса уровня переменные и все не статические переменные экземпляра переменные». Тогда у вас не будет никакой путаницы!

i.e. Для статической переменной Все ссылки, сделанные в коде, имеют переменную точку в том же месте памяти. А для нестатической переменной новое распределение памяти выполняется всякий раз, когда создается новый экземпляр этого класса (поэтому каждая ссылка, сделанная в коде на переменную, указывает на другое место памяти, выделенное для вызова экземпляра класса).