1. Си / Говнокод #17978

    +138

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    if(argc > 6){
	for(arg = argv[6]; strlen((char *)arg) >= 17 &&
					isdigit(arg[0]) &&
					isdigit(arg[1]) &&
					isdigit(arg[3]) &&
					isdigit(arg[4]) &&
					isdigit(arg[6]) &&
					isdigit(arg[7]) &&
					isdigit(arg[9]) &&
					isdigit(arg[10]) &&
					isdigit(arg[12]) &&
					isdigit(arg[13]) &&
					isdigit(arg[15]) &&
					isdigit(arg[16])
					; arg+=18){

		int t1, t2;
		struct period *sp;

		t1 = (arg[0] - '0') * 10 + (arg[1] - '0');
		t1 = (t1 * 60) + (arg[3] - '0') * 10 + (arg[4] - '0');
		t1 = (t1 * 60) + (arg[6] - '0') * 10 + (arg[7] - '0');
		t2 = (arg[9] - '0') * 10 + (arg[10] - '0');
		t2 = (t2 * 60) + (arg[12] - '0') * 10 + (arg[13] - '0');
		t2 = (t2 * 60) + (arg[15] - '0') * 10 + (arg[16] - '0');
		if(t2 < t1) break;
		sp = myalloc(sizeof(struct period));
		if(sp){
			sp->fromtime = t1;
			sp->totime = t2;
			sp->next = acl->periods;
			acl->periods = sp;
		}
		if(arg[17]!=',') break;
	}
}

    Получение временных диапазонов, из https://github.com/z3APA3A/3proxy/blob/devel/src/3proxy.c#L1470
    Основное говно в том, что нифига не проверяются входные данные т.е. секунд и минут может быть больше 59, а часов может быть больше 23. И проверку if(t2 < t1) лучше заменить на if(t2 <= t1)
    А еще там в том исходнике в качестве отступов используются и пробелы и табы. Вообще там очень много говнокода

    Запостил: j123123, 12 Апреля 2015

    Комментарии (5) RSS

    • ava Fike 12.04.2015 15:15 # 0

      myalloc. ну отлично.
      Ответить

    • ava tirinox 12.04.2015 16:19 # 0

      Вот это совпадение! Этот чел у нас 1 год информатику в школе вел!
      Он ведь еще и стихи пишет!
      Ответить

      • ava tirinox 12.04.2015 16:25 # 0

        Оппа, он и в одной компании со мной работает.
        Ответить

        • ava Fike 12.04.2015 16:59 # +5

          %ссылка на статью про раздвоение личности%
          Ответить

    • ava xyilka 12.04.2015 21:44 # −1

      cleaned
      Ответить

    Добавить комментарий