Как будет выглядеть защита от изменения вектора нулевой длины?

Question

[Егор Неймлес]

Здравствуйте. Написал класс с использованием вектора. Но вот я выбираю редактирование и у меня, например, заполнено три строки в таблице. Редактирую первую строку, а изменяется всегда последняя. Вроде как нужно написать защиту от изменения вектора нулевой длины. Но как это сделать, не знаю. Пожалуйста, если можете, покажите, как это делается. Заранее спасибо.

Файл Sport.h

#pragma once
#include <iostream>
#include <string>
#include <vector>
#include <iomanip>
#include <fstream>

using namespace std;
typedef string str;

class Sport
{
private:
  str Name;
  char Team;
  float Bal;
  int Mesto;
  vector<Sport> SP;
public:
  Sport();
  Sport(str name, char team, float bal, int mesto, vector<Sport> sp);
  void setName(const str&);
  void setTeam(char);
  void setBal(float);
  void setMesto(int);

  const str& getName() const;
  const char getTeam() const;
    const float getBal() const;
  const int getMesto() const;

  void Menu();
  void Show();
  void Add();
  void Del();
  void Remove();
  
  ~Sport();
};

файл Sport.cpp

#include "Sport.h"
#include <iostream>
#include <string>
#include <vector>
#include <iomanip>

using namespace std;


Sport::Sport()
{
  Name = "Имя не указано";
  Team = '\0';
  Bal = 0.0;
  Mesto = 0;
}

Sport::~Sport()
{
}

Sport::Sport(str name, char team, float bal, int mesto, vector<Sport> sp)
  : Name(name), Team(team), Bal(bal), Mesto(mesto), SP(sp)
{ }

void Sport::setName(const str& name)
{
  Name = name;
}

void Sport::setTeam(char team)
{
  Team = team;
}

void Sport::setBal(float bal)
{
  Bal = bal;
}

void Sport::setMesto(int mesto)
{
  Mesto = mesto;
}

const str& Sport::getName() const
{
  return Name;
}

const char Sport::getTeam() const
{
  return Team;
}

const float Sport::getBal() const
{
  return Bal;
}

const int Sport::getMesto() const
{
  return Mesto;
}

void Sport::Menu()
{
  cout << "1 - Добавить нового участника\n";
  cout << "2 - Редактировать данные об участнике\n";
  cout << "3 - Удалить участника\n";
  cout << "0 - Выход\n";
}

