Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+132
static inline uint64_t eight_bytes_to_uint64(uint8_t *buf) {
return (uint64_t) four_bytes_to_uint32(buf) << 32 | four_bytes_to_uint32(buf + sizeof(uint32_t));
}
static inline uint32_t four_bytes_to_uint32(uint8_t *buf) {
return (uint32_t) two_bytes_to_uint16(buf) << 16 | two_bytes_to_uint16(buf + sizeof(uint16_t));
}
static inline uint16_t two_bytes_to_uint16(uint8_t *buf) {
return buf[0] << 8 | buf[1];
}
Чё-то вспомнились Schemeовые времена и ФП.
Buffer overflow исключён на 146%.
codemonkey,
25 Ноября 2014
+134
//// AEM_* AkelEdit control messages
if (uMsg >= WM_USER)
{
if (uMsg >= AEM_CANUNDO)
goto CanUndo;
//Text retrieval and modification
if (uMsg == AEM_EXSETTEXTA)
{
AESETTEXTA *at=(AESETTEXTA *)lParam;
return AE_SetTextAnsi(ae, at->nCodePage, at->pText, at->dwTextLen, at->nNewLine);
}
if (uMsg == AEM_EXSETTEXTW)
{
AESETTEXTW *at=(AESETTEXTW *)lParam;
return AE_SetText(ae, at->pText, at->dwTextLen, at->nNewLine, FALSE);
}
if (uMsg == AEM_SETTEXTA)
{
return AE_SetTextAnsi(ae, CP_ACP, (char *)lParam, wParam, AELB_ASINPUT);
}
if (uMsg == AEM_SETTEXTW)
{
return AE_SetText(ae, (wchar_t *)lParam, wParam, AELB_ASINPUT, FALSE);
}
if (uMsg == AEM_APPENDTEXTA)
{
AEAPPENDTEXTA *at=(AEAPPENDTEXTA *)lParam;
AE_AppendTextAnsi(ae, at->nCodePage, at->pText, at->dwTextLen, at->nNewLine);
return 0;
}
if (uMsg == AEM_APPENDTEXTW)
{
AEAPPENDTEXTW *at=(AEAPPENDTEXTW *)lParam;
AE_AppendText(ae, at->pText, at->dwTextLen, at->nNewLine);
return 0;
}
if (uMsg == AEM_REPLACESELA)
{
AEREPLACESELA *rs=(AEREPLACESELA *)lParam;
AE_ReplaceSelAnsi(ae, rs->nCodePage, rs->pText, rs->dwTextLen, rs->nNewLine, rs->dwFlags, rs->ciInsertStart, rs->ciInsertEnd);
return 0;
}
if (uMsg == AEM_REPLACESELW)
{
AEREPLACESELW *rs=(AEREPLACESELW *)lParam;
AE_ReplaceSel(ae, rs->pText, rs->dwTextLen, rs->nNewLine, rs->dwFlags, rs->ciInsertStart, rs->ciInsertEnd);
return 0;
}
if (uMsg == AEM_GETTEXTRANGEA)
{
AETEXTRANGEA *tr=(AETEXTRANGEA *)lParam;
return AE_GetTextRangeAnsi(ae, tr->nCodePage, tr->lpDefaultChar, tr->lpUsedDefChar, &tr->cr.ciMin, &tr->cr.ciMax, tr->pBuffer, tr->dwBufferMax, tr->nNewLine, tr->bColumnSel, tr->bFillSpaces);
}
И далее около сотни if'ов в таком духе.
AkelPad
gost,
25 Ноября 2014
+133
int size;
size = EXPR;
if (size > INT_MAX || size <= 0) {
return NULL;
}
// ...Ндя. Семь лет уже. Теперь всё понятно...
bot,
23 Ноября 2014
+139
SGELISTENTRY *sgeListAdd(SGELIST *l, const char *id, void *data) {
SGELISTENTRY *ret;
sgeNew(ret, SGELISTENTRY);
l->numberOfEntries++;
if (l!=NULL) {
ret->prev=l->last;
} else {
ret->prev=NULL;
}
if (l!=NULL && l->last!=NULL) {
l->last->next=ret;
}
ret->next=NULL;
ret->id=strdup(id);
ret->data=data;
if (l==NULL) return ret;
if (l->first==NULL) l->first=ret;
l->last=ret;
return ret;
}Эх, проверяй, не проверяй, один хрен все грохнется при l==NULL
Pythoner,
21 Ноября 2014
+136
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
+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
Follow us!