Jak oznaczyć nieużywany parametr funkcji constexpr?

Question

Rozważmy to klasyczny przykład:

template <typename T, std::size_t N> 
constexpr std::size_t arraySize(T (&array)[N]) noexcept { return N; } 

Teraz to działa prawidłowo, ale jest jeden kłopot, gcc daje ostrzeżenie:

warning: unused parameter ‘array’ [-Wunused-parameter] 

znane rozwiązania:

• robi” t praca: jeśli dodaję klasyczną funkcję (void)arr;, otrzymam error: body of constexpr function ‘...‘ not a return-statement.
• Niezadowalający: Mogę mieć arraySize(T (&)[N]), ale chcę, aby wymienić argument z dwóch powodów:
  1. To sprawia, że ​​komunikat o błędzie kompilatora bardziej zrozumiałe.
  2. Bardziej subiektywnie, myślę, że czyni to kod jaśniejszym, szczególnie dla tych, którzy nie żyją i oddychają tą składnią.
• Niezły: W tym konkretnym przykładzie, mogę również return sizeof(array)/sizeof(array[0]);, ale takie podejście nie jest uniwersalnym rozwiązaniem, a także myślę return N; jest o wiele ładniejszy, zdecydowanie łatwiejsze w oku.
• Dobre, ale nie zawsze możliwe: przełącz na C++ 14 i kompilator, który obsługuje go w pełni. Następnie dozwolone jest ciało funkcji constexpr, takie jak { (void)array; return N; }.

Jak mogę dobrze usunąć ostrzeżenie o nieużywanym parametrze, używając C++ 11?

Answer 1

sugeruję następujące (ab) zastosowanie operatora przecinkami:

return (void)array, N; 

Answer 2

Najlepszym rozwiązaniem jest ten, który jest zabroniony z drugim punktem, to znaczy z użyciem T(&)[N].

Innym możliwym podejściem, począwszy od komentarza, jest zastosowanie następujące wartości zwracanej:

return array ? N : N; 

jestem całkiem pewny, że kompilator będzie pozbyć się go i nie będzie problemu z wydajnością w środowisku wykonawczym .

Answer 3

Spróbuj tego. I skorzystać z tej metody czasami

template <typename T, std::size_t N> 
constexpr std::size_t arraySize(T (& /*array*/)[N]) noexcept { return N; } 

Answer 4

jak @ fnc12 i @skypjack zarówno wskazują obecnie, idiomatyczne sposób uciszyć nieużywany parametr ostrzeżenie kompilatora jest nie nadać nazwę parametru.

constexpr std::size_t arraySize(T (& /*array*/)[N]) noexcept { return N; } 

Zastosowanie komentarzy rozwiązuje sprzeciw czytelności. Nie naprawia to nazwy w komunikacie kompilatora, ale twierdzę, że najbardziej prawdopodobną sytuacją jest "niezadeklarowany identyfikator", który jest dość łatwy i oczywisty do rozwiązania, gdy zauważysz, że identyfikator jest komentowany.

Jeśli naprawdę nie jesteś ustawiony w sposób idiomatyczny, po prostu pomiń ostrzeżenie (lokalnie, jeśli pozwala na to twój kompilator).

Suppressing warnings in GCC using #pragma GCC diagnostic.

Suppressing warnings in Visual Studio using #pragma warning suppress

ja odradzam dodawanie referencji „obojętne” w kodzie, aby zamknąć kompilatora. Odwoływanie się do nieużywanego parametru bez przyczyny, niż do stłumienia ostrzeżenia kompilatora, niepotrzebnie zwiększa złożoność kodu i może zmylić przyszłych opiekunów z tego powodu.

Answer 5

Unnamed argument jest dobrym rozwiązaniem.

Więc można użyć pośrednią funkcję:

template <typename T> 
constexpr void avoid_warning_for_unused_parameter(T&&) noexcept{} 

a następnie:

template <typename T, std::size_t N> 
constexpr std::size_t arraySize(T (&array)[N]) noexcept 
{ 
    avoid_warning_for_unused_parameter(array); 
    return N; 
} 

dla C++ 11, trzeba włamać trochę więcej:

template <typename... Ts> 
constexpr auto return_first_and_avoid_warning_for_unused_parameters(T&&t, Ts&&) noexcept 
-> decltype(t) 
{ 
    return t; 
} 

template <typename T, std::size_t N> 
constexpr std::size_t arraySize(T (&array)[N]) noexcept 
{ 
    return return_first_and_avoid_warning_for_unused_parameters(N, array); 
} 

Answer 6

gcc dostarcza unused attribute, który może być używany w następujący sposób:

constexpr std::size_t arraySize(__attribute__((unused)) T (&array)[N]) noexcept { return N; } 
           ^^^^^^^^^^^^^^^^^^^^^^^ 

Ben Deane recently tweatted about a C++11 way stłumić to ostrzeżenie za pomocą lambdy które w połączeniu z operatorem przecinkami będzie wyglądać mniej więcej tak:

#define UNUSED(x) [&x]{}() 

//... 

return UNUSED(array), N; 

Answer 7

Dla nowych pracowników Google:

C++ 17 dodaje atrybut, który mógłby [[maybe_unused]] używaj tak:

template <typename T, std::size_t N> 
constexpr std::size_t arraySize(T (&array)[N] [[maybe_unused]]) noexcept { return N; }