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

[NewLamer&Programer]

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

п.с. особенно просьба к BDA посмотрите тему пожалуйста

[Bugrimov]

Т.е. советы других участников форума не интересуют???

[Базиля]

Если проблема с поиском, почему бы сам фрагмент поиска не выложить? Зачем нам нужен ваш проект целиком?

[Bugrimov]

Базиля замечание по существу.
Действительно взглянуть на реализацию поиска или автор считает что необходим весь проект для решения проблемы с поиском...

[NewLamer&Programer]

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

Т.е. советы других участников форума не интересуют???

интересуют просто BDA уже в теме, несколько раз с этой курсовой хорошо помог

по поводу поиска я просто предполагаю что ошибка может быть и в функции Menu использующей функцию Search, но ладно:

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 (PB[i].Family == Text) return i; else return NULL;
		case 2: if (PB[i].Name == Text) return i; else return NULL;
		case 3: if (PB[i].Patronymic == Text) return i; else return NULL;
		case 4: switch(SubField)
				{
					case 1: if (PB[i].Phone.Home == Text) return i; else return NULL;
					case 2: if (PB[i].Phone.Work == Text) return i; else return NULL;
					case 3: if (PB[i].Phone.Mobile == Text) return i; else return NULL;
				}
		case 5: switch(SubField)
				{
					case 1: if (PB[i].Address.Country == Text) return i; else return NULL;
					case 2: if (PB[i].Address.City == Text) return i; else return NULL;
					case 3: if (PB[i].Address.Street == Text) return i; else return NULL;
					case 4: if (PB[i].Address.N_Home == Text) return i; else return NULL;
					case 5: if (PB[i].Address.N_Apartment == Text) return i; else return NULL;
				}
		case 6: if (PB[i].Email == Text) return i; else return NULL;
	}
}

[Базиля]

А кто вам сказал, что таким образом можно сравнивать значения строк?
Для этого есть библиотечная функция strcmp.
В вашем же случае вы сравниваете именно указатели, а не сами значения ваших строк.
P.S. Это первое что бросилось в глаза.

[NewLamer&Programer]

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

А кто вам сказал, что таким образом можно сравнивать значения строк?
Для этого есть библиотечная функция strcmp.
В вашем же случае вы сравниваете именно указатели, а не сами значения ваших строк.
P.S. Это первое что бросилось в глаза.

ну на самом деле мне в этом месте уже про это говорили, но сошлись на том что как сейчас вроде правильно тем не менее... это даже работало... потом были просто еще другие изменения... ну ок, переписал, все равно не работает:

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; else return NULL;
		case 2: if (strcmp(PB[i].Name,Text) == 0) return i; else return NULL;
		case 3: if (strcmp(PB[i].Patronymic,Text) == 0) return i; else return NULL;
		case 4: switch(SubField)
				{
					case 1: if (strcmp(PB[i].Phone.Home,Text) == 0) return i; else return NULL;
					case 2: if (strcmp(PB[i].Phone.Work,Text) == 0) return i; else return NULL;
					case 3: if (strcmp(PB[i].Phone.Mobile,Text) == 0) return i; else return NULL;
				}
		case 5: switch(SubField)
				{
					case 1: if (strcmp(PB[i].Address.Country,Text) == 0) return i; else return NULL;
					case 2: if (strcmp(PB[i].Address.City,Text) == 0) return i; else return NULL;
					case 3: if (strcmp(PB[i].Address.Street,Text) == 0) return i; else return NULL;
					case 4: if (strcmp(PB[i].Address.N_Home,Text) == 0) return i; else return NULL;
					case 5: if (strcmp(PB[i].Address.N_Apartment,Text) == 0) return i; else return NULL;
				}
		case 6: if (strcmp(PB[i].Email,Text) == 0) return i; else return NULL;
	}
}

[Базиля]

ну на самом деле мне в этом месте уже про это говорили, но сошлись на том что как сейчас вроде правильно тем не менее... это даже работало..

Ну что значит "вроде правильно", там шло сравнение указателей, о какой правильности шла речь?

Вопрос можно?
Вот исходя из вашего кода, каков вообще смысл цикла в этой п/п?
При корректном значении MainField и SubField вы не дойдете даже до второй записи.

Далее, возвращать значение NULL из функции исходя из того, что строки не равны по значениям не есть корректно, даже просто потому что нумерация идет с того же 0.

Я полагаю вы используете возвращаемое значение функцией для индексации нужной записи из структуры.

Я предлагаю сменить прототип вашей функции на такой:

bool Search(TPhoneBook *PB, int COUNT, char Text[100], int MainField, int SubField, int &index);

Функция будет возвращать значение true если такая запись существует, соотв. в index будет содержаться ее индекс, иначе функция возвращает false.

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

Но опять таки, вся проблема у вас в том, о чем я писал изначально в этом сообщении.

[NewLamer&Programer]

Базиля,
1. про bool это тема, пожалуй сделаю
2. про цикл, в смысле какой смысл? в том то и дело что вы смотрите лучше проект, по одной функции мало что сделаешь, у меня есть массив структур, цикл проверяет все записи в массиве

[Базиля]

NewLamer&Programer

2. про цикл, в смысле какой смысл? в том то и дело что вы смотрите лучше проект, по одной функции мало что сделаешь, у меня есть массив структур, цикл проверяет все записи в массиве

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

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

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