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

    +63.4

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    case PLUG_ACTION_GET_MESSAGE: {
    	msgIn++;
    	//при входящем плюсует еще и к исходящим, недоработка движка?
    	msgOut--;
    }
    			
    case PLUG_ACTION_SEND_MESSAGE: {
    	msgOut++;
    }

    Запостил: guest, 21 Июня 2009

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

    • Они должны были как в C# сделать такую конструкцию недопустимой.
      Ответить
    • показать все, что скрытоЕсли это кусок кода менеджера сообщений, msgOut - это длина очереди отправленных сообщений, а mgsIn - количество полученных и обработанных сообщений, то вполне себе нормальный код. Просто переменные надо называть своими именами, и не называть однотипно совершенно разные по смыслу переменные.
      Ответить
    • Hz:
      ээээ, а может он просто break забыл написать? поэтому и плюсует кисходящим при входящем сообщении о.О
      Ответить
    • вот, а С# не даст забыть break :)
      Ответить
      • >вот, а С# не даст забыть break :)
        ну и зря. Столько конструкций языка зафейлили...
        Ответить
    • Romanius:
      А если такая ситуация, что break как раз не нужен?
      Ответить
    • hz:
      Тогда надо менять местами кейс блоки, а лучше рубить руки разработчику
      Ответить
    • test
      Ответить

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