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

    +78

    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
    public synchronized void add(Object obj){

		if (!sequence.isEmpty()) {
			if (sequence.size() == maxSize) {
				sequence.remove(0);
				sequence.add(obj);		
			}
			else 
				sequence.add(obj);		
		} 
		
		if (sequence.isEmpty()) 
			sequence.add(obj);  
		
			
		notifyAll();
	}

    Альтернативный else.

    Запостил: Connector, 09 Ноября 2011

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

    • ava guestGovno 10.11.2011 07:09 # +1

      У меня есть такое в проекте. Появляется при написании функции не за один раз. Строка 12 означает "если, неважно что там делали раньше, но список всё равно остался пуст, то...", и даёт возможность не зависеть от строк 3-10.
      Ответить

      • ava ingenuus 10.11.2011 08:43 # +4

        Рефакторить такое сразу же надо, а не оправдывать говно.
        public synchronized void add(Object obj) {
    if(sequence.size() == maxSize) {
        sequence.remove(0);
    }
    sequence.add(obj);
}
        Ответить

    • ava Lemming 25.08.2021 05:37 # 0

      "Значит, опасности нет, тогда почему я до сих пор здесь?"
      Ответить

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