이 글의 풀이는 효율보다는 객체 지향에 의의를 두고 작성한 풀이입니다.
실제 업무에서 유지보수 및 개선을 염두하는 느낌으로 작성했습니다.
그리고 문제에서 보장한 입력 조건에 대해선 검증을 하지 않습니다.
질문과 피드백은 언제나 환영합니다!
문제: 동영상 재생기
문제 링크: https://school.programmers.co.kr/learn/courses/30/lessons/340213
(문제는 간단한 참고용으로 복붙만 했습니다. 문제를 안보셨다면 링크타고 가독성좋은 곳에서 보시는걸 추천합니다)
문제 설명
당신은 동영상 재생기를 만들고 있습니다. 당신의 동영상 재생기는 10초 전으로 이동, 10초 후로 이동, 오프닝 건너뛰기 3가지 기능을 지원합니다. 각 기능이 수행하는 작업은 다음과 같습니다.
10초 전으로 이동: 사용자가 "prev" 명령을 입력할 경우 동영상의 재생 위치를 현재 위치에서 10초 전으로 이동합니다. 현재 위치가 10초 미만인 경우 영상의 처음 위치로 이동합니다. 영상의 처음 위치는 0분 0초입니다.
10초 후로 이동: 사용자가 "next" 명령을 입력할 경우 동영상의 재생 위치를 현재 위치에서 10초 후로 이동합니다. 동영상의 남은 시간이 10초 미만일 경우 영상의 마지막 위치로 이동합니다. 영상의 마지막 위치는 동영상의 길이와 같습니다.
오프닝 건너뛰기: 현재 재생 위치가 오프닝 구간(op_start ≤ 현재 재생 위치 ≤ op_end)인 경우 자동으로 오프닝이 끝나는 위치로 이동합니다.
동영상의 길이를 나타내는 문자열 video_len, 기능이 수행되기 직전의 재생위치를 나타내는 문자열 pos, 오프닝 시작 시각을 나타내는 문자열 op_start, 오프닝이 끝나는 시각을 나타내는 문자열 op_end, 사용자의 입력을 나타내는 1차원 문자열 배열 commands가 매개변수로 주어집니다. 이때 사용자의 입력이 모두 끝난 후 동영상의 위치를 "mm:ss" 형식으로 return 하도록 solution 함수를 완성해 주세요.
제한사항
- video_len의 길이 = pos의 길이 = op_start의 길이 = op_end의 길이 = 5
- video_len, pos, op_start, op_end는 "mm:ss" 형식으로 mm분 ss초를 나타냅니다.
- 0 ≤ mm ≤ 59
- 0 ≤ ss ≤ 59
- 분, 초가 한 자리일 경우 0을 붙여 두 자리로 나타냅니다.
- 비디오의 현재 위치 혹은 오프닝이 끝나는 시각이 동영상의 범위 밖인 경우는 주어지지 않습니다.
- 오프닝이 시작하는 시각은 항상 오프닝이 끝나는 시각보다 전입니다.
- 1 ≤ commands의 길이 ≤ 100
- commands의 원소는 "prev" 혹은 "next"입니다.
- "prev"는 10초 전으로 이동하는 명령입니다.
- "next"는 10초 후로 이동하는 명령입니다.
Code
def convert_string_to_sec(time_str: str) -> int:
m, s = time_str.split(":")
return int(m) * 60 + int(s)
def convert_sec_to_string(s: int) -> int:
return f"{str(s//60).zfill(2)}:{str(s%60).zfill(2)}"
class Video:
def __init__(self, metadata: dict):
self.len: int = convert_string_to_sec(metadata["length"])
self.opening_start: int = convert_string_to_sec(metadata["opening_start"])
self.opening_end: int = convert_string_to_sec(metadata["opening_end"])
self.current_pos: int = convert_string_to_sec(metadata["current_pos"])
self.metadata = metadata
class VideoPlayer:
def __init__(self, video: Video):
self._video = video
self._adjust_cur_pos()
def input_command(self, cmd: str):
cmd_interpreter = {"next": 10, "prev": -10}
self._video.current_pos += cmd_interpreter[cmd]
self._adjust_cur_pos()
def get_current_pos(self) -> str:
return convert_sec_to_string(self._video.current_pos)
def _adjust_cur_pos(self):
if self._video.current_pos < 0:
self._video.current_pos = 0
elif self._video.current_pos > self._video.len:
self._video.current_pos = self._video.len
if self._video.opening_start <= self._video.current_pos < self._video.opening_end:
self._video.current_pos = self._video.opening_end
def solution(video_len, pos, op_start, op_end, commands):
video_metadata = {
"length": video_len,
"opening_start": op_start,
"opening_end": op_end,
"current_pos": pos,
}
video = Video(video_metadata)
video_player = VideoPlayer(video)
for cmd in commands:
video_player.input_command(cmd)
return video_player.get_current_pos()코드 해석
문제에서 객체는 비디오와 비디오를 다루는 비디오플레이어 2가지입니다.
비디오 객체는 비디오가 가지는 데이터를 다루고, 비디오 플레이어 객체는 비디오를 컨트롤하는 역할을 가진다고 볼 수 있습니다.
비디오 객체는 값을 검증, 보관, 표현하는 책임을 가지고, 비디오플레이어 객체는 비디오의 값에 접근하여 읽기, 쓰기 등의 책임을 맡습니다.
Video
- 비디오가 가지는 원본 데이터를 메타데이터로 저장합니다.
- 원본데이터는 문자열이지만 커맨드에 따라 값을 조정해야하기 때문에 각 시간값들은 초 단위로 가집니다.
- 각 메타데이터 값들을 초 단위의 int 타입의 속성값을 가집니다.
datetime을 사용하지 않는 이유는 재생 관련 시간이 날짜에 해당하지 않는 점, (해당 문제에서는) 초 단위에서 해결 가능하기 때문에 int값으로 문제를 해결합니다.
VideoPlayer
- 비디오가 가진 값을 조작하기 위한 객체로 봅니다.
VideoPlayer객체를 통해Video값을 다룹니다input_command(): 커맨드를 받아 현재 재생위치를 조정합니다._adjust_cur_pos(): 문제에서 주어진 조건에 따라 재생 위치를 조정합니다.
convert_string_to_sec(), convert_sec_to_string()
"%M:%S"값과 초단위의 int값을 변환하는 함수입니다.- 위의 두 클래스에 종속시켜도 무방해 보이지만, 함수의 역할이 비디오에 국한된게 아니라는 생각이 들어 유틸리티 함수 정도로 생각하여 모듈 레벨의 함수로 정의했습니다.
TODO
- 실제 업무로 맡아야했다면 비디오의 메타데이터 값을 검증하는 부분이 수행되어야 합니다.
- 줄여쓴 변수명이 있습니다. 통용되는 수준의 변수명인지, 해석의 여지가 다분한지 확인이 필요합니다.
convert_string_to_sec(),convert_sec_to_string()함수명이 명확하지 않습니다. 명확한 함수명으로 변경하거나 간단한 주석이 필요해 보입니다.- 비디오 저장 과정이 구현되지 않았습니다. 필요하다면
Video객체에서 현재 값을 메타데이터에 저장하거나 내보내는 부분이 구현되어야 합니다.
마무리
처음에는 비디오 데이터 중 불변값(재생 시간, 오프닝 시간대)을 비디오 객체 비디오플레이어 객체에다가 현재 재생 위치를 추가했다.
하지만 내가 보았던 영상들 대부분은 이전 재생 위치를 기억하고 그 때부터 재생이 된다는게 생각이나서 현재 재생 위치(아마도 마지막 재생 위치 정도일 듯 하다)를 메타데이터로 포함시켰다.
input_command() 가 아닌 input_commands()를 통해 커맨드 리스트를 그대로 넣을까도 생각했지만 단위테스트를 고려해서 조금 더 작은 단위로 구현했다. 하지만 커맨드가 항상 리스트로 들어온다면 input_commands()로 구현하는 것이 맞을 듯 하다.
분명 레벨1 문제인데 레벨2보다 더 고생해서 작성했다.. 역시 고통은 구현보단 머리싸매기에서 많이 오는것 같다.
