Подсчитать количество одинаковых слов в строке

[Chainik!]

В чем ошибка?? Ругается на строку с функцией strcmp. Как исправить

#include "stdafx.h"
#include "iostream"
#include "conio.h"
using namespace std;
const int n = 100;
//-----------------------------------------------------------------
bool same_word(char *s)
{
	for(int i = 0; i < strlen(s); i++)
	{
		bool b = true;
		for(int j = i + 1; j < strlen(s); j++)
		{
			if(stcmp(s[i],s[j]))
				b = false;
		}
		if(!b)
			return true;
	}
	return false;
	
}
//-----------------------------------------------------------------
int _tmain(int argc, _TCHAR* argv[])
{
	char s[n] = "Hello word my name is", *p;
	int count = 0;
	/*cout << "Vvedite stroku: ";
	gets(s);*/
	p = strtok(s, " ");
	while(p)
	{
		if(same_word(p))
			count++;
		p = strtok(NULL," ");
	}
	cout << "\n\nKolichestvo odinakovix slov: " << count;
	_getch();
	return 0;
}

[Bugrimov]

Первое что режет глаз. Отсутствие хэдэр файла для работы со строками.

#include <cstring>

Не

if(stcmp(s[i],s[j]))

может вы имели в виду

if(strcmp(s[i],s[j]))

Зачем указывать размер строки (n)

char s[n] = "Hello word my name is",

напишите

char s[] = "Hello word my name is";

и все...
Еще к вам вопрос, что вы передаете в функцию

same_word(p)

.... по вашей задумке... ?????

[Chainik!]

Всё равно ругается компилятоор

[Igor95]

В strcmp требуется передавать адрес, а не значение.
s[i] - значение типа char.
s[i] годилось бы, если бы s было указателем на указатель. Тогда s[i] было бы адресом, хранящимся в указателе, на который указывает s.

&s[i] - указатель, т.е адрес, указывающий на ячейку памяти, где хранится значение типа char.
Только вот &s[i] - не строка. А функция strcmp будет искать символ окончания строки. Т.е передавать &s[i] в качестве параметра туда, где требуется строка небезопасно.

Подумайте, даже само имя функции strcmp говорит о том, что нужно передать адреса начала строк.

___________________________________ ___________________________________ __________

Да, и программа у вас на C, а подключается библиотека C++ - iostream. Компилятору-то все равно, а вот программистов этот факт будет смущать.
___________________________________ ___________________________________ __________

Насчет

if (strcmp(pFirstString, pSecondString))

Возвращаемое значение этой функции - 0, если строки равны. Т.е для if выполнится false.
Возвращаемое значние > 0, если первая строка больше второй. Т.е для if выполнится true.
Возвращаемое значение < 0, если вторая строка больше первой. Т.е для if выполнится true.

[Igor95]

Еще к вам вопрос, что вы передаете в функцию
Код:


same_word(p)

.... по вашей задумке... ?????

2Bugrimov: strtok(char*, const char*) возвращает указатель на char.

P.S - хотел прикрепить это сообщение к своему предыдущему. Проблема в том, что не пойму, как удалить это сообщение.

[Bugrimov]

Сообщение от Igor95

2Bugrimov: strtok(char*, const char*) возвращает указатель на char.

P.S - хотел прикрепить это сообщение к своему предыдущему. Проблема в том, что не пойму, как удалить это сообщение.

Передали вы указатель и что??????? Что с этим указателем должна делать функция.....
Кстати, в стандарт на Си уже входит булевый тип???????????? C++ и есть...

[Igor95]

Все, что нужно для решения задачи (в разумных пределах)
Или я чего-то не понял?
C не поддерживает ОО парадигмы

[Bugrimov]

Спрошу по другому

strlen(s)

- что возвращает функция????
Теперь мне все ясно!!!!!!!!!

[Igor95]

Сообщение от Bugrimov

Спрошу по другому

strlen(s)

- что возвращает функция????

Длину строки.
strlen(const char*) - требует указатель на char. Константа s является адресом начала массива и хранит его адрес. При передаче s в качестве аргумента компилятор неявно приводит его к указателю. Поэтому все будет отлично работать.

[BDA]

#include <iostream>
#include <string>
#include <sstream>
#include <algorithm>
#include <iterator>
#include <map>

using namespace std;

int
main()
{
    string sentence = "Hello word my name is word Word";
    istringstream iss(sentence);
    map<string, int> words;
    for_each(istream_iterator<string>(iss), istream_iterator<string>(), [&words](string i) {++words[i];});
    for_each(words.begin(), words.end(), [](pair<const string, int> &i) {cout << i.first << " - " << i.second << endl;});
}

C++11
Правда, я не понял, что значит "Подсчитать количество одинаковых слов в строке". Влияет ли одинаковый регистр или нет? По вышеприведенному коду получается, что если слово встречалось ранее, то нужно увеличить счетчик одинаковых слов, т.е., например, в строке "привет привет" 1 одинаковое слово.