🚀 go-pugleaf

On Tuesday, January 2, 2018 at 11:08:07 PM UTC+1, Wojciech Muła wrote:
> On Tuesday, January 2, 2018 at 2:17:11 PM UTC+1, Maciek wrote:
> > Cześć. Mam typ enum w którym jest kilka wartości. Następnie testuje coś za pomocą instrukcji switch / case.
> > Czy jeśli nie testuje wszystkich możliwych wartości które są w tym typie, a tylko te które w danym momencie mnie interesują, to czy jest powód do ostrzeżenia:
> > 
> > /Users/Maciek0/Projekty/books/books/window.cpp:223:11: Enumeration values 'b_n', 'ean', and 'isbn' not handled in switch
> 
> To może oznaczać, że Twój typ enum obejmuje za wiele przypadków.
> Powinieneś mieć typ, który zawiera tylko takie wartości, jakie Cię
> interesują w danym miejscu programu. Załatwianie wszystkiego jednym
> enumem i używanie defaultów to proszenie się o kłopoty.

Ja bardzo często w enum mam 'dodatkowe' pole na końcu typu. Np.:

enum Enum {
 field_0,
 field_1,
 field_2,
 .......
 filed_N,
 size_of_enum,
};

W niektórych przypadkach, po dodaniu pola field_N+1, wszystkie algorytmy
działają bez przepisywania, bo np. mam gdzieś tablicę:

QVector<...> table( size_of_enum , init_value );

Niestety w swich-case z reguły nigdy nie wykorzystuję pola size_of_enum i
kompilator krzyczy...


 

> Niestety, typy wyliczeniowe są w C++ koślawe (z mikro-poprawką na
> enum class), 
Chyba nie używałem nigdy enum class, może pora się zapoznać. 


> więc modelowanie za ich pomocą jest męczące.
Ale i tak często ułatwia rozwój aplikacji i zwiększa przejrzystość. A to 
co powyżej opisałem z size_of_enum na końcu, jeszcze bardziej pomaga.

Pozdrawiam