1. C# / Говнокод #17077

    +133

    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
    [DataContract]
        class outColumnsList
        {
...
            [DataMember]
            public decimal? FK_ID_OBJECT { get; set; }
...
            [DataMember]
            public decimal? IS_PK { get; set; }
            [DataMember]
            public decimal? IS_EDIT { get; set; }
..
        }
....
                List<outColumnsList> l = new List<outColumnsList>();
                while (r.Read())
                {
                    outColumnsList l1 = new outColumnsList();
                    if (r.GetOracleValue(0).ToString()!="null")
                    l1.ID_OBJECT = Convert.ToDecimal(r.GetOracleValue(0).ToString());
                    l1.NM_DISPLAY = r.GetOracleValue(1).ToString();
                    l1.NM_SOURCE = r.GetOracleValue(2).ToString();
                    if (r.GetOracleValue(3).ToString() != "null")
                    l1.DISPLAY_SIZE = Convert.ToDecimal(r.GetOracleValue(3).ToString());
                    if (r.GetOracleValue(4).ToString() != "null")
                    l1.FK_ID_OBJECT = Convert.ToDecimal(r.GetOracleValue(4).ToString());
                    if (r.GetOracleValue(5).ToString() != "null")
                    l1.ORDER_DISPLAY = Convert.ToDecimal(r.GetOracleValue(5).ToString());
                    if (r.GetOracleValue(6).ToString() != "null")
                    l1.IS_EDIT = Convert.ToDecimal(r.GetOracleValue(6).ToString());
                    if (r.GetOracleValue(7).ToString() != "null")
                    l1.IS_PK = Convert.ToDecimal(r.GetOracleValue(7).ToString());
                    l1.FK_FIELD = r.GetOracleValue(8).ToString();
                    l1.NM_SOURCE_FK = r.GetOracleValue(10).ToString();
                    l.Add(l1);
                }

    это потрясающе. в качестве идентификатора использовать дробные числа... хорошо хоть не с плавающей запятой.
    а так же, их же, для булевых....

    а то, как они из базы читались, так это вообще сказка.
    все ToString(), и все ConvertToDecimal... и проверка, на null еще тоже на уровне. про IsDBNull явно не слышал ничего

    Запостил: Lokich, 07 Ноября 2014

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

    • ava kegdan 08.11.2014 14:12 # +1

      l1.NM_DISPLAY

      за такое и по ебалу можно дать
      Ответить

    • ava bormand 08.11.2014 14:21 # 0

      > в качестве идентификатора использовать дробные числа
      Ну может быть там по задаче какая-нибудь прописанная в каком-нибудь нормативном документе херня в духе 1.53, а строки юзать не хотелось?
      Ответить

      • ava Lokich 10.11.2014 16:33 # +1

        нет, потому, что бэкэнд программист был быдлоговнокодером, который про оптимизацию ничего никогда не слышал. он вместо int полей делал number, а в он дробный.
        его как-то мало волновали последствия, размеры таблицы, индексов и прочих вещей. в итоге мы имеем несколько таблиц по 100кк записей, который соединяются по number полям в каскаде представлений.
        Ответить

    • ava Vince 13.11.2014 12:07 # +2

      это прям про проект, который я сейчас поддерживаю. все что должно быть int - оно decimal. Ни одного Foreign Key, а primary key получается как Select Max(id) from somtable
      Ответить

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