Массив указателя и обратный вызов

Question

У меня есть небольшая проблема с несколькими простыми строками кода.
Эти строки я называл мой метод:

char** paras = new char*; 
inputLength = charUtils::readParameterFromConsole(paras, paraCount, stringBeginningIndex); 

Метод выглядит следующим образом:

int charUtils::readParameterFromConsole(char** &inputs, int &paraCount, int &stringBeginningIndex) { 
    char input[BUFFER_STRING_LENGTH]; 

    cin.getline(input, BUFFER_STRING_LENGTH); 

    if(strlen(input) > 0) 
    { 
     bool stringBeginning = false; 
     char* part = ""; 
     string partString = ""; 

     for(int i = 0; i < paraCount; i++) 
     { 
      if (i == 0) 
       part = strtok(input, " "); 
      else 
       part = strtok(NULL, " "); 

      inputs[i] = part; 
     } 
    } else 
    { 
     cout << "Error! No Input!" << endl; 
    } 

    cout << &inputs[0] << endl; 
    cout << inputs[0] << endl; 

    return strlen(input); 
} 

В методе readParameterFromConsole являются значения правильны, но в вызывающем методе они Арен» t корреляции. Я столкнулся с этой проблемой, так как я реорганизовал код и создаю новый класс.

Может ли кто-нибудь дать мне совет, пожалуйста?

Answer 1

Ваш код, как я пишу это:

int charUtils::readParameterFromConsole(char** &inputs, int &paraCount, int &stringBeginningIndex) { 
    char input[BUFFER_STRING_LENGTH]; 

    cin.getline(input, BUFFER_STRING_LENGTH); 

    if(strlen(input) > 0) 
    { 
     bool stringBeginning = false; 
     char* part = ""; 
     string partString = ""; 

     for(int i = 0; i < paraCount; i++) 
     { 
      if (i == 0) 
       part = strtok(input, " "); 
      else 
       part = strtok(NULL, " "); 

      inputs[i] = part; 
     } 
    } else 
    { 
     cout << "Error! No Input!" << endl; 
    } 

    cout << &inputs[0] << endl; 
    cout << inputs[0] << endl; 

    return strlen(input); 
} 

Основная проблема заключается в том, что вы устанавливаете inputs[i] =указатель в локальный массив. Этот массив больше не существует, когда функция возвращается. Неопределенное поведение, если вы используете любой из этих указателей.

Как я понимаю, в результате вы получите массив «слов».

Это легко организовать (примечание: код нетронутых руками компилятора):

#include <vector> 
#include <string> 
#include <sstream> 
#include <stdexcept> 

bool throwX(char const s[]) { throw std::runtime_error(s); } 

typedef std::vector<std::string> StringVector; 

std::string lineFromUser() 
{ 
    std::string line; 
    std::getline(cin, line) 
     || throwX("lineFromUser failed: std::getline failed"); 
    return line; 
} 

void getWordsOf(std::string const& s, StringVector& result) 
{ 
    std::istringstream stream(s); 
    std::string  word; 
    StringVector  v; 

    while(stream >> word) 
    { 
     v.push_back(word); 
    } 
    result.swap(v); 
} 

StringVector wordsOf(std::string const& s) 
{ 
    StringVector result; 
    getWordsOf(s, result); 
    return result; 
} 

// Some call, like 
StringVector const words = wordsOf(lineFromUser()); 

Опять же, это от кода манжеты, пожалуйста, просто исправить синтаксическую erors.

Приветствия & НТН.,

Answer 2

Вы передаете назад указатели на стек выделенной переменной, input, когда вы говорите inputs[i] = part, потому что part является указателем на input передал обратно strtok.

http://www.cplusplus.com/reference/clibrary/cstring/strtok/