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

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

+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

printf("Enter item code: ");                                        //Prompts user
scanf ("%14s", codenew1);                                           //Read user input
len = strlen(codenew1);                                             //Read each character into variable len

while (len != strspn(codenew1, "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz"))
{
    printf ("Name contains non-alphabet characters. Try again!: "); //Prompts user to try again
    scanf ("%14s", codenew1);                                       //Reads user input
    len = strlen(codenew1);                                         //Read each character into variable len
}                                                                   //Endwhile
strncpy(codenew, codenew1,2);                                       //Copy the first two characters from the variable codenew1
codenew[2] = 0;                                                     //Store first two characters into variavle codenew

for ( j = 0; j < num_items; j++)                                    //Loop for num_items times
{                                                                   //Beginning of for loop
    if (strcmp(array[j].code1, codenew) == 0)                       //If codenew is found in file
    {                                                               //Beginning of if statement
        price[i] = item_qty[i] * array[j].price1;                   //Calculating the price of an item
        printf("Price : %d", price[i]);                             //Prints price
        printf("\nEnter '%s' to confirm: ", array[j].itemname1);    //Confirming the item
        scanf("%19s", item_name1[i]);
        while (strcmp(item_name1[i], array[j].itemname1 )!=0)       //Looping until both item names are the same
        {                                                           //Begin while loop
            printf("Item name is not %s ,Try Again!: ", array[j].itemname1);    //Prompt user to try again
            scanf ("%19s", item_name1[i]);                              //Reads item name into an array
            len = strlen(item_name1[i]);                                //Reads each character into variable len
        }                                                               //End while loop
        len = strlen(item_name1[i]);                                    //Read each character into variable len
        while (len != strspn(item_name1[i], "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz"))    //While len contains non alphabetic characters
        {                                                                       //Beginning while
            printf ("Name contains non-alphabet characters. Try again!: ");     //Prompts user to try again
            scanf ("%19s", item_name1[i]);                                      //Read user input
            len = strlen(item_name1[i]);                                        //Read each character into variable len
        }                                                                       //End while
        strncpy(item_name[i], item_name1[i], 20);                               //Copy the first two characters from the variable codenew1
        item_name[i][20] = 0;                                                   //Store first 20 characters in variable item_name[i]
        total_price+= price[i];                                     //Calculate total price
        break;                                                      //Terminates loop
    }                                                               //End of if statement
    else 
        if (strcmp(array[j].code1, codenew) != 0)                       //If codenew is found in file
        {
            printf("Invalid input! Try again.");
            goto Here;
        }
}                                                           //End of for loop

Бесценные комментарии!
http://stackoverflow.com/questions/29045067/error-check-files

myaut, 14 Марта 2015

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

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

+128

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

#include <stdio.h>
int main()
{
int i,g;
char a[26];
a[0]='a';
a[1]='b';
a[2]='c';
a[3]='d';
a[4]='e';
a[5]='f';
a[6]='g';
a[7]='h';
a[8]='i';
a[9]='j';
a[10]='k';
a[11]='l';
a[12]='m';
a[13]='n';
a[14]='o';
a[15]='p';
a[16]='q';
a[17]='r';
a[18]='s';
a[19]='t';
a[20]='u';
a[21]='v';
a[22]='w';
a[23]='x';
a[24]='y';
a[25]='z';
for(i=0;i<25;i++)
{
for(g=0;g<i+1;g++)
{
printf("%c",a[g]);
}
if(g!=25)
printf("\n");

}
return 0;
}

К предыдущему коду.

alexmir, 12 Марта 2015

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

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

+135

1
2
3
4
5

#define   CONC_LINE(a)  CONC(a, __LINE__)
#define   CONC(a, b)  __CONC(a, b)
#define __CONC(a, b)  a##b
// ...
int CONC_LINE(name);

Способ присовокупить к названию переменной номер строки, где её объявили.
Объясните кто-нибудь, почему #define CONC_LINE (a) __CONC ( a, __LINE__ ) дает: name__LINE__?

refactor, 11 Марта 2015

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

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

+145

1
2
3
4
5
6
7

/* */
        int fooBar() {
            /* do something */

/* - */     return NULL;
/* + */     return 0ULL;
        }

> src/foobar.c:42:3: warning: return makes integer from pointer without a cast
> return NULL;

П О Ч И Н Е Н О

Elvenfighter, 11 Марта 2015

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

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

+135

1
2
3
4
5
6
7

int overflow(const char *str)
{
    char buf[9000];
    int res = atoi(buf);
    itoa(res, buf, 10);
    return strcmp(buf, str);
}

По мотивам #17745.

gost, 09 Марта 2015

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

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

+134

1
2
3
4
5
6
7

UInt64 unpackSizeSpec =
        p->UnpackPositions[p->FolderStartFileIndex[folderIndex + 1]] -
        p->UnpackPositions[p->FolderStartFileIndex[folderIndex]];
    size_t unpackSize = (size_t)unpackSizeSpec;

    if (unpackSize != unpackSizeSpec)
      return SZ_ERROR_MEM;

