с++ syntax error

[Tomson]

Или я правильно ошибаюсь?

ASSERT((IDM_ABOUTBOX &amt; 0xFFF0) == IDM_ABOUTBOX);

syntax error : ')'

if(filename.Left(2).CompareNoCase("a:")!=0 &amt;&amt; filename.Left(2).CompareNoCase("b:")!=0)

syntax error : missing ')' before ';'

[Алексей++]

"& a m t ;" - это какой то символ заменился при вставке в пост

[Tomson]

amp?)

[marat_]

--

[marat_]

а где предыдущий ответ?

[Алексей++]

marat_, я ничего не удалял...

Tomson, там же написано syntax error : missing ')' before ';' , без кода не понятно, где то синтаксическая ошибка

[Tomson]

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

[Алексей++]

Tomson, а что за amp , amt ? )))
может проще так:
ASSERT((IDM_ABOUTBOX & 0xFFF0) == IDM_ABOUTBOX);

if(filename.Left(2).CompareNoCase("a:")!=0 && filename.Left(2).CompareNoCase("b:")!=0)

[Tomson]

хм, да, для этого случая помогло, но там такое не один раз

[Tomson]

тут тоже
virtual BOOL SelChanged(LPCSTR lpcsSelection, CString&amt; csStatusText;)
 syntax error : missing ')' before ';'
что бы это могло быть?

[Вад]

Tomson, а почему ";" перед скобкой стоит? =)

[RXL]

Вад, а он, видимо, копипастит из HTML-кода...

[Алексей++]

Tomson,

virtual BOOL SelChanged(LPCSTR lpcsSelection, CString& csStatusText);

а ты откуда копируешь то, на самом деле ?

[Tomson]

да, каюсь за невнимательность, все победил, осталось несколько варненгов и еще кое-что

exe1.cpp(54) : warning C4996: 'CWinApp::Enable3dControls' was declared deprecated
        c:\program files\microsoft visual studio 8\vc\atlmfc\include\afxwin.h(4471) : see declaration of 'CWinApp::Enable3dControls'
        Message: 'CWinApp::Enable3dControls is no longer needed. You should remove this call.'
exe1.cpp(61) : warning C4244: 'initializing' : conversion from 'INT_PTR' to 'int', possible loss of data
exe1Dlg.cpp
exe1dlg.cpp(140) : warning C4390: ';' : empty controlled statement found; is this the intent?

AFX_DEPRECATED("CWinApp::Enable3dControls is no longer needed. You should remove this call.")
         BOOL Enable3dControls()

К стати, если сискол выключить, то обламывается на варненгах

[Алексей++]

1) Message: 'CWinApp::Enable3dControls is no longer needed. You should remove this call.'
 - компилятор рекомендует удалить вызов этой функции

2)conversion from 'INT_PTR' to 'int', possible loss of data
 - покажи строчку, где это предупреждение

3) warning C4390: ';' : empty controlled statement found; is this the intent?
- тоже покажи строчку, скорее всего логически не правильно поставил точку с запятой , типа

if(...);
{
}

[Tomson]

CExe1Dlg dlg;
m_pMainWnd = & dlg;
int nResponse = dlg.DoModal(); <-  warning C4244: 'initializing' : conversion from 'INT_PTR' to 'int', possible loss of data
if (nResponse == IDOK)
{

[Алексей++]

Tomson, покажи, как перегрузил виртуальную DoModal() в классе CExe1Dlg

и  от чего производен CExe1Dlg тоже покажи

[Tomson]

да вроде не перезагружал

Код:

class CExe1Dlg : public CDialog
{
public:
CExe1Dlg(CWnd* pParent = NULL);
enum { IDD = IDD_EXE1_DIALOG };
CEdit m_edit2;
CEdit m_edit1;
CButton m_button1;
CButton m_button3;
CButton m_buttonok;
CStatic m_ico;
CString m_FileName;
CString m_FileName1;
CString m_path2;

public:
virtual BOOL PreTranslateMessage(MSG* pMsg);
protected:
virtual void DoDataExchange(CDataExchange* pDX);

А вот тут перезагружал....

Код:

class CDirDialog
{
public:

CDirDialog();
virtual ~CDirDialog();

BOOL DoModal(CWnd *pwndParent = NULL);
...etc...
private:
virtual BOOL SelChanged(LPCSTR lpcsSelection, CString& csStatusText)
{
	return TRUE; 
};
static int __stdcall CDirDialog::BrowseCtrlCallback(HWND hwnd, UINT uMsg, LPARAM lParam, LPARAM lpData);
};

[Алексей++]

Tomson, не "перезагружал" , а "перегружал"

ну класс CDirDialog вообще даже и не произведён от CDialog , поэтому он вообще лесом, а кроме того ты в примере выше его и не показывал. Перепутал ?


ну а почему рагается на

CExe1Dlg dlg;
int nResponse = dlg.DoModal();

не пойму что то.... Где то ты что то не показал нам, мне кажется


typedef long INT_PTR, *PINT_PTR;

а DoModal вертает int , так что колись, где что делал ))

