코린이 c++ 코드 관련으로 질문드립니다.

Question

#includeclass stack {public:int* create_stack();void push_stack(int* top, int size, int stack[]);void print_stack(int* top, int stack[]);void pop_stack(int* top, int stack[]);};int stack::* create_stack(){int size;static int dico = 0;std::cout << "스택의 크기를 정해주세요" << std::endl;std::cin >> size;if (dico > 0)int* stack1 = new int[size];else {int* stack1 = new int[size];delete[] stack1;int* stack1 = new int[size];}dico++;return stack1;}void stack::push_stack(int* top, int size, int stack[]){int num, a;if ((*(top)+1) == size)std::cout << "스택이 가득 차서 출력이 불가합니다" << std::endl;else {std::cout << "추가할 숫자를 적어주십시오" << std::endl;std::cin >> num;stack[*top] = num;top++;}}void stack::print_stack(int* top, int stack[]){if (top != 0) {for (int i = 0; i < (*top); i++)std::cout << stack[i] << std::endl;}elsestd::cout << "현재 스택에 자료가 없기 때문에 출력이 불가합니다." << std::endl;}void stack::pop_stack(int* top, int stack[]) {if (top == 0)std::cout << "현재 스택에 자료가 없어서 삭제 작업이 불가능합니다." << std::endl;else {top--;}}int main() {int count, size, top;count = 0;int* stack_top;top = 0;stack_top = ⊤int* backup;stack stk = stack();stk.create_stack();while (count != 4) {std::cout << "작동하실 명령을 선택해주세여" << std::endl;std::cout << "1번 스택 생성(이전 스택은 삭제),2번 스택에 자료 추가,3번 스택에 자료를 삭제" << std::endl;std::cout << "4번 현스택에 자료를 출력 " << std::endl;std::cin >> count;if (count == 1)backup = stk.create_stack();if (count == 2)stk.push_stack(stack_top, backup);if (count == 3)stk.pop_stack(stack_top, backup);}delete stk;} 제가 조금만 배우고 만든 c++ 스택 관련 코드입니다.제가 배움이 부족한 상태에서 코드를 만들어서 코드가 간결하지 못하고 오류가 많아서 어느 부분을 고쳐야 할지 모르겠습니다.배움이 부족해서 너무 무례하지만 고쳐야할 부분을 알려주시면 감사합니다.

posj · Answer

저두 초보라 제가 아는선에서 설명드리겠습니다!create_stack 안에서 stack1이 return 되지못하는이유는if문 안에서 지역변수로 선언되었기에 if문을 나올시 해당 변수가 해제되기때문입니다.if문 밖에 선언해주시거나 stack1의 내용을 저장할 변수를 선언하는것이 좋을것같네요main 부분에 push_stack 함수가 에러가나는이유는push_stack의 매개변수를 3개를 넘기게 되었있는데 2개만 넘겨주어서 그렇습니다.push_stack의 받는 매개변수의 수를 줄이거나 올바른 값을 넣어주세요~create_stack() 부분에 else로 실행되는 부분에서 메모리를 할당해주고계시지만 new를 쓰시고 delete는 안하셨네요 ~그리구 main단에서 push시 if문보다는 switch 문으로 하시는게 좋을것같습니다!

bot99 · Answer

#include class stack { public: stack(void); void create_stack(); void push_stack(void); void print_stack(void); int pop_stack(void); private: int * stack1; int top; int size; }; stack::stack(void) { stack1 = NULL; top = 0; size = 0; } void stack::create_stack(void) { int new_size; std::cout << "스택의 크기를 정해주세요" << std::endl; std::cin >> new_size; if ( stack1 ) delete[] stack1; stack1 = new int[new_size]; top = 0; size = new_size; } void stack::push_stack(void) { int num, a; if ( top == size ) std::cout << "스택이 가득 차서 출력이 불가합니다. (size:" << size << ")" << std::endl; else { std::cout << "추가할 숫자를 적어주십시오" << std::endl; std::cin >> num; stack1[top++] = num; } } void stack::print_stack(void) { if (top != 0) { for (int i = 0; i < top; i++ ) std::cout << i << ": " << stack1[i] << std::endl; } else std::cout << "현재 스택에 자료가 없기 때문에 출력이 불가합니다." << std::endl; } int stack::pop_stack(void) { if (top == 0) std::cout << "현재 스택에 자료가 없어서 삭제 작업이 불가능합니다." << std::endl; else { std::cout << "스택에서 뽑아낸 자료: " << stack1[top - 1] << std::endl; } return stack1[--top]; } int main() { int count = 1; stack stk; stk.create_stack(); while (count != 0) { std::cout << "작동하실 명령을 선택해주세여" << std::endl; std::cout << "1번 스택 생성(이전 스택은 삭제),2번 스택에 자료 추가,3번 스택에 자료를 삭제" << std::endl; std::cout << "4번 현스택에 자료를 출력 " << std::endl; std::cout << "명령>>"; std::cin >> count; if (count == 1) stk.create_stack(); if (count == 2) stk.push_stack(); if (count == 3) stk.pop_stack(); if (count == 4) stk.print_stack(); } return 0; }

코린이 c++ 코드 관련으로 질문드립니다.

2개의 답변

기술 목록