C++, truncate a char array

Question

Я работаю над проектом, где у меня есть класс Time, и мне нужно отформатировать время.

void Time::FormatTime(char *string, unsigned int max_string_len) { 
    ostrstream fd; 
    ft << hour << ":" << minutes; 
    cout << ft.str() << endl;  
} 

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

Я бы не хотел использовать STL, если это возможно.

Благодаря

Answer 1

Всегда используйте strncpy(), чтобы безопасно усекать строку в char[N].

void Time::FormatTime(char *str, unsigned int max_string_len) { 
    if (max_string_len == 0) return; 
    ostringstream ft; // strstream is obsolete, use stringstream 
    ft << hour << ":" << minutes; 
    strncpy(str, ft.str().c_str(), max_string_len); 
    str[ max_string_len - 1 ] = 0; 
} 

Answer 2

Если первый параметр может быть изменен, то вы можете просто место 0 в правильном положении, чтобы быть строкой терминатор, однако это довольно неприятная (но быстро) решение. В противном случае вы захотите создать новый массив символов и скопировать только правильное количество символов, не забывая выделять дополнительный символ для нулевого терминатора.

Answer 3

Это делает то, что вы хотите:

void Time::FormatTime(char *string, unsigned int max_string_len) { 
     ostrstream fd; 
     ft << hour << ":" << minutes; 
     std::string str = ft.str(); 
     str.resize(max_string_len); 
     strcpy(string, str.c_str()); 
} 

EDIT: Я обновил мой код, благодаря полезной обратной связи potatoswatter и quanmrana.

Answer 4

if(str.length() > max_string_len) 
{ 
    str = str.substr(0, max_string_len); 
} 

strncpy(string, str.c_str(), max_string_len); 

Answer 5

Из ваших требований вы, похоже, намерены писать C вместо C++, но я думаю, это ваше дело. Однако для вашей спецификации кажется, что правильный ответ strftime().

Answer 6

Я наполовину угадывая, что вы хотите, но я думаю, что это что-то вроде следующего, где я предполагающей или изменяющегося:

• ft действительно должен быть fd
• Я использовал ostringstream объект вместо ostrstream
• max_string_len является размер целевого буфера в символах (включая символ, который будет содержать «\ 0»)
• вы действительно не Inte отлаивается в отформатированной строке, отправляемой в std::cout, но хочу ее в предоставленном буфере вызывающего абонента
• Я изменил имя параметра string на s, поэтому его не путают с типом std:string.

Код:

void Time::FormatTime(char *s, unsigned int max_string_len) { 
    if (max_string_len == 0) { 
     return; // no buffer, bail out 
    } 

    std::ostringstream fd; 

    fd << hour << ":" << minutes; 

    size_t len = fd.str().copy(s, max_string_len - 1); // leave room for the null terminator 

    s[len] = '\0'; 
} 

Если вы действительно не хотите какой-либо части STL, то следующий C-стиле кода следует сделать трюк:

void Time::FormatTime(char *s, unsigned int max_string_len) { 

    snprintf(s, max_string_len, "%02u:%02u", hour, minutes); 
} 

Это проще , но разработчикам на C++ не нравится использовать семейство printf(), потому что он не является типичным.