Почему не удается передать строковой литерал в функцию, принимающую char*?

Question

[Vizik]

Код:

#include <iostream>
using namespace std;

char *get_substr(char *sub, char *str);

int main() {
	setlocale(LC_ALL, "RUS");
	char * substr;

	substr = get_substr("один", "два три четыре");
	cout << "Подстрока найдена: " << substr;

	return 0;
}

char *get_substr(char *sub, char *str) {
	int t;
	char *p, *p2, *start;
	
	for(t = 0; str[t]; t++) {
		p = &str[t];
		start = p;
		p2 = sub;
		while (*p2 && *p2 == *p) {
			p++;
			p2++;
		}
		if (!*p2)
		return start;
	}
	
	return 0;
}

Comments

[Roman]

что делать?

а что нужно?

https://en.cppreference.com/w/cpp/algorithm/search

Хотя видимо ничего...
Треш и Хардкор

get_substr(const_cast<char*>("один"), const_cast<char*>("два три четыре"));

[Евгений Шатунов]

Vizik , для начала нужно определиться зачем тебе нужен именно char* и почему ты не можешь работать конкретно с типом const char*?

[Roman]

Евгений Шатунов, Потому что копипаста.
www.cyberforum.ru/cpp-beginners/thread50536.html
Этот код гуляет по интернету в той или чуть иной модификации лет 10.

[Евгений Шатунов]

Roman , флуд, конечно, и удалят, но... напомнило =)

[RabraBabr]

Roman, Нашел первоисточник, точно это Шилдт. Тока глянул сразу понял - злобнокодинг в его духе (знакомый подчерк). В гугле нашел https://books.google.ru/books?id=WoTEdWshaWAC&pg=P...

Я так думаю этому безобразию не 10 лет, а так все 30.

[Roman]

RabraBabr,

The C programming language - First Edition / Язык Си первая редакция
Год издания: 1978
Автор: Kerninghan, Brian W.
Издательство: Bell Telephone Laboratories
ISBN: 0-13-110163-3
Язык: Английский

Картинка

[RabraBabr]

Roman, теперь понятно откуда уши растут. Но в современном мире неокрепшим умам лучше этого не видеть.

Answer 1

[Евгений Шатунов]

Все дело в том, что строковой литерал в C++ имеет константный тип.
Когда ты делаешь вызов get_substr("один", "два три четыре"), ты в функцию передаешь два строковых литерала, тип которых - const char[ N ], где N - это длина строки литерала включая терминальный символ '\0'.
Неявное преобразование типа из const char* в char* недопустимо, поэтому компилятор и пишет тебе ошибку.

На самом деле, тебе в твоем тексте char* вообще не нужен, т.к. полностью все операции у тебя не приводят к изменению состояния строки. Заменить char* на const char* будет и логичнее, и понимаемость кода тоже улучшит.

Answer 2

[Roman]

А еще сто раз говорили, что копипаста это зло.
Надо разорвать этот порочный круг.

Копирайт
The C programming language 1978 Kerninghan, Brian W.

#include <stdio.h>

int index(char[], char[]);

int main()
{
  char* str = "foo bar baz";
  char* str2 = "barz";
  
  if(index(str, str2) != -1)
    printf("found");
  else
    printf("not found");
  return 0;
}

int index(char s[], char t[])
{
  int i, j, k;
  for (i = 0; s[i] != '\0'; i++)
  {
    for (j = i, k = 0; t[k] != '\0' && s[k] == t[k]; j++, k++)
      ;
    if(t[k] == '\0')
      return i;
  }
  return -1;
}