Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 48
+52
template<typename T> Light *create_instance(args ¶meters) { return new T(parameters); }
static map<string, Light* (*)(args &)> light_factory;
template<typename T> GeometricPrimitive * create_instance(args ¶meters) { return new T(parameters); }
static map<string, GeometricPrimitive* (*)(args &)> geometry_factory;Откопал в архиве студенческих времен. Ray Tracing. abstract factory по именам.
codemonkey,
17 Ноября 2014
+135
static enum rc (*request_functions[])(void) = {
ko,
koko,
kokoko,
illegal_request
};
static inline enum rc illegal_request(void) { return ILLEGAL_REQUEST; }
reply.rc = request_functions[cmd.opcode < NKEYS(request_functions) ? cmd.opcode : ILLEGAL_REQUEST]();Вызываем функцию по опкоду с абортом в случае index_out_of_bounds.
codemonkey,
12 Ноября 2014
+135
void *list_find_data(struct node *llist, void *data, int (*cmp_func)(void *, void *))
{
struct node* it_node = NULL;
if (NULL != cmp_func)
{
for(it_node = llist; (NULL != it_node); it_node = it_node->next)
{
if (NULL != it_node->data)
{
if (0 == cmp_func(data, it_node->data))
{
break;
}
}
}
}
return ((NULL != it_node) && (NULL != it_node->data)) ? it_node->data : NULL;
}Велосипедисты жгут. О glibопараше не наслышаны.
codemonkey,
04 Ноября 2014
+133
if (strcmp(request_name, REQUEST_1) == 0)
function_1();
else if (strcmp(request_name, REQUEST_2) == 0)
function_2();
else if (strcmp(request_name, REQUEST_3) == 0)
function_3();
/* И так далее */
else if (strcmp(request_name, REQUEST_N) == 0)
function_n();Собственно, как можно ЭТО рефакторить в более приемлемый вид и можно ли заменить на branchless?
codemonkey,
15 Октября 2014
−115
ls -laF /usr/lib/x86_64-linux-gnu/libcurl.so.4.2.0
if [ "$?" -ne 0 ]; then
current_location=$PWD
gzip -dc < curl-7.22.0.tar.gz | tar -xf -
cd curl-7.22.0
./configure --prefix=/usr
make
make install
cd $current_location
else
echo "libcurl.so.4.2.0 already exist!"
fiКлюч -f? Нет, не слышали.
codemonkey,
12 Октября 2014
+133
/*Checks whether the path exists and the path is directory, otherwise creates a new directory*/
RET_VALS check_create_directory(const char* const dir_path, void (*print_func)(int, char *,...))
{
RET_VALS ret_val;
struct stat sb;
ret_val = RET_OK;
if (NULL != dir_path)
{
if(NULL != print_func)
{
print_func(DBG_INFO, "%s - Checking %s existence\n", __FUNCTION__, dir_path);
}
else
{
dbgprintln("Checking %s existence", dir_path);
}
if (0 != stat(dir_path, &sb) || (false == S_ISDIR(sb.st_mode)))
{
ret_val |= RET_DIR_MISSING;
if(NULL != print_func)
{
print_func(DBG_INFO, "%s - %s is missing\n", __FUNCTION__, dir_path);
}
else
{
dbgprintln("%s is missing", dir_path);
}
errno = 0;
if (0 == mkdir(dir_path, S_IRWXU | S_IRGRP | S_IXGRP | S_IROTH | S_IXOTH))
{
ret_val |= RET_DIR_CREATE_OK;
if(NULL != print_func)
{
print_func(DBG_INFO, "%s - %s is successfully created\n", __FUNCTION__, dir_path);
}
else
{
dbgprintln("%s is successfully created", dir_path);
}
}
else
{
char err_msg[_K];
sprintf(err_msg, "Failed to create %s, error %04d - %s", dir_path, errno, strerror(errno));
ret_val |= RET_DIR_CREATE_FAILED;
//dbg_ffln(DBG_ERROR, "Failed to create %s", dir_path);
if(NULL != print_func)
{
print_func(DBG_INFO, "%s - %s\n", __FUNCTION__, err_msg);
}
else
{
dbgprintln("%s", err_msg);
}
}
}
else
{
ret_val |= RET_DIR_ALREADY_EXIST;
if(NULL != print_func)
{
print_func(DBG_INFO, "%s - %s already exists\n", __FUNCTION__, dir_path);
}
else
{
dbgprintln("%s already exists", dir_path);
}
}
}
else
{
ret_val = RET_DIR_MISSING;
if(NULL != print_func)
{
print_func(DBG_ERROR, "%s - No directory name is provided", __FUNCTION__);
}
else
{
dbgprintln("No directory name is provided");
}
}
return ret_val;
}Продолжаем раскопки. Вообще весь .с файл можно сюда выложить.
codemonkey,
12 Октября 2014
+136
DIR_STRUCT * fill_dir_struct(DIR_STRUCT * dir_stc, _ptr_by_val_(char *) base_dir_path, _ptr_by_val_(char *) offset_dir)
{
DIR_STRUCT * sub_dir;
DIR * srcdir;
struct stat st;
char full_sub_dir_path[MAX_PATH_SIZE];
struct dirent * dent;
sprintf(full_sub_dir_path, "%s/%s", base_dir_path, offset_dir);
dbgprintln("Directory %s", full_sub_dir_path);
if(NULL != (dir_stc = malloc(sizeof(DIR_STRUCT))))
{
memset(dir_stc, 0x00, sizeof(DIR_STRUCT));
if (NULL != (srcdir = opendir(full_sub_dir_path)))
{
if(NULL != (dir_stc->dir_path = malloc(strlen(offset_dir) + 1)))
{
strcpy(dir_stc->dir_path, offset_dir);
for( ;(NULL != (dent = readdir(srcdir))); )
{
if((0 != strcmp(dent->d_name, ".")) &&
(0 != strcmp(dent->d_name, "..")) &&
(0 <= fstatat(dirfd(srcdir), dent->d_name, &st, 0)))
{
if (S_ISDIR(st.st_mode)) //Directory
{
if(NULL != (sub_dir = fill_dir_struct(sub_dir, full_sub_dir_path, dent->d_name)))
{
dir_stc->sub_dirs = list_append_node(dir_stc->sub_dirs, sub_dir);
dir_stc->dir_files_size += sub_dir->dir_files_size;
dir_stc->sub_dir_files_count += sub_dir->sub_dir_files_count;
}
else
{
dir_stc = destroy_dir_struct(dir_stc);
break;
}
}
else //File
{
dbgprintln("File %s, size %lu bytes", dent->d_name, st.st_size);
dir_stc->dir_files = list_append_node(dir_stc->dir_files, allocate_and_copy_char_buffer(dent->d_name, strlen(dent->d_name) + 1));
dir_stc->dir_files_size += st.st_size;
dir_stc->sub_dir_files_count++;
}
}
}//End of for(;NULL != (dent = readdir(srcdir)); )
closedir(srcdir);
}
else//End of if(NULL != (dir_stc->dir_path = malloc(strlen(offset_dir) + 1)))
{
dir_stc = destroy_dir_struct(dir_stc);
}
}//End of if (NULL != (srcdir = opendir(full_sub_dir_path)))
else
{
dir_stc = destroy_dir_struct(dir_stc);
}
}//End of if(NULL != (dir_stc = malloc(sizeof(DIR_STRUCT))))
return dir_stc;
}
Восьмикратный индент, Йода нотейшн, const *(x) const в аргументах. Есть подозрение, что велосипедисты не знали о scandir.
Это кладезь говна.
codemonkey,
08 Октября 2014
+138
if ((NULL != (string_array = (0 == *length) ? malloc((*length + 1) * REG_SIZE) : realloc(string_array, (*length + 1) * REG_SIZE))))А кому-то это говно меинтейнить надо будет...
codemonkey,
08 Октября 2014
Follow us!