DaniWeb Forum Index > Software Development > C++

C++ RSS

Bug in getting the current date

Please support our C++ advertiser: Intel Parallel Studio Home

Thread Solved

•

Join Date: Dec 2006

Posts: 19

Reputation: jobra is an unknown quantity at this point

Solved Threads: 0

jobra

Offline

Newbie Poster

Bug in getting the current date

Dec 13th, 2006

Hi

I am new to c++. I tried to get current date but when I compile the program, it gives me an error. This is my program

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
#include <iostream>
#include <ctime>
#include <stdio.h>
int main ()
{
  char sdate [10];
  _strdate(sdate);
  std::cout<<"Current Date:"<<sdate;
  return 0;
}
#include <iostream>
#include <ctime>
#include <stdio.h>
int main ()
{
  char sdate [10];
  _strdate(sdate);
  std::cout<<"Current Date:"<<sdate;
  return 0;
}

when I complie in cygwin, the error is
test.cpp:8: error: â was not declared in this scope

Where are am I making a mistake. Please help me. Its urgent. Thanks a lot in advance.

Last edited by ~s.o.s~; Dec 13th, 2006 at 10:22 pm. Reason: Added code tags learn to use them yourself.

•

Join Date: May 2006

Posts: 1,580

Reputation: Infarction has a spectacular aura about

Solved Threads: 52

Infarction Infarction is offline

Offline

Battle Programmer

Re: Bug in getting the current date

Dec 13th, 2006

1) Where is _strdate() defined? I'm not finding it...
2) Why are you using <iostream> and <stdio.h> (which should be <cstdio>)?

•

Join Date: Apr 2006

Posts: 5,051

Reputation: John A is a splendid one to behold

Solved Threads: 332

John A

Offline

Vampirical Lurker

Re: Bug in getting the current date

Dec 13th, 2006

You need to include time.h to be able to use _strdate() - but please note that this is a non-standard function (doesn't work in Unix).

And please use code tags.

Last edited by John A; Dec 13th, 2006 at 10:19 pm.

"Technological progress is like an axe in the hands of a pathological criminal."

All my posts may be freely redistributed under the terms of the MIT license.

•

Join Date: Jun 2006

Posts: 7,617

Reputation: ~s.o.s~ has much to be proud of

Solved Threads: 466

~s.o.s~

Offline

Failure as a human

Re: Bug in getting the current date

Dec 13th, 2006

Here is the reference site.

Here is a small example:

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
#include <iostream>
#include <ctime>
using namespace std ;
 
int main( )
{
    char buffer[BUFSIZ] = { '\0' } ;
    time_t now = time( &now ) ;
    struct tm* local_time = new tm( ) ;
    local_time = localtime( &now ) ;
 
    strftime( buffer, BUFSIZ, "%d / %m / %Y", local_time ) ;
    cout << buffer ;
}
#include <iostream>
#include <ctime>
using namespace std ;

int main( )
{
    char buffer[BUFSIZ] = { '\0' } ;
    time_t now = time( &now ) ;
    struct tm* local_time = new tm( ) ;
    local_time = localtime( &now ) ;

    strftime( buffer, BUFSIZ, "%d / %m / %Y", local_time ) ;
    cout << buffer ;
}

I don't accept change; I don't deserve to live.

•

Join Date: Aug 2005

Posts: 15,408

Reputation: Ancient Dragon has a reputation beyond repute

Solved Threads: 1469

Ancient Dragon Ancient Dragon is offline

Offline

Still Learning

Re: Bug in getting the current date

Dec 14th, 2006

or this, which is almost identical to what ~S.O.S.~ posted. Like most things in programming there is always more than one way to do it. Unless you are instructed to do otherwise you can choose whichever method you want. One is not any better than the other unless you make the buffer too small to hold the result. In that case neither version will work correctly.

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
#include <iostream>
#include <ctime>
using namespace std ;
 
int main( )
{
    char buffer[BUFSIZ] = { '\0' } ;
    time_t now = time( &now ) ;
    struct tm* local_time = new tm( ) ;
    local_time = localtime( &now ) ;
    sprintf(buffer,"%02d/%02d/%04d",
          local_time->tm_mday,
          local_time->tm_mon+1,
          local_time->tm->year+1900); 
    cout << buffer ;
}
#include <iostream>
#include <ctime>
using namespace std ;
 
int main( )
{
    char buffer[BUFSIZ] = { '\0' } ;
    time_t now = time( &now ) ;
    struct tm* local_time = new tm( ) ;
    local_time = localtime( &now ) ;
    sprintf(buffer,"%02d/%02d/%04d",
          local_time->tm_mday,
          local_time->tm_mon+1,
          local_time->tm->year+1900); 
    cout << buffer ;
}

Don't PM me with questions -- you might get a nasty PM in response. If you have a question then post it in one of the forums.

•

Join Date: Apr 2004

Posts: 4,362

Reputation: Dave Sinkula has a brilliant future

Solved Threads: 241

Dave Sinkula Dave Sinkula is offline

Offline

long time no c

Re: Bug in getting the current date

Dec 14th, 2006

Isn't this leaking memory (aside from the fact that no attempt is made to delete the lost memory)?

Help with Code Tags

(Toggle Plain Text)

    struct tm* local_time = new tm( ) ;
    local_time = localtime( &now ) ;

It is also a good habit to check return values.
Time - Displaying Current
Change the format fed to strftime, as already mentioned, for the date.

"One of the methods used by statists to destroy capitalism consists in establishing controls that tie a given industry hand and foot, making it unable to solve its problems, then declaring that freedom has failed and stronger controls are necessary." --Ayn Rand