임베디드 소스 리뷰

377
-1

Published on

0 Comments
0 Likes
Statistics
Notes
  • Be the first to comment

  • Be the first to like this

No Downloads
Views
Total Views
377
On Slideshare
0
From Embeds
0
Number of Embeds
0
Actions
Shares
0
Downloads
1
Comments
0
Likes
0
Embeds 0
No embeds

No notes for slide

임베디드 소스 리뷰

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

×