void Sport::Show()
{
  system("cls");
  cout << "------------------------------------------------------------------------------------------------" << endl;
  cout << "Ведомость спортивних состязаний" << endl;
  cout << "------------------------------------------------------------------------------------------------" << endl;
  cout << "  №" << setw(26) << "Фамилия участника" << setw(20) << "Код команды" << setw(25) << "Количество балов" << setw(20) << "Место в итоге" << endl;
  cout << "------------------------------------------------------------------------------------------------" << endl;
  for (int i = 0; i < SP.size(); i++) {
    cout << "  " << i + 1 << '\t' << setw(15) << SP[i].Name << setw(20) << SP[i].Team << setw(24) << SP[i].Bal << setw(20) << SP[i].Mesto << endl;
    if (i != SP.size() - 1)
    {
      cout << " -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  -  - " << endl;
    }
  }
  cout << "------------------------------------------------------------------------------------------------" << endl;
  cout << " Примечание: D - Динамо, C - Спартак, S - Шахтер" << endl;
  cout << "------------------------------------------------------------------------------------------------" << endl;
}
void Sport::Add()
{
  int n;
  Sport t;
  SP.push_back(t);
  cout << "Введите фамилию участника: ";
  cin >> SP[SP.size() - 1].Name;
  do {
    n = 0;
    cout << "Введите код команды (D - Динамо, C - Спартак, S - Шахтер):";
    cin >> SP[SP.size() - 1].Team;
    if (SP[SP.size() - 1].Team != 'D' && SP[SP.size() - 1].Team != 'C' && SP[SP.size() - 1].Team != 'S')
    {
      n = 1;
      cout << "Вы ввели некорректный код команды, повторите ввод:\n";
    }
  } while (n == 1);
  n = 0;
  do {
    n = 0;
    cout << "Введите количество балов: ";
    cin >> SP[SP.size() - 1].Bal;

    if (!cin.good() || SP[SP.size() - 1].Bal < 0)
    {
      n = 1;
      cout << "Вы ввели некорректное число, повторите ввод:\n";
    }
  } while (n == 1);
  n = 0;
  do {
    n = 0;
    cout << "Введите итоговое место: ";
    cin >> SP[SP.size() - 1].Mesto;
    if (SP[SP.size() - 1].Mesto < 0)
    {
      n = 1;
      cout << "Вы ввели некорректное число, повторите ввод:\n";
    }
  } while (n == 1);
  n = 0;
  Show();
}
void Sport::Del()
{
  int n;
  cout << "Введите номер строки, которую Вы желаете удалить: ";
  cin >> n;
  SP.erase(SP.begin() + n - 1);
  Show();
}
void Sport::Remove()
{
  int p;
  int n;
  cout << "Введите номер строки, которую Вы хотите отредактировать: ";
  cin >> n;
  cout << "Введите фамилию участника: ";
  cin >> SP[n - 1].Name;
  do {
    p = 0;
    cout << "Введите код команды (D - Динамо, C - Спартак, S - Шахтер)::";
    cin >> SP[SP.size() - 1].Team;
    if (SP[SP.size() - 1].Team != 'D' && SP[SP.size() - 1].Team != 'C' && SP[SP.size() - 1].Team != 'S') {
      n = 1;
      cout << "Вы ввели некорректный код команды, повторите ввод:\n";
    }
  } while (p == 1);
  p = 0;
  do {
    p = 0;
    cout << "Введите количество балов: ";
    cin >> SP[SP.size() - 1].Bal;
    if (SP[SP.size() - 1].Bal < 0)
    {
      p = 1;
      cout << "Вы ввели некорректное число, повторите ввод:\n";
    }
  } while (p == 1);
  p = 0;
  do {
    p = 0;
    cout << "Введите итоговое место: ";
    cin >> SP[SP.size() - 1].Mesto;
    if (SP[SP.size() - 1].Mesto < 0)
    {
      p = 1;
      cout << "Вы ввели некорректное число, повторите ввод:\n";
    }
  } while (p == 1);
  p = 0;
  Show();
}

Файл main.cpp

#include "Sport.h"
#include <iostream>
#include <conio.h>
#include <string>
#include <vector>

using namespace std;


int main() {
  setlocale(LC_ALL, "Russian");
  int c;
  Sport B;

  do {
    B.Menu();
    cout << "Ваш выбор: ";
    cin >> c;
    switch (c) {
    case 1:
      B.Add();
      break;
    case 2:
      B.Remove();
      break;
    case 3:
      B.Del();
      break;
    case 0:
      break;
    default: cout << "Некорректный вариант выбора!" << endl;
    }
  } while (c != 0);

  _getch();
  return 0;
}

Comments

[Евгений Шатунов]

Егор Неймлес , зачем у тебя std::vector<Sport> лежит внутри class Sport?
Где во всем этом фарше место, на которое стоит обратить внимание? Ты не указал этого ни в комментариях на строчках, ни в пояснении к вопросу.

Answer 1

[Павел Язовских]

Remove крайне странное название для "режима редактирования". Edit назовите, что ли :-)
Думаю, проблема в этом: в Remove вы сначала читаете номер n строки для редактирования, но все данные продолжаете вбивать в SP[SP.size() - 1]
У вектора есть полезные фишки: ссылка на последний элемент: SP.back(), проверка на пустоту: SP.empty()
Код будет гораздо проще читать, если на элемент, с которым предполагается работа получить ссылку, и все действия делать уже с ней. Т.е. вместо SP[SP.size() - 1] ввести ссылку Sport& curSport = SP[SP.size() - 1]; и дальше работать с ней.
Помимо проверки вводимых данных неплохо бы ещё сделать проверку выхода за границы массива. Вот тут, например:

cout << "Введите номер строки, которую Вы хотите отредактировать: ";
  cin >> n;
  cout << "Введите фамилию участника: ";
  cin >> SP[n - 1].Name;