c_str () строка - все остальные строки, кроме первой, теряются

Question

c_str () строка - все остальные строки, кроме первой, теряются

0

У меня есть строка, которая содержит несколько строк текста.

Например,

f.html                            Sat Oct 19 22:59:47 2013         23675
folder                            Mon Nov  4 19:36:14 2013         4096
readdirpractice.cpp               Tue Nov  5 03:00:10 2013         1203
server.cpp                        Mon Nov  4 21:22:27 2013         11369
photo.jpg                         Wed Oct 23 01:45:04 2013         4360
qq                                Sun Nov  3 01:54:36 2013         66031
server.cpp~                       Mon Nov  4 21:22:25 2013         11368
myhttp.cpp                        Sun Nov  3 01:43:09 2013         1816
getoptpractice.cpp~               Sun Nov  3 01:15:25 2013         1324

Это то, как обычно выглядит строка.

Когда я преобразовал этот stringstream в строку, информация была такой же.

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

Он имеет только

f.html                            Sat Oct 19 22:59:47 2013         23675

после преобразования.

Почему это и как я могу это исправить?

Я прикрепляю часть моего кода.

if(is_dir) {
    char dirname[1024];
    strncpy(dirname, requests[1].c_str(), sizeof(dirname));
    dirname[sizeof(dirname)-1] = 0;

    DIR           *d;
    struct dirent *dir;
    d = opendir(dirname);
    stringstream ss;
    if (d) {
        while ((dir = readdir(d)) != NULL) {
            char* file = dir->d_name;
            if(file[0] != '.') {
                struct stat sb;

                if (stat(file, &sb) == -1) {
                    cerr << "stat error" << endl;
                    exit(EXIT_FAILURE);
                }

                char* lm = ctime(&sb.st_mtime);
                string lastmod(lm);
                lastmod.at(lastmod.size()-1) = '\0';
                string spacing = "                                  ";
                ss << file << spacing.substr(0, spacing.size() - strlen(file)) << lastmod << spacing.substr(0, spacing.size() - lastmod.size()) << sb.st_size << '\n';
            }
        }
        closedir(d);
    }
    //cout << ss.str() << endl; // for testing
    char msg2[10000];
    strncpy(msg2, ss.str().c_str(), sizeof(msg2));
    msg2[sizeof(msg2)-1] = 0;
    msg = msg2;

Eric Na 05 нояб. 2013, в 10:14

Источник

0

@kfsone stream << std::endl эквивалентен stream << '\n' << std::flush . Почему OP должен использовать std::endl ?
milleniumbug 05 нояб. 2013, в 08:55
0

Что такое "сообщение"? Какова цель "msg2"? Является ли msg std :: string? Если это так, избавьтесь от msg2 и используйте msg.assign(ss.str().c_str(), std::min(ss.str().size(), 10000);
kfsone 05 нояб. 2013, в 08:55
0

endl выполняет сброс, который обычно не нужен. Используется для файла, он вызывает системный вызов (ядро). Это не очень хорошая практика.
Potatoswatter 05 нояб. 2013, в 08:58
0

msg определяется над if(is_dir) как char *msg . Это возвращаемое значение.
Eric Na 05 нояб. 2013, в 08:59
0

«Это возвращаемое значение» №, его неопределенное поведение . Если это функция, возвращающая адрес в msg , вы возвращаете адрес буфера символов ( msg2 ), который собирается исчезнуть.
WhozCraig 05 нояб. 2013, в 09:00
0

Да, endl vs '\ n' должен был иметь знак вопроса в конце. Вот почему это был комментарий, а не ответ ;-P
kfsone 05 нояб. 2013, в 09:01
0

@WhozCraig Как я могу использовать msg в моем коде, чтобы заменить msg2 ? Могу ли я напрямую передать msg без выделения пробелов (т.е. без части msg2[10000] )
Eric Na 05 нояб. 2013, в 09:06
0

@user2418202 user2418202 Отправьте его в качестве параметра вашей функции. Сделайте так, чтобы вызывающая сторона дала вам адрес буфера возврата, а также размер. Единственными другими параметрами являются статический (фу) или динамический (выполнимый, но вызывающий должен освободить его позже). Хорошо. общеизвестно, что глобал тоже работает, но это даже страшнее, чем статический.
WhozCraig 05 нояб. 2013, в 09:06
0

Могу ли я сделать что-то вроде msg = char [10000]?
Eric Na 05 нояб. 2013, в 09:09
0

Просто понял, что не могу, но что-то похожее?
Eric Na 05 нояб. 2013, в 09:10
0

Есть причина, по которой вы не можете позволить вызывающей стороне передавать буфер соответствующего размера в качестве параметра. Т.е. void myFunc(char result[], size_t maxlen) и использовать этот буфер и информацию о размере в качестве цели? Честно говоря, я бы, вероятно, malloc() решение malloc() для этого конкретного экземпляра только потому, что содержимое может быть таким динамичным. Как я думаю, буфер вызывающих программ может быть недостаточно большим, и у вас остается мало выбора, кроме как информировать их через возвращаемое значение как таковое. То, что вы делаете, это ваш выбор, но то, что вы делаете сейчас , не является правильным решением.
WhozCraig 05 нояб. 2013, в 09:11
0

Причина в том, что есть else часть, которая является более сложной. Я динамически выделил для него пробелы в else части (я скопировал некоторый пример кода). Это тоже не рекомендуется?
Eric Na 05 нояб. 2013, в 09:14
1

Вместо того, чтобы выделять динамическую память, почему бы не использовать класс, который инкапсулирует динамическую строковую память, такую как std::string . Заставьте вашу функцию принимать std::string& msg чтобы вызывающая сторона предоставляла std :: string, а затем все, что ваша функция должна сделать, это заполнить ее. (см. пример в моем ответе)
kfsone 05 нояб. 2013, в 09:19
0

@kfsone Одна из причин заключается в том, что это функция C-интерфейса, реализованная в библиотеке C ++, вызываемая из кода C, но я согласен, что в идеале наилучшим решением будет просто возврат std::string .
WhozCraig 05 нояб. 2013, в 09:29
0

@WhozCraig Это правда, но он уже использует sstream в коде, переключение на строку поможет ему минимизировать контактную поверхность с C-строками.
kfsone 05 нояб. 2013, в 18:25
0

@kfsone возможно я не правильно сказал. Он, безусловно, может делать все, что захочет, в теле этой вещи. Дело в том что я делал просто , что есть причины , доступ-интерфейс в этой функции может и не быть разрешено иметь измы C ++. То, что происходит в Вегасе, безусловно, может остаться в Вегасе, но способ добраться туда и обратно - это другая проблема, и это может быть одной из причин, почему ОП делают это таким образом.
WhozCraig 05 нояб. 2013, в 18:38
0

@WhozCraig Понятно - в моем предыдущем комментарии, вероятно, отсутствует слово «вместо». Я не из тех, кто похож на С! охотники на ведьм :)
kfsone 05 нояб. 2013, в 18:46

Показать ещё 15 комментариев

Теги:

c++

string

c

3 ответа

1

Эти строки выглядят подозрительными:

string lastmod(lm);
lastmod.at(lastmod.size()-1) = '\0';

Вставка нулевого символа в середину экземпляра класса строки, вероятно, создает неожиданное поведение (например, преждевременное завершение строки при печати).

Я не думаю, что вам нужно это сделать. Просто попробуйте удалить строку lastmod.at.

selbie 05 нояб. 2013, в 06:46

1

В соответствии с комментариями вы указали, что "msg" - это возвращаемое значение.

char msg2[10000];
strncpy(msg2, ss.str().c_str(), sizeof(msg2));
msg2[sizeof(msg2)-1] = 0;
msg = msg2;

"msg2" создается как локальная переменная, то есть в стеке. Затем вы копируете свой струнный поток в эту область стека, беря адрес строки (в стеке), а затем оставляете свою функцию, после чего эта область стека освобождается и используется другими частями вашего приложения.

Вам потребуется либо выделить память в куче для строки, либо вам нужно будет передать хранилище из вызывающей функции.

Поскольку вы уже выполняете строковые манипуляции, использование std :: string для инкапсуляции вне служебных накладных расходов управления будет наиболее логичным. В качестве аргумента возьмите ссылку на стандартную строку:

void getDirListing(std::string& msg) {
}

и затем позвоните

std::string dirListing = "";
getDirListing(dirListing);

kfsone 05 нояб. 2013, в 06:11

Ещё вопросы

@kfsone stream << std::endl эквивалентен stream << '\n' << std::flush . Почему OP должен использовать std::endl ?
Что такое "сообщение"? Какова цель "msg2"? Является ли msg std :: string? Если это так, избавьтесь от msg2 и используйте msg.assign(ss.str().c_str(), std::min(ss.str().size(), 10000);
endl выполняет сброс, который обычно не нужен. Используется для файла, он вызывает системный вызов (ядро). Это не очень хорошая практика.
msg определяется над if(is_dir) как char *msg . Это возвращаемое значение.
«Это возвращаемое значение» №, его неопределенное поведение . Если это функция, возвращающая адрес в msg , вы возвращаете адрес буфера символов ( msg2 ), который собирается исчезнуть.
Да, endl vs '\ n' должен был иметь знак вопроса в конце. Вот почему это был комментарий, а не ответ ;-P
@WhozCraig Как я могу использовать msg в моем коде, чтобы заменить msg2 ? Могу ли я напрямую передать msg без выделения пробелов (т.е. без части msg2[10000] )
@user2418202 user2418202 Отправьте его в качестве параметра вашей функции. Сделайте так, чтобы вызывающая сторона дала вам адрес буфера возврата, а также размер. Единственными другими параметрами являются статический (фу) или динамический (выполнимый, но вызывающий должен освободить его позже). Хорошо. общеизвестно, что глобал тоже работает, но это даже страшнее, чем статический.
Могу ли я сделать что-то вроде msg = char [10000]?
Просто понял, что не могу, но что-то похожее?
Есть причина, по которой вы не можете позволить вызывающей стороне передавать буфер соответствующего размера в качестве параметра. Т.е. void myFunc(char result[], size_t maxlen) и использовать этот буфер и информацию о размере в качестве цели? Честно говоря, я бы, вероятно, malloc() решение malloc() для этого конкретного экземпляра только потому, что содержимое может быть таким динамичным. Как я думаю, буфер вызывающих программ может быть недостаточно большим, и у вас остается мало выбора, кроме как информировать их через возвращаемое значение как таковое. То, что вы делаете, это ваш выбор, но то, что вы делаете сейчас , не является правильным решением.
Причина в том, что есть else часть, которая является более сложной. Я динамически выделил для него пробелы в else части (я скопировал некоторый пример кода). Это тоже не рекомендуется?
Вместо того, чтобы выделять динамическую память, почему бы не использовать класс, который инкапсулирует динамическую строковую память, такую как std::string . Заставьте вашу функцию принимать std::string& msg чтобы вызывающая сторона предоставляла std :: string, а затем все, что ваша функция должна сделать, это заполнить ее. (см. пример в моем ответе)
@kfsone Одна из причин заключается в том, что это функция C-интерфейса, реализованная в библиотеке C ++, вызываемая из кода C, но я согласен, что в идеале наилучшим решением будет просто возврат std::string .
@WhozCraig Это правда, но он уже использует sstream в коде, переключение на строку поможет ему минимизировать контактную поверхность с C-строками.
@kfsone возможно я не правильно сказал. Он, безусловно, может делать все, что захочет, в теле этой вещи. Дело в том что я делал просто , что есть причины , доступ-интерфейс в этой функции может и не быть разрешено иметь измы C ++. То, что происходит в Вегасе, безусловно, может остаться в Вегасе, но способ добраться туда и обратно - это другая проблема, и это может быть одной из причин, почему ОП делают это таким образом.
@WhozCraig Понятно - в моем предыдущем комментарии, вероятно, отсутствует слово «вместо». Я не из тех, кто похож на С! охотники на ведьм :)

Potatoswatter · Accepted Answer · 2013-11-05T06-33-00.000Z

std::string вероятно, имеет встроенные символы '\0' (NUL), которые преждевременно завершают последовательность char, когда она интерпретируется как строка C.

Попробуйте std::remove NUL байты с помощью std::remove.

Изменить - это может быть вашей проблемой:

             lastmod.at(lastmod.size()-1) = '\0';

Это просто просит неприятностей. std::string точно хранит ваш байт NUL и копирует его в выходной файл.

+1 и @ user2418202 примите во внимание мой комментарий в общих комментариях выше. Если вы думаете, что это «работает», исправляя это, но возвращаете указатель на локальный буфер в функции, подумайте еще раз.
Но я не могу думать о том, как я собираюсь делать то, что вы говорите. Я определенно проведу некоторое время позже, но это не так просто
@ user2418202 Это так. это просто немного больше работы. Может показаться много, если вы не знакомы с тем, как, но, как и все, с практикой становится легче.