Морской бой: расставляется кораблей больше, чем положено

void PlayerTable::fillRandom()
{
    int j = 0;
    reset(); // сбросить всю таблицу
    int i1 = 0;
    int i2 = 0;
    int i3 = 0;
    int i4 = 0;
    while (i1 != 4) // 1 палубные
    {
        Ship temp(1); // создает 1 палубник с рандомными координатами
        ++j;
        if (addShip(temp)) { // если удалось добавить на карту (не пересекается ни с кем и т.д.) возвращает true, иначе false
            _ships.push_back(temp); // если удачно добавился на карту - добавляем этот корабль в вектор
            ++_goodship; // счетчик удачно добавленных кораблей
            ++i1;
        }
    }
    while (i2 != 3) // 2 палубные
    {
        ++j;
        Ship temp(2);
        if (addShip(temp)) {
            _ships.push_back(temp);
            ++_goodship;
            ++i2;
        }
    }
    while (i3 != 2) // 3 палубные
    {
        Ship temp(3);
        ++j;
        if (addShip(temp)) {
            _ships.push_back(temp);
            ++_goodship;
            ++i3;
        }
    }
    while (i4 != 1) // 4 палубные
    {
        Ship temp(4);
        ++j;
        if (addShip(temp)) {
            _ships.push_back(temp);
            ++_goodship;
            ++i4;
        }
    }
}

_ships.push_back(std::move(temp));

bool PlayerTable::addShip(Ship s)
{
    if (!s) { // if something wrong with s._fails
        ++_errship;
        return false;
    }
 
    switch (s.turn())
    {
    case HORIZONT: // горизонтальный поворот корабля
        for (int i = s.begin().column; i <= s.end().column; ++i)
            if (_tablestate[s.end().row][i] == FALSE) { // check tablestate
                ++_errship;
                return false;
            }
 
        for (int i = s.begin().column; i <= s.end().column; ++i)
            _table[s.end().row][i] = SHIP; // make a ship
 
        for (int i = s.begin().column - 1; i < s.end().column + 2; ++i) {
            _tablestate[s.end().row - 1][i] = FALSE; //
            _tablestate[s.end().row][i]     = FALSE; // make a safe place around ship
            _tablestate[s.end().row + 1][i] = FALSE; //
        }
        break;
 
    case VERT: // вертикальный поворот
        for (int i = s.begin().row; i <= s.end().row; ++i) {
            if (_tablestate[i][s.end().column] == FALSE) { // check tablestate
                ++_errship;
                return false;
            }
            _table[i][s.end().column] = SHIP; // make a ship
        }
        for (int i = s.begin().row - 1; i < s.end().row + 2; ++i) {
            _tablestate[i][s.end().column - 1] = FALSE; //
            _tablestate[i][s.end().column]     = FALSE; // make a safe place around ship
            _tablestate[i][s.end().column + 1] = FALSE; //
        }
        break;
 
    default:
        return false; break;
    }
    
    _ships.push_back(s);
    _goodship++;
    return true;
}

 _ships.push_back(s);