Из недр lzma sdk.
По заверениям разработчиков:
> ... может использоваться любым современным 32-битным процессором (или, при определенных условиях 16-битным CPU).
Очевидно что этот код вернет SZ_ERROR_MEM на 32-разрядной платформе, если unpackSizeSpec выходит за границы 32-битного числа.
На самом то деле все логично - пользователь просто посылается на хуй, если его нищебродское устройство не тянет такого толстого архива, но тогда утверждение разработчиков о безпроблемном использовании на любой кофеварке становится пиздежью.

Pythoner, 08 Марта 2015

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

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

+138

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

typedef enum { SUCCESS = 0, NOTNUMBER = 1, BOUNDS = 2, EOFREACHED = 3 } rdstat_t;

int flush_line(FILE *stream) {
  int ch;
  do {
    ch = fgetc(stream);
  } while (ch != EOF && ch != '\n');
  return ch;
}

/* enough to contain (1 + length of ASCII string of SIZE_MAX
   decimal representation) characters */
#define SIZE_T_BUF_SIZE 100

/* reads until newline or EOF, writes (size_t *out) if success */
rdstat_t read_size_t(FILE *stream, size_t *out) {
  char str_SIZE_MAX[SIZE_T_BUF_SIZE];
  snprintf(str_SIZE_MAX, SIZE_T_BUF_SIZE, "%zu", SIZE_MAX);
 
  char buf[SIZE_T_BUF_SIZE];
  if (!fgets(buf, SIZE_T_BUF_SIZE, stream)) 
    return EOFREACHED;
  if (buf[strlen(buf) - 1] != '\n') {
    flush_line(stream);
    return NOTNUMBER;
  }

  buf[strlen(buf) - 1] = 0;
  char *ch;
  for (ch = buf; *ch; ++ch)
    if (!isdigit(*ch)) 
      return NOTNUMBER;

  if (strlen(buf) == strlen(str_SIZE_MAX) && strcmp(buf, str_SIZE_MAX) > 0) 
      return BOUNDS;
    
  sscanf(buf, "%zu", out);
  return SUCCESS;
}

Наваял функцию для чтения size_t из файла с защитой от дурака (писал код будучи больным, чесслово).
Ничего умнее печати SIZE_MAX в строчку и лексикографического сравнения для проверки границ введенного числа не придумал %)

vlitomsk, 06 Марта 2015

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

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

+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

int start()
  {
 string filename = "GBPJPY.hst";
 int i_unused[30];
   
 int FILE = FileOpenHistory(filename, FILE_READ|FILE_BIN);
 FileSeek(FILE, 0, SEEK_SET);
 
 int version    = FileReadInteger (FILE, LONG_VALUE);
 string c_copyright  = FileReadString (FILE, 64);
 string name    = FileReadString (FILE, 12);
 int period     = FileReadInteger (FILE, LONG_VALUE);
 int i_digits   = FileReadInteger (FILE, LONG_VALUE);
 int timesign    = FileReadInteger (FILE, LONG_VALUE);       //timesign
 datetime last_sync   = FileReadInteger (FILE, LONG_VALUE);       //last_sync
 FileReadArray (FILE, i_unused, 0, 13);
 
 Print("Version = ", version);
 Print("c_copyright = ", c_copyright);
 Print("Equity = ", name);
 Print("period = ", period);
 Print("i_digits = ", i_digits);
 Print("timesign = ", TimeToStr(timesign, TIME_DATE|TIME_SECONDS));
 Print("last_sync = ", last_sync);
 Print("i_unused = ", i_unused[0]);
 Print("i_unused = ", i_unused[1]);
 Print("i_unused = ", i_unused[2]);
 Print("i_unused = ", i_unused[3]);
 Print("i_unused = ", i_unused[4]);
 Print("i_unused = ", i_unused[5]);
 Print("i_unused = ", i_unused[6]);
 Print("i_unused = ", i_unused[7]);
 Print("i_unused = ", i_unused[8]);
 Print("i_unused = ", i_unused[9]);
 Print("i_unused = ", i_unused[0]);
 Print("i_unused = ", i_unused[11]);
 Print("i_unused = ", i_unused[12]);
 Print("Time    = ", FileReadInteger (FILE, LONG_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Volume   = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Time    = ", FileReadInteger (FILE, LONG_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Volume   = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Time    = ", FileReadInteger (FILE, LONG_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Volume   = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Time    = ", FileReadInteger (FILE, LONG_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Price    = ", FileReadDouble (FILE, DOUBLE_VALUE));
 Print("Volume   = ", FileReadDouble (FILE, DOUBLE_VALUE));
 FileClose(FILE);   
 return(0);
}

Язык программирования торгового терминала MetaTrader 4 - MQL 4 - имеет C-нотацию. Многое роднит его с языком C, поэтому пример размещаю сюда.
У меня складывается мнение, что, как и Forex - дно (днище) в мире электронной торговли, так и основная масса кода, написанного на MQL4 - образец того, как
НЕ нужно программировать! Чего стоит одна только вырвиглазная разметка вкупе с корявым непоследовательным именованием... характерная, в том числе,
и для официально поставляемых с терминалом примеров кода. Заранее прошу прощения за многострочный пример

AndreySt, 02 Марта 2015

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

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

+137

1

(a != b) ? a ? b = true : b = !b : b = (a == b) && a;

refactor, 01 Марта 2015

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

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