Красивый и безопасный код в С

ivank · Отправлено: **17:55, 06-07-2004** | #2

Извращаться - так красиво.

Код:

int len;
char *sz;

len = strlen(data) + 1;
sz = (char*)alloca(len);

strcpy(sz, data);
/* или */
memcpy(sz, data, len);

P.S. Ненавижу венгерскую нотацию.

DAnG · Отправлено: **07:19, 07-07-2004** | #3

перефразируем

Код:

#define strNcpy(dest,src,maxsize) \
   { \
      strncpy(dest, src, maxsize - 1); \
      dest[maxsize-1]=0; \
   }

P.S. на с++ все красивее будет, однако

hasherfrog · Конфигурация компьютера

ivank
Фишка в том, что использование динамической памяти накладывает на программиста обязательство следить за использованными ресурсами. Убирать за собой мусор - тоже своего рода искусство. Но если предположить, что всё будет почищено , то ещё вариант:

Код:

char *psz; 
psz = strdup(pszSomeData); 
//... 
if (psz) free(psz); psz = NULL;

ivank · Отправлено: **20:35, 07-07-2004** | #5

hasherfrog
Так alloca - не динамисечкая память, в этом вся фишка

Она на стеке память выделяет. Правда, может возникнуть stack overflow.

hasherfrog · Конфигурация компьютера

ivank
Да, я лоханулся, alloca а не alloc. Не заметил во-первых, а во-вторых, я никогда ей не пользуюсь из-за man alloca:

Цитата:

Эта функция не регламентируется стандартами POSIX или SUSv3.

А для меня POSIX - критично.

ivank · Отправлено: **18:49, 08-07-2004** | #7

Ну... Я на самом деле тоже, поскольку пользую C++, а там и (относительно) умные указатели сделать несложно.

Я, кстати, не знаю ни одной юниксообразной операционка , где этой ф-ии не было бы. В Windows оно тоже есть, только alloca_ зовётся. Так что, имхо, из-за непереносимости её не пользовать довольно таки странно.

P.S. В C99 есть массивы перменного размера. Точнее, не совсем переменного. Поскольку, по сути это та же alloca, но присутствующая в стандарте.

hasherfrog · Конфигурация компьютера

Цитата ivank:

из-за непереносимости её не пользовать довольно таки странно.

И всё-таки траблы будут. Не у всех, конечно. У м$ про _alloc сказано: Security Note: In Windows XP, if _alloca is called inside a try/catch block, you must call _resetstkoflw in the catch block.
Но в общем случае, в принципе, в С можно юзать. Жаль, что strdupa в RTL у м$ нет. Это был бы идеальный вариант.

hasherfrog · Конфигурация компьютера

Ну вот, сам же и словил багу.

Код:

char sz[SZSIZE]; 
strncpy(sz, pszSomeData, sizeof(sz) - 1); 
strcat(sz, '\0');

Компилятор ни слова не сказал про одинарные кавычки в последней строке. Поскольку я привык уже к его "многомудрости", искать ошибку пришлось минут двадцать...