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

    +118

    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
    private String compileOwnerName() {
		return Joiner.on(' ').skipNulls().
		String firstName = ownerFirstName.getValue();
		String lastName = ownerLastName.getValue();
		
		final StringBuilder buffer = new StringBuilder();
		boolean first = true;
		
		if (firstName != null) {
			firstName = firstName.trim();
			
			if (!"".equals(firstName)) {
				buffer.append(firstName);
				first = false;
			}
		}

		if (lastName != null) {
			lastName = lastName.trim();
			
			if (!"".equals(lastName)) {
				if (!first) {
					buffer.append(" ");
				}
				
				buffer.append(lastName);
			}
		}
		
		return buffer.toString();
	}

    В итоге заменено однострочником на Perl:

    return Joiner.on(' ').skipNulls().join(Strings.emptyToNull(ownerFirstName.getValue().trim()),
    
	ownerLastName.getValue().trim()));

    Запостил: someone, 05 Июля 2012

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

    • ava someone 05.07.2012 12:02 # 0

      Точнее, конечно же, к ownerLastName тоже emptyToNull.
      Ответить

    • ava rat4 05.07.2012 12:57 # +6

      >return Joiner.on(' ').skipNulls().

      Что-что?
      Ответить

      • ava someone 05.07.2012 14:01 # 0

        Guava.
        Ответить

        • ava roman-kashitsyn 05.07.2012 14:08 # +2

          @rat4 как бы намекнул, что код в топике перестаёт компилиться начиная со второй строки.
          Ответить

      • ava Lure Of Chaos 05.07.2012 14:08 # 0

        a null b null c
        Ответить

      • ava someone 05.07.2012 15:25 # +3

        Это начало нового кода. Этой строки там не должно было быть. Но редактировать и удалять код на этом сайте нельзя...
        Ответить

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