Почему я не должен использовать Objective C 2,0 средства доступа в init/dealloc? [закрытый]

Что такое NullPointerException?

Хорошим местом для начала является JavaDocs . Они охватывают это:

Брошено, когда приложение пытается использовать null в случае, когда требуется объект. К ним относятся:

  • Вызов метода экземпляра нулевого объекта.
  • Доступ или изменение поля нулевого объекта.
  • Выполнение длины null, как если бы это был массив.
  • Доступ или изменение слотов с нулевым значением, как если бы это был массив.
  • Бросать нуль, как если бы это было значение Throwable.

Приложения должны бросать экземпляры этого класса для указания других незаконных видов использования нулевого объекта.

blockquote>

Также, если вы попытаетесь использовать нулевую ссылку с synchronized, который также выдаст это исключение, за JLS :

SynchronizedStatement:
    synchronized ( Expression ) Block
  • В противном случае, если значение выражения равно null, NullPointerException.
blockquote>

Как это исправить?

Итак, у вас есть NullPointerException. Как вы это исправите? Возьмем простой пример, который выдает NullPointerException:

public class Printer {
    private String name;

    public void setName(String name) {
        this.name = name;
    }

    public void print() {
        printString(name);
    }

    private void printString(String s) {
        System.out.println(s + " (" + s.length() + ")");
    }

    public static void main(String[] args) {
        Printer printer = new Printer();
        printer.print();
    }
}

Идентифицирует нулевые значения

. Первый шаг - точно определить , значения которого вызывают исключение . Для этого нам нужно выполнить некоторую отладку. Важно научиться читать stacktrace . Это покажет вам, где было выбрано исключение:

Exception in thread "main" java.lang.NullPointerException
    at Printer.printString(Printer.java:13)
    at Printer.print(Printer.java:9)
    at Printer.main(Printer.java:19)

Здесь мы видим, что исключение выбрано в строке 13 (в методе printString). Посмотрите на строку и проверьте, какие значения равны нулю, добавив протоколирующие операторы или используя отладчик . Мы обнаруживаем, что s имеет значение null, а вызов метода length на него вызывает исключение. Мы видим, что программа перестает бросать исключение, когда s.length() удаляется из метода.

Трассировка, где эти значения взяты из

Затем проверьте, откуда это значение. Следуя вызовам метода, мы видим, что s передается с printString(name) в методе print(), а this.name - null.

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

Где установлен this.name? В методе setName(String). С некоторой дополнительной отладкой мы видим, что этот метод вообще не вызывается. Если этот метод был вызван, обязательно проверьте порядок , что эти методы вызывают, а метод set не будет называться после методом печати. ​​

Этого достаточно, чтобы дать нам решение: добавить вызов printer.setName() перед вызовом printer.print().

Другие исправления

Переменная может иметь значение по умолчанию setName может помешать ему установить значение null):

private String name = "";

Либо метод print, либо printString может проверить значение null например:

printString((name == null) ? "" : name);

Или вы можете создать класс, чтобы name всегда имел ненулевое значение :

public class Printer {
    private final String name;

    public Printer(String name) {
        this.name = Objects.requireNonNull(name);
    }

    public void print() {
        printString(name);
    }

    private void printString(String s) {
        System.out.println(s + " (" + s.length() + ")");
    }

    public static void main(String[] args) {
        Printer printer = new Printer("123");
        printer.print();
    }
}

См. также:

Я все еще не могу найти проблему

Если вы попытались отладить проблему и до сих пор не имеете решения, вы можете отправить вопрос для получения дополнительной справки, но не забудьте включить то, что вы пробовали до сих пор. Как минимум, включите stacktrace в вопрос и отметьте важные номера строк в коде. Также попробуйте сначала упростить код (см. SSCCE ).

41
задан Community 23 May 2017 в 12:01
поделиться

4 ответа

Это - все об использовании идиоматически последовательного кода. При копировании всего кода соответственно существуют ряды правил, которые гарантируют, что использование средства доступа в init/dealloc безопасно.

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

- (NSMutableDictionary *) myMutableDict {
    if (!myMutableDict) {
        myMutableDict = [[NSMutableDictionary alloc] init];
    }

    return myMutableDict;
}

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

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

19
ответ дан Louis Gerbarg 27 November 2019 в 00:43
поделиться

Это - в основном инструкция для уменьшения потенциала для ошибок.

В этом случае существует (возможность), что Ваш метод set/метод get может непреднамеренно сделать прямые или косвенные предположения о состоянии объекта. Эти предположения могли быть проблемой, когда объект посреди того, чтобы быть установкой или уничтожен.

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

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

@implementation Example

-(void) setFoo:(Foo*)foo
{
   _foo = foo;
  [_observer onPropertyChange:self object:foo];
}

-(void) dealloc
{
   ...
   self.foo = nil;
}

@end
29
ответ дан Andrew Grant 27 November 2019 в 00:43
поделиться

Вы ответили на свой собственный вопрос:

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

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

не ясно, тем не менее, почему это обращено конкретно в Objective C 2 средства доступа? Те же принципы применяются, используете ли Вы объявленные свойства или средства доступа записи сами.

15
ответ дан mmalc 27 November 2019 в 00:43
поделиться

Возможно, у установщика есть логика, которая должна работать, или, возможно, реализация использовала ivar с именем, отличным от получателя / установщика, или, возможно, два ivar, которые необходимо освободить и / или иметь свои значение установлено равным нулю. Единственный верный способ - вызвать сеттера. Установщик обязан быть написан таким образом, чтобы нежелательные побочные эффекты не возникали при вызове во время инициализации или освобождения.

Из «Какао-паттернов проектирования», Бак, Яктман, стр 115: «... нет практическая альтернатива использованию средств доступа, когда вы используете синтезированные переменные экземпляра в современной среде выполнения Objective-C или ... "

2
ответ дан 27 November 2019 в 00:43
поделиться