글 읽기 - 런타임 에러가 자꾸 나네요ㅜㅜ 이유를 모르겠습니다.

pjy101102 5년 전 0

1>c:\users\pjy10\source\repos\project5\project5\소스.cpp(99): warning C4715: 'Stack::pop': 모든 제어 경로에서 값을 반환하지는 않습니다.

라는 에러 때문에 혹시

catch (RuntimeException &e) { cout << e.getMessage() << endl; } }

부분에서 return이 없어서 그런가 해서 return 하도록 수정도 해봤는데 그대로네요.,.

도움 주시면 감사하겠습니다.

#include <iostream>
#include <stdio.h>
#include <string>
using namespace std;

class RuntimeException { // generic run-time exception
private:
	int errorMsg;	// string 타입의 errorMsg 생성
public:
	RuntimeException(const int& err) { errorMsg = err; }
	int getMessage() const { return errorMsg; }
};
inline ostream& operator<<(ostream& out, const RuntimeException& e) { return out << e.getMessage(); }

class Node {
public:
	int data;
	Node *prevNode;
	Node(int data);
	~Node();
};

Node::Node(int data) {
	this->data = data;
}

Node::~Node() {}

class Stack {
public:
	Node * topNode;
	int size;

	Stack();
	~Stack();

	int empty();
	void push(int data);
	int top();
	int pop();
	int getsize();
};

Stack::Stack() {
	this->topNode = NULL;
	this->size = 0;
}

Stack::~Stack() {
	Node *top = this->topNode;
	while (top) {
		Node* old = top;
		top = top->prevNode;
		delete old;
	}
}

void Stack::push(int data) {
	if (this->topNode == NULL) {
		this->topNode = new Node(data);
	}
	else {
		Node* old = this->topNode;
		this->topNode = new Node(data);
		this->topNode->prevNode = old;
	}
	size++;
}

int Stack::empty() {
	if (this->topNode == NULL)
		return 1;
	else
		return 0;
}

int Stack::pop() {
	try {
		if (this->topNode == NULL) throw RuntimeException(-1);

		int data = this->topNode->data;
		size--;

		if (size == 0) {
			delete this->topNode;
			this->topNode = NULL;
			cout << data << endl;
			return data;
		}
		else {
			this->topNode = this->topNode->prevNode;
			cout << data << endl;
			return data;
		}
	}
	catch (RuntimeException &e) {
		cout << e.getMessage() << endl;
	}
}

int Stack::top() {
	try {
		if (this->topNode == NULL) throw RuntimeException(-1);

		int data = this->topNode->data;
		cout << data << endl;
		return data;
	}
	catch (RuntimeException &e) {
		cout << e.getMessage() << endl;
	}
}


int Stack::getsize() {
	return size;
}

/*
* testing
*/
int main() {

	int N, data;
	string op;
	Stack S;

	cin >> N;

	for (int i = 0; i <N; i++) {
		cin >> op;
		if (op == "push") {
			cin >> data;
			S.push(data);
		}
		else if (op == "pop") {
			S.pop();
		}
		else if (op == "top") {
			S.top();
		}
		else if (op == "size") {
			cout << S.size << endl;
		}
		else if (op == "empty") {
			cout << S.empty() << endl;
		}
	}
	return 0;
}

djm03178 5년 전 0

새로운 Node가 만들어질 때 prevNode에 대한 초기화를 하지 않고 있습니다. 그 때문에 60번째 줄에서 새로 만들어진 Node의 prevNode는 무슨 값일지 알 수 없습니다. 이것만 초기화해주면 맞습니다.

컴파일 시에 뜨는 워닝은 비록 catch문에 걸리더라도 함수는 리턴을 해야 하는 것이 원칙인데 아무것도 반환하지 않고 있기 때문입니다. 애초에, top과 pop 모두 반환한 값을 사용하는 곳이 없으니 처음부터 void로 선언하고 아무것도 리턴하지 않아도 됩니다. 좀 더 좋은 클래스 디자인이라면, top이나 pop이라는 기능 자체에 '출력한다'는 의미는 없으므로 두 함수는 값만을 리턴하고 출력은 하지 않는 편이 좋아 보입니다.

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

문제

문제

출처

ICPC

런타임 에러가 자꾸 나네요ㅜㅜ 이유를 모르겠습니다.

pjy101102 5년 전 0

djm03178 5년 전 0

Baekjoon Online Judge

채점 현황

문제

유저 대회

고등학교 대회

출처

대학교 대회

도움말