В чем ошибка? (non-static method cannot be referenced...)

[djdance]

Не понимаю. Вот сервис1 и вот класс1. Пытаюсь в классе1 взять getApplicationContext.
Во-первых, его там нет (как быть?). Ладно, делаю паблик запрос через сервис1.
И компилер ругается

java: non-static method getContext() cannot be referenced from a static context
что я делаю не так?


вот код обоих:

Код: Выделить всё

public class сервис1 extends Service {
...
    public Context getContext() {
        return getApplicationContext();
    }

--------------------------------
public class класс1 {
...
    void работа(){
        SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(сервис1.getContext());

а если сделать getContext() статиком, то ругается

java: non-static method getApplicationContext() cannot be referenced from a static context

[Finch]

Типичная ошибка начинающего =) Подсказыть не стану ты сам должен понять =)

[djdance]

а зачэм тогда форум? я не смог, не просто же так пришёл

public функцию из класса -> в активити вызывал.
а тут не получается
это какие-то ограничения на направления? о_0

опять же делаю иногда скажем переменную паблик - а снаружи её не видно, или видно, но нулл.
че за такое. Паблик не паблик? Как перекидывать переменные - через преференции неудобно.

[rezak90]

он имеет в виду (и я с ним согласен) в том что вам нужно начать с основ java если задаетесь такими вопросами (особенно "делаю иногда скажем переменную паблик - а снаружи её не видно, или видно, но нулл.").

[Finch]

а зачэм тогда форум? я не смог, не просто же так пришёл

public функцию из класса -> в активити вызывал.
а тут не получается
это какие-то ограничения на направления? о_0

опять же делаю иногда скажем переменную паблик - а снаружи её не видно, или видно, но нулл.
че за такое. Паблик не паблик? Как перекидывать переменные - через преференции неудобно.

Обращаться к полям на прямую - правило дурного тона. Существует инкапсуляция.

У тебя есть класс, в нём есть поля, обращаться к ним нужно через геттеры и сеттеры

Код: Выделить всё

public class User {
    private String mName = "Вася Пупкин";
    private String[] mNames;
    
    //Тут не будет NullPointerException
    public String getName() {
        return mName;
    }

    public void setName(String name) {
        mName = name;
    }

    //А тут будет NullPointerException, знаешь почему? Уверен что у тебя такие же ошибки, когда твои паблик переменные возвращают null
    public String[] getNames() {
        return mNames;
    }
}

П.С. а форум затем чтоб не учить основам языка, а помогать решить какие-то задачи или подводные камни
Тебе нужно выучить основы Java и тогда твои ошибки пропадут сами собой =)

[neoksi]

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

[Finch]

И больше того, я тебе дам ссыль на русскоязычную статью с хорошим объяснением + и - разных реализаций паттерна singleton =)
П.С. разве это не чудо иметь такой форум в своей обойме? =)

[djdance]

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

спасибо neoksi, я примерно понимаю про один экземпляр класса и причину появления null. Примерно вижу, как это заворачивают, да. Мне надо понять, всегда ли так работать, или классы все-таки могут шарить переменные по-простому.

спасибо Finch, я знаю про инкапсуляцию, и даже сделал ее (в нулевом посте она же?) - getContext().
И моё непонимание как раз в том, что компилер ругается на нон-паблик независимо от оформления функции.

Смотрите, поменял в сервисе1 ответ функции на нулл:
public Context getContext() {
return null;//getApplicationContext();
}

При этом в классе1 вызов остался такой:
SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(сервис1.getContext());

Компилер все равно говорит:
java: non-static method getContext() cannot be referenced from a static context

То есть я бы понял, если бы сервис1 вызывал класс1 и пытался из класса1 взять паблик.
Но у меня же наоборот. Используемый класс1 пытается забрать данные из сервиса1 (родительского).
Что же мне, и родителя обёртывать в инстанс? Непонятно.

[Finch]

Лады ленивец, держи намёк на понимание

Код: Выделить всё

private static Context mContext;
	
	public сервис1() {
		mContext = getApplicationContext();
	}
	
	public static Context getContext(){
		return mContext;
	}

ток скажи можно ли так делать?)

[djdance]

Finch,

да, я тоже к этому пришел. Или делать сеттер в классе1. Работает, спасибо. Не знаю, почему нельзя так делать.

Но - зачем нужно определение ПЕРЕМЕННОЙ областью видимости "паблик", если она не видна и вообще "дурной тон"?

Вот в статьях по синглетону есть пример оборачивания на дельфи. Смешно, но там такой метод излишен - если переменная типа паблик, она ВСЕГДА видна. Да, тоже в общем дурной тон её дергать напрямую. Но ситуация более однозначна, чем в джаве. Зачем же тут нужен "паблик" для переменных?

[djdance]

вот эта статья мне помогла сейчас.

http://ru-code-android.livejournal.com/4594.html

[KamiSempai]

djdance, Вы не понимаете разницы между объектом и классом объекта. Задумайтесь над этим.

[djdance]

KamiSempai

класс - абстракция, объект - созданная из неё работающая структура.
это не даёт мне ответа на вопрос, ЗАЧЕМ нужен паблик для переменных, если их все равно нельзя брать.

[KamiSempai]

класс - абстракция, объект - созданная из неё работающая структура.
это не даёт мне ответа на вопрос, ЗАЧЕМ нужен паблик для переменных, если их все равно нельзя брать.

Это делает код менее безопасным. Объект теряет контроль над своими переменными, в любой части кода ни могут быть изменены, даже за пределами этого объекта. Создание гетеров и сетеров помогает обработать ситуации которые могут привести к исключению. Например, если взять пример приведенный выше, где должно произойти обращение к нулл. Добавим в метод проверку на существование объекта и создание его в случает отсутствия. Все, теперь у нас ни кода не будет возвращен null. И не надо делать подобную проверку в других местах. В итоге меньше кода, меньше ошибок, больше здоровых нервов.

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

Запомните, глобальные переменные - это темная сторона силы, она облегчит вам путь к достижению цели, но она же вас и погубит. Настоящие джедаи ни кода не пользуются глобальными переменными.

[neoksi]

...
Запомните, глобальные переменные - это темная сторона силы, она облегчит вам путь к достижению цели, но она же вас и погубит. Настоящие джедаи ни кода не пользуются глобальными переменными.

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

[rezak90]

поддерживаю на счёт не использования глобальных, в андроиде на такой случай есть bundle который в свою очередь можно сделать singleton что в принципе часто и делается (это если не избежать прямой передачи из бандла в бандл, например: передать данные из фрагмента активти А в фрагмент активити Б).

[Finch]

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

[Mikhail_dev]

Нередко использую глобальные переменные. getInstance() генерирует что-либо, парсит к примеру конфиг, дает значения переменным и после можно спокойно работать. Это так называемый ребут приложения, когда поправили настройки запуска. Другое дело, что они должны быть приватными

[djdance]

стоп-стоп-стоп, товарищи, вопрос я ставлю - извините, в третий раз - именно так:

зачем нужны ПАБЛИК переменные? зачем писать publict static mVar, если мы этим пабликом никогда не пользуемся, а работаем через геттеры и сеттеры.

[rezak90]

потому что в первую очередь запись "publict static mVar" чревато утечкой памяти