c언어 진짜 기초만 배운 학생인데 제가 짠 프로그램에 대해서 피드백해주시면 감사하겠습니다

Question

#include <stdio.h>#pragma warning(disable:4996)int main(){ int emnum, age, sec, min, hour, resec, salary, rsalary, ssalary, asalary; double tariff, notariff, tariff1, tariff2, tax, batax, sbtax; char code, grade; printf(&#34;사원번호(정수 두 자리)를 입력해주세요 : &#34;); scanf(&#34;%d&#34;, &emnum); printf(&#34;나이(정수)를 입력해주세요 : &#34;); scanf(&#34;%d&#34;, &age); printf(&#34;부서코드(문자 한개)를 입력해주세요 : &#34;); scanf(&#34;
%c&#34;, &code); printf(&#34;급여등급(R,S,A)을 입력해주세요 : &#34;); scanf(&#34;
%c&#34;, &grade); printf(&#34;근무시간을 초 단위로 입력해주세요 : &#34;); scanf(&#34;%d&#34;, &sec); min &#61; sec / 60; hour &#61; min / 60; resec &#61; sec % 60; salary &#61; 0; tax &#61; 0; if (resec > 0) min &#43;&#61; 1; switch (code) { case 'a': case 'A': break; case 'b': case 'B': break; case 'c': case 'C': break; case 'd': case 'D': break; case 'e': case 'E': break; default:printf(&#34;해당하는 부서가 없습니다&#34;); } rsalary &#61; (hour 10000) &#43; (min 10000 / 6); ssalary &#61; (hour 10000) &#43; (min 8000 / 6); asalary &#61; (hour 10000) &#43; (min 5000 / 6); switch (grade) { case 'R': case 'r': salary &#61; rsalary; break; case 'S': case 's': salary &#61; ssalary; break; case 'A': case 'a': salary &#61; asalary; break; default:printf(&#34;해당하는 등급이 없습니다&#34;); } sbtax &#61; salary; tariff &#61; sbtax * 0.8; tariff1 &#61; sbtax * 0.95; tariff2 &#61; sbtax * 0.9; notariff &#61; sbtax; if (age >&#61; 65) batax &#61; tariff1; else if (age >&#61; 55) batax &#61; tariff2; else if (age >&#61; 25) batax &#61; tariff; else if (age >&#61; 18) batax &#61; tariff2; else batax &#61; notariff; tax &#61; sbtax - batax; printf(&#34;
사원번호 : %d번
&#34;, emnum); switch (code) { case 'a': case 'A': printf(&#34;부서 : 총무부서&#34;); break; case 'b': case 'B': printf(&#34;부서 : 인사부서&#34;); break; case 'c': case 'C': printf(&#34;부서 : 기획부서&#34;); break; case 'd': case 'D': printf(&#34;부서 : 영업부서&#34;); break; case 'e': case 'E': printf(&#34;부서 : 회계부서&#34;); break; } printf(&#34;
근무시간 : %d시간 %d분
&#34;, hour, min); switch (grade) { case 'R': case 'r': printf(&#34;급여등급 : R&#34;); break; case 'S': case 's': printf(&#34;급여등급 : S&#34;); break; case 'A': case 'a': printf(&#34;급여등급 : A&#34;); break; } printf(&#34;
세전급여액 : %.f
&#34;, sbtax); printf(&#34;세금 : %.f
&#34;,tax); printf(&#34;세후급여액 : %.f
&#34;, batax); return 0;}안녕하세요 컴공 1학년입니다. 급여관리 프로그램을 만들어봤는데 만드는 당시에는 나쁘지 않다고 생각이 들었는데 막상 다 만들고 전체를 둘러보니 뭔가 조잡하고 필요없는 부분이 있을 것 같은 느낌이 들어서 질문드려봅니다. c언어를 배운지 얼마 안돼서 위에서 사용된 간단한 구문들 밖에 못써서 최대한 초보가 사용하는 것들로 피드백을 해주시면 감사하겠습니다.

mirheeoj · Answer

