jujj, de szééép... nasty little csak-néha-végtelen ciklus :D

if (days > 366)

Ekkora gányolást!

Mi a lósz*r ez???

Ha jól látom: input: napok (ORIGINYEAR január 1 óta)
Output: struktúra day, dayofweek, month, year-el kitöltve.

De ki volt ezt képes újra megírni (pláne így), amikor erre van millió library függvény?

@bodzasfanta:
Mármint az "if (days > 366)"-nak a nem létező else ágára gondoltál, ugye? (ami a days = 366-ot kezelné.)

@John42: igen én sem teljesen értettem, hogy miért nem volt jó nekik egy GetSystemTime hívás.. WinCE-ben implementálva van, és a Zune oprendszere CE alapú, ha jól tudom...

a hibas kod egy Freescale chip drivereben talalhato, igy a bug nem Zune specifikus...pl Toshiba Gigabeat S szeria szinten megdoglott abban a boldog oraban :)

akkurvannyát, ezt melyik barom írta?! így kell:

while (days > (d = IsLeapYear(year) ? 366 : 365)) {
++year;
days -= d;
}

rövidebb, egyszerűbb, olvashatóbb, van jelentése a feltételnek, és mellesleg működik.

@szadba...: inicializaciot vegyuk ki azert a ciklus ele ;)

"Zune"-os kodban azert vicces a fuckup

1: while (days > 365)
2: if (days > 366) :))))

synapse

pastie.org/349916

// Global Variables
//These macro define some default information of RTC
#define ORIGINYEAR 1980 // the begin year
#define MAXYEAR (ORIGINYEAR + 100) // the maxium year
#define JAN1WEEK 2 // Jan 1 1980 is a Tuesday
#define GetDayOfWeek(X) (((X-1)+JAN1WEEK)%7)


Lesz itt még baj 2080-ban is :DDD

@synapse: szeirntem félreérted ezt a bonyolult kódot.. milyen inicializációt és minek? gondold végig, mert megalázlak.

d = IsLeapYear(year) ? 366 : 365); d) {
++year;
days -= d;
}

Alazol? Ajajjj :D

synapse

d = IsLeapYear(year) ? 366 : 365);
while (days > d) {
++year;
days -= d;
}

Na, igy. Kurva tacspad :D

synapse

ciklus előtt inicializáció mánia by synapse :) de jogos :)

Hat ugyanazt kiertekelni minden iteracioban nem igazan ertelmes. De azert erzem, hogy ezert meg nagyon, NAGYON meg leszek alazva :D:D:D:D:D

Amugy a multkori meglett EQ?

synapse

Synapse drága...
A year minden egyes cikluslépésben növekszik. Befejezzem a gondolatot?

PS: 1980 szökőév.

nem segfaultol és 3oprendeszeren 4gépen, 12processként fut mint teszt, szóval működik egyenlőre :)

@synapse: na mert bezzeg faszságot írni kommentbe, na az meg pláne igazán értelmes.
mert ha szólok, hogy faszságot írtál (mint ahogy tényleg), akkor ahelyett hogy elgondolkodnál, hogy talán valóban fasz vagy, nagyarccal visszaszólsz, és mondod tovább a magad baromságát. na ez NAGYON gáz.
minthogy annyira ostoba vagy, hogy egy három értékadásból egy egy összehasonlításból álló kódot nem bírsz felfogni, hát megszánlak, és elmagyarázom.
a ciklus invariánsa, hogy a keresett nap a year év jan.1-étől számolva, a day-edik nap. ez minden interáció előtt és után teljesül. mivel a ciklusfeltételben a d mindig pontosan azt az értékes kapja, ahány napja a year évnek van, a ciklusfeltétel azt jelenti, hogy "a keresett nap nem a year évben van". ha ez teljesül, akkor ugrik a következő évre. ezt jelenti a "++year;" a year tehát minden kurva iterációnál változik. (gondolom még soha nem találkoztál a ++ operátorral, és ez okozta a problémát.) hogy az invariánst megtartsa, ezután a napok számát éppen annyival csökkeni, ahány nap az adott évben van. azaz d-vel. miven a days minden interációban szigorúan csökken, a ciklus garantáltan véget ér.
tehát a kódom helyes.
te meg hülye vagy.
és túl nagy az arcod.

