Как я могу обновить значение вектора или карты

Question

Существует 12 скоростных ограничений. Я хотел бы сделать статистику c, которая учитывает их отдельно. Как я могу их обновить? Я не хочу вставлять новый элемент в каждую итерацию. Я просто хочу обновлять их значение на каждой итерации. Есть ли метод, который лучше моего? Я начал кодировать так:

if (pSeg->pRule->b_IsRealSpeedLimitValid)
{

    realSpeedLimit validSpeedLimit;

    speedLimitMap.insert(std::pair<int, realSpeedLimit>(10, validSpeedLimit));
    speedLimitMap.insert(std::pair<int, realSpeedLimit>(20, validSpeedLimit));
    speedLimitMap.insert(std::pair<int, realSpeedLimit>(30, validSpeedLimit));
    speedLimitMap.insert(std::pair<int, realSpeedLimit>(35, validSpeedLimit));
    speedLimitMap.insert(std::pair<int, realSpeedLimit>(40, validSpeedLimit));
    speedLimitMap.insert(std::pair<int, realSpeedLimit>(50, validSpeedLimit));
    speedLimitMap.insert(std::pair<int, realSpeedLimit>(60, validSpeedLimit));
    speedLimitMap.insert(std::pair<int, realSpeedLimit>(70, validSpeedLimit));
    speedLimitMap.insert(std::pair<int, realSpeedLimit>(80, validSpeedLimit));
    speedLimitMap.insert(std::pair<int, realSpeedLimit>(90, validSpeedLimit));
    speedLimitMap.insert(std::pair<int, realSpeedLimit>(100, validSpeedLimit));
    speedLimitMap.insert(std::pair<int, realSpeedLimit>(110, validSpeedLimit));

    switch (pSeg->pRule->Num_RealSpeedLimit)
    {
    case 10:
        auto search = speedLimitMap.find(10);
        if (search != speedLimitMap.end())
        {
            ++validSpeedLimit.totalSegmentCount;
            validSpeedLimit.isValid = true;
            segLength += validSpeedLimit.totalSegmentLength;
        }
        break;
    case 20:
        break;
    case 25:
        break;
    case 30:
        break;
    case 40:
        break;
    case 50:
        break;
    case 60:
        break;
    case 70:
        break;
    case 80:
        break;
    case 90:
        break;
    case 100:
        break;
    case 110:
        break;
    default:
        break;
    }

}

Answer 1

Да, здесь есть нагрузки, которые вам не нужны.

switch (pSeg->pRule->Num_RealSpeedLimit)
{
case 10:
    auto search = speedLimitMap.find(10);

Зачем повторять 10? У вас уже есть переменная !

auto search = speedLimitMap.find(pSeg->pRule->Num_RealSpeedLimit);

Теперь вообще нет необходимости в switch, поскольку он будет выполняться одинаково для всех этих чисел.

У вас также есть ошибка в том, что вы не используете итератор, полученный с помощью .find, а просто модифицируете локальную переменную validSpeedLimit, которую вы скопировали в каждый элемент карты. Я предполагаю, что это неправильно.

---

Дальнейшее улучшение вашего .insert действительно, но многословно. Вместо этого вы можете использовать одно из следующих:

speedLimitMap.emplace(10, validSpeedLimit));
speedLimitMap[10] = validSpeedLimit;

Поскольку validSpeedLimit создается по умолчанию, нам даже не нужны все эти копии; просто default-construct на месте:

speedLimitMap.emplace(10, {});

---

Трудно догадаться, что делает ваша программа, потому что вы не указали контекст, но (при условии, что все ваши case должны повторяться первое, так как вы говорили об «итерациях») вот что я сделаю:

auto& rule = *pSeg->pRule;
if (rule.b_IsRealSpeedLimitValid)
{
    static constexpr const int keys[] = {
       10, 20, 30, 350, 40, 50, 60, 70, 80, 90, 100, 110
    };

    for (const auto i : keys)
       speedLimitMap.emplace(i, {});

    if (rule.Num_RealSpeedLimit)
    {
        if (auto it = speedLimitMap.find(rule.Num_RealSpeedLimit); it != speedLimitMap.end())
        {
            auto& validSpeedLimit = it->second;

            ++validSpeedLimit.totalSegmentCount;
            validSpeedLimit.isValid = true;
            segLength += validSpeedLimit.totalSegmentLength;
        }
    }
}

Теперь я предполагаю, что в действительности вы используете эту карту позже, в противном случае логика c хранить все эти элементы на самом деле не имеет никакого смысла.

Answer 2

Один из вариантов - сделать так, чтобы карта автоматически запускалась. И после этого у вас есть правильный номер, так почему бы просто не использовать его?

if (pSeg->pRule->b_IsRealSpeedLimitValid) {

realSpeedLimit validSpeedLimit;

for (unsigned int i = 1; i <= 12; ++i) {
    speedLimitMap.insert(std::pair<int, realSpeedLimit>(i*10, validSpeedLimit));
}

auto search = speedLimitMap.find(pSeg->pRule->Num_RealSpeedLimit);

if (search != speedLimitMap.end()) {
    ++validSpeedLimit.totalSegmentCount;
    validSpeedLimit.isValid = true;

    segLength += validSpeedLimit.totalSegmentLength;
} else {
    std::cerr << "Your number: " << pSeg->pRule->Num_RealSpeedLimit<< " not in database" << std::endl;
}

}