Refator: Remove callback field

[cheesecrust]

🔗 Related Issue

• https://github.com/jam2in/arcus-works/issues/623

⌨️ What I did

• 다른 Impl 에서 구현하고 있는 방식과 동일하게 callback 을 Impl class 필드에서 저장하고 쓰는 방식에서 handleLine()시에 상위 객체에서 가져와서 캐스팅을 하는 방식으로 통일하였습니다.

[oliviarla]

@jhpark816
가독성 측면에서 저는 기존 방식이 더 나은 것 같긴 한데, 어떻게 생각하시나요?

[jhpark816]

리뷰 완료

[jhpark816]

질문입니다.
타입 캐스팅을 해야 하나요?

[cheesecrust]

네 현재 getCallback() 호출 시 OperationCallback 타입을 반환하게 됩니다.
이때 handleLine 메서드에서 OperationCallback을 extend 한 CollectionPipedInsertOperation.Callback 의 메서드인 gotStatus 를 사용하고 있습니다.
따라서 타입 캐스팅을 하여 이를 호출 할 수 있도록 해야합니다.

[jhpark816]

변수 타입이 CollectionPipedInsertOperation.Callback이므로, gotStatus() 호출하는 데 문제가 없지 않나요?

   CollectionPipedInsertOperation.Callback cb = getCallback();

[cheesecrust]

getCallback에서는 OperationCallback 타입을 반환하는데 이는 Callback의 상위 타입이기 때문에 다운 캐스팅을 해야합니다.
따라서 컴파일러는 해당 객체가 실제로 Callback 타입인지 알 수 없기 때문에 명시적 형 변환을 통해 캐스팅을 해주어야합니다.

[jhpark816]

@cheesecrust
생성자에 아래 코드가 있습니다.

this.cb = (Callback) cb;

그러면, 아래 코드도 가능한가요?

CollectionPipedInsertOperation.Callback cb = (Callback) getCallback();

[cheesecrust]

예 가능합니다만 정확한 타입 표현을 위해서

CollectionPipedInsertOperation.Callback cb =
            (CollectionPipedInsertOperation.Callback) getCallback();

위 처럼 하였습니다.

[cheesecrust]

@cheesecrust
1번 방식으로 구현한 연산을 살펴 보니, handleLine() 메소드가 여러번 호출될 수 있다는 것입니다.
그 수만큼 getCallback() 호출하는 비용을 조금이라도 줄이려는 목적으로 구현해 둔 것입니다.
따라서, 1번 방식도 그대로 유지해 두는 걸로 합시다.

https://github.com/jam2in/arcus-works/issues/623#issuecomment-2511333536