добавление строковых объектов в массив через цикл

Question

То, что я пытаюсь сделать, это создать класс массива шаблонов, который будет хранить значения типа данных в массиве. У меня он отлично работает со значениями int, однако работа со строковыми объектами начинает разрушаться.

Я вынул блок кода и попробовал его самостоятельно, и я получаю ту же ошибку. Я уверен, что узнал об этом, и я почти уверен, что ответ прост, пытаясь обернуть голову темпами, в которых мы изучаем C++, иногда немного сумасшедшим!

Моя лучшая догадка прямо сейчас заключается в том, что мне нужно будет токенизировать строку и искать пробелы. Я склоняюсь к тому, чтобы думать о вещах, которые приводят к большей путанице, поэтому я ищу ответ здесь!

Код:

// Test String: Hello World this is a String Object 
int stringSize = 7; 
int count = 0; 

string s[stringSize]; 

cout << "\nEnter " << stringSize << " one-word string values:\n"; 

while (count < stringSize) { 

    string tmpVal; 

    cin >> tmpVal; 
    s[count] = tmpVal; 

    count ++; 
} 

Answer 1

string s[stringSize]; является незаконным, потому что stringSize не является константой. Вы должны либо использовать динамическую память (то есть string* s = new string [stringSize];), включите stringsize в качестве аргумента шаблона (не делайте этого, он фактически не решает проблему), используйте фиксированное значение размера или используйте существующую структуру (я бы предложил vector, как в ответе Билла). Приведенный ниже код работает отлично на мой компилятор:

int main(int argc, char *argv[]) { 
int stringSize = 7; 
int count = 0; 
string* s = new string [stringSize]; 
cout << "\nEnter " << stringSize << " one-word string values:\n"; 
while (count < stringSize) { 
    string tmpVal; 
    cin >> tmpVal; 
    s[count] = tmpVal; 
    count ++; 
    } 
    delete[] s; 
} 

Answer 2

Я немного запутался, что именно вы ищете, но я предлагаю смотреть в стандартную библиотеку.

Может быть что-то вроде:

list<string> s; 

, а затем, в использовании петли push_back.

Answer 3

Почему бы не прочитать во всей строке, а затем найти все пробелы и использовать метод substr, разделите строку?

Вам понадобятся следующие методы: getline() find_first_of() substr()

Кроме того, поиск вокруг этого сайта для разделения строк в C++ даст вам много советов.

Answer 4

Прежде всего, размер вашего массива должен быть постоянным:

const int stringSize = 7; 

Во-вторых, как сказал dbrien, вы должны использовать зЬй :: вектор, если вы делаете это для опыта обучения:

std::string tmpVal; 
std::vector<std::string> s; 
cout << "\nEnter " << stringSize << " one-word string values:\n"; 

while (cin >> tmpVal) 
{ 
    s.push_back(tmpVal); 
} 

Answer 5

Во-первых, размер массива должен быть постоянным, поэтому она должна быть константной ИНТ stringsize = 7; Кроме того, я бы предложил использовать std :: vector, а не std :: list, дополнительно. Какова была ошибка?

Answer 6

Не уверен, что ошибка, вы получаете, но это неправильно, потому что вы должны использовать постоянное интегральное значение для распределения массивов в стеке .. Изменение:

int stringSize = 7; 
int count = 0; 

string s[stringSize]; 

...чтобы:

const int stringSize = 7; 
int count = 0; 

string s[stringSize]; 

Вы можете и, вероятно, следует также использовать вектор вместо использования массивов C-стиля, или пытается передать свернуть свой собственный шаблонного класс массива:

vector<string> s; 
const int stringSize = 7; 

cout << "\nEnter " << stringSize << " one-word string values:\n"; 

while (s.size() < stringSize) { 

    string tmpVal; 

    cin >> tmpVal; 
    s.push_back(tmpVal); 
} 

Answer 7

Так что получается, что это было компилятор. Я использую Xcode и получить:

cin_cout(7307) malloc: *** error for object 0x1000072c0: pointer being freed was not allocated 

*** установить контрольную точку в malloc_error_break для отладки

Запуск и тот же блок в Visual C++, казалось, хорошо ... Извините за моей глупости и благодаря любезно для вся быстрая обратная связь!

Answer 8

Я также смущен, каков ваш реальный вопрос, потому что ваш код работает. Однако, FWIW, я бы предложил следующее. Изменения следующие: (1) использование const (уже предложенное другими), (2) использование size_t, (3) изменение имени переменной stringSize в numStrings (из-за этого я был сбит с первого взгляда) и (4) избегая строковой копии.

#include <iostream> 
#include <string> 
using namespace std; 

int main() 
{ 
    const size_t numStrings = 7; 
    size_t count = 0; 

    string s[ numStrings ]; 

    cout << "\nEnter " << numStrings << " one-word string values:\n"; 

    while (count < numStrings) { 
     cin >> s[ count ]; 
     count++; 
    } 

    return 0; 
}