Поиск по ключевым словам

@marina2 · Регистрация: 08.11.2009

Author24 — интернет-сервис помощи студентам

Я использую функцию

C++

1 2	if (_tcsstr(pBook->ptszWord, m_Find)=NULL) bFound = FALSE;

Можно так писать, просто у меня в этом месте происходит сбой.

Или можно по другому написать?

Евгений М. · 18.07.2010, 08:16

Думаю можно.
http://msdn.microsoft.com/en-u... S.71).aspx

Поправка:
Ой... Извиняюсь. Для TCHAR есть только _tcsstr.

@Том Ардер · 18.07.2010, 17:10

Не рекомендуется путать = и ==

a = b;

if( a == b )

@marina2 · 18.07.2010, 19:39 **[ТС]**

А может быть проблема не в коде, который я создала, а в коде Visual C++, или например в асемблере?

@alex_x_x · 18.07.2010, 19:48

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

А может быть проблема не в коде, который я создала, а в коде Visual C++, или например в асемблере?

да, с вероятностью 0,000001%

@M128K145 · 18.07.2010, 21:16

marina2, вам Том Ардер правильно указал на вашу ошибку

@marina2 · 19.07.2010, 04:42 **[ТС]**

Дело не в этом,я исправила.

Когда программа находит нужное слово, все происходит нормально, а на следующем компоненте происходит сбой.

@M128K145 · 19.07.2010, 08:18

marina2, так что за компонент? Код можно увидеть?

@marina2 · 19.07.2010, 17:29 **[ТС]**

Имеется список элементов книги. Производится поиск по ключевым словам. Пользователь ввел слово в Edit и поиск проходит в соответствующих полях.

C++

for(UINT nItem = 0; nItem < m_nBooksMapSize; nItem++)
{
    if(m_pBookdMap[nItem].pBookData)
    {
        pBook = m_pBookdMap[nItem].pBookData;
        if(pBook->dwParent)
        {
            pChapter = (PL_CHAPTER)pBook->dwParent;
            if(pChapter->ptszChapterDesc)
            {
                bFound = TRUE;
 
                if( pBook->ptszCaption && pFindData->ptszCaption)
                {
                    if(StrCmpNI(
                        pBook->ptszCaption, pFindData->ptszCaption, 
                        min(_tcslen(pBook->ptszCaption), 
                        _tcslen(pFindData->ptszCaption))) != 0)
                    //if(_tcsicmp(pBook->ptszCaption, pFindData->ptszCaption) != 0)
                    {
                        bFound = FALSE;
                                                
                    }
                }// if(bFound && pBook->ptszCaption && pFindData->ptszCaption)
 
                if(bFound && pFindData->ptszWord)
                {
                    CString m_Find=pFindData->ptszWord;
                    if (_tcsstr(pBook->ptszWord, m_Find)==NULL)
                        bFound = FALSE;
                }
            }
        }
    }
}

@M128K145 · 19.07.2010, 20:20

marina2, я так понимаю, что все переменные которые стоят в ифах - типа int или bool?
Сделайте проверку на NULL, вполне возможно что до этого вы где-то удалили объект

@marina2 · 19.07.2010, 21:17 **[ТС]**

Нет.

Вот структура списка.

Компонент книга

C

typedef struct stChapter
{
    BYTE btDistrictNum;
    BYTE btMode;
    TCHAR * ptszChapterDesc;
    PL_BOOK_DATA pBooksHead;
    BYTE btChanged;
    DWORD dwBooksCount;
    stChapter * pNext;
} L_CHAPTER, * PL_CHAPTER;
 
Элементы книги
 
typedef struct stBook
{
    TCHAR * ptszAuthor;
    TCHAR * ptszCaption;
    UINT uiYear;
    TCHAR * ptszEdition;
    TCHAR * ptszCity;
    UINT uiPages;
    BYTE btSex;
    UINT uiNumber;
    TCHAR * ptszPeriod;
    TCHAR * ptszAux;
    TCHAR * ptszWord;
    BYTE btChanged;
    stBook * pNext;
    LONG_PTR dwParent;
} L_BOOK_DATA, * PL_BOOK_DATA;

А вот проекция в памяти этого списка

C

typedef struct stBookToBaseMap
{
    PL_BOOK_DATA pBookData;
} D_BOOK_TO_BASE, * PD_BOOK_TO_BASE;

По шагам проверяла, ничего не удаляется

Программа грохается в этом месте

Assembler

  mov     al,[edi]            ; al is next char from str1
 
  add     esi,1               ; increment pointer into str1
 
  cmp     al,dl

Добавлено через 19 минут
Может это происходит из-за того, что много элементов в базе?

@M128K145 · 19.07.2010, 23:23

marina2, приаттачьте весь проект в архиве, я посмотрю

@marina2 · 20.07.2010, 03:37 **[ТС]**

Вот. Мне кажется, там что-то с памятью.

@Том Ардер · 21.07.2010, 14:58

Источник неприятностей с памятью может скрываться здесь (ИМХО):

C++

void CDistrictApp::RewriteCStringT(TCHAR * ptsBuffer, const TCHAR * ptsString, DWORD dwBufferLength)
{
....
            memset( ptsBuffer, 0, sizeof(TCHAR)*(dwBufferLength) );
            _tcsncpy_s(ptsBuffer, dwBufferLength, ptsString, nLength);
.....
}

