Use asserts instead of couting a segfault

[richelbilderbeek]

Is your feature request related to a problem? Please describe.

I find here:

void game::remove_tile(...) 
{
  //...
  try {
    if(m_tiles[i].get_id() == m_selected.at(0)){
       m_selected.pop_back();
    }
  } catch (std::out_of_range) {
    std::cout << "segmentation fault\n";
  }
}

This is problematic for multiple reasons:

• It hides something that is an error
• It's a non-standard idiom

Describe alternatives you've considered

Add asserts and use the operator[] instead:

  assert(i < static_cast<int>(m_tiles.size());
  assert(0 < static_cast<int>(selected.size())
  if(m_tiles[i].get_id() == m_selected[0]){
     m_selected.pop_back();
  }

Additional context

None.