1. Java / Говнокод #16193

    +73

    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
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    // Current Month Days
            for (int i = 1; i <= _nDaysInMonth; i++) {
                if (_myPrefs.getIfConfigured()) {
                    if (_regionWithHolidays != null && _regionWithHolidays.length > 0) {

                        String holidayDateStringFormat = String.valueOf(i) + "-" + getMonthAsString(nCurrentMonth) + "-" + _strCurrentSelectedYear;
                        for (int index = 0; index < _regionWithHolidays.length; index++) {
                            if (_regionWithHolidays[index].getHolidayDate().equals(holidayDateStringFormat)) {
                                list.add(String.valueOf(i) + "-RED" + "-" + getMonthAsString(nCurrentMonth) + "-" + yy);
                            }
                        }
                    }
                }

                if (i == getCurrentDayOfMonth()) {
                    list.add(String.valueOf(i) + "-BLUE" + "-" + getMonthAsString(nCurrentMonth) + "-" + yy);
                } else {
                    list.add(String.valueOf(i) + "-WHITE" + "-" + getMonthAsString(nCurrentMonth) + "-" + yy);
                    if (_myPrefs.getIfConfigured()) {
                        if (_regionWithHolidays != null && _regionWithHolidays.length != 0) {
                            String otherDates = String.valueOf(i) + "-" + getMonthAsString(nCurrentMonth) + "-" + _strCurrentSelectedYear;
                            for (int index = 0; index < _regionWithHolidays.length; index++) {
                                if (_regionWithHolidays[index].getHolidayDate().equals(otherDates)) {
                                    list.remove(String.valueOf(i) + "-WHITE" + "-" + getMonthAsString(nCurrentMonth) + "-" + yy);
                                }
                            }
                        }
                    }
                }
            }

            // some code

            for (int i = 1; i <= _nDaysInMonth; i++) {
                String[] day_color = list.get(i).split("-");
                if (day_color[1].equals("WHITE")) {
                    //active dates in current month
                    dayNumberView.setTextColor(getResources().getColor(R.color.darkGrey_font));
                } else if (day_color[1].equals("BLUE")) {
                    //current date in current month
                    holder.tvTime.setTextColor(getResources().getColor(R.color.whitetranslucent));
                    dayNumberView.setTextColor(getResources().getColor(R.color.whitetranslucent));
                    cell.setBackgroundResource(R.color.blue_font);
                } else if (day_color[1].equals("RED")) {
                    //active dates in current month
                    dayNumberView.setTextColor(getResources().getColor(R.color.red));
                }
            }

    Это приложение с десятками тысяч пользователей. Мне выпала честь править в нем баги. На сколько я могу судить задача этого куска была отобразить календарь на текущий месяц на экране и подсветить WHITE - обычные дни, BLUE - текущий, RED - выходные и праздничные. Итак в чем соль:
    1) сама соль метода - в одном цикле создается список строк вида НОМЕР-ЦВЕТ-МЕСЯЦ-ГОД, чуть ниже в аналогичном цикле эти строки разбиваются по "-" и сравнивается по строковому значению цвета. Кроме того день может быть или текущим или праздничным, но никак не одновременно.
    2) два практически одинаковых куска кода по 10 строк - строки 4-13 и 19-28, первый при определенных условия добавляет ПРАЗДНИЧНЫЙ день в календарь, потом день в 16-18 строках день всегда добавляется еще раз этот день, либо обычный(WHITE) либо текущий(BLUE), и выполняется второй кусок и если проходят те же условия т.е. фактические если был добавлен праздничный день(RED) то удаляется добавленный ОБЫЧНЫЙ день. Баг был в том что если текущий день был еще и праздничным то они задваивались. Сделать по нормальному - если день уже добавлен, не добавлять еще раз или сделать continue главному циклу. Не говоря уже о том чтобы добавить break после 9 строки, видно автор не знал про эти операторы.

    Запостил: TAX, 19 Июня 2014

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

    • ava 3Doomer 19.06.2014 13:36 # +3

      здесь не хватает свича по стрингу
      Ответить

    • ava bormand 19.06.2014 14:51 # 0

      > Сделать по нормальному
      Запилить функцию isHolyday() и... всё ;)
      Ответить

    • ava bormand 19.06.2014 14:52 # +2

      > если текущий день был еще и праздничным то они задваивались
      Но баг никто не замечал годами т.к. в выходные никто прогу не юзал?
      Ответить

      • ava Lure Of Chaos 20.06.2014 14:49 # +4

        упорно делали вид, что не замечали и гуляли в два раза больше остальных
        Ответить

        • ava absolut 21.06.2014 07:26 # +1

          Больше выходных - меньше багов. Но спутники все равно падают.
          Ответить

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