Dates...

[ustk]

@Lindon That seems convoluted because of the leap years of course. Now having a time in ms will allow you to do the same thing in two lines...

[Lindon]

@ustk said in Dates...:

@Lindon That seems convoluted because of the leap years of course. Now having a time in ms will allow you to do the same thing in two lines...

yeah - ish.... not really a problem - the diffDates was the worst offender - I dont think time in milliseconds is going to help with that - in fact I can see it being worse.. We really need a function that tells me how long to the end of the year for a given year...

as well as something sensible like

myDate = aGivenDate + 30

[ustk]

@Lindon Time in milliseconds take into account all leaps from 01-01-1970. So if you subtract two dates you'll get the exact number of milliseconds between the two, that you can convert up to the scale you want -> msDiff/1000/60/60/24 for days for instance.

[Lindon]

@ustk yeah I get that - and Im not obsessed about leap years - its not that hard to do...I'd just like a nice way to say

reg myDate = Time(Engine.getSystemTime);

and save all that text parsing...which would mean implementing Time::getCurrentTime()
or Time::currentTimeMillis()

..and whilst we are there:

Time::fromISO8601(StringRef iso8601) would be really helpful too...

So I guess I can already see a few more methods we could add right off the bat to make this waay more useful.

[ustk]

@Lindon Just adding a parameter to the actual function (or make a separate one...) and it's a one liner:

[Lindon]

@ustk good plan. I look forward to it. :-)

[ustk]

So I have finally created a separated class as Dave suggested that I weirdly called Timing...
Time couldn't be used as a class name because of a million conflicts, but other ideas are welcome

For now I have added 4 functions, let me know if you need more

Since it is just for a few conversion jobs, I don't think we actually need a Time object, but who knows... (well, except Christoph obviously :) )

[Christoph Hart]

Hmm, Timing is a pretty bad name for this as I would expect it to measure some benchmark stuff or handle MIDI delays etc.

What's the problem with Time? I'm not aware of any existing classes with this name and even if, I would prefer ´Date` as an API class (there's also a popular JS library with this name so it's a bit more close to our mother language.

[Christoph Hart]

reg leapYears = [1992,1996,2000,2004,2008,2012,2016,2020,2024,2028,2032]; //add more if you need em

Lol

inline function isLeapYear(year) 
{
    return year % 4 == 0;
}

[ustk]

@Christoph-Hart Simply because this was as far as my english brought me today

I tried Time but I encountered some conflicts with Juce Time, or I'm just idiot in the way I did it...

Date sounds good :thumbs_up:

[Christoph Hart]

@ustk ah your class doesn‘t need to be called Time in C++ the scripting name is determined by getObjectName().

[ustk]

@Christoph-Hart If you say so :man_shrugging: Could you say it again in french please?

EDIT: Oh ok I see :thumbs_up:

[d.healey]

Yeah I think date is a good choice since that's familiar to us Javascript people :)

[ustk]

@Christoph-Hart @d-healey @Lindon Pull request made (along with a few others)

[Lindon]

@Christoph-Hart said in Dates...:

reg leapYears = [1992,1996,2000,2004,2008,2012,2016,2020,2024,2028,2032]; //add more if you need em

Lol

inline function isLeapYear(year) 
{
    return year % 4 == 0;
}

except of course this is wrong.... you will need

year % 100

and

year % 400

[Christoph Hart]

@Lindon huh? This would mean that theres a leap year every 400 years so please tell everyone who has their birthday on the 29th of February that they won‘t live to see another birthday party…

[Lindon]

@Christoph-Hart - er you should go read up on what makes a leap year, here's the code (which I couldnt be bothered writing and testing last time out - naturally lazy...):

inline function isLeapYear(year) 
{
    if(year % 400 == 0 )
    {
           return true;
    }else{
        if(year % 100 == 0)
        {
             return false;
        }else{
            return year % 4 == 0;
       }
    }
}

So 2000 is a leap year - but 1900 isnt.

[Dan Korneff]

@Lindon here's a little script online:
https://www.programiz.com/javascript/examples/check-leap-year

inline function isLeapYear(year) 
{
     if ((0 == year % 4) && (0 != year % 100) || (0 == year % 400)) 
     {
          return true;
     }  
     else 
     {
          return false;
     }
}

[d.healey]

@Dan-Korneff Why not?

inline function isLeapYear(year) 
{
    return (0 == year % 4) && (0 != year % 100) || (0 == year % 400));
}

[Dan Korneff]

@d-healey That works too, and you save 4 lines of code.