안녕하세요. 여러분들에게 학습관련 조언을 구합니다.

일단 첨부하신 파일에서 학번이랑 이름은 빼시는게 어떨까요...

그것만으로도 신상 털릴수도 있을 것 같네요 -_-

직접 짠 소스코드의 생김새에 정답은 없습니다. 굳이 기준을 제시한다면, 수행 속도가 빠를 수록 좋고, 남들이 알아보기 쉬울수록 좋습니다. 나머지는 뭐 언어 종류와 개발 방법론에 따라 달라지겠지만 크게는 이 두가지가 중요해요.

첨부화일을 봤는데, 실행시켜보지는 않았지만, 일단 컴파일 잘 되고 설계한 대로 작동한다면 나쁘진 않습니다.

소스코드를 보니, C++치고는 객체지향적인 모습이 좀 없어 보인다는 느낌이 들긴 하네요.

하지만, 어쨌든 학생이 실력이 있건 없건 학생의 부족한 부분을 제대로 이야기해주지 못하고 포기하는 교수님이라면, 역시 그분에게서 배울 것은 없겠죠.
차라리 학생의 인간성이 나빠서 포기한다면 그 교수님을 이해하겠어요.

뭔가 부족한 부분이 느껴진다면, 코딩보다는 알고리즘과 자료구조를 집중적으로 파서 내공을 쌓으세요. 코딩은 그냥 하다보면 느는 거죠 뭐.

교수가 상황을 몰라서 오해하는건 아닐까요. 비교적 고학번인데 기초 수업에 들어와 있으니까요.

만약, 저렇게 극단적인 이야기를 한 이유가 오직 이 소스 때문이라면, 수업을 바꾸시는걸 추천합니다.

바꿀수 없다면, 일단 저위의 피드백을 바탕으로 함께 발표한 다른 학생들중 괜찮은 학점의 학생들에게 밥이라도 사면서 학생들의 소스 리뷰를 부탁하고 차이점을 깨달으면 좋겠네요.

만약 그 차이점을 인지하고 다시 교수에게 가서 의도한바가 맞는지 확인하면 좀 더 다른 형태의 피드백을 줄 수도 있으리라 생각합니다.

후자가 이상적이라고 생각하는데, 저도 졸업 이후이니 3자의 시각에서 '그때 이러이러 하면 되었구나'라는 생각이 드는거구요. 아마 글쓴이의 상황이라면 극도의 자괴감에 빠졌을 것 같습니다. 어느 분야나 쉽지 않습니다. 힘내세요.

소스 보니까 각이 안나오는 것은 사실이긴 합니다. 처음하니까 잘 못하는 게 당연합니다. 조언조차 거절하다니 그 교수 참 그러네요.

그래서, 간략하게 몇가지만 짚어볼까 합니다.

void main(void) {
... 중략...
 
  FileStream employeeLoad;
  employeeLoad.datFileLoader(thisYear, thisMonth); // employee.dat파일의 자료를 로드 한다.
}

객체 이름(employeeLoad)도 그렇고, 함수 이름(datFileLoader)도 그렇고, 주석도 그렇고, 이 부분을 읽는 사람은 당연히 저 함수 속에서 employee 데이터를 employee.dat라는 파일에서 읽어들이는 기대하게 됩니다. 그리고는 생각하겠죠. "응? 로드만 하고 끝나네?"

이 의문을 풀기 위해 datFileLoader() 함수 내용을 찾아들어가보면, 그곳에서 자료를 읽어들이는 작업 뿐만 아니라, 데이터를 계산해서, 화면에 출력하고, 다른 파일로 저장하는 내용까지, 전부 그 안에 뭉뚱그려져 있다는 걸 발견하게 됩니다. 뭔가 로딩을 하는 것 같은 이름을 가진 함수에서, 계산과 저장까지 다 하다니, 대략 좋지 않죠.

비슷한 맥락에서, IncludeData 클래스나 FileStream 클래스 같은 경우, 그 이름만 봐서는 그게 어떤 역할을 수행하는 클래스인지 짐작하기 어렵거나(IncludeData), 이름과 실제 수행하는 역할과 매칭이 되지 않습니다(FileStream).

등등.

