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

    +75

    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
    /**
 * Workaround ObjectInputStream for maintaining backward compatibility with serialization.
 * 
 * In the future, please, please, PLEASE assign each serializable class an explicit serialVersionUID.
 * 
 */
public final class DecompressibleInputStream extends ObjectInputStream {
	private static final Logger logger = Logger.getLogger(DecompressibleInputStream.class);

    public DecompressibleInputStream(InputStream in) throws IOException {
        super(in);
    }

    protected ObjectStreamClass readClassDescriptor() throws IOException, ClassNotFoundException {
        ObjectStreamClass resultClassDescriptor = super.readClassDescriptor();
        Class<?> localClass;
        
        try {
            localClass = Class.forName(resultClassDescriptor.getName()); 
        } catch (ClassNotFoundException e) {
            logger.error("No local class for " + resultClassDescriptor.getName(), e);
            return resultClassDescriptor;
        }
        
        ObjectStreamClass localClassDescriptor = ObjectStreamClass.lookup(localClass);
        
        if (localClassDescriptor != null) { // only if class implements serializable
            final long localSUID = localClassDescriptor.getSerialVersionUID();
            final long streamSUID = resultClassDescriptor.getSerialVersionUID();
            
            if (streamSUID != localSUID &&
            		(localClass == ByteArraySerial.class || localClass == Vector2D.class)) {
            	// Workaround: check for serialVersionUID mismatch with two specific classes
                logger.error(String.format("Overriding serialized class version mismatch for %s: " +
                		"local serialVersionUID = %s, stream serialVersionUID = %s",
                		localClass.getName(), localSUID, streamSUID));
                resultClassDescriptor = localClassDescriptor; // Use local class descriptor for deserialization
            }
        }
        
        return resultClassDescriptor;
    }
}

    Продукт использует в качестве бинарного формата сохранённых файлов встроенную сериализацию. При этом ранние версии полагались на встроенный serialVersionUID.

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

    Запостил: lucidfox, 24 Октября 2011

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

    • ava bot-minurast 24.10.2011 13:28 # +1

      ну да. а java - это портянка по горизонтали
      Ответить

    • ava roman-kashitsyn 24.10.2011 13:34 # +1

      > PLEASE assign each serializable class an explicit serialVersionUID
      PLEASE READ "Effective Java" before you start working on this project!
      Ответить

    • ava roman-kashitsyn 24.10.2011 13:40 # 0

      Требование обратной совместимости очень часто вынуждает писать говнокод. Ибо кто-то когда-то не подумал дважды прежде, чем что-то сделать.
      Ответить

      • ava Lure Of Chaos 24.10.2011 14:06 # +2

        да хотя бы однажды, но хорошо и прочитав пару книжечек по теме
        Ответить

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