OOP 원칙을 존중하는 코드 리팩토링

Question

이것은 인터뷰 질문입니다. 그들은 코드를 제공하고 그것을 비판하고보다 OOP 방식으로 재 배열하도록 요청합니다.

public abstract class Storage{ 
    public static Vector<IStorage> objects = /*...*/; // Create a vector IStorage empty, I do not know how 

    public Storage(String xmlFile){ 
     ReadFromXMLFile(xmlFile); 
     objects.addElement((IStorage)this); 
    } 
    protected abstract ReadFromXMLFile(String xmlFile); 

} 

public interface IStorage{ 
    public String getAuthor(); 
    public String getEditor(); 
    public String getName(); 
    public boolean isBook(); 
    public boolean isFood(); 
} 

// Exemple d'implémentation 
public class Novel implements IStorage extends Storage{ 
    private String author; 
    private String editor; 
    private String name; 
    private boolean historical; 

    public Novel(String xmlFile){ 
     super(xmlFile); 
    } 

    public String getAuthor(){return author;} 
    public String getEditor(){return editor;} 
    public String getName(){return name;} 

    public boolean isHistorical(){return historical;} 
    public void setHistorical(boolean b){historical = b;} 

    public void setAuthor(String a){author = a;} 
    public void setEditor(String e){editor = e;} 
    public void setName(String n){name = n;} 

    public boolean isBook(){return true;} 
    public boolean isFood(){return false;} 

    public boolean needRefrigeration(){return false;} 

    private ReadFromXMLFile(String xmlFile){ 
     // Loads the object from an XML file 
// The implementation of this method is not given 
    } 
} 

잘못된 OOP 프로그래밍을 감지하지 못했습니다. 이 코드에서 무엇이 잘못되었으며 어떻게 변경할 수 있습니까?

감사합니다. 문제가있는 곳입니다

Answer 1

public class Novel implements IStorage extends Storage 

.....

Answer 2

는 심지어 컴파일되지 않습니다이 코드처럼 보인다.

Novel은 Storage를 확장하고 추상화가 아니기 때문에 ReadFromXMLFile을 구현해야하지만 개인용 메서드로 구현하므로 Storage 생성자는이를 호출 할 수 없습니다. 그 옆에

는 Storage의 생성자는 IStorage에 this을 캐스팅하기 때문에 스토리지의 하위 클래스, IStorage을 구현하는 것으로 가정합니다.

objects.addElement((IStorage)this); 

따라서, Storage는 IStorage를 구현해야합니다.