Side Effects of C Macros (diskuse)

[⧠ A = 0]

А какой же это блог?

[Dušan Kreheľ]

Definície (lokalných) funkcií nedefinuj v main(), ale mimo main(), pred definície globalných premenných.

Píš štylom, že main() funkcia je ako posledná v zdrojáku, alebo na konci je posledná najdôležitejšia funkcia v zdrojáku, ak sa tam nenachádza funkcia main.

Tá podmienka if je zbytočná:

if (str[strlen(str) - 1] == '\n')
   str[strlen(str) - 1] = '\0';

Lebo:

fgets()  reads in at most one less than size characters from stream and
       stores them into the buffer pointed to by s.

echo -e "123456"|./check_end_fgets 
49 '1'
50 '2'
51 '3'
0 ''

Testovací kód:

#include <stdio.h>
#include <stdlib.h>

void check_end_fgets()
{
	char buffer[4];
	int a;
	char *p;
	
	if(fgets(buffer, 4, stdin) == NULL)
		return;
	
	for(p=buffer,a=0;a<4 && *p != '\n'; a++,p++)
		printf("%d '%c'\n", *p, *p);
	
}

int main(void)
{
	check_end_fgets();
	return 0;
}

[Dušan Kreheľ]

Máš chybu:

echo "LLLLLLLLLLLLLL"|./cod
upper macro: 7
ctype function: 14

Oprav si chybu vo funkcií uppermacro().

[sad]

Tu chybu jsem tam udělal schválně. Jedná se o srovnání dvou stejných funkcí, které vrací počet velikých písmen s tím rozdílem, že jedna používá vlastní makro myupper() a druhá funkci isupper() ze standardní knihovny.

Problém je v tom, že se ta dekrementace do makra přenese dvakrát:

((*str++) >= 'A' && (*str++) <= 'Z')

[sad]

Inkrementace.

[Michal Kubeček]

Proto je např. v manuálové stránce ke getc() upozornění, že na rozdíl od fgetc() může být implementována jako makro, které argument vyhodnocuje víc než jednou.

[sad]

Tak to jsem nevěděl. Díky za upozornění.

[deda.jabko]

Hodil by se k tomu nejaky komentar...

[Dušan Kreheľ]

Pozri komentár na konci zdrojáku.

[Dušan Kreheľ]

Poznáš príkaz:

cpp source.c -o source_cpp.h

[deda.jabko]

Ano, znam preprocesor i jeho ulohu a to do te miry, abych vedel, ze vystupem preprocesoru neni hlavickovy soubor, ale soubor ciste v jazyce C.

Jinak ma poznamka se tykala toho, ze bez dalsiho komentare je ten clanek zbytecny. Trochu zkuseny programator na prvni pohled vidi, v cem je problem. Zacatecnikovi to moc nerekne a ani nevi, jak se z toho poucit pri psani maker.

[kralyk z abclinuxu]

#define myupper(c) ((c) >= 'A' && (c) <= 'Z')

se dá nahradit něčim jako

#define myupper(c) ((unsigned long)(c - 'A') <= 'Z' - 'A')

(ale je to trochu prasárna)

[pc2005]

C makra? Tohle byla taky vtipná situace. Stringifikace jména argumentu pro vnitřní makro .

[Odin1918]

Pouzivat makra na toto je antipattern a stejne tak je antipattern inkrementovat argument v ramci volani procedury, ktera je v tomto pripade jeste k tomu makrem. Vede to k errorprone kodu, ktery se neda cist. Vice radek muze byt nekdy citelnejsi.