с++ 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]

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

[Алексей++]

а чё аж 6 метров ? )) Ты папки debug и release то удалил из проекта ? Они при пересылке ни к чему.

Сейчас то я скачаю, на будущее имей в виду

[Алексей++]

проект не могу открыть, у меня VC++6
попробуй удалить строку
LTEXT "rrrr",IDC_STATIC,26,160,105,8

перекомпилировать всё и снова кинуть контрол на форму

[Tomson]

удалил и он дальше ругнулся
BLOCK "FileInfo"
error RC2104 : undefined keyword or key name: FileInfo
еще раз удалил, дальше...
BLOCK "080404b0"
error RC2108 : expected numerical dialog constant

Возможно ли, что lvalue функция какая-нибуть?

[Алексей++]

ты в какой студии пишешь ?

а всё подряд удалять то не надо - не сработало, надо вернуть обратно и разбираться дальше

[Tomson]

студия 2005

[Алексей++]

у менянет такой. Может, кто отзовётся и поможет.
А пока попробуй пересоздать проект, кода то ещй наверное немного

[Вад]

У меня 2005я на работе, только:
1. в чём проблема, довести до собираемого состояния?
2. кидать лучше в аську, т.к. на работе юзаю только корпоративную почту, а светить её неохота

[Tomson]

->1. в чём проблема, довести до собираемого состояния?
да вот застрял на ресурсах
->2. кидать лучше в аську, т.к. на работе юзаю только корпоративную почту, а светить её неохота
а в почту та что в профиле?

[Вад]

В exe1.rc проекта заменил китайскую кодировку на русскую - ресурсы стали собираться.

Код:

#if !defined(AFX_RESOURCE_DLL) || defined(AFX_TARG_CHS)
    //#ifdef _WIN32
    //LANGUAGE LANG_CHINESE, SUBLANG_CHINESE_SIMPLIFIED
    //#pragma code_page(936)
#ifdef _WIN32
LANGUAGE LANG_RUSSIAN, SUBLANG_DEFAULT
#pragma code_page(1251)
#endif //_WIN32

Дальше уже проблемы линковки - может, сам разберёшься

[Tomson]

А ты без Enable3dControls()  собрал?

[Вад]

Tomson, я собирал в точности в таких конфигурациях, которые ты скинул. У меня проблема исчезла при замене кодировки. Осталась только проблема с линковкой, связанная с тем, что ты зачем-то CAboutDlg в двух cpp-шниках определил

[Tomson]

Чорд, не выходит, если убираю все что после комента "used for App About" в exe1Dlg.cpp или changedircolor.cpp все ломается совсем(

[Вад]

Объяви свой класс CAboutDlg в заголовочном файле. Для простоты, лучше в отдельном: AboutDlg.h - и соответственно определи методы в .cpp. Убери все остальные объявления этого класса из других .cpp, а затем делай себе на здоровье include из обоих .cpp, где у тебя он раньше был напрямую объявлен. Наведи порядок, словом

[Tomson]

странно, а от виндовса зависить может? я на 2к3 собираю. Вот это запихал в хедер

Код:

class CAboutDlg : public CDialog
{
public:
CAboutDlg();
enum { IDD = IDD_ABOUTBOX };
protected:
virtual void DoDataExchange(CDataExchange* pDX);  
protected:
	DECLARE_MESSAGE_MAP()
};
CAboutDlg::CAboutDlg() : CDialog(CAboutDlg::IDD)
{
}
void CAboutDlg::DoDataExchange(CDataExchange* pDX)
{
	CDialog::DoDataExchange(pDX);
}
BEGIN_MESSAGE_MAP(CAboutDlg, CDialog)
END_MESSAGE_MAP()

но все равно все те же ошибки линковки

[Вад]

Не зависит от винды.

Код:

CAboutDlg::CAboutDlg() : CDialog(CAboutDlg::IDD)
{
}
void CAboutDlg::DoDataExchange(CDataExchange* pDX)
{
	CDialog::DoDataExchange(pDX);
}
BEGIN_MESSAGE_MAP(CAboutDlg, CDialog)
END_MESSAGE_MAP()

- вот это запихивать в хедер совсем не стоило, надо было в .cpp-файл. Читай внимательнее
Ошибка линковки - потому что ты этот хедер в двух местах включаешь, а у тебя вне объявления класса методы определены - ругается на переопределение методов (они же в двух объектниках в итоге оказываются). Вынеси в .cpp - и будет тебе счастье.

[Tomson]

вынес, только вот такое все равно осталось
error LNK2019: unresolved external symbol "public: __thiscall CMainFrame::CMainFrame(void)" (??0CMainFrame@@QAE@XZ) referenced in function "public: virtual int __thiscall CchangedircolorApp::InitInstance(void)" (?InitInstance@CchangedircolorApp@@UAEHXZ)
 fatal error LNK1120: 1 unresolved externals

[Вад]

Это означает, что конструктор для CMainFrame, вызываемый в CchangedircolorApp::InitInstance, не определён. Посмотри, действительно ли это так (но линковщик врать не должен ). Я тоже посмотрю, только до работы доберусь.

P.S. Ну вот, .cpp для CMainFrame в проект вообще не включён оказался почему-то. Внимательнее надо быть