Си — Говнокод.ру

Си / Говнокод #17139

+136

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44

len += sprintf(event_xml_msg, XML_TAG_START, XML_KOKOKO_HTTP_PROTOCOL);

	// Set <monitor-event>
	len += sprintf(strend_ptr(event_xml_msg), XML_TAG_START, XML_MONITOR_EVENT_NODE_TREE);

	// Set <date>
	len += xml_string_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_DATE, dt.date_b);

	// Set <time>
	len += xml_string_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_TIME, dt.time_b);

	// Set <product> Ex. "VersAtive"
	len += xml_string_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_PRODUCT, product_type);

	// Set <entity code>
	// Supposed to work for all union types
	len += xml_int_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_CODE, event_code);

	// Set <severity>
	//	len += xml_int_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_SEVERITY, severity);
	memset(severity_str, 0, sizeof(severity_str));
	get_severity_string(severity, severity_str);
	len += xml_string_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_SEVERITY, severity_str);

	// Set event entity name
	len += xml_string_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_ENTITY_TYPE, entity_name);

	// Set event description
	if((len + strlen(description)) > (payload_size - footer_size))
	{
		// TODO HANDLE
		printf("Message description overflows buffer size.\n");
		return false;
	}
	len += xml_cdata_string_add_tag(event_xml_msg, XML_MONITOR_EVENT_NODE_DESCRIPTION, description);

	// Set params
	add_xml_entity_params(event_xml_msg, entity_params);

	// Close <monitor-event>
	sprintf(strend_ptr(event_xml_msg), XML_TAG_END, XML_MONITOR_EVENT_NODE_TREE);

	// Close <HTTPProtocol>
	len += sprintf(strend_ptr(event_xml_msg), XML_TAG_END, XML_KOKOKO_HTTP_PROTOCOL);

В проекте широко используется libmxml, а вот блять использовать его по назначению велосипедики не могут.

codemonkey, 20 Ноября 2014

Комментарии (56)

Си / Говнокод #17096

+135

1
2
3
4
5
6
7
8
9

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

Комментарии (38)

Си / Говнокод #17087

+135

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90

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

main(){    
    
    FILE *filein, *fileout;
    
    char fname1[15], fname2[15];
    int n, m;
    int i, j;
    
    printf("Enter name of input file: "); scanf("%s", &fname1);
    printf("Enter name of output file: "); scanf("%s", &fname2);
    
    filein=fopen(fname1, "r");
    fileout=fopen(fname2, "w");
    if(filein==NULL){
      fprintf(stderr, "\nError: can't open file \"%s\"\n\n", fname1);
      fclose(filein);
      fclose(fileout);
      getch();
      }
    else{
         fscanf(filein, "%d%d", &n, &m);
         char ch[m][n];
         int array[m][n];
         int V[m][n];
         
         for(i=0;i<m;i++){
            for(j=0;j<n;j++){
               V[i][j]=0;
               }
            }
         
         for(i=0;i<m;i++){
            for(j=0;j<n;j++){
               fscanf(filein, "%s", &ch[i][j]);
               if(ch[i][j]=='+') array[i][j]=1;
               else if(ch[i][j]=='-') array[i][j]=0;
               }
            }
         
         int sum;
         for(i=0;i<m;i++){
            sum=0;
            for(j=0;j<n;j++){
               sum+=array[i][j];
               }
               if(sum==1){
                         for(j=0;j<n;j++){
                            V[i][j]=array[i][j];
                            }
                         }
               else continue;
            }
            
         int mm=0, c;
         for(i=0;i<m;i++){
            for(j=0;j<n;j++){
               mm+=V[i][j];
               }
            }
         c = m-mm;
         
         int VoteArray[n];
         
         for(j=0;j<n;j++){
            sum=0;
            for(i=0;i<m;i++){
               sum+=V[i][j];
               VoteArray[j]=sum;
               }
            }
         
         float percent[n];
         float per;
         per = 100/(float)c;
         for(i=0;i<n;i++){
            
            if((percent[i]=per*VoteArray[i])>=7.0) fprintf(fileout, "%d ", i+1);
            }
            
         }
    
    fclose(fileout);
    fclose(filein);
    puts("\nMission comleted\nPress any key...\n");
    getch();
}

Фу, блять

Sushev, 09 Ноября 2014

Комментарии (25)

Си / Говнокод #17061

+135

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19

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

Комментарии (10)

Си / Говнокод #16887

+131

1
2
3
4
5
6
7

struct tm * localtime (const time_t * timer);

A pointer to a tm structure with its members filled with the values
 that correspond to the local time representation of timer.

 The returned value points to an internal object whose validity or
 value may be altered by any subsequent call to gmtime or localtime.

Я нуб, впервые вижу такой способ вернуть структуру.

TarasB, 18 Октября 2014

Комментарии (24)

Си / Говнокод #16854

+133

1
2
3
4
5
6
7
8
9

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

Комментарии (151)

Си / Говнокод #16838

+133

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85

/*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

Комментарии (3)

Си / Говнокод #16835

+134

1
2
3
4
5

#ifdef __APPLE__
#define OPENGL_LIBRARY "/System/Library/Frameworks/OpenGL.framework/Libraries/libGL.dylib"
#else
#define OPENGL_LIBRARY "libGL.so"
#endif

А вообще, у мака есть переменные окружения?

Pythoner, 10 Октября 2014

Комментарии (9)

Си / Говнокод #16822

+136

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63

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

Комментарии (33)

Си / Говнокод #16821

+138

1

if ((NULL != (string_array = (0 == *length) ? malloc((*length + 1) * REG_SIZE) : realloc(string_array, (*length + 1) * REG_SIZE))))

А кому-то это говно меинтейнить надо будет...

codemonkey, 08 Октября 2014

Комментарии (9)

Говнокод: по колено в коде.

Си / Говнокод #17139

Си / Говнокод #17096

Си / Говнокод #17087

Си / Говнокод #17061

Си / Говнокод #16887

Си / Говнокод #16854

Си / Говнокод #16838

Си / Говнокод #16835

Си / Говнокод #16822

Си / Говнокод #16821