Post on 12-Jun-2015
임베디드 소스 리뷰
0892030 임준석 (A)0892050 김도일 (N)0892068 신건영 (N)
Embeded System Source Review
소스는 김도일 학생이 객체지향언어 1 에서 과제로 짰던 , ‘ 비행기 예약 시스템’ 을 리뷰해 보았다 .
Embeded System Source Review
소스 (AirlineBook.h)
변수명 함수명 올바르지 못함 .
Embeded System Source Review
소스 (main.cpp)
air.set() 함수이름 역할과 맞지않음
air.cancle() 도 비슷하지만… 함수이름 역할과 맞지않음 air.print() 역시… .
Embeded System Source Review
소스 (Airlinebook.cpp)
Console.set_aitNO()?? Set 하는데 인자를 받지않음…Console.set_getNO()?
?
Embeded System Source Review
소스 (Console.h)
Console 에 역할을 수행 할 때멤버변수가 필요없다 .( 저장공간이 필요없다 .)
Embeded System Source Review
소스 (Console.cpp)
AirlineBook.cpp에서 두 개의 함수를 부름 . 비효율적 . 읽기도 힘듬 .
Embeded System Source Review
소스 (Console.h)
모든 함수가 get set 을 따로함 . 불필요……
Embeded System Source Review
소스 (Schedule.h)
마찬가지로 함수명이 역할과 맞지 않음 .
Embeded System Source Review
소스 (Schedule.h)
20 은 뭐고 8 은 뭔지 모르겠음 .
i++ k++ 분명 같은 로직인데 분리해뒀음 .
Embeded System Source Review
마찬가지로 함수명이 역할과 맞지 않음 .
소스 (Seat.h)
Embeded System Source Review
소스 (Seat.cpp)
User_name 이라는 변수로 예약자리인지 확인함 .
Embeded System Source Review
로직이 별로 많지 않은 프로그램이었지만 , 생각보다 문제가 많았다 . 혼자서 짤 때는 자기자신은 이해할 수 있는 변수명 , 함수명 이지만 , 다른사람이 소스를 읽어보면 , 역할에 맞지 않은 함수명 , 변수명을 사용하여 , 그 이름만으로도 , 코드를 읽는데 어려움이 있었다 .
결과