рекурсивно вызывать метод, но регистрировать только второй раз?

Question

У меня есть метод ниже, который расшифровывает URL-адреса для значения, переданного вместе с использованием кодировки.

public String decodeValue(String value, Charset charset) { 
    if (!Strings.isNullOrEmpty(value)) { 
     try { 
     value = URLDecoder.decode(value, charset.name()); 
     } catch (UnsupportedEncodingException ex) { 
     // log error 
     return null; 
     } 
    } 
    return value; 
    } 

Теперь, если URLDecoder.decode линия бросает UnsupportedEncodingException в первый раз, то я хочу, чтобы запустить же value против ниже три линий:

value = value.replaceAll("%(?![0-9a-fA-F]{2})", "%25"); 
value = value.replaceAll("\\+", "%2B"); 
value = URLDecoder.decode(value, charset.name()); 

И если затем снова URLDecoder.decode линия бросает исключение во второй раз, то я буду регистрировать но только второй раз и возвращает значение null в противном случае возвращает значение, которое декодируется.

Каков наилучший и элегантный способ сделать это?

Answer 1

Самый простой способ - сделать частную версию вашей сигнатуры функции, которая включает дополнительный флаг.

private String decodeValue(String value, Charset charset, boolean isFirstTime) { 
    if (!Strings.isNullOrEmpty(value)) { 
     try { 
     value = URLDecoder.decode(value, charset.name()); 
     } catch (UnsupportedEncodingException ex) { 
     if (isFirstTime) { 
      value = value.replaceAll("%(?![0-9a-fA-F]{2})", "%25"); 
      value = value.replaceAll("\\+", "%2B"); 
      return decodeValue(value, charset.name(), false); 
     } else { 
      // log error 
      return null; 
     } 
     } 
    } 
    return value; 
    } 

Тогда просто передать true в первый раз и false в рекурсивном вызове. Внутри функции выполняются только следующие три строки, если передано true.

Публичная версия может пройти true.

public String decodeValue(String value, Charset charset) { 
     decodeValue(value, charset, true); 
    } 

Answer 2

Здесь вы идете:

public String decode(String value, Charset charset) { 
    if (!Strings.isNullOrEmpty(value)) { 
     try { 
     value = URLDecoder.decode(value, charset.name()); 
     } catch (UnsupportedEncodingException ex) { 
     try { 
     value = value.replaceAll("%(?![0-9a-fA-F]{2})", "%25"); 
     value = value.replaceAll("\\+", "%2B"); 
     value = URLDecoder.decode(value, charset.name()); 
     } catch (UnsupportedEncodingException ex) { 
     // log error 
     return null; 
     } 
     } 
    } 
    return value; 
    } 

Надеется, что это решит вашу проблему.

Answer 3

Добавление флага - опция. Да, это легче; но многие утверждают, что наличие флагов просто говорят: плохая практика.

Вы просто пытаетесь свести к минимуму эти вещи.

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

Answer 4

Хотя он не является рекурсивным, вы можете использовать цикл while и флаг.

public String decode(String value, Charset charset) { 
    boolean first = true; 
    while(!Strings.isNullOrEmpty(value)) { 
     try { 
      return value = URLDecoder.decode(value, charset); 
     } catch (UnsupportedEncodingException e) { 
      if(first == false) { 
       // Log error. 
       return null; 
      } 
      value = value.replaceAll("%(?![0-9a-fA-F]{2})", "%25").replaceAll("\\+", "%2B"); 
     } 
     first = false; 
    } 
    return value; 
} 

Answer 5

Вот версия без дополнительных флагов, дублируется код, рекурсии и петли:

public String decodeValue(final String value, Charset charset) throws UnsupportedEncodingException { 
    String result; 
    if (!Strings.isNullOrEmpty(value)) { 
     UnsupportedEncodingException[] lastException = new UnsupportedEncodingException[1]; 
     result = Stream.<Function<String, String>>of(
       Function.identity(), 
       s -> { 
        s = s.replaceAll("%(?![0-9a-fA-F]{2})", "%25"); 
        s = s.replaceAll("\\+", "%2B"); 
        return s; 
       }) 
       .map(f -> f.apply(value)) 
       .map(appliedValue -> { 
        try { 
         return URLDecoder.decode(appliedValue, charset.name()); 
        } catch (UnsupportedEncodingException e) { 
         lastException[0] = e; 
         return null; 
        } 
       }) 
       .filter(Objects::nonNull) 
       .findFirst() 
       .orElseThrow(() -> lastException[0]); 
    } 
    return result; 
} 

Здесь я просто взять поток с 2-мя функциями: тождественное и функция делает коррекцию строки. Затем я применяю эти функции к исходной строке и пытаюсь декодировать. Если первая попытка успешна, тогда код не будет применять функцию коррекции и просто вернет правильный результат. Если декодер выдает исключение после коррекции значения, тогда «findFirst» не найдет значения. Затем мы бросаем последнее пойманное исключение.