Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+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
#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
+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
+131
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
+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
+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
+134
#ifdef __APPLE__
#define OPENGL_LIBRARY "/System/Library/Frameworks/OpenGL.framework/Libraries/libGL.dylib"
#else
#define OPENGL_LIBRARY "libGL.so"
#endifА вообще, у мака есть переменные окружения?
Pythoner,
10 Октября 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
+133
//str will destroy
char* strdup_(char* str){wat?
LispGovno,
07 Октября 2014
Follow us!