Проблема с поиском. С

[NewLamer&Programer]

Сообщение от Базиля

NewLamer&Programer

В Цикле НЕ проверяются все записи.
В этом и вся проблема.

Не вижу смысла смотреть проект, если уже видно что функция работает некорректно.

Даже могу угадать, функция возвращает всегда 1-ую запись, так?

не угадал, всегда последнюю, и тогда объясните чем же вам так не нравиться проверка в цикле и как по вашему надо проверять...

[Базиля]

Функция не может возвращать последнюю запись.
Она всегда будет возвращать 0 индекс, т.е. первую запись.

Читаем назначение инструкции return и все вопросы отпадут.

[NewLamer&Programer]

а вот таки возвращает, я же говорю, возможно дело в функции menu, использующей функцию Search, да посмотрите вы уже весь код... кстати до меня доперло что за бред про bool, в том то и смысл что мне надо возвращать не истина ложь, а номер найденной записи, просто NULL это я пытался как то обозначить то что запись не найдена, я не знал что это и есть 0, да думаю выход использовать отрицательное значение, и наконец опять же повторюсь ЧЕМ ВАМ НЕ НРАВИТЬСЯ ЦИКЛ??? почему по вашему он будет всегда возвращать первую запись???

[BDA]

Вопрос:

Сообщение от NewLamer&Programer

почему по вашему он будет всегда возвращать первую запись???

Ответ:

Сообщение от Базиля

Читаем назначение инструкции return и все вопросы отпадут.

return

[NewLamer&Programer]

BDA, не врубаюсь, да return осуществляет выход из функции, но если только попадется по условию ! а оно не попадется до тех пор пока не будет найдена (или не найдена) запись

п.с. да про "else return NULL" это лишнее, модифицировал так, но теперь все время пишит запись не найдена (то есть возвращает -1)

int Search(TPhoneBook *PB, int COUNT, char Text[100], int MainField, int SubField)
{
	for (int i = 0; i < COUNT-1; i++)
	switch(MainField)
	{
		case 1: if (strcmp(PB[i].Family,Text) == 0) return i;
		case 2: if (strcmp(PB[i].Name,Text) == 0) return i;
		case 3: if (strcmp(PB[i].Patronymic,Text) == 0) return i;
		case 4: switch(SubField)
				{
					case 1: if (strcmp(PB[i].Phone.Home,Text) == 0) return i;
					case 2: if (strcmp(PB[i].Phone.Work,Text) == 0) return i;
					case 3: if (strcmp(PB[i].Phone.Mobile,Text) == 0) return i;
				}
		case 5: switch(SubField)
				{
					case 1: if (strcmp(PB[i].Address.Country,Text) == 0) return i;
					case 2: if (strcmp(PB[i].Address.City,Text) == 0) return i;
					case 3: if (strcmp(PB[i].Address.Street,Text) == 0) return i;
					case 4: if (strcmp(PB[i].Address.N_Home,Text) == 0) return i;
					case 5: if (strcmp(PB[i].Address.N_Apartment,Text) == 0) return i;
				}
		case 6: if (strcmp(PB[i].Email,Text) == 0) return i;
	}
	return -1;
}

[BDA]

1) После каждого case нужен break, иначе будет сравнение и во всех последующих case:

case 1: if (strcmp(PB[i].Family,Text) == 0) return i;
break;
case 2: if (strcmp(PB[i].Name,Text) == 0) return i;
break;

http://msdn.microsoft.com/ru-ru/library/k0t5wee3.aspx
2) Может оказаться так, что text и поле действительно содержат разный текст (символы переноса строки, например).

[NewLamer&Programer]

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

1) После каждого case нужен break, иначе будет сравнение и во всех последующих case:

case 1: if (strcmp(PB[i].Family,Text) == 0) return i;
break;
case 2: if (strcmp(PB[i].Name,Text) == 0) return i;
break;

зачем? ну и пусть сравнивает во всех case, так и было запланировано как бы... если найдет то выйдет из функции совсем, а не найдет продолжить сравнивать, это логично...

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

2) Может оказаться так, что text и поле действительно содержат разный текст (символы переноса строки, например).

если это намек на добавление ветки default, то насчет MainField можете не беспокоится, туда никогда не попадет ничего кроме этих чисел, а Text используется только в сравнении...

[BDA]

Сообщение от NewLamer&Programer

зачем? ну и пусть сравнивает во всех case, так и было запланировано как бы... если найдет то выйдет из функции совсем, а не найдет продолжить сравнивать, это логично...

Вы предлагаете пользователю выбрать, по какому параметру осуществлять поиск. Если не сделать break, то поиск будет осуществляться не только по 1 параметру, но и по всем последующим.

Сообщение от NewLamer&Programer

если это намек на добавление ветки default, то насчет MainField можете не беспокоится, туда никогда не попадет ничего кроме этих чисел, а Text используется только в сравнении...

Это не намек на ветку default. "Привет" и "Привет\n" - разные строки. Для тестирования добавьте 1 запись в массив, затем в функции поиска побайтово выведите Text и поле, с которым собираетесь сравнивать.

[NewLamer&Programer]

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

Вы предлагаете пользователю выбрать, по какому параметру осуществлять поиск. Если не сделать break, то поиск будет осуществляться не только по 1 параметру, но и по всем последующим.

имхо бред, как он будет по следующим осуществляться, если MainField допустим равен 2, то он глянет - ага равен 1, если нет, то продолжит, равно ли 2? да, значит значит надо сравнить, если тексты не равны, он продолжит, MainField = 3? если нет продолжит 4 и т.д...

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

Это не намек на ветку default. "Привет" и "Привет\n" - разные строки. Для тестирования добавьте 1 запись в массив, затем в функции поиска побайтово выведите Text и поле, с которым собираетесь сравнивать.

если у меня окажутся разные строки то он и вернет значение -1, что не понятно то? другой вопрос что такие символы как \n в строке все равно не будут присутствовать, т.к. строка вводиться с клавиатуры, откуда они там возьмутся, вот ввел ты и нажал Enter, через scanf строка считалась, откуда там такие символы возьмуться?

[BDA]

Сообщение от NewLamer&Programer

имхо бред

Мое имхо - подтягивайте теорию. switch

Сообщение от NewLamer&Programer

если у меня окажутся разные строки то он и вернет значение -1, что не понятно то?

Так Ваша жалоба в том, что он должен найти запись, а он не находит. Мое предположение - строки различны, хотя Вы предполагаете, что одинаковы.