그래서 총평을 하자면, 컴파일한 결과물이 의도한 출력을 낼지는 모르겠으나(안해봄), 소스코드가 개념적으로 잘 조직 되어있지 않습니다. (크게 나눠봤을 때) A B C를 해야하는 프로그램인데, "B", "C"라고 이름 붙은 부분은 없이, "A"라는 이름 붙은 부분만 존재하고, 그 "A"를 한다고 이름 붙은 부분에서 A B C를 다 하고 있는 식이죠.

다음과 같은 구조를 생각해보시면 어떨까 싶네요.

void main(void) {
... 중략...
 
  // 피고용인의 데이터를 가지는 빈 객체 정의
  EmployeeData employee;
 
  // employee.dat라는 파일에서 객체 내부로 데이터를 읽어들임
  employee.load("employee.dat"); 
 
  // employee 내부에 저장된 데이터와 함수 인자로 주어진 날짜 정보를 가지고
  // 급여 정보를 계산 한 뒤, 이 급여 정보를 갖는 Payroll 객체를 생성.
  Payroll payroll = employee.calcPayroll(thisYear, thisMonth);
 
  // Payroll 정보를 출력
  payroll.display();
 
  // Payroll 정보를 'payroll-<연도>-<날짜>.dat' 라는 파일로 저장
  payroll.save();
}

현재 만드신 프로그램을 *조금씩 조금씩* 바꿔서, 대략 이런 비스무리한 형태를 갖는 프로그램으로 변형하는 연습을 해보시면 어떨까 싶네요.

한꺼번에 싹 다 바꾸다 뭔가 엉키면 도전 의욕을 잃을 수도 있으니, 조금 바꿔서 컴파일 -> 실행 -> 결과 확인 하는 걸 반복하다보면, 어느 순간 프로그램의 전체적인 구조가 내 머리속에 그려지는 순간이 있을 겁니다.

OOP 책 하나 사서 읽어보시고요. 거기 나온 개념들을 다 외우거나 절대화할 필요는 없고, 지금은 대략 그게 뭔지 감을 잡는다 정도로 생각하시면 될 듯 합니다.

처음 치곤 잘하신 소스 같네요..다들 보시는 눈이 다른지라..
제가 보기엔 어느정도 감을 잡으신듯 한데..

잡힐듯 말듯 하는 부분에서 조금..헤매신거 같은데..

이런 경우 실행 계획을 조금 명확히 해서 스스로 이해를 높이고 코드로
구현 하시는게 좋습니다.

먼저 무얼 하실지..

클래스 설계 경우 includeData 같은 경우 자료 구조를 의도 하신듯 한데
완성도를 높이실려면 데이터 재사용성(저장 후 사용)을 위해 저장공간(array,vector,etc)에
저장을 하시는게 좋을듯 하고..

FileStream의 경우는 클래스 네임처럼 데이터의 로드/저장을 주목적으로 하시는게 좋을듯
하네요.
당연히 연산자 오버로딩 하셔서 includeData자료 구조를 받는(vector/array,etc를 받는게 좋겠죠)
메서드가 있어서 거기까지 처리를 해주신다면 의도한대로 되지 않을까 싶네요.

Calcurator 클래스는 유틸리티 클래스인 경우인듯 한데..이 경우는 현직에 계시는 분들은
머 여러가지 방식으로들 구현 하시겠지만..저 같은 경우는 includeData쪽에 합치는게
좋지 않을까 싶네요..

사용되는 메서드의 파라미터들이 모드 자료구조에 있는 내용들이니..
굳이 따로 떼어놓을 필요가 있을까 싶은데..( 이 부분은 구현 하시는 분들마다의 성향이라서..)

일단 제가 보기엔 나쁘지 않네요.. 단지 경험이 부족하셔서 하시는 실수들이..
교수님 말씀에 너무 신경 쓰지 마시고 꾸준히 하시면 좋을듯 하네요.

혹시나 필요 하시면 연락 주세요.

dif공공일 앳 네이버 닷컴

한글 숫자는 아라비안으로 수정해서.. :)

ps. 그리고 덧붙여서 화면 표현쪽은 따로 작업을 하시는게 좋을듯 하네요 :)