ptsBuffer в такой конструкции может не содержать заключительного символа "конец строки" (== '\0'). Тогда в полях

C++

1 2	TCHAR * ptszAuthor; TCHAR * ptszCaption;

и подобных им могут оказаться "лишние" символы.

Необходимо: в отладчике поставить точки прерывания и проверить содержимое подобных текстовых полей.
А в этом фрагменте кода я бы добавил:

C++

1	*(ptsBuffer + nLength) = '\0';

@marina2 · 24.07.2010, 20:27 **[ТС]**

Ничего не выходит.

А что означает эти знаки: * &?

C++

1	TCHAR *& ptsBuffer

@Egeni · 24.07.2010, 20:33

знаки & и * означают взять адрес и получить число расположеное по адресу

@marina2 · 24.07.2010, 21:16 **[ТС]**

Когда они стоят рядом?

@CyBOSSeR · 24.07.2010, 22:35

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

Когда они стоят рядом?

Нет, когда они стоят рядом - это бред.
& - взятие адреса.
* - разыменование указателя.
Данная тема есть в любой книге по С++.

@marina2 · 31.07.2010, 13:19 **[ТС]**

*(ptsBuffer + nLength) = '\0'

Эта строчка совсем не компилируется. Когда по шагам проходишь код, компилятор его перепрыгивает.
А ведь все строчки оканчиваются нулевым символом.

@_Hellboy · 31.07.2010, 15:52

Не может быть, чтобы она не компилировалась)
Можно на крайний случай дописать *(char *)(ptsBuffer + nLength) = '\0'

Евгений М. 1080 / 1007 / 106 Регистрация: 28.02.2010 Сообщений: 2,889
	18.07.2010, 08:16	2
	Думаю можно. http://msdn.microsoft.com/en-u... S.71).aspx Поправка: Ой... Извиняюсь. Для TCHAR есть только _tcsstr. 1

@Том Ардер $Эксперт по математике/физике$ 4217 / 3412 / 396 Регистрация: 15.06.2009 Сообщений: 5,818
	18.07.2010, 17:10	3
	Не рекомендуется путать = и == a = b; if( a == b ) 0

@marina2 Рожденная для битвы 293 / 71 / 15 Регистрация: 08.11.2009 Сообщений: 1,265
	18.07.2010, 19:39 [ТС]	4
	А может быть проблема не в коде, который я создала, а в коде Visual C++, или например в асемблере? 0

@alex_x_x бжни 2473 / 1684 / 135 Регистрация: 14.05.2009 Сообщений: 7,162
	18.07.2010, 19:48	5
	Сообщение от marina2 А может быть проблема не в коде, который я создала, а в коде Visual C++, или например в асемблере? да, с вероятностью 0,000001% 0

@M128K145 8384 / 3617 / 419 Регистрация: 03.07.2009 Сообщений: 10,709
	18.07.2010, 21:16	6
	marina2, вам Том Ардер правильно указал на вашу ошибку 0

@marina2 Рожденная для битвы 293 / 71 / 15 Регистрация: 08.11.2009 Сообщений: 1,265
	19.07.2010, 04:42 [ТС]	7
	Дело не в этом,я исправила. Когда программа находит нужное слово, все происходит нормально, а на следующем компоненте происходит сбой. 0

@M128K145 8384 / 3617 / 419 Регистрация: 03.07.2009 Сообщений: 10,709
	19.07.2010, 08:18	8
	marina2, так что за компонент? Код можно увидеть? 0

@M128K145 8384 / 3617 / 419 Регистрация: 03.07.2009 Сообщений: 10,709
	19.07.2010, 20:20	10
	marina2, я так понимаю, что все переменные которые стоят в ифах - типа int или bool? Сделайте проверку на NULL, вполне возможно что до этого вы где-то удалили объект 0

@M128K145 8384 / 3617 / 419 Регистрация: 03.07.2009 Сообщений: 10,709
	19.07.2010, 23:23	12
	marina2, приаттачьте весь проект в архиве, я посмотрю 0

@Egeni 10 / 10 / 2 Регистрация: 17.06.2010 Сообщений: 107 Записей в блоге: 1
	24.07.2010, 20:33	16
	знаки & и * означают взять адрес и получить число расположеное по адресу 0

@marina2 Рожденная для битвы 293 / 71 / 15 Регистрация: 08.11.2009 Сообщений: 1,265
	24.07.2010, 21:16 [ТС]	17
	Когда они стоят рядом? 0

@CyBOSSeR 2348 / 1721 / 149 Регистрация: 06.03.2009 Сообщений: 3,675
	24.07.2010, 22:35	18
	Сообщение от marina2 Когда они стоят рядом? Нет, когда они стоят рядом - это бред. & - взятие адреса. * - разыменование указателя. Данная тема есть в любой книге по С++. 1

@marina2 Рожденная для битвы 293 / 71 / 15 Регистрация: 08.11.2009 Сообщений: 1,265
	31.07.2010, 13:19 [ТС]	19
	*(ptsBuffer + nLength) = '\0' Эта строчка совсем не компилируется. Когда по шагам проходишь код, компилятор его перепрыгивает. А ведь все строчки оканчиваются нулевым символом. 0

@_Hellboy 5 / 5 / 0 Регистрация: 14.07.2010 Сообщений: 17
	31.07.2010, 15:52	20
	Не может быть, чтобы она не компилировалась) Можно на крайний случай дописать (char )(ptsBuffer + nLength) = '\0' 0