BOJ 1620, 나는야 포켓몬 마스터 이다솜

입력

첫째 줄에는 도감에 수록되어 있는 포켓몬의 개수 N이랑 내가 맞춰야 하는 문제의 개수 M이 주어져. N과 M은 1보다 크거나 같고, 100,000보다 작거나 같은 자연수인데, 자연수가 뭔지는 알지? 모르면 물어봐도 괜찮아. 나는 언제든지 질문에 답해줄 준비가 되어있어.

둘째 줄부터 N개의 줄에 포켓몬의 번호가 1번인 포켓몬부터 N번에 해당하는 포켓몬까지 한 줄에 하나씩 입력으로 들어와. 포켓몬의 이름은 모두 영어로만 이루어져있고, 또, 음… 첫 글자만 대문자이고, 나머지 문자는 소문자로만 이루어져 있어. 아참! 일부 포켓몬은 마지막 문자만 대문자일 수도 있어. 포켓몬 이름의 최대 길이는 20, 최소 길이는 2야. 그 다음 줄부터 총 M개의 줄에 내가 맞춰야하는 문제가 입력으로 들어와. 문제가 알파벳으로만 들어오면 포켓몬 번호를 말해야 하고, 숫자로만 들어오면, 포켓몬 번호에 해당하는 문자를 출력해야해. 입력으로 들어오는 숫자는 반드시 1보다 크거나 같고, N보다 작거나 같고, 입력으로 들어오는 문자는 반드시 도감에 있는 포켓몬의 이름만 주어져. 그럼 화이팅!!!

출력

첫째 줄부터 차례대로 M개의 줄에 각각의 문제에 대한 답을 말해줬으면 좋겠어!!!. 입력으로 숫자가 들어왔다면 그 숫자에 해당하는 포켓몬의 이름을, 문자가 들어왔으면 그 포켓몬의 이름에 해당하는 번호를 출력하면 돼. 그럼 땡큐~

내 접근

1. 이름와 번호가 매핑이 되야하네 Map을 써야할 수 있겠다.
2. 입력되는 포켓몬은 도감 번호 오름차순으로 이름이 입력되네, 입력에서 순서가 보장되는데, unordered_map을 써서 Lookup 시간을 줄일 수 있을까?
3. 출력에서는 도감 번호로 조회하는 경우도 있고, 이름으로 조회하는 경우도 있네, key가 번호인 map, 이름인 map 두 개를 만들어서 하는게 나을까?

구현

#include <iostream>
#include <stdexcept>
#include <string>
#include <sys/types.h>
#define IO std::cin.tie(NULL), std::ios_base::sync_with_stdio(false)

#include <unordered_map>

void solve() {
  int n, m;

  std::cin >> n >> m;

  std::unordered_map<std::string, int> nameToId;
  std::unordered_map<int, std::string> idToName;
  nameToId.reserve(n);
  idToName.reserve(n);

  std::string x;

  for (int i = 1; i <= n; i++) {
    std::cin >> x;
    nameToId.insert({x, i});
    idToName.insert({i, x});
  }

  for (int i = 0; i < m; i++) {
    std::cin >> x;
    if (nameToId.count(x)) {
      std::cout << nameToId.at(x) << '\n';
    } else {
      try {
        int number = std::stoi(x);
        if (idToName.count(number)) {
          std::cout << idToName.at(number) << '\n';
        }
      } catch (const std::invalid_argument &e) {
        continue;
      }
    }
  }
}

int main() {
  IO;
  solve();
  return 0;
}

개선점

try-catch는 비용이 크다

문제 입력은 문자열은 오로지 문자열, 숫자도 마찬가지 보장이 있으니, m만큼 입력되는 문자열 s에 대해 std::stoi와 try-catch보다 더 간단하게 판별할 수 있을 것이다.

if (std::isdigit(s[0])) {
	int id = std::stoi(x);
}

idToName 또한 unordered_map일 필요가 있었을까?

도감 번호는 입력 순서와 동일하므로 vector를 사용하고 도감번호를 인덱스로 취급해서 참조해도 같은 결과가 나왔을 것.

개선한 구현

#include <cctype>
#include <iostream>
#include <string>
#include <sys/types.h>
#define IO std::cin.tie(NULL), std::ios_base::sync_with_stdio(false)

#include <unordered_map>
#include <vector>

void solve() {
  int n, m;

  std::cin >> n >> m;
  std::unordered_map<std::string, int> nameToId;
  std::vector<std::string> idToName(n);
  nameToId.reserve(n);

  std::string x;

  for (int i = 0; i < n; i++) {
    std::cin >> x;
    nameToId.insert({x, i});
    idToName[i] = x;
  }

  for (int i = 0; i < m; i++) {
    std::cin >> x;
    if (std::isdigit(x[0])) {
      int number = std::stoi(x);
      std::cout << idToName[number - 1] << '\n';
    } else {
      auto it = nameToId.find(x);
      if (it != nameToId.end()) {
        std::cout << it->second + 1 << '\n';
      }
    }
  }
}

int main() {
  IO;
  solve();
  return 0;
}