Ошибка «векторного индекса за пределами диапазона»

Question

В настоящее время я пытаюсь получить «вариационные функции» и просто пытаюсь загрузить 4 имени в вектор-строке, а затем распечатать их. Когда я делаю это с помощью типа «int» и использую числа, он работает нормально, но когда я использую вектор строки, я получаю ошибку.

#include "stdafx.h" 
#include<cstdio> 
#include<cstdarg> 
#include<string> 
#include<vector> 
#include<iostream> 
using namespace std; 



int count; 

vector<string> namesVector; 

void names(int count, ...) 
{ 
    va_list namesList; 

    int i; // for loop 

    va_start(namesList, count); 

    for (i = 0; i < count; i++) 
    { 
     string currentElement; 
     currentElement = va_arg(namesList, string); 

     namesVector[i] = currentElement; 

    } 

    va_end(namesList); 
} 


int main() 
{ 
    int nameCount = 4; 

    names(nameCount,"jon", "maggie", "joan", "alfred"); 

    for (int i = 0; i < nameCount; i++) 
    { 
     cout << "Name at element " << i << " is: " << namesVector[i] << endl; 
    } 

} 

Answer 1

C++ позволяет использовать только типы с возможностью копирования в виде вариационных аргументов. Поскольку std::string является конструктивным из указателя char*, который указывает на буфер с нулевым завершением, вы можете использовать char* вместо std::string. Просто замените

currentElement = va_arg(namesList, string); 

с

currentElement = va_arg(namesList, char*); 

в вашем коде. Чтобы избавиться от этого ограничения, рассмотрите variadic templates, которые генерируют код во время компиляции для любого используемого вами типа.

Ваш код также содержит ошибку времени выполнения. Это:

namesVector[i] = currentElement; 

, вероятно, сбой вашей программы, поскольку вы не выделяли какую-либо память в векторе. Векторы на самом деле являются динамическими массивами, поэтому вы должны либо передать аргумент размера соответствующему конструктору, либо вызвать resize на вектор. В случае, если вы вы можете сделать ни один из него, а просто использовать push_back method:

namesVector.push_back(currentElement);