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

    +1

    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
    ...
    private enum CSPTitle {
        CONTENT_SECURITY_POLICY, X_CONTENT_SECURITY_POLICY, X_WEBKIT_CSP;

        public String getName() {
            return WordUtils.capitalizeFully(this.name(), new char[] { '_' }).replace('_', '-');
        }
    }
...
   public Map<String, String> getHeaders(boolean disableXWebkitCspHeader, StringBuilder cspHeaderBodyBuilder){
        Map<String, String> cspHeaders = new HashMap<>();
        for (CSPTitle cspTitle : CSPTitle.values()) {
            if (disableXWebkitCspHeader && CSPTitle.X_WEBKIT_CSP.equals(cspTitle)) {
                continue;
            }

            String cspHeaderBody = cspHeaderBodyBuilder.toString();
            if (CSPTitle.X_CONTENT_SECURITY_POLICY.equals(cspTitle)) {
                cspHeaderBody = processXCSPHeader(cspHeaderBody);
            }
            cspHeaders.put(cspTitle.getName(), cspHeaderBody.trim());
        }
        return cspHeaders;
  }
....

    Запостил: reizy, 14 Ноября 2017

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

    • ava reizy 14.11.2017 21:13 # 0

      По-сути одинаковое недешевое действие в getName вместо конструктора. Заголовок X-Webkit-CSP по результату кривой. Единственное использование enumа - цикл в котором два из трех значений обрабатываются по-особому. И да, Builder.toString() тоже один раз было бы достаточно вызвать.
      Ответить

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