챗지피티의 제안입니다. 제가 보기엔 전부 맞는 얘기네요. --Here are some suggestions to improve the given C code:1. Add comments: Add comments throughout the code to explain the purpose of each section or variable. This will make the code more readable and easier to understand.2. Use meaningful variable names: Replace variable names like emnum, sec, min, etc., with more descriptive names. This will enhance code readability and maintainability.3. Format the code properly: Use consistent indentation and spacing to improve code readability.4. Validate user input: Perform input validation to handle invalid inputs gracefully. For example, if the user enters an invalid department code or grade, provide appropriate error messages and handle the situation accordingly.5. Separate code into functions: Divide the code into separate functions to improve modularity and reusability. For example, you can create separate functions for calculating the salary, calculating the tax, and displaying the employee details.6. Avoid magic numbers: Instead of using hardcoded constants like 10000, 8000, and 6 in the salary calculations, define them as named constants or variables. This makes the code more maintainable and allows for easier modifications in the future.7. Use meaningful constants: Instead of using the characters 'R', 'S', 'A', 'r', 's', 'a' to represent grade codes, use meaningful constants or enums. This improves code readability and avoids confusion.8. Encapsulate related data: Consider using structures or classes to encapsulate related data, such as employee information. This can make the code more organized and easier to work with.9. Handle default cases explicitly: In the switch statements for department code and grade, add a default case that handles invalid inputs explicitly. Display an error message and handle the situation accordingly.10. Update variable assignments: In the salary calculation section, update the expressions to use multiplication operators (`) instead of division operators (/). For example, rsalary = (hour 10000) + (min * 10000 / 6)`.11. Use printf format specifiers correctly: In the printf statements, ensure that the format specifiers match the data types being printed. For example, use %f for printing double values.By implementing these improvements, you can make the code more readable, maintainable, and robust.

Tomson · Answer

switch문이 너무 많은 거 같아요. 제 생각에는 맵핑을 이용하면 코드가 더 간결해질 거 같은데요, 즉 'R', 'S'와 같은 키에 맞는 문자열들을 미리 자료구조에 저장하는 방식을 쓰면 좋을 거 같아요.

갈리오스 · Answer

변수명이 너무 많아요!sbtax = salary;tariff = sbtax * 0.8;tariff1 = sbtax * 0.95;tariff2 = sbtax * 0.9;tariff 들은 한번만 쓰고 다시 안쓰니까 계산 부분에 적고 주석 처리하는 것이 더 깔끔해 보입니다.case는 default를 웬만하면 정해주는 것이 좋아요. default가 없으면 의도치 않은 동작을 할 수도 있는거니까 없는 부분들에 추가하면 좋을 것 같습니다.

가누 · Answer

변수 초기화입니다. scanf로 받는다고 하지만 개인적으로 변수 선언할 때 초기화도 같이 하는 편입니다.구조체를 사용해보세요. 예를 들어 아래 처럼요. 가독성도 뛰어나고 유지보수도 용이할 겁니다.typedef struct _People{unsigned int age;unsigned int salary;unsigned int ssalary;}People;변수 타입을 다양화해보세요. 위에서 보인 것처럼 음수가 아닐 경우에 unsigned 예약어를 써보시는 것도 좋을 것 같아요. 메모리 절약 차원에서는 나이는 int가 아닌 short를 쓰는 것도 방법입니다. 단 속도 차원에서는 32bit에서는 short보다 int가 더 빠릅니다..저는 if가 한 줄일 때도 중괄호를 넣는 편입니다. 가독성 측면에서..상수는 되도록 define 등으로 정의해서 사용해주세요. 코드가 길어지고 오래되면 기억이 잘 안나요.main 코드에 모두 적지 마시고. 기능 별로 함수를 나누어보세요.예외처리를 넣어보세요. 예를 들면 사원번호가 정수 두 자리인데 세 자리를 입력했다던지, 나이를 음수값으로 입력했다던지 예외적인 상황에 대항 처리를 넣어보세요. 사용자에게 로그를 남겨주면서 프로그램을 종료시켜도 되고, 재 입력하게끔 해도 됩니다. try count를 주고 3번까지는 입력할 기회를 주어도 되구요.위 예외처리와 더불어서 디폴트값을 넣어보세요. 예를 들면 급여 등급을 입력하지 않았을 때 기본 S로 저장하는 등 입력하지 않은 값에 대한 디폴트를 해주는 것 말이에요.제가 드릴 수 있는 짧은 코멘트는 여기까지입니다! ㅎㅎ

c언어 진짜 기초만 배운 학생인데 제가 짠 프로그램에 대해서 피드백해주시면 감사하겠습니다

4개의 답변

Q&A 목록