임베디드 소스 리뷰

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

로직이 별로 많지 않은 프로그램이었지만 , 생각보다 문제가 많았다 . 혼자서 짤 때는 자기자신은 이해할 수 있는 변수명 , 함수명 이지만 , 다른사람이 소스를 읽어보면 , 역할에 맞지 않은 함수명 , 변수명을 사용하여 , 그 이름만으로도 , 코드를 읽는데 어려움이 있었다 .

결과