글 읽기 - C) 여기서 개선할 수 있는 부분이 있을까요?

toppi6741 2년 전 0

일단 정답처리는 나왔는데 뭔가 찜찜하고 아쉬운 느낌이 들어서 고수분들이 보시기에 개선점이 있는지 궁금하네요

피드백 부탁드립니다!

#include <stdio.h>
#include <string.h>

void push(int arr[],int n);
int pop(int arr[]);
int top(int arr[]);
int size(int arr[]);
int empty(int arr[]);

int arrsize = 0;

int main() {
  int arr[10001],n,comm;
  char cmPush[5] = "push";
  char cmPop[4] = "pop";
  char cmSize[5] = "size";
  char cmEmpty[6] = "empty";
  char cmTop[4] = "top";
  char command[6];
  scanf("%d",&n);
  for(int i = 0;i<n;i++) {
    scanf("%s",command);
    if(!strcmp(command,cmPush)) {
      scanf("%d",&comm);
      push(arr,comm);
      }
    if(!strcmp(command,cmPop)) {
      printf("%d\n",pop(arr));
      
    }
    if(!strcmp(command,cmSize)) {
      printf("%d\n",size(arr));
    }
    if(!strcmp(command,cmEmpty)) {
      printf("%d\n",empty(arr));
    }
    if(!strcmp(command,cmTop)) {
      printf("%d\n",top(arr));
    }
  }
}


void push(int arr[],int n) {
  arr[arrsize]=n;
  arrsize++;
}
int pop(int arr[]) {

  if(!arrsize) return -1;
  int n = arr[arrsize-1];
  arr[arrsize-1]=0;
  arrsize--;
  return n;
}
int top(int arr[]) {
  if(!arrsize) return -1;
  return arr[arrsize-1];
}
int size(int arr[]) {
  return arrsize;
}
int empty(int arr[]) {
  if(!arrsize) return 1;
  return 0;
}

jjang36524 2년 전 1

arr은 전역 변수로 선언하는 게 좋아 보입니다. 스택 크기가 터지면 런타임 에라가 날 수도 있고, 초기화가 안 되어 있어 문제가 생길 수도 있습니다.

댓글을 작성하려면 로그인해야 합니다.

문제

문제

출처

ICPC

C) 여기서 개선할 수 있는 부분이 있을까요?

toppi6741 2년 전 0

jjang36524 2년 전 1

Baekjoon Online Judge

채점 현황

문제

유저 대회

고등학교 대회

출처

대학교 대회

도움말