Side Effects of C Macros

11.3.2017 20:27 | Přečteno: 1125× | Ostatní | poslední úprava: 11.3.2017 20:38

After typing Ctrl-D the fgets() function will read EOF and the program will be terminated.

#include <stdio.h>
#include <string.h>
#include <ctype.h>

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

enum {
	MAX = 100
}; 

int main()
{	
	int uppermacro(char *str);
	int ctypefun(char *str);
	
	
	char str[MAX];
	int nupper;	
	
	while (fgets(str, MAX, stdin) != NULL) {
	
		if (str[strlen(str) - 1] == '\n')
			str[strlen(str) - 1] = '\0';
	
		nupper = uppermacro(str); 
		printf("upper macro: %d\n", nupper);
	
		nupper = ctypefun(str);
		printf("ctype function: %d\n", nupper);
	}
	
	return 0;
}

int uppermacro(char *str)
{
	int nupper = 0;
	
	while (*str != '\0')	{
		if (myupper(*str++))	
			nupper++;		
	}
	
	return nupper;	
}	

int ctypefun(char *str)
{
	int nupper = 0;
	
	while (*str != '\0')	{
		if (isupper(*str++))
			nupper++;		
	}
	
	return nupper;	
}

        /* 
	while (myupper(c = getchar())) ...	ERROR
	while (isupper(c = getchar())) ...	WORKS
	while ((c = getchar()) != EOF &&
	isupper(c)) ... 			SAFER
	*/

Takový blog sem ještě patří?        

Komentáře

[⧠ 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.