/u/22283/crop5e2c64c410533.png?f=community)
Siditamentis astuentis pactum.
/u/142011/crop65b383c6c6c2f_cropped.png?f=community)
:strip_icc():strip_exif()/u/6143/rincewind.jpg?f=community)
“Build a man a fire, and he'll be warm for a day. Set a man on fire, and he'll be warm for the rest of his life.”
/u/47683/Asterix%252060x60.png?f=community)
:strip_icc():strip_exif()/u/593726/crop5dcaa04cb6842_cropped.jpeg?f=community)
/u/200133/artemis_square_e_60-60.png?f=community)
Deel 2 was inderdaad vooral hopen dat de implementatie van alle checks correct was, anders zit er niet veel anders op dan testcases schrijven om te zien welke check het issue is.
Wel geinig om te zien hoe erg de oplossingen in Kotlin en C# lijken op wat ik zelf geschreven heb. En dat het bij veel mensen net als bij mij copy/paste van een vorige dag is en dan omgebouwd wordt, ik zie bij @Woy nog een functie om bomen te tellen
Wel geinig om te zien hoe erg de oplossingen in Kotlin en C# lijken op wat ik zelf geschreven heb. En dat het bij veel mensen net als bij mij copy/paste van een vorige dag is en dan omgebouwd wordt, ik zie bij @Woy nog een functie om bomen te tellen
Niet waar, ik zou nooit Copy/Paste doen:
En dat het bij veel mensen net als bij mij copy/paste van een vorige dag is en dan omgebouwd wordt, ik zie bij @Woy nog een functie om bomen te tellen
“Build a man a fire, and he'll be warm for a day. Set a man on fire, and he'll be warm for the rest of his life.”
Ja je hebt gelijk, het is een fullmatch, niet iets wat ik gewend ben bij de meeste regex libraries, maar dan kun je mijn comment negeren inderdaad
Op zich wel vreemd overigens, want in regex termen matched hij wel gewoon.
[ Voor 6% gewijzigd door Woy op 04-12-2020 09:10 ]
“Build a man a fire, and he'll be warm for a day. Set a man on fire, and he'll be warm for the rest of his life.”
Mjah, ik heb in al heel wat talen van regex API's gebruik gemaakt, en gegeven hoe verschillend die API's soms werken vind ik het lastig om daar een soort universeel beeld uit te halen van hoe-het-zou-moeten-zijn(tm).:
[...]
Ja je hebt gelijk, het is een fullmatch, niet iets wat ik gewend ben bij de meeste regex libraries, maar dan kun je mijn comment negeren inderdaad
Op zich wel vreemd overigens, want in regex termen matched hij wel gewoon.
Stel niet uit tot morgen wat je vandaag nog tot morgen kunt uitstellen...
:strip_icc():strip_exif()/u/189778/crop58e3af7a85b51.jpeg?f=community)
:strip_exif()/u/29138/caffeine.gif?f=community)
Deel 1 van dag 4 weer in native Excel opgelost :+: één kolom per field en dan kijken of er genoeg gevuld zijn. Voor deel 2 wel de VBA-editor aangezwengeld om de waarden te valideren. Ging vrij vlot moet ik zeggen! En gelijk het Like-statement ontdekt voor eenvoudige regex in VBA, nice 
.
.
Zo scherp als een voetbal!
:strip_icc():strip_exif()/u/490608/crop5e346de55c670_cropped.jpeg?f=community)
:strip_icc():strip_exif()/u/140697/crop6180fb9103afa_cropped.jpg?f=community)
:strip_icc():strip_exif()/u/26612/droptikkels_icon.jpg?f=community){kind=icon}
Nou, valt me niet tegen dat m'n part 2 oplossing in één keer goed was. Vreesde al dat te moeten gaan debuggen
:strip_icc():strip_exif()/u/206680/crop5fc7b8ec33719_cropped.jpeg?f=community)
Ik was ook bijna die route op gegaan (maar dan Apps Script ivm Google Spreadsheets) maar uiteindelijk toch met wat funky formules toch ook deel 2 volledig in de sheet kunnen doen. Wel stuk omslachtiger dan dag 2 & 3.:
Deel 1 van dag 4 weer in native Excel opgelost :+: één kolom per field en dan kijken of er genoeg gevuld zijn. Voor deel 2 wel de VBA-editor aangezwengeld om de waarden te valideren. Ging vrij vlot moet ik zeggen! En gelijk het Like-statement ontdekt voor eenvoudige regex in VBA, nice
:strip_icc():strip_exif()/u/279799/B4nqRMq_opt.jpg?f=community)
Ik zat met deel 2 vast omdat m'n antwoord niet klopte maar niet uit kon vogelen waarom.
spoiler: title
re.match(r"[0-9]{9}",passport["pid"])
Er is blijkbaar een pid met 10+ cijfers waar dit ook op matched
Er is blijkbaar een pid met 10+ cijfers waar dit ook op matched
:strip_icc():strip_exif()/u/133678/133963684346bda9327d0a6-1.jpg?f=community)
Duurde iets langer voor deel 2 dan nodig door een foutje in mijn validator (copy paste foutje ), pas toen ik de validators onder unittest had was het erg snel duidelijk.
Maar wel tevreden met code, kan netter / leaner als ik anderen hun C# zie maar dat blijft voorlopig wel zo. Stopte nu ook alles in een List omdat ik het vermoeden had dat deel 2 bepaalde records moest ophalen en daar iets mee moest doen, daarom beetje overkill code maar ach.
https://github.com/osxy/A...2020/AoC2020/Days/Day4.cs
), pas toen ik de validators onder unittest had was het erg snel duidelijk.Maar wel tevreden met code, kan netter / leaner als ik anderen hun C# zie maar dat blijft voorlopig wel zo. Stopte nu ook alles in een List omdat ik het vermoeden had dat deel 2 bepaalde records moest ophalen en daar iets mee moest doen, daarom beetje overkill code maar ach.
https://github.com/osxy/A...2020/AoC2020/Days/Day4.cs
[ Voor 18% gewijzigd door Osxy op 04-12-2020 10:13 ]
"Divine Shields and Hearthstones do not make a hero heroic."
Het lijkt erop dat je elke regel als een paspoort ziet? In het probleem zijn passports over meerdere regels verspreid met een witregel ertussen.
Edit: nevermind, ik zag een if over het hoofd. Dan weet ik het zo direct nog niet
[ Voor 7% gewijzigd door ThoNohT op 04-12-2020 10:33 ]
:strip_icc():strip_exif()/u/19784/crop5602e6447372a_cropped.jpeg?f=community)
1 opmerking.. die RegexOptions.Compiled maakt het instantiëren van de Regex een factor (10?) trager;
dus beter maak je die buiten je loop aan of haal je de flag eruit
Edit: Juist, verkeerd gelezen. Er komt een Func<> uit die functie die meermaals uitgevoerd kan worden.
[ Voor 63% gewijzigd door nescafe op 04-12-2020 11:12 ]
* Barca zweert ook bij fixedsys... althans bij mIRC de rest is comic sans
“Build a man a fire, and he'll be warm for a day. Set a man on fire, and he'll be warm for the rest of his life.”
Ik zie niet waar een regex meerdere malen aangemaakt wordt? Per validator maximaal 1 keer:
[...]
1 opmerking.. die RegexOptions.Compiled maakt het instantiëren van de Regex een factor (10?) trager;
dus beter maak je die buiten je loop aan of haal je de flag eruit
“Build a man a fire, and he'll be warm for a day. Set a man on fire, and he'll be warm for the rest of his life.”
/u/102105/crop56f481fe1f74f.png?f=community)
Ik maak niet per paspoort validators aan hoor
“Build a man a fire, and he'll be warm for a day. Set a man on fire, and he'll be warm for the rest of his life.”
/u/12668/hydra.png?f=community)
Ah fuck; goeie. Ik lees ze nu per regel in ipv als 1 grote string. Ga ik ff mee aan de slag.:
Je kan splitten op 2 newlines achter elkaar en dan de lines uit die blokken flatmappen
Edit: Done! Veel beter
[ Voor 4% gewijzigd door Hydra op 04-12-2020 11:42 ]
https://niels.nu
:strip_icc():strip_exif()/u/5080/veldsla.jpg?f=community)
Ik heb mijn code wat gefatsoeneerd. Ik zat vanmorgen om 6 uur achter m'n pc, en wilde een snelle tijd zetten. Maar goed.. dat levert dus lelijke en ondoordachte code op. Voor mij althans. En snel was ik ook niet. 
Nu wat herschikt: Here you go
Als ik de functionele oplossing van anderen bekijk munten die soms uit in eenvoud en compactheid. Ik vind mijn code verbose en ook niet elegant.
Nu wat herschikt: Here you go
Als ik de functionele oplossing van anderen bekijk munten die soms uit in eenvoud en compactheid. Ik vind mijn code verbose en ook niet elegant.
Siditamentis astuentis pactum.
Misschien sowieso een tip om niet alle oplossingen in 1 file te doen. Ik wil niet weten hoe die file er na dag 25 uitziet
:strip_icc():strip_exif()/u/145979/7613.jpg?f=community)
:strip_icc():strip_exif()/u/36083/crop5dca9e34ce091_cropped.jpeg?f=community)
Gewoon langer.:
Ik wil niet weten hoe die file er na dag 25 uitziet
Zolang de individuele methods niet onoverzichtelijk worden geen probleem, vind ik.
Maar misschien wordt de grootte inderdaad een keer te gek. Dan kan ik altijd nog uitsplitsen over één klasse per dag. Ik leef de 'premature optimization is the root of all evil' regel ;-)
Siditamentis astuentis pactum.
/u/3106/crop601874b34ff83_cropped.png?f=community)
Erg grappig al die verschillende oplossingen.
Vandaag was een beetje een lastige ten opzichte van de voorgaande dagen. Zelf een tijdje vast gezeten met String compare in Java (hoezo == werkt niet). Moet echt meer in Java gaan doen.
Vind de oplossingen van @Varienaja wel mooier dan die van mij. Ik maak gewoon een lange lijst met if-statements en dat is natuurlijk niet echt OOP.
Vandaag was een beetje een lastige ten opzichte van de voorgaande dagen. Zelf een tijdje vast gezeten met String compare in Java (hoezo == werkt niet
). Moet echt meer in Java gaan doen.Vind de oplossingen van @Varienaja wel mooier dan die van mij. Ik maak gewoon een lange lijst met if-statements en dat is natuurlijk niet echt OOP.
:strip_icc():strip_exif()/u/91018/dgtw.jpg?f=community)
Mwoah
Probeer deze eens:code:
1
| iyr:2010 hgt:15cm8 hcl:b6652a# ecl:blu byr:1944 eyr:2021 pid:093l54719 |
Drie fouten maar jouw code vindt 'm goed. De testdata is de eerste paar dagen nog niet zo heel kritisch, ik verwacht dat je hier volgende week niet meer mee wegkomt.
code:
1
| if ($string >= 59 && $string <= 76) { |
Vind ik persoonlijk erg lelijk
Over een tijdje ga je jezelf ook uitschelden dat je zulke namen gebruikt dus ik zou hier iets zorgvuldiger mee gaan doen Ook geeft je code erg veel notices. Zet deze aan (als je dat nog niet aan hebt staan, error_reporting(E_ALL) bovenaan je script) en zorg dat je die nooit te zien krijgt. Meestal wijst dat op een programmeerfout en PHP is enorm vergevingsgezind. In dit geval is het inderdaad een programmeerfout, vier zelfs. In een andere taal was je script keihard gecrasht. In PHP kan het zomaar betekenen dat er iets wordt uitgerekend wat helemaal niet kan, waardoor je antwoord fout wordt. Dat is ook iets wat je in de toekomst flink tegen kan gaan werken als je met grotere opdrachten bezig bent.
Je goto-oplossing kan, ik ben zelf niet tegen gebruik van goto, maar dan pas als je weet waar je het precies voor nodig hebt en in die heel specifieke gevallen correct toepast. Dit is m.i. niet zo'n geval. Het valideren van een paspoort is prima iets wat je in een functie zou kunnen zetten, met een serie if(iets) return false; if(ietsanders) return false; return true; die effectief hetzelfde doet als je goto, maar veel duidelijker/leesbaarder is en waar je geen gekke side-effects krijgt als je onder je end-label toch nog wat code neerzet.
FYI er is een subtiel verschil tussen class en instance attributes. De attributen die je buiten de constructor in de klasse definieert horen niet bij de instance. Ze bestaan ook uberhaupt niet in de instance als attributen tot je er iets aan toewijst, wat je in de constructor lijkt te doen. Voor immutables (zoals None), die je alleen maar met assignments "wijzigt", gaat dat dus prima. Maar bij het wijzigen van mutables doe je geen directe assignment maar een getattr om een magic method aan te roepen, dan wordt er doorgezocht naar class attributes als ze in de instance niet bestaan. Bijvoorbeeld met een map dict (zo heet dat ding in Python, foei ik) gaat dat fout.
code:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
| class A:
clvarNone = None
clvarMap = {}
def __init__(self):
self.invarNone = None
self.invarMap = {}
def __str__(self):
return ", ".join([str(a) for a in [self.clvarNone, self.invarNone, self.clvarMap, self.invarMap]])
def add(self, val):
self.clvarNone = val
self.clvarMap[val] = val
self.invarNone = val
self.invarMap[val] = val
a = A()
b = A()
print(f"a: {a}")
print(f"b: {b}")
a.add("a")
print(f"a: {a}")
print(f"b: {b}")
b.add("b")
print(f"a: {a}")
print(f"b: {b}")
print(A.clvarNone)
print(A.clvarMap) |
Output:
code:
1
2
3
4
5
6
7
8
| a: None, None, {}, {}
b: None, None, {}, {}
a: a, a, {'a': 'a'}, {'a': 'a'}
b: None, None, {'a': 'a'}, {}
a: a, a, {'a': 'a', 'b': 'b'}, {'a': 'a'}
b: b, b, {'a': 'a', 'b': 'b'}, {'b': 'b'}
None
{'a': 'a', 'b': 'b'} |
In 99% van de gevallen verwacht je niet dat b in a te zien is en andersom. Daarom is het het beste jezelf aan te leren alle members in de __init__ te maken met self.var = val, in plaats van var = val daarbuiten.
Maar valid_ecl kan dan opzich wel prima een class attribute zijn
[ Voor 11% gewijzigd door DataGhost op 04-12-2020 13:50 ]
Die je dan wel volkomen verkeerd interpreteert overigens, zoals veel developers. De hele quote is:
We should forget about small efficiencies, say about 97% of the time: premature optimization is the root of all evil. Yet we should not pass up our opportunities in that critical 3%
Hij heeft het specifiek over echt kleine winsten waarvan je niet weet of je ze nodig gaat hebben. Je code netjes leesbaar maken valt daar dus niet onder, je weet al lang dat je dat nodig gaat hebben.
Developers die die quote misbruiken is een beetje een pet-peeve, sorry
https://niels.nu
Uncle Bob is Clojure fan en alles is slechter dan Clojure. Je haalt veel nuttige info uit z'n praatjesen boeken, en hij is een erg goeie spreker, maar je moet wat 'ie zegt vaak met een grote korrel zout nemen. In z'n praatjes is hij vaak ook veel genuanceerder dan in z'n boeken.
Yup! Als we nu nog even Java de nek omdraaien en gewoon Kotlin gaan gebruiken komt alles goed
https://niels.nu
Daar heb je wel een punt. We gaan erachter komen
Engineering is like Tetris. Succes disappears and errors accumulate.
:strip_exif()/u/627282/charmander-kerst.gif?f=community)
Without nipples, boobs are pointless - 365 project - In mijn hoofd is het alle dagen Kerstmis - What type of bees make milk? Boobies! - What type of bees are scary? BoooOOOOOooobeees! - Cactusliefhebster
Hehe, ja die had ik nog kunnen doen..:
[...]
spoiler:gr[yn]
? Of ging je niet voor de kortste
Anyone who gets in between me and my morning coffee should be insecure.
Tja. Ik moet in jouw oplossing 100 regels meer code lezen dan in de mijne, terwijl ik het wel leesbaar heb gehouden. Ik had veel meer op 1 regel kunnen pakken puur om regels te besparen, maar mijn doel is vooral 'mooie' code te schrijven (beauty is in the eye of the beholder natuurlijk) die ook leesbaar is. En minder 'gedoe' vind ik vaak beter.
De beste code is die niet geschreven hoeft te worden
https://niels.nu
Ik vind jouw oplossing ook zeker mooi en beter leesbaar, maar niet helemaal te vergelijken. Ik heb nogal wat zaken erbij gezet die absoluut overbodig zijn om tot de oplossing te komen, maar mooie vingeroefening is
Ik bedoel alleen dat je wel super compacte code kan schrijven met oneliners, maar dat het er niet meteen beter leesbaar van wordt.
Leesbaar is leesbaar. Compact is leuk als je beperkte storage hebt
Kan Dag 4 ook zo doen:
code:
1
2
3
4
5
6
7
8
| object Day04s : Day {
private val fields = mapOf("byr" to { f: String -> f.toInt() in 1920 .. 2002}, "iyr" to { f: String -> f.toInt() in 2010 .. 2020}, "eyr" to { f: String -> f.toInt() in 2020 .. 2030}, "hgt" to ::heightValid, "hcl" to { f: String -> "#[0-9a-f]{6}".toRegex().matches(f) }, "ecl" to { f: String -> "amb|blu|brn|gry|grn|hzl|oth".toRegex().matches(f) }, "pid" to { f: String -> "[0-9]{9}".toRegex().matches(f) })
private val passports = read().filter { pass -> fields.keys.all { pass.containsKey(it) } }
private fun read() : List<Map<String, String>> = resourceString(2020, 4).split("\n\n").map { pass -> pass.split("[ \\n]".toRegex()).filterNot(String::isBlank).map { kv -> kv.split(":").let { (k, v) -> k to v } }.toMap() }
private fun heightValid(hgt: String) : Boolean = "([0-9]+)(in|cm)".toRegex().matchEntire(hgt)?.groupValues?.let { (_, a, u) -> if(u == "in") { a.toInt() in 59..76 } else { a.toInt() in 150..193 } } ?: false
override fun part1(): Int = passports.size
override fun part2(): Int = passports.count { pass -> fields.all { (k, f) -> f(pass.getValue(k)) } }
} |
https://niels.nu
Nee dat snap ik hoor, het is geen kritiek op jouw code. Ik snap waarom je het doet. Alleen zie ik juist als Java dev wel vaak dit soort code in productie; compleet overengineered waarbij je eerst een half uur zit te kijken voordat je erachter bent dat je beter 70% kan wegsnoeien.:
Ik vind jouw oplossing ook zeker mooi en beter leesbaar, maar niet helemaal te vergelijken. Ik heb nogal wat zaken erbij gezet die absoluut overbodig zijn om tot de oplossing te komen, maar mooie vingeroefening is
Het gaat niet om compact en onliners perse. Maar om minder visuele overhead. Java bijvoorbeeld is een taal waarin veel overhead zit die je niet nodig hebt. Meer code betekent vaak minder duidelijk. Dus minder code is dan vaak goed. Maar je kunt natuurlijk doorslaan (zoals in m'n voorbeeld) en gaan code-golven. Dat zie je veel in Scala code; die is vaak onleesbaar omdat de leesbaarheid ongeschikt geacht werd.
Ik probeer dus ook met AoC wel idiomatisch Kotlin te schrijven en hou me ook aan m'n eigen formatter settings.
https://niels.nu
Die deur staat echt wagenwijd open
. Leesbaarheid is er in gradaties. Verbose code kan nog steeds best wel goed leesbaar zijn. Dat wil niet zeggen dat het niet leesbaarder kan door de code minder verbose en daarmee meer to-the-point te maken. En dat kost tijd en moeite, en it's totally worth it!Ik moet hierbij altijd denken aan deze quote van Blaise Pascal:
Paradoxaal genoeg is het moeilijker en tijdrovender om makkelijk leesbare code te schrijven dan om moeilijk leesbare code te schrijven.
Stel niet uit tot morgen wat je vandaag nog tot morgen kunt uitstellen...
:strip_icc():strip_exif()/u/137799/pp.jpg?f=community)
Ik ga persoonlijk alleen wel wat minder goed op 1 letter variabelen die niet echt beschrijven wat voor data ze houden, zoals: p, r, q en s in jouw geval.
Wel een gevalletje persoonlijke voorkeur, kan voor de rest de linq uitwerking wel waarderen
Daar heb je zeker een punt. Die moet ik eigenlijk nog renamen, maar is een kwestie van luiheid en het feit dat dit de enige plek is waar ik er echt mee weg kom.:
[...]
Ik ga persoonlijk alleen wel wat minder goed op 1 letter variabelen die niet echt beschrijven wat voor data ze houden, zoals: p, r, q en s in jouw geval.
Wel een gevalletje persoonlijke voorkeur, kan voor de rest de linq uitwerking wel waarderen
edit: variabelen hernoemd
Mother north, how can they sleep while their beds are burning?
:strip_icc():strip_exif()/u/569624/crop57cdca5e3392e_cropped.jpeg?f=community)
Bij deel 2 van vandaag loop ik vast, alle gegeven voorbeelden kloppen en ook de unit tests komen er keurig doorheen, alleen bij de complete set gaat het ergens fout 
https://github.com/ProAce...de/tree/master/2020/Day_4
https://github.com/ProAce...de/tree/master/2020/Day_4
Je checkt de pid alleen op lengte, niet of het een nummer is. Weet niet of dat het probleem is, maar dat zie ik zo.:
Bij deel 2 van vandaag loop ik vast, alle gegeven voorbeelden kloppen en ook de unit tests komen er keurig doorheen, alleen bij de complete set gaat het ergens fout
https://github.com/ProAce...de/tree/master/2020/Day_4
[ Voor 26% gewijzigd door P_Tingen op 04-12-2020 18:31 ]
... en gaat over tot de orde van de dag
:strip_icc():strip_exif()/u/585680/olliolli-buttonjpg-df0ded_60w.jpg?f=community)
Ik heb geen ervaring in Go, maar ik begrijp je code wel. Het is lastig om je bug te vinden. Regex zien er OK uit, misschien daar nog start / end aangeven, maar dat is het volgens mij niet. Heb in mijn puzzle input gekeken, en daar zie ik geen hcl met meer character staan dan aangegeven, maar dat zegt natuurlijk niet alles. Even voor de zekerheid een $ of \b achterzetten.:
Bij deel 2 van vandaag loop ik vast, alle gegeven voorbeelden kloppen en ook de unit tests komen er keurig doorheen, alleen bij de complete set gaat het ergens fout
https://github.com/ProAce...de/tree/master/2020/Day_4
Zou het in het converteren van string naar int mis kunnen gaan in checkdigit? Ik had hier ook iets vreemds mee en m'n fix was geloof ik extra nog een lengte check doen van de string.
[ Voor 11% gewijzigd door Reynouts op 04-12-2020 19:45 ]
Dag 4 ook weer afgerond. https://github.com/Werner...blob/main/src/04/index.ts
//edit
@ProAce Ik gok dat je fout in de checkDigits functie zit.
Je min en max waardes zijn ook geldig dus ik denk dat je meer success hebt als je hier <= en >= doet.
//edit2
wacht nevermind, jij checkt precies andersom. Ik check of de waarde groter of gelijk is dan min, jij of het kleiner is. Iets te snel door de code gescanned.
//edit
@ProAce Ik gok dat je fout in de checkDigits functie zit.
code:
1
| if (temp < min) || (temp > max) |
Je min en max waardes zijn ook geldig dus ik denk dat je meer success hebt als je hier <= en >= doet.
//edit2
wacht nevermind, jij checkt precies andersom. Ik check of de waarde groter of gelijk is dan min, jij of het kleiner is. Iets te snel door de code gescanned.
[ Voor 88% gewijzigd door WernerL op 04-12-2020 19:10 ]
Roses are red, violets are blue, unexpected '{' on line 32.
:strip_icc():strip_exif()/u/237526/crop5e05c1d265781_cropped.jpeg?f=community)
Vandaag al een keer eerder voorbij gekomen volgens mij, weet je zeker dat de haar kleur regex niet ook kleuren met 7 characters goedkeurd?:
Bij deel 2 van vandaag loop ik vast, alle gegeven voorbeelden kloppen en ook de unit tests komen er keurig doorheen, alleen bij de complete set gaat het ergens fout
https://github.com/ProAce...de/tree/master/2020/Day_4
Ik heb werkelijk geen kennis van Go, maar gelukkig leest het ongeveer zoals Java.:
Bij deel 2 van vandaag loop ik vast, alle gegeven voorbeelden kloppen en ook de unit tests komen er keurig doorheen, alleen bij de complete set gaat het ergens fout
https://github.com/ProAce...de/tree/master/2020/Day_4
spoiler:
Regel 78 doe je '(len(input) - (invalidPasswordsP1 + invalidPasswordsP2))', dit moet alleen P2 zijn.
Regel 38/39 moet je 'validity = false' toevoegen.
https://repl.it/@Postman/GoTest
Regel 38/39 moet je 'validity = false' toevoegen.
https://repl.it/@Postman/GoTest
Hoop dat het duidelijk is wat ik bedoel
:strip_icc():strip_exif()/u/1007893/crop5f84470ecb79a.jpeg?f=community)
Thanks voor het meedenken! . Heb de begin/eind check toegevoegd maar dit mocht helaas niet baten.
Op het moment dat de conversie faalt is het blijkbaar geen getal en wordt het passport afgekeurd. Op een 64 bits systeem returned go ook een 64 bits int, dat zou het probleem niet mogen zijn lijkt me. Als ik snel de input scan dan lijken hier ook geen absurd grote waardes in te staan.
Mijn ervaring met regex begon vandaag
. Heb de begin/eind check toegevoegd maar dit mocht helaas niet baten.Hoe ik het opgebouwd had, om alles in een loop te kunnen berekenen, is door een counter bij te houden van de passports die niet valid zijn bij P1 en een counter van de passports die vervolgens niet zouden gelden voor P2. Vandaar de optelling van invalids. De uitkomsten van mijn input zijn voor P1: 237 (valid) en P2: 161 (too low)
`(^[0-9]{9}$)`:
Bij deel 2 van vandaag loop ik vast, alle gegeven voorbeelden kloppen en ook de unit tests komen er keurig doorheen, alleen bij de complete set gaat het ergens fout
https://github.com/ProAce...de/tree/master/2020/Day_4
Zit je probleem er niet in dat je begin en eind helemaal voor een achteraan moeten staan?
(Waardoor je pid matched)
Edit: nee dus, volgende keer toch maar eerst gaan slapen zodat mijn Nederlands wat beter is en dan eerst zelf opzoeken wat ik niet zeker weet..
Ik dacht dat ^ alleen helemaal vooraan mocht staan om het begin te matchen.
[ Voor 18% gewijzigd door jammo op 05-12-2020 08:55 ]
spoiler:
Was ik bijna binary-search aan het implementeren zeg. 
helemaal niet nodig!
helemaal niet nodig!Niet moeilijk, niet veel code, toch 20 minuten bezig geweest..
Siditamentis astuentis pactum.
De strategie om gewoon blind het maximaal mogelijke 127*8+7 als oplossing in te geven zonder ook maar iets te programmeren is helaas mislukt. 
[ Voor 8% gewijzigd door Varienaja op 05-12-2020 06:41 ]
Siditamentis astuentis pactum.
En dat was dag 5
Leuke opdracht, ik had hem niet direct door (het is nog vroeg..).
Leuke opdracht, ik had hem niet direct door (het is nog vroeg..).
spoiler:
Het duurde even voor ik doorhad dat het gewoon om het parsen van een binair getal gaat, daarna was het simpel.
@Varienaja Slim! Ik wist niet dat je met Integer.parseInt ook een binair getal kunt parsen. Daarmee had mijn code nog een paar regels korter gekund
.
@Varienaja Slim! Ik wist niet dat je met Integer.parseInt ook een binair getal kunt parsen. Daarmee had mijn code nog een paar regels korter gekund
.Stel niet uit tot morgen wat je vandaag nog tot morgen kunt uitstellen...
Thanks! Dat was inderdaad de fout. Ik dacht alleen dat de repl.it ook met mijn input runde en daarmee niet klopte
Beter lezen is toch wel key hier.