if (self = [super init]) - ostrzeżenie LLVM! Jak sobie z tym radzisz?

Przed Xcode 4 z LLVM minął kompilator niezauważony. Przypisanie w ramach warunku jest całkowicie celowe i jest to kakao.if (self = [super init]) - ostrzeżenie LLVM! Jak sobie z tym radzisz?

Xcode 4 z wybranym kompilatorem LLVM nigdy nie zawodzi, a nie tylko w czasie kompilacji, zaraz po wpisaniu pojawia się żółta ikona ostrzegawcza. Wyłączenie ostrzeżeń jako błędów i ignorowanie ostrzeżenia nie wydaje się dobrym pomysłem. Przeniesienie przypisania z nawiasów powoduje marnowanie miejsca. Konieczność wyłączenia tego ostrzeżenia z pragmą przy każdym nowym projekcie stanie się żmudna.

Jak sobie z tym radzisz? Jaki będzie nowy idiom?

Źródło

2011-02-09 Adam Eberbach

Podczas formatowania kodu "marnowanie miejsca" niekoniecznie jest marnotrawstwem. Wcięcie linii w celu odzwierciedlenia struktury programu "marnuje przestrzeń", ale ty tego nie robisz? – JeremyP

Marnowanie miejsca w stosunku do idiomu przypisania/oceny wyłącznie dla siebie. Dużo zależy mi na tym, jak gęsty jest kod i nie mam nic przeciwko dodatkowym liniom, kiedy to sprawia, że rzeczy są wyraźniejsze. –

Uwielbiam podwójny negatywny JeremyP. –

To jest rzeczywiście bardzo stara ostrzeżenie, to było po prostu domyślnie wyłączone z GCC i Clang 1.6. Xcode powinien dać ci sugestię, jak to naprawić - mianowicie podwoić nawiasy.

if ((self = [super init])) { ... }

Dodatkowa para parens mówi kompilatorowi, że rzeczywiście zamierzałeś wykonać zadanie w trybie warunkowym.

Źródło

2011-02-09 01:10:18

A więc tak - po prostu założyłem, że wiem, na co narzeka, więc nie przeczytałem tej podpowiedzi. –

Wystarczy użyć dwóch par nawiasów, aby było jasne dla kompilatora, że jesteś nadającego na celu:

if ((self = [super init]))

Źródło

2011-02-09 01:09:25 Chuck

Jeśli utworzyć metodę init, z nowszych makr tekstowych Xcode, można zauważyć, że nowy błogosławiony sposób zrobić init:

- (id)init { 
    self = [super init]; 
    if (self) { 
     <#initializations#> 
    } 
    return self; 
}

ten sposób unika się ostrzeżenie. Chociaż osobiście w moim własnym kodzie, jeśli natknę się na to, po prostu stosowałem metodę, którą pokazał Kevin.

Coś dobrego do poznania!

Źródło

2011-02-09 03:46:18

Nie jestem pewien, czy możesz nazwać to "błogosławioną drogą".Apple zawsze był niekonsekwentny w zakresie stylów kodowania. –

Nie powiedziałem, że poprzednie style nie były kiedyś błogosławione ... W informatyce żadne błogosławieństwo nie trwa wiecznie. Więc może to bardziej "urok". –

Zarówno odpowiedzi Kevina, jak i Kendall są poprawne. Różnica między nimi polega na tym, że Kendall sprawdza, czy działa inicjalizacja superklasy. Aaron Hillegass wyjaśnia w swojej książce, że niektóre inicjatory zwrócą zero, jeśli inicjalizacja superklasy nie powiedzie się. Mam wrażenie, że taki przypadek byłby patologiczny (awaria podczas uruchamiania), dlatego nie wydaje się, aby istniały przykłady ponownego inicjowania w przypadku zerowego powrotu. Jednak jestem tu poza moją głębią i tylko spekuluję na ten temat. Chciałbym zobaczyć autorytatywne dogłębne wyjaśnienie. –

Wyświetl nawigator projektu i wybierz projekt. W wyświetlonym oknie głównym wybierz "Wszystkie". W sekcji "Kompilator LLVM 2.0 - Ostrzeżenia" wybierz "Inne flagi ostrzegawcze". Dodaj flagę "Wno-idiomatyczne-nawiasy" zarówno dla "Debugowania" i "Release". Teraz wyczyść i przekompiluj. enter image description here

Źródło

2011-03-10 19:59:20 shadowmatter

Jak kilka innych osób zasugerowało, że powinieneś dodać dodatkowy zestaw nawiasów.

jestem daleko od zwykłego guru ekspresji więc nie krępuj się to posprzątać ale to znaleźć i zamienić w Xcode stałej około 95% moich wystąpień:

Replace: if\s*\({1}\s*self\s*={1}(.*)\){1} 
With: if ((self =\1))

Bądź ostrożny, ponieważ będzie to również znaleźć, jeśli (self == ...), więc użyj podglądu i odznacz te lub napraw moje wyrażenie :)

I zacznij używać self = ...; jeśli (samo), to jest czystsze.

Źródło

2011-07-26 20:10:15

if (self = [super init]) - ostrzeżenie LLVM! Jak sobie z tym radzisz?

Odpowiedz

Powiązane problemy