Jak znaleźć wszystkie nieużywane klasy w Intellij Idea?

126

Istnieje inspekcja „Niewykorzystana deklaracja”, dzięki której można znaleźć cały niewykorzystany kod w Intellij Idea. (zobacz to pytanie ) Ale chcę znaleźć wszystkie nieużywane klasy, a nie metody, zmienne itp. Tylko klasy. (na liście wyników 3000 trudno znaleźć tylko klasy). Jak mogę to zrobić?

java intellij-idea dead-code

— wiśnia
źródło

Po prostu uruchom inspekcję, kliknij prawym przyciskiem myszy i zobacz to na liście menu.

— Cherry

To dwa pytania - lepiej by były jako oddzielne pytania. Również serialVersionXXX jest złym pomysłem w przypadku większości projektów.

— Inżynier oprogramowania,

To zły pomysł, ponieważ niewielu programistów wie, do czego służy, i prawie zawsze jest nadużywany. W Javie niezwykle rzadko używa się mechanizmu serializacji do odczytu i zapisu obiektów na poziomie bajtów, a jeśli to zrobisz, domyślna wersja szeregowa jest zwykle wystarczająca, aby zapewnić bezpieczeństwo. Zakodowana na stałe wersja serialVersion musi być aktualizowana ręcznie za każdym razem, gdy zmienia się interfejs klasy i za każdym razem, gdy zmienia się lista pól - niezmiennie programiści nie robią tego, ponieważ po prostu nie rozumieją, do czego służy serialVersion. Głównym powodem, dla którego widzisz to w kodzie, jest zaćmienie.

— Inżynier oprogramowania

218

Naciśnij Ctrl+ Shift+ A(w Mac Command+ Shift+ A)
Wpisz „nieużywana deklaracja”
Kliknij dwukrotnie „Niewykorzystana deklaracja”

Ustawienia pojawią się

Kliknij Java / Deklaracja nadmiarowości / Nieużywana deklaracja
w prawym dolnym rogu wybierz „Ustawienia edytora w locie”
odznacz pola wyboru, ..., sprawdź parametry. Należy zaznaczyć tylko klasy kontrolne.
wciśnij OK

Ustawienia zostaną zamknięte

Na pasku menu kliknij Analizuj / Uruchom inspekcję według nazwy (lub Ctrl+ Alt+ Shift+ I- w Mac Command+ Option+ Shift+ I)
Wstaw tekst „Nieużywana deklaracja”
Wybierz „Nieużywana deklaracja Java | Nadmiarowość deklaracji”

Rozpoczyna się wyszukiwanie

Po zakończeniu sprawdź stan zadania u dołu Pomysłu: ciesz się wynikami i wspaniałym uczuciem czyszczenia pomieszanego kodu. :)

— BlondCode
źródło

Opublikowanie prawie tej samej odpowiedzi w ciągu kilku minut na kilka pytań sugeruje, że jedno z nich jest duplikatem drugiego. oznacz je jako takie.

— Jaap,

3

Jeden z nich jest przeznaczony specjalnie do użytku z IntelliJ Idea, a drugi dla ogólnej Javy. Nie powiedziałbym, że to duplikaty.

— BlondCode,

7

Ignoruje konfigurację „Only Check Classes”, więc otrzymuję mnóstwo niechcianych wyników (nieużywane metody, pola itp., A co gorsza, wiele z nich to fałszywe alarmy z kilku powodów ...).

— thelawnmowerman

6

Nastąpiła zmiana w Studio 3.0 Opcja sprawdzania tylko klas następuje po kroku: Wybierz "Nieużywana deklaracja Java |

— Nadmiar

4

Nie musimy zmieniać ustawień globalnych, aby przeprowadzić inspekcję z innymi parametrami. Akcja Uruchom inspekcję według nazwy spowoduje wyświetlenie ustawień inspekcji.

— TWiStErRob

11

Nie sądzę, żeby to było wykonalne. Podejrzewam, że ta funkcja jest celowo pomijana w IDE, ponieważ nie można jej bezpiecznie używać tak, jak inne refaktoryzacje typu „usuń nieużywane XXX”.

Nieużywane deklaracje, których szuka IDEA (i AFAIK, NetBeans), dotyczą prywatnych członków i zmiennych lokalnych: rzeczy, które nie są dostępne, nawet dynamicznie, spoza tej klasy lub zakresu. (Cóż, przynajmniej bez robienia rzeczy z Reflection lub hakowania JVM, których nie powinieneś robić.) Bez względu na to, co zewnętrzny kod robi z twoją biblioteką, nie spowoduje to użycia tych rzeczy, ponieważ ich zakres jest ograniczony, a IDE może to wszystko zobaczyć. Kompilator może to ustalić, patrząc tylko na Twój kod.

W przypadku klas, nawet jeśli nie mają publicdostępu, można do nich dynamicznie odwoływać się za pomocą Class.forName(), co faktycznie dzieje się w kodzie aktywnym. Więc nawet jeśli najwyraźniej nie są używane w kodzie projektu, mogą być używane w zależności od tego, co Ty lub kod zewnętrzny korzystający z biblioteki działa. Więc IDE nie może zagwarantować, że usunięcie tych klas nie zmieni zachowania obserwowalnego zewnętrznie.

Dlatego uważam, że IDEA po prostu nie zapewnia takiego zachowania: może dać użytkownikom fałszywe oczekiwania dotyczące bezpieczeństwa, a ich usunięcie nie jest bezpieczną refaktoryzacją.

— Andrew Janke
źródło

6

Cóż, wyszarza nazwę klasy podczas faktycznego przeglądania pliku i daje zamiar „usunięcia nieużywanej klasy”. Brakuje tylko możliwości ich automatycznego znalezienia. Z pewnością istnieją sposoby, które mogą zepsuć sytuację, ale dotyczy to wielu refaktoryzacji.

— Rob Fletcher

2

Class.forName () zapewne nie różni się, z punktu widzenia bezpieczeństwa analizy, od używania refleksji do odczytu / ustawienia pól prywatnych w klasie. Więc jeśli IntelliJ zrobi to dla pól, to nie jest akceptowalna odpowiedź, dlaczego nie zrobi tego również dla klas. W obu przypadkach wynik analizy nie gwarantuje 100% bezpieczeństwa.

— Dogs

1

Może powinieneś przyjrzeć się Unused Symbolinspekcji z następującymi ustawieniami:

wprowadź opis obrazu tutaj

— Vic
źródło

Ha! Chyba nie :-) Wyłączyłem wszystko w grupie "nieużywana deklaracja" i zostawiłem tylko "Nieużywany symbol" tylko z "sprawdź klasy". Ale wyniki się nie zmieniają. :-(

— Cherry

@Cherry, przykro mi to słyszeć ... Może powinieneś zgłosić usterkę do jetbrains

— Vic

1

Nie jestem pewien, czy to odpowie na twoje pytanie, ale w przeszłości korzystałem z narzędzia Fortify do uruchamiania reguł przeglądu kodu w projekcie, które precyzyjnie wskazuje na niewykorzystane importy, martwy kod, nieużywane klasy itp. Jest to płatne oprogramowanie, ale ja Jestem pewien, że będą dostępne darmowe wtyczki / oprogramowanie do tego samego.

— słoneczny
źródło