Meget STOR spørgsmål omkring MFC Appl.

void log(char *text);

burde være nok som erklæring i Logging.h

så får jeg en
D:\c++ projects\Update\UpdateDlg.cpp(136) : error C2065: 'log' : undeclared identifier

jeg har included min Logging.h fil i UpdateDLG.cpp

Indeholder UpdateDlg.cpp linie 136:

log("hello");

?

Du har ikke nogle #IFDEF eller /* */ i Logging.h som laver ged i det ?

her er min komplettet logging.cpp fil

// Logging.cpp : implementation file
//

#include "stdafx.h"
#include "Update.h"
#include "Logging.h"

#ifdef _DEBUG
#define new DEBUG_NEW
#undef THIS_FILE
static char THIS_FILE[] = __FILE__;
#endif

/////////////////////////////////////////////////////////////////////////////
// Logging

Logging::Logging()
{
}

Logging::~Logging()
{
}

BEGIN_MESSAGE_MAP(Logging, CWnd)
    //{{AFX_MSG_MAP(Logging)
        // NOTE - the ClassWizard will add and remove mapping macros here.
    //}}AFX_MSG_MAP
END_MESSAGE_MAP()


/////////////////////////////////////////////////////////////////////////////
// Logging message handlers

    CTime theTime;

int datetime()
{
    theTime = CTime::GetCurrentTime();
    theTime.Format( "%H:%M:%S : " );
    return -1;
}

void log(char *text)
{
    // open log file
    FILE *fp = fopen("log\\logfile.txt", "a");
    if (fp)
    {
    // time log
    CString d = datetime();
    // write to  logfile
    fprintf(fp,"%s",d);
    fprintf(fp," => %s\n",text);
    // close log file
    fclose(fp);
    }
    return;
}

void errorlog(char *text)
{
    // open log file
    FILE *fp = fopen("log\\errorlog.txt", "a");
    if (fp)
    {
    //time log
    CString d = datetime();
    // write to logfile
    fprintf(fp,"%s",d);
    fprintf(fp," => %s\n",text);
    // close log file
    fclose(fp);
    }
    return;
}

og min complettet logging.h fil

#if !defined(AFX_LOGGING_H__E5FCD6A1_A249_47B8_9A0B_77B978CCB1C9__INCLUDED_)
#define AFX_LOGGING_H__E5FCD6A1_A249_47B8_9A0B_77B978CCB1C9__INCLUDED_

#if _MSC_VER > 1000
#pragma once
#endif // _MSC_VER > 1000
// Logging.h : header file
//

/////////////////////////////////////////////////////////////////////////////
// Logging window

class Logging : public CWnd
{
// Construction
public:
    Logging();
    void log(char *text);

// Attributes
public:

// Operations
public:

// Overrides
    // ClassWizard generated virtual function overrides
    //{{AFX_VIRTUAL(Logging)
    //}}AFX_VIRTUAL

// Implementation
public:
    virtual ~Logging();

    // Generated message map functions
protected:
    //{{AFX_MSG(Logging)
        // NOTE - the ClassWizard will add and remove member functions here.
    //}}AFX_MSG
    DECLARE_MESSAGE_MAP()
};

/////////////////////////////////////////////////////////////////////////////

//{{AFX_INSERT_LOCATION}}
// Microsoft Visual C++ will insert additional declarations immediately before the previous line.

#endif // !defined(AFX_LOGGING_H__E5FCD6A1_A249_47B8_9A0B_77B978CCB1C9__INCLUDED_)

void log(char *text);

er da ikke erklæret i den Logging.h fil !

i min dlg fil linie 136 stod der

log("Write sucess");

bare for at teste

Du har erklæret en:
  void log(char *text);
som member af Logging klassen.

Det er noget helt andet.

Flyt:

void log(char *text);

ud af Logging klassen, så slipper du af med den fejl !

Alternativt skal i Logging.cpp lave log om til Logging::log og have fat
i et Logging objekt der hvor skal logge noget.

ja men jeg ville jo gerne have det sådan her så jeg kan holde styr på ting..

f.eks alt med log ligger i logging.cpp
alt med f.eks winsock kommer i en ny fil f.eks Winsock.cpp

lagde mærke til nogle steder hvor de bruger en slags templates (sådan de senere kan hente filerne ind og bruge dem i andre projecter)

syntes det var sådan de gjorde det..

Du kan da godt have:

void log(char *text);

i Logging.h den skal bare udenfor class Logging : public CWnd { }

Ellers må du lave implementeringen og kaldet om til objekt orienteret.

ejda hvorfor sagde du ikke det med det samme, nu virker det jo lol..

anyways, skriv lige et svar, du har fortjent de points (for det her manglede jeg 100% at lære)

så helt klart, du har fortjent dem.

ok