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

    +93

    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
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78
    79. 79
    80. 80
    81. 81
    82. 82
    83. 83
    84. 84
    85. 85
    package java.util.logging;
public class XMLFormatter extends Formatter 
{
    // Append a two digit number.
    private void a2(StringBuffer sb, int x) {
	if (x < 10) {
	    sb.append('0');
	}
	sb.append(x);
    }
    // Append the time and date in ISO 8601 format
    private void appendISO8601(StringBuffer sb, long millis) {
	Date date = new Date(millis);
	sb.append(date.getYear() + 1900);
	sb.append('-');
	a2(sb, date.getMonth() + 1);
	sb.append('-');
	a2(sb, date.getDate());
	sb.append('T');
	a2(sb, date.getHours());
	sb.append(':');
	a2(sb, date.getMinutes());
	sb.append(':');
	a2(sb, date.getSeconds());
    }
    private void escape(StringBuffer sb, String text) {
	if (text == null) {
	    text = "<null>";
	}
	for (int i = 0; i < text.length(); i++) {
	    char ch = text.charAt(i);
	    if (ch == '<') {
		sb.append("<");
	    } else if (ch == '>') {
		sb.append(">");
	    } else if (ch == '&') {
		sb.append("&");
	    } else {
		sb.append(ch);
	    }
	}
    }
    public String format(LogRecord record) {
	StringBuffer sb = new StringBuffer(500);
	sb.append("<record>\n");

	sb.append("  <date>");
	appendISO8601(sb, record.getMillis());
	sb.append("</date>\n");

	sb.append("  <millis>");
	sb.append(record.getMillis());
	sb.append("</millis>\n");

	sb.append("  <sequence>");
	sb.append(record.getSequenceNumber());
	sb.append("</sequence>\n");

	String name = record.getLoggerName();
	if (name != null) {
	    sb.append("  <logger>");
	    escape(sb, name);
	    sb.append("</logger>\n");
	}

	sb.append("  <level>"); 
	escape(sb, record.getLevel().toString());
	sb.append("</level>\n");

	if (record.getSourceClassName() != null) {
	    sb.append("  <class>");
	    escape(sb, record.getSourceClassName());
	    sb.append("</class>\n");
	}

	if (record.getSourceMethodName() != null) {
	    sb.append("  <method>");
	    escape(sb, record.getSourceMethodName());
	    sb.append("</method>\n");
	}

	sb.append("  <thread>");
	sb.append(record.getThreadID());
	sb.append("</thread>\n");
 ....

    Я конечно всё понимаю, неоднократно были ситуации когда для скорости сам собирал xml руками.
    Но тут и вовсе такое ощущение будто школьник писал.
    http://kickjava.com/src/java/util/logging/XMLFormatter.java.htm

    Запостил: 3.14159265, 15 Октября 2012

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

    • ava Lure Of Chaos 15.10.2012 18:05 # +4 

      if (ch == '<') {
		sb.append("<");
	    } else if (ch == '>') {
		sb.append(">");
	    } else if (ch == '&') {
		sb.append("&");
	    } else {
		sb.append(ch);
	    }
      кусок особенно шикарен
      Ответить

      • ava bormand 15.10.2012 18:41 # +3

        Там походу когда-то были &lt;, &gt; и т.п.
        Ответить

        • ava Elvenfighter 15.10.2012 18:42 # +1

          Может быть это уютненький уже поескейпил?
          Ответить

          • ava 3.14159265 15.10.2012 19:01 # 0

            Угу, это у меня браузер походу поэскейпил, людям на смех.
            Но это код не спасает, забыли про: apos,quot.
            Ответить

            • ava 3.14159265 15.10.2012 19:07 # −3

              Ответить

            • ava wvxvw 15.10.2012 19:08 # 0

              А их и не нужно экранировать, как раз. Кстати, & тоже не всегда нужно экранировать, в такой ситуации: <foo bar="foo & bar"/> нужно только непосредственно один амперсанд. Но проблема в том, что большинству писателей парсеров на этот момент положить, и все равно будет ошибка.
              Ответить

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