Как улучшить код сравнения строк?

Question

[re3r0]

#include <stdio.h>

int compare_word(char *word_one, char *word_two){
  for (int i = 0; (*(word_one + i) != '\0') || (*(word_two + i) != '\0'); ++i)
    if (*(word_one + i) != *(word_two + i))
      return 0;

  return 1;
}


int main(){

	char word_one[] = "helloworld";
	char word_two[] = "helloworld !";

  if (compare_word(&word_one[0], &word_two[0]))
    printf("%s\n", "all is ok !");
  else
    printf("%s\n", "all is not ok !");
  return 0;
}

Либо скажите что дальше улучшать некуда и моя душа успокоится.

Comments

[LoliDeveloper]

У вас может быть сегфолт из-за разности длин строк

Answer 1

[CityCat4]

Ну посмотри уже исходник strcmp. Потому что у тебя если первая строка например "I love toster and spent here many hours" а вторая "I love toster" - то ты выхватываешь SIGSEGV, потому что первая строка еще идет, а вторая - уже кончилась (а для проверки условия завершения цикла нужно вычислить оба условия)

Comments

[re3r0]

нене цикл повторяется пока одна из двух строк не кончится так что все должно прекрасно работать (лично у меня работает, и с вашим примером SIGSEGV не вылетает)

(*(word_one + i) != '\0') || (*(word_two + i) != '\0') это условие собственно и нужно что бы не сесть на бутылку

(посмотрел я исходник strcmp и маслину поймал от непонимание что там творится)

[CityCat4]

re3r0, Да там все просто как полено и вторая строка вообще не проверяется. Взяли символы по переданным указателям, указатели постинкрементом сдвинули. Если первая строка кончилась, вернуть разницу между первым и вторым символами (считая их беззнаковыми целыми). Если нет, то крутить цикл дальше, пока указатели указывают на одинаковые символы. strcmp() считается небезопасной и вполне может привести к SIGSEGV, поэтому я вот обычно использую strncmp() (сравнение по фиксированной длине)