[Tomson]

->> а DoModal вертает int , так что колись, где что делал ))

void CExe1Dlg::OnSysCommand(UINT nID, LPARAM lParam)
{
if ((nID & 0xFFF0) == IDM_ABOUTBOX)//; <-- убрал тчк. с запятой
{
CAboutDlg dlgAbout;
dlgAbout.DoModal();
}
else
{
CDialog::OnSysCommand(nID, lParam);
}
}
->а кроме того ты в примере выше его и не показывал. Перепутал ?
ээ, кого не показывал?

[Алексей++]

всё логично было, тогда уж так сделай

Код:

void CExe1Dlg::OnSysCommand(UINT nID, LPARAM lParam)
{
  if (nID == IDM_ABOUTBOX)
  {
    CAboutDlg dlgAbout;
    dlgAbout.DoModal();
    return;
  }

  CDialog::OnSysCommand(nID, lParam);
}

что перепутал - не знаю, почему компилятор ругаецо на типы у тебя ?

[Алексей++]

блин, кстати, а чего это ты сравниваешь с IDM_ABOUTBOX ? Зачем ?

The framework calls this member function when the user selects a command from the Control menu, or when the user selects the Maximize or the Minimize button.

afx_msg void OnSysCommand(
   UINT nID,
   LPARAM lParam
);
Parameters
nID
Specifies the type of system command requested. This parameter can be any one of the following values:
SC_CLOSE   Close the CWnd object.
SC_HOTKEY   Activate the CWnd object associated with the application-specified hot key. The low-order word of lParam identifies the HWND of the window to activate.
SC_HSCROLL   Scroll horizontally.
SC_KEYMENU   Retrieve a menu through a keystroke.
SC_MAXIMIZE (or SC_ZOOM)   Maximize the CWnd object.
SC_MINIMIZE (or SC_ICON)   Minimize the CWnd object.
SC_MOUSEMENU   Retrieve a menu through a mouse click.
SC_MOVE   Move the CWnd object.
SC_NEXTWINDOW   Move to the next window.
SC_PREVWINDOW   Move to the previous window.
SC_RESTORE   Restore window to normal position and size.
SC_SCREENSAVE   Executes the screen-saver application specified in the [boot] section of the SYSTEM.INI file.
SC_SIZE   Size the CWnd object.
SC_TASKLIST   Execute or activate the Windows Task Manager application.
SC_VSCROLL   Scroll vertically.
lParam
If a Control-menu command is chosen with the mouse, lParam contains the cursor coordinates. The low-order word contains the x coordinate, and the high-order word contains the y coordinate. Otherwise this parameter is not used.
SC_HOTKEY   Activate the window associated with the application-specified hot key. The low-order word of lParam identifies the window to activate.
SC_SCREENSAVE   Execute the screen-save application specified in the Desktop section of Control Panel.

[Tomson]

всякие события окна..
afx_msg void OnSysCommand(UINT nID, LPARAM lParam);
....
strAboutMenu.LoadString(IDS_ABOUTBOX);
if (!strAboutMenu.IsEmpty())
{
pSysMenu->AppendMenu(MF_SEPARATOR);
pSysMenu->AppendMenu(MF_STRING, IDM_ABOUTBOX, strAboutMenu);
........
// IDM_ABOUTBOX must be in the system command range.
ASSERT((IDM_ABOUTBOX & 0xFFF0) == IDM_ABOUTBOX);
ASSERT(IDM_ABOUTBOX < 0xF000);

[Алексей++]

и работает ?

по моему тебе надо

virtual BOOL OnCommand(
   WPARAM wParam,
   LPARAM lParam
);
Parameters
wParam
The low-order word of wParam identifies the command ID of the menu item, control, or accelerator. The high-order word of wParam specifies the notification message if the message is from a control. If the message is from an accelerator, the high-order word is 1. If the message is from a menu, the high-order word is 0.
lParam
Identifies the control that sends the message if the message is from a control. Otherwise, lParam is 0.
Return Value
An application returns nonzero if it processes this message; otherwise 0.

[Tomson]

Ну должно работать, по сути похоже они одно и тоже делают.

[Алексей++]

Tomson, ну а ты пробовал ?

[Tomson]

Пока нет, да наверно перекраивать придется немало, пробую разрулить то что есть:)

[Tomson]

кароче это победил, дело было в ресурсах...
Только теперь проблемс плавно перешел на сами ресурсы...
 error RC2104 : undefined keyword or key name: \u0410\u0446\u0428\u0408\u042b\u0459\u0433\u0430
и дело не в кирилице
LTEXT "rrrr",IDC_STATIC,26,160,105,8
undefined keyword or key name: rrrr

[Алексей++]

проект покажи ) Я так не смогу разобраться. Это повально....

[Tomson]

отправил в мыл, к стати)