Как объявить пустой символ * и динамически увеличить его размер?

Question

Как объявить пустой символ * и динамически увеличить его размер?

0

Скажем, я пытаюсь сделать следующее (это подпункт того, чего я пытаюсь достичь):

int compareFirstWord(char* sentence, char* compareWord){
      char* temp; int i=-1;
      while(*(sentence+(++i))!=' ') { *(temp+i) = *(sentence+i); }
      return strcmp(temp, compareWord); }

Когда я побежал compareFirstWord("Hi There", "Hi"); , Я получил ошибку на линии копирования. Он сказал, что я использовал temp uninitialized. Затем я использовал char* temp = new char[]; В этом случае функция вернула 1, а не 0. Когда я отлаживался, я видел темп, начинающийся с некоторых случайных символов длиной 16, и strcmp терпит неудачу из-за этого.

Есть ли способ объявить пустой символ * и увеличить размер динамически только до длины и содержимого того, что мне нужно? Любой способ заставить функцию работать? Я не хочу использовать std::string.

anna.sarp 12 июль 2014, в 01:19

Источник

7

Используйте std::string .
chris 11 июль 2014, в 23:16
3

Тогда используйте std::vector .
chris 11 июль 2014, в 23:26
2

Почему вы не хотите использовать std::string ? Если вы на самом деле не можете использовать его достаточно справедливо, но если вы можете использовать его и просто не хотите, я думаю, вы должны добавить символ в строку с помощью str.push_back(a_char) вы можете зарезервировать для строка с str.reserve(size) и проверка выделенного пространства с помощью str.capacity() а затем str.shrink_to_fit() уменьшает емкость строк, чтобы она соответствовала фактическому str.size() который можно установить / сбросить с помощью str.resize(size) , если вам нужен char * эквивалент строки, вы можете просто вызвать str.c_str() и очистить строку с помощью str.clear()
James 12 июль 2014, в 01:20
0

Спасибо за очень четкое объяснение!
anna.sarp 12 июль 2014, в 23:56

Показать ещё 2 комментария

Теги:

c++

c

char

strcmp

dynamic-memory-allocation

2 ответа

0

temp - неинициализированная переменная.

Похоже, вы пытаетесь извлечь первое слово из предложения в вашем цикле.

Чтобы сделать это таким образом, вам сначала нужно будет инициализировать temp как минимум до тех пор, пока ваше предложение.

Кроме того, ваше предложение может не иметь места в нем. (Что относительно периода, \t,\r,\n? Имеют ли это значение?)

Кроме того, вы должны завершить temp с нулевым символом.

Вы можете попробовать:

int len = strlen(sentence);
char* temp = new char[len + 1];
int i = 0;

while(i < len && *(sentence+(i))!=' ') {
   *(temp+i) = *(sentence+i); 
   i++;
}
*(temp+i) = '\0';
int comparable = strcmp(temp, compareWord);
delete temp;
return comparable;

Также рассмотрите возможность использования isspace (* (sentence+ (i))), который, по крайней мере, поймает все пробелы. В общем, однако, я бы использовал библиотеку или STL... Зачем изобретать колесо...

jazeee 11 июль 2014, в 22:24

0

После инициализации temp вы также можете использовать strcpy, а затем установить первый пробел с помощью \ 0
jazeee 11 июль 2014, в 23:27
0

В зависимости от того, доверяете ли вы источнику своих предложений, имейте в виду, что они могут быть не в правильной форме, с нулевым терминатором. Это может привести к переполнению буфера или другим проблемам. Я считаю, что strlen требует строки с нулевым символом в конце.
jazeee 11 июль 2014, в 23:28
1

Кстати, у вас есть утечка памяти.
Jarod42 11 июль 2014, в 23:31
0

Да, извините, я забыл, не забудьте удалить temp в конце функции перед возвратом.
jazeee 11 июль 2014, в 23:34
1

[] отсутствуют после delete , это должно быть delete [] temp;
Jarod42 11 июль 2014, в 23:37
0

Спасибо @jazeee, главная ошибка - не заканчиваться на '\ 0'.
anna.sarp 11 июль 2014, в 23:38
1

Извините - это ужасный ответ. Никогда не используйте это в реальном коде. Это подвержено ошибкам. Единственное, что искупает этот ответ, это последнее предложение, но я готов поспорить, что OP игнорирует это.
Konrad Rudolph 11 июль 2014, в 23:39
0

Я склонен согласиться с тем, что код не очень хорош, на самом деле. Я пытался сохранить стиль OP.
jazeee 12 июль 2014, в 00:05

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

Ещё вопросы

Почему вы не хотите использовать std::string ? Если вы на самом деле не можете использовать его достаточно справедливо, но если вы можете использовать его и просто не хотите, я думаю, вы должны добавить символ в строку с помощью str.push_back(a_char) вы можете зарезервировать для строка с str.reserve(size) и проверка выделенного пространства с помощью str.capacity() а затем str.shrink_to_fit() уменьшает емкость строк, чтобы она соответствовала фактическому str.size() который можно установить / сбросить с помощью str.resize(size) , если вам нужен char * эквивалент строки, вы можете просто вызвать str.c_str() и очистить строку с помощью str.clear()
Спасибо за очень четкое объяснение!
После инициализации temp вы также можете использовать strcpy, а затем установить первый пробел с помощью \ 0
В зависимости от того, доверяете ли вы источнику своих предложений, имейте в виду, что они могут быть не в правильной форме, с нулевым терминатором. Это может привести к переполнению буфера или другим проблемам. Я считаю, что strlen требует строки с нулевым символом в конце.
Да, извините, я забыл, не забудьте удалить temp в конце функции перед возвратом.
[] отсутствуют после delete , это должно быть delete [] temp;
Спасибо @jazeee, главная ошибка - не заканчиваться на '\ 0'.
Извините - это ужасный ответ. Никогда не используйте это в реальном коде. Это подвержено ошибкам. Единственное, что искупает этот ответ, это последнее предложение, но я готов поспорить, что OP игнорирует это.
Я склонен согласиться с тем, что код не очень хорош, на самом деле. Я пытался сохранить стиль OP.

Jarod42 · Accepted Answer · 2014-07-11T20-54-00.000Z

В C вы можете:

int compareFirstWord(const char* sentence, const char* compareWord)
{
    while (*compareWord != '\0' && *sentence == *compareWord) {
        ++sentence;
        ++compareWord;
    }
    if (*compareWord == '\0' && (*sentence == '\0' || *sentence == ' ')) {
        return 0;
    }
    return *sentence < *compareWord ? -1 : 1;
}

С std::string вы просто имеете:

int compareFirstWord(const std::string& sentence, const std::string& compareWord)
{
    return sentence.compare(0, sentence.find(" "), compareWord);
}