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

    +74

    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
    public static final int TYPE_A_OK = 0;
public static final int TYPE_R_OPEN = 1;
public static final int TYPE_R_STOPSID = 2;
public static final int TYPE_A_STOPSID = 3;
//....
public static final int TYPE_R_ALARM = 26;
HashMap<String, Integer> typesMap = new HashMap<String, Integer>();

//в конструкторе

public Data() {
typesMap.put("A_OK", TYPE_A_OK);
typesMap.put("R_OPEN", TYPE_R_OPEN);
typesMap.put("R_STOPSID", TYPE_R_STOPSID);
//...
typesMap.put("R_ALARM", TYPE_R_ALARM);
//...
}


//в одном из методов

public boolean processPacket(Packet pack) {
//...
StringTokenizer strt = new StringTokenizer(body, "\n");
		try {
			id = strt.nextToken();
			sign = strt.nextToken();
			type = typesMap.get(strt.nextToken());
			try {
				commBody = strt.nextToken();
			} catch (Exception e) {
				// System.out.println("ERR: " + id + "; " + sign + "; " + type
				// + "; ");
			}
		} catch (Exception e) {
			System.err.println(sdf.format(Calendar.getInstance().getTime()) +"packet parsing error");
			outBody += "A_ERR\n" + e.getMessage();
		}

switch (type) {
		case TYPE_A_OK:
			dontsend = true;
			break;
		case TYPE_R_OPEN:
			outBody = processROpen(comm);
			break;
		case TYPE_R_CLOSE:
			outBody = processRClose(comm);
			break;
//...
case TYPE_R_ALARM:
			outBody = processRAlarm(comm);
			break;
default:
			outBody += "A_ERR";
			break;
}
//...
if(debug)
System.err.println(outBody);
//...
return true;
}

    ява она такая. Вот так. Вынужденно-китайский код. Не умеет свич со строками работать... и не хочется с хэшем заморачиваться. А скоро типов будет больше...

    Запостил: ark, 14 Апреля 2011

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

    • ava ark 14.04.2011 19:01 # 0

      можно впихнуть хак типа
      switch(type.hashCode()){
      "R_OPEN".hashCode:блабла

      но оно именно хак, никто не гарантирует уникальности хэша
      Ответить

    • ava Lure Of Chaos 14.04.2011 20:35 # 0

      и все же .split() и через if..else if было бы более читаемо (строки все равно сравниваются по хэшам)
      Ответить

      • ava RaZeR 14.04.2011 20:57 # −1

        Здесь говнокод, а не бюро советов!
        Ответить

        • ava Lure Of Chaos 14.04.2011 21:00 # +2

          где здесь говнокод? это не бюро советов, а комментарий.
          Ответить

      • ava wvxvw 18.04.2011 18:36 # 0

        Вообще, по-уму можно было comm создавать разных типов и несколько оверлоад для метода-приемщика (команды?), тогда и расширяемость, и код человеческий.
        Ответить

        • ava Lure Of Chaos 18.04.2011 19:08 # 0

          если по уму - то Packet body оформлять не монолитным стрингом, а хотя бы Enum. А уж потом можно свитчкейсить, оверлоадить и иначе как угодно дизайнпаттернить
          Ответить

    • ava guest 14.04.2011 21:25 # +1

      Объясните жабаобезьянке, что есть такая вещь, конечный автомат называется.
      Про комбинаторный парсинг говорить не стоит: мозжечок поциента слабенький, он, чего доброго, не выдержит такой ментальной нагрузки.
      Ответить

    • ava guest 14.04.2011 21:47 # 0

      Расскажите этому индусу про Enum и Enum.valueOf()
      Ответить

      • ava Lure Of Chaos 14.04.2011 21:54 # 0

        не поможет.
        Ответить

      • ava guest 17.04.2011 21:08 # 0

        valueOf в этом куске по неведомым причинам валит OpenJDK. Яебу, почему. ССЗБ, да. Но вот так приходится извращаться.
        Ответить

    • ava gegMOPO4 18.04.2011 15:40 # 0

      enum Types {A_OK, R_OPEN, ...}. Потом Types.valueOf() (завернуть в try...catch) и switch. Или прямо переопределить метод для каждого типа.

      В Java7 будет switch по строкам.
      Ответить

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