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

    +144

    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
    #include <stdio.h>

// Говнокод здесь
#define ABS(x) ((x) ? (x) : (-(x)))

int main () {
	char ch = -128;
	short sm = -32768;
	int i = -2147483647; i--;                                    // здесь
	long long l = -9223372036854775807LL; l--; // и здесь декременты нужны
	                                                                             // для подавления варнингов
	
	printf("char:\t\tABS(%hhd) = %hhd\n", ch, ABS(ch));
	printf("short:\t\tABS(%hd) = %hd\n", sh, ABS(sh));
	printf("int:\t\tABS(%d) = %d\n", i, ABS(i));
	printf("long long:\tABS(%lld) = %lld\n", l, ABS(l));

	return 0;
}

    Может быть, баян, но...
    Всем знакомый макрос ABS способен сделать большую гадость.
    Вот вывод программы:
    char: ABS(-128) = -128
    short: ABS(-32768) = -32768
    int: ABS(-2147483648) = -2147483648
    long long: ABS(-9223372036854775808) = -9223372036854775808

    Запостил: whiskey, 12 Ноября 2010

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

    • ava whiskey 12.11.2010 12:05 # 0

      Извините, что табы уползли в комментах.
      Ответить

    • ava Arigato 12.11.2010 12:07 # +5

      > #define ABS(x) ((x) ? (x) : (-(x)))

      Чего-то я не понял, но это ни какой не abs
      Ответить

      • ava burdakovd 12.11.2010 12:49 # +6

        Да, вероятно имелось в виду #define ABS(x) ((x>0) ? (x) : (-(x)))

        Но действительно есть такая фишка, что для minValue нет противоположного положительного значения, т.к. числа кодируются несимметрично.

        Ну и в разных языках в этом случае происходят разные забавные вещи.

        Мораль: нефиг считать на грани диапазонов.
        Ответить

      • ava whiskey 12.11.2010 12:54 # −8

        показать все, что скрытоЕщё какой abs. Эта конструкция используется в куче прог и библиотек.
        Ответить

        • ava absolut 12.11.2010 13:18 # +3

          То, что указано у вас в 4ой строке дает обычное повторение значения.
          Для целых типов по крайней мере.
          Ответить

    • ava whiskey 12.11.2010 12:57 # −2

      Я, в принципе, ожидал, что заминусуют. Код, действительно, не очень тянет на говнокод. Просто хотел лишний раз обратить внимание на то, как простое решение может породить сложную проблему.
      Ответить

      • ava sectus 12.11.2010 13:28 # +5

        Чему должны быть равны соответствующие значения при правильном, а не простом решении?
        char: ABS(-128) = ?
        short: ABS(-32768) = ?
        int: ABS(-2147483648) = ?
        long long: ABS(-9223372036854775808) = ?
        Ответить

        • ava 3.14159265 12.11.2010 14:16 # 0

          2sectus
          +1.

          я вот только что доказывал другу что в .NETe говна хватает и вспомнил винрарнейший говнокод 3117
          Ответить

    • ava rat4 12.11.2010 13:56 # 0

      Чем не нравится стандартный abs из stdlib.h? А за переполнениями программист должен следить сам
      Ответить

    • ava Анонимус 12.11.2010 15:14 # 0

      ну что поделаешь, если кол-во отрицательных и положительных чисел, кодируемых конечным числом байт -- не равно. не вводить же два ноля?)
      Ответить

      • ava eth0 12.11.2010 20:57 # −1

        Ввести "минус ноль". abs(-0) = 0, (-0) x (-0) = 0, и всё в том же духе.
        Ответить

        • ava absolut 12.11.2010 22:40 # 0

          >Ввести "минус ноль"
          А потом париться со сравнением +0 и -0.
          Ответить

          • ava eth0 13.11.2010 00:40 # +2

            Пора вводить тег "ирония". Чуть тоньше, чем "лопата", и не катит уже.
            Ответить

        • ava absolut 13.11.2010 13:06 # 0

          кстати, вот вам http://govnokod.ru/4604#comment52552
          Ответить

    • ava bugmenot 12.11.2010 15:42 # −1

      надо бы кастить результат в unsigned
      Ответить

      • ava da4ever 13.11.2010 08:47 # −1

        и получать нежданчик на выходе?
        замечательно!
        Ответить

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