On 18/05/2023 20:05, Marek wrote:
>> Aby były *bezpieczne*.
> Co jest niebezpieczne w moim przykładzie?

PA1 ^=1; ?

Wybrałeś sobie prymitywny przykład, więc niebezpieczeńśtwo jest mniejsze.

Mój był inny.

Może tak:

setupUart( F_UART_SINGLE_BIT_MULTIPLY | F_UART_CONTROL | UART_SPEED_9600 );

Ten kod zawiera błąd.

Jest - mniej więcej - podobny do faktycznego kodu w systemie, gdzie
szukałem kiedyś niedziałajacego UARTu, tylko tych flag było nascie o
bardzo różnych nazwach.

Wyjaśnię na czym polega problem:

#define UART_SPEED_9600 4
#define F_UART_SPEED_9600 (1<<UART_SPEED_9600)

Już rozumiesz? Funkcja zaakceptowała #define z i bez F_. A powinna tylko
z F_, bo tak jest zaimplemnetowana, ze przyjmuje maski a nie numery
bitów. Ta wiedza, maska czy numer bitu, nie istnieje nigdzie, bo to
prymitywny C. Ba, nawet nie wiadomo do której to fukcji są flagi. To
tylko nazwane numerki.

Nikt tego nie kontroluje, funkcja przyjmuje cokolwiek i jedyna nadzieja
w białku, które pisze kod, że się nie pomyli. Naiwna.

Takich funkcji w kodzie są setki.

> Niebezpieczne to jest właśnie przekombinowanie  jak to zrobić "lepiej".

Być może inaczej definicujesz słowo "niebezpiecznie".

Funkcjonalnośc została przerobiona tak, że nie dało się już podać flagi
bez F_ mimo, że obie były liczbami. Wymagało to napisania
kilkudziesięciu lini w C++ i zmiany z #define na class enum.

C++ pozwolił mi zatkać źródło błędu którego nie da się w sposób sensowny
zatkać w samym C z uwagi na prymitywizm wyrażania intencji.

Bezkosztowo. Podczas implementacji nie ucierpiał ani jeden mnemonik
asemblera a przy okazji znalazły się jeszcze dwa takie same przypadki
dla innych elementów systemu, gdzie pomylono flagi.

Po kilku latach pytałem jeszcze kolegi, mieli to w użyciu cały czas i
używali dla nowych funkcji.

Deklaracja dla klienta kodu zmieniał się z:
setupUart( int _flags )
na
setupUart( flags<UartFlags> _flags )

Gdzie UartFlags było wyliczniem co wolno za pomocą prostego template.
Całosc redukuje się do 1 liczby na kompilacji, jak dla C.

I tyle. Reszta nikogo nie interesowała.

Dzieki C++ jestem w stanie napisać teraz kod, który w wypadku pomylenia
flagi, zatrzyma kompilację, zamiast wesoło zrzucić problem na frajera z
debuggerem.

Rozumiesz już po co jest ten C++?