OverWorld/LostWorld

Gaikotsu

яжпрограммист
Легенда
Победитель в номинации 2023
Победитель в номинации 2022
Победитель в номинации 2021
Участник Новогоднего Фонда 2021
Эксперт
Знаток
Просветитель
Магистр реакций
Знаток письма
Куратор Данных
Медаль Благодарности
Старожил II степени
Старожил I степени
Победитель в номинации 2020
Победитель в номинации 2019
Клиент разработчик
Преподаватель
За веру и верность форуму
Победитель в номинации 2018
Медаль за активность на Форуме
За заслуги перед форумом
Web разработчик
Разработчик
За знание датапака
За знание ядра
Сообщения
1 538
Розыгрыши
0
Решения
24
Репутация
6 036
Реакции
2 628
Баллы
2 188
я удивлен что никто не выкладывает эталонный пример того, как не надо писать, из сырцов овера/лв, который так и кочует из сборки в сборку и никто его почему-то не исправляет :)
метод из класса ExProperties
Код:
    public static boolean parseBoolean(String s)
    {
        switch (s.length()) {
            case 1: {
                char ch0 = s.charAt(0);
                if (ch0 == 'y' || ch0 == 'Y' || ch0 == '1')
                {
                    return true;
                }
                if (ch0 == 'n' || ch0 == 'N' || ch0 == '0')
                {
                    return false;
                }
                break;
            }
            case 2: {
                char ch0 = s.charAt(0);
                char ch1 = s.charAt(1);
                if ((ch0 == 'o' || ch0 == 'O') &&
                        (ch1 == 'n' || ch1 == 'N') )
                {
                    return true;
                }
                if ((ch0 == 'n' || ch0 == 'N') &&
                        (ch1 == 'o' || ch1 == 'O') )
                {
                    return false;
                }
                break;
            }
            case 3: {
                char ch0 = s.charAt(0);
                char ch1 = s.charAt(1);
                char ch2 = s.charAt(2);
                if ((ch0 == 'y' || ch0 == 'Y') &&
                        (ch1 == 'e' || ch1 == 'E') &&
                        (ch2 == 's' || ch2 == 'S') )
                {
                    return true;
                }
                if ((ch0 == 'o' || ch0 == 'O') &&
                        (ch1 == 'f' || ch1 == 'F') &&
                        (ch2 == 'f' || ch2 == 'F') )
                {
                    return false;
                }
                break;
            }
            case 4: {
                char ch0 = s.charAt(0);
                char ch1 = s.charAt(1);
                char ch2 = s.charAt(2);
                char ch3 = s.charAt(3);
                if ((ch0 == 't' || ch0 == 'T') &&
                        (ch1 == 'r' || ch1 == 'R') &&
                        (ch2 == 'u' || ch2 == 'U') &&
                        (ch3 == 'e' || ch3 == 'E') )
                {
                    return true;
                }
                break;
            }
            case 5: {
                char ch0 = s.charAt(0);
                char ch1 = s.charAt(1);
                char ch2 = s.charAt(2);
                char ch3 = s.charAt(3);
                char ch4 = s.charAt(4);
                if ((ch0 == 'f' || ch0 == 'F') &&
                        (ch1 == 'a' || ch1 == 'A') &&
                        (ch2 == 'l' || ch2 == 'L') &&
                        (ch3 == 's' || ch3 == 'S') &&
                        (ch4 == 'e' || ch4 == 'E') )
                {
                    return false;
                }
                break;
            }
        }

        throw new IllegalArgumentException("For input string: \"" + s + "\"");
    }
 

я удивлен что никто не выкладывает эталонный пример того, как не надо писать, из сырцов овера/лв, который так и кочует из сборки в сборку и никто его почему-то не исправляет :)
метод из класса ExProperties
Код:
    public static boolean parseBoolean(String s)
    {
        switch (s.length()) {
            case 1: {
                char ch0 = s.charAt(0);
                if (ch0 == 'y' || ch0 == 'Y' || ch0 == '1')
                {
                    return true;
                }
                if (ch0 == 'n' || ch0 == 'N' || ch0 == '0')
                {
                    return false;
                }
                break;
            }
            case 2: {
                char ch0 = s.charAt(0);
                char ch1 = s.charAt(1);
                if ((ch0 == 'o' || ch0 == 'O') &&
                        (ch1 == 'n' || ch1 == 'N') )
                {
                    return true;
                }
                if ((ch0 == 'n' || ch0 == 'N') &&
                        (ch1 == 'o' || ch1 == 'O') )
                {
                    return false;
                }
                break;
            }
            case 3: {
                char ch0 = s.charAt(0);
                char ch1 = s.charAt(1);
                char ch2 = s.charAt(2);
                if ((ch0 == 'y' || ch0 == 'Y') &&
                        (ch1 == 'e' || ch1 == 'E') &&
                        (ch2 == 's' || ch2 == 'S') )
                {
                    return true;
                }
                if ((ch0 == 'o' || ch0 == 'O') &&
                        (ch1 == 'f' || ch1 == 'F') &&
                        (ch2 == 'f' || ch2 == 'F') )
                {
                    return false;
                }
                break;
            }
            case 4: {
                char ch0 = s.charAt(0);
                char ch1 = s.charAt(1);
                char ch2 = s.charAt(2);
                char ch3 = s.charAt(3);
                if ((ch0 == 't' || ch0 == 'T') &&
                        (ch1 == 'r' || ch1 == 'R') &&
                        (ch2 == 'u' || ch2 == 'U') &&
                        (ch3 == 'e' || ch3 == 'E') )
                {
                    return true;
                }
                break;
            }
            case 5: {
                char ch0 = s.charAt(0);
                char ch1 = s.charAt(1);
                char ch2 = s.charAt(2);
                char ch3 = s.charAt(3);
                char ch4 = s.charAt(4);
                if ((ch0 == 'f' || ch0 == 'F') &&
                        (ch1 == 'a' || ch1 == 'A') &&
                        (ch2 == 'l' || ch2 == 'L') &&
                        (ch3 == 's' || ch3 == 'S') &&
                        (ch4 == 'e' || ch4 == 'E') )
                {
                    return false;
                }
                break;
            }
        }

        throw new IllegalArgumentException("For input string: \"" + s + "\"");
    }
Этот бред нужно вообще переделать на рефлекшен который будет автоматически заполнять переменную. Без такой херни.
И посадить все на хмл.
 
Назад
Сверху Снизу