"A year minden egyes cikluslépésben növekszik. Befejezzem a gondolatot?"

Ahaha, hat ezt bizony kurvara beneztem, igazad van. :)

synapse

@szádba szarok

ja, beneztem, kurvaokos vagy gratulalok. Viszont a stilusod miatt kijarna neked egy alapos szajbaveres.

synapse

@Szádba szarok

A kérdéses évben (ami végtelenre viszi az eredetit), a tied hibás eredményt hoz.

Jó lenne tudni, hogy a days paraméterrel történik e valami érdekes, mielőtt megkapja a fv.

@Udi: az eredeti a szökőévet követő évek jan1-eitől száll áll. pontosan milyen paraméterre ad hibás eredményt az enyém? mit kéne, és mit ad?

@szadba szarok:
a szokoevek utolso napjan szall el az eredeti kod, de ettol fuggetlenul szvsz rendben van a cilklus amit irtal...

talan ez meg olvashatobb es egyszerubb:
while (days > 365 + IsLeapYear(year)) {
days -= 365 + IsLeapYear(year);
year++;
}

hogy csak egyszer kelljen IsLeapYear-t hivni:

while (days > (d = 365 + IsLeapYear(year))) {
days -= d;
year++;
}

@gadget: igaz, dec31.. (most nézem, 1-től kezdi a számozást..) amúgy a fene tudja, hogy a bool true az int-ként mennyit ért... lehet hogy az IsLeapYear() 59-et ad vissza szökőévre.. kurvára nem lepődnék meg azon se, ha így lenne :D

@szádba szarok:
bool int-re castolva mindig 0/1

utananeztem az IsLeapYear fv. implementaciojanak (egy ujabb csodakod)...bool helyett direktbe int-et ad vissza (0/1), szal vegul is mukodhet a fenti ciklusom

bit-player.org/2009/the-zune-bug

@gadget:
"bool int-re castolva mindig 0/1"
hogyne, c++-ban. meg 200 másik nyelven is. c-ben pont nem. ott nincs is beépített bool típus.

@szádba szarok:
tenyleg, errol el is feledkeztem...de mivel int 0/1-et ad vissza, a kodomnak mukodnie kell

ha a hivatalos fixet ugyanaz a zseni koveti el aki az eredeti kodot, akkor ugyis csak else agon egy break; lesz a javitas :)

egy apró megjegyzés a bool dologgal kapcsolatban: ha jól látom, a kódban sehol sincs bool, mindenhol BOOL van, ami meg int..

@szádba szarok:

Én javában teszteltem, és így inicializáltam az inputot:
private static final Calendar epoch = new GregorianCalendar(1980,1,1,0,0);
private static final GregorianCalendar d = new GregorianCalendar(2008,12,31,0,0);

aztán
days = Math.round((float)(teszt.d.getTimeInMillis()-teszt.epoch.getTimeInMillis())/1000/60/60/24);

Nem szép, tudom. Valami nem volt kerek, mert 2009.1.1 inputtal futott végtelen ciklusra. Miután kilőttem a hibát tesztelni kezdtem a többi algoritmust is. Az eredeti 0-t ad, a tied pedig nem lép be az utolsó iterációba, és 365 lesz a végeredmény.
Egyik se helyes megoldás.

Ha a days-hez hozzáadsz egyet inikor, akkor is más eredmények jönnek ki.

Fura probléma, és nincs is kedvem már többet foglalkozni vele. Annyit sejtek, hogy a probléma magja, az hogy 1980.1.1 az általuk használt epoch.

Én ezt írtam:
while (days >365){
days -= 365;
if(d.isLeapYear(year)&&days>1)days--;
year++;
}
PS: teljesség kedvéért, a te algoritmusod javában:
int day = 0;
while (days > (day = d.isLeapYear(year) ? 366 : 365)) {
year++;
days -= day;
}

@gadget:

while (days > (d = 365 + (IsLeapYear(year)!=0?1:0))) {
days -= d;
year++;
}

és mindenki boldog :D

@Udi:
ays = Math.round((float)(teszt.d.getTimeInMillis()-teszt.epoch.getTimeInMillis())/1000/60/60/24);
ez 0-tól sorszámozza a napookat, az eredeti meg 1-től.

Poppe tanár úr igen büszke lenne! :)