Xcode 8 / Swift 3: „Wyrażenie typu UIViewController? jest nieużywany ”

Mam następującą funkcję, która skompilowała się wcześniej, ale generuje ostrzeżenie za pomocą Xcode 8.

func exitViewController()
{
    navigationController?.popViewController(animated: true)
}

„Wyrażenie typu„ UIViewController? ”Nie jest używane”.

Dlaczego to mówi i czy można to usunąć?

Kod wykonuje się zgodnie z oczekiwaniami.

TL; DR

popViewController(animated:)zwraca UIViewController?, a kompilator wyświetla to ostrzeżenie, ponieważ nie przechwytujesz wartości. Rozwiązaniem jest przypisanie go do podkreślenia:

_ = navigationController?.popViewController(animated: true)

Swift 3 Change

Przed wersją Swift 3 wszystkie metody miały domyślnie „wynik do odrzucenia”. Ostrzeżenie nie pojawi się, jeśli nie uchwycisz, co zwróciła metoda.

Aby powiedzieć kompilatorowi, że wynik powinien zostać przechwycony, trzeba było dodać @warn_unused_resultprzed deklaracją metody. Byłby używany do metod, które mają zmienną postać (np. sortI sortInPlace). Dodałbyś, @warn_unused_result(mutable_variant="mutableMethodHere")żeby poinformować o tym kompilator.

Jednak w Swift 3 zachowanie jest odwrócone. Wszystkie metody ostrzegają teraz, że zwracana wartość nie jest przechwytywana. Jeśli chcesz powiedzieć kompilatorowi, że ostrzeżenie nie jest konieczne, dodajesz @discardableResultprzed deklaracją metody.

Jeśli nie chcesz używać zwracanej wartości, musisz jawnie powiedzieć kompilatorowi, przypisując ją do znaku podkreślenia:

_ = someMethodThatReturnsSomething()

Motywacja do dodania tego do Swift 3:

• Zapobieganie możliwym błędom (np. Używanie sortmyślenia modyfikuje kolekcję)
• Wyraźny zamiar nie przechwytywania lub konieczności przechwytywania wyniku dla innych współpracowników

Interfejs API UIKit wydaje się być opóźniony, nie dodając @discardableResultdo całkowicie normalnego (jeśli nie bardziej powszechnego) użycia popViewController(animated:)bez przechwytywania wartości zwracanej.

Czytaj więcej

• SE-0047 Propozycja szybkiej ewolucji
• Zaakceptowana propozycja z poprawkami

Kiedy życie daje ci cytryny, zrób przedłużenie:

import UIKit

extension UINavigationController {
    func pop(animated: Bool) {
        _ = self.popViewController(animated: animated)
    }

    func popToRoot(animated: Bool) {
        _ = self.popToRootViewController(animated: animated)
    }
}

Pamiętaj, że dodanie czegoś takiego @discardableResult func pop(animated: Bool) -> UIViewController?spowoduje ostrzeżenie, którego próbujesz uniknąć.

Dzięki rozszerzeniu możesz teraz pisać:

func exitViewController()
{
    navigationController?.pop(animated: true)
}

func popToTheRootOfNav() {
    navigationController?.popToRoot(animated: true)
}

Edycja: Dodano także popToRoot.

W Swift 3 ignorowanie wartości zwracanej przez funkcję, która ma zadeklarowaną wartość zwrotną, powoduje wyświetlenie ostrzeżenia.

Jednym ze sposobów rezygnacji z tego jest oznaczenie funkcji @discardableResultatrybutem. Ponieważ nie masz kontroli nad tą funkcją, to nie zadziała.

Inną metodą pozbycia się ostrzeżenia jest przypisanie wartości _. To informuje kompilator, że wiesz, że metoda zwraca wartość, ale nie chcesz zachować jej w pamięci.

let _ = navigationController?.popViewController(animated: true)

Chociaż to, work correctly if kept as it isalenumber of warning increases.

Rozwiązaniem jest po prostu, replace it with underscore ( _ )choć wydaje się brzydkie.

Eg.  _ = navigationController?.popViewController(animated: true)

W tej sytuacji użyj discardableResult .

Zgodnie z <Swift Programming Language>, rozdział Language Reference - Atrybuty.

discardableResult

Zastosuj ten atrybut do deklaracji funkcji lub metody, aby ukryć ostrzeżenie kompilatora, gdy funkcja lub metoda zwracająca wartość zostanie wywołana bez użycia jej wyniku.

https://developer.apple.com/library/content/documentation/Swift/Conceptual/Swift_Programming_Language/Attributes.html#//apple_ref/doc/uid/TP40014097-CH35-ID347

Demo znajduje się również w <Swift Programming Language>, rozdział Language Guide - Methods.

@discardableResult
    mutating func advance(to level: Int) -> Bool {
    ...
return true
}

Ponieważ niekoniecznie jest to błąd w kodzie, który wywołuje metodę forward (to :) w celu zignorowania wartości zwracanej, funkcja ta jest oznaczona atrybutem @discardableResult. Aby uzyskać więcej informacji o tym atrybucie, zobacz Atrybuty.

https://developer.apple.com/library/content/documentation/Swift/Conceptual/Swift_Programming_Language/Methods.html#//apple_ref/doc/uid/TP40014097-CH15-ID234

Jeśli chcesz iść drogą rozszerzeń takich jak odpowiedź CodeReaper, powinieneś użyć @descardableResult. Zachowuje to wszystkie możliwości, ale wycisza ostrzeżenie.

import UIKit

extension UINavigationController {
    @discardableResult func pop(animated: Bool) -> UIViewController? {
        return self.popViewController(animated: animated)
    }

    @discardableResult func popToRoot(animated: Bool) -> [UIViewController]? {
        return self.popToRootViewController(animated: animated)
    }
}

Innym sposobem jest rozpakowanie self.navigationController?wartości i wywołanie popViewControllerfunkcji.

    if let navigationController = navigationController {
        navigationController.popViewController(animated: true)
    }