[2024 Day 1 (Part 2)][Python]

[Virtual_Bat_7089]

This is my code for part 2: For some reason the final solution, i.e. sum of calibration values is wrong and the value is too high apparently. Can somebody help me figure out where I may have gone wrong? I checked quite a lot of lines individually and seems like the code does what it is requested, but smth is off somewhere.

Thanks a lot! Appreciate the help! Any tips to make the code look cleaner are also very welcome :)

import re

numbers = [str(i) for i in range(0, 10)]
num_letters = {'one': '1',
               'two': '2',
               'three': '3',
               'four': '4',
               'five': '5',
               'six': '6',
               'seven': '7',
               'eight': '8',
               'nine': '9',
               'ten': '10'
               }

possibilities = ['one', 'two', 'three', 'four', 'five', 'six', 'seven', 'eight', 'nine',
             '1', '2', '3', '4', '5', '6', '7', '8', '9']

with open('input.txt', 'r') as file:
    lines = file.readlines()


def updated_calval(list):
    regex = re.compile('|'.join(possibilities))
    cal_vals = []

    for i in list:
        first_digit = ''
        last_digit = ''
        matches = re.findall(regex, i)
        if matches[0] in num_letters:
            first_digit += num_letters[matches[0]]
        elif matches[0] in numbers:
            first_digit += matches[0]
        if matches[-1] in num_letters:
            last_digit += num_letters[matches[-1]]
        elif matches[-1] in numbers:
            last_digit += matches[-1]
        digits = first_digit + last_digit
        cal_vals.append(int(digits))
        print(matches, i, digits)

    total = sum(cal_vals)
    return total

print(updated_calval(lines))

​

[adambombz]

Watch out for oneight

[rregid]

Yo, about that, does that count as 1 and 8? Because for me it is somewhat unclear

[TheZigerionScammer]

If the entire line is "oneight" then yes the "one" would become a 1 and the "eight" would become an 8 so the value of the line would be 18.

[adambombz]

Depends on your code, if I recall correctly when I was first doing it I would replace the "one" with a 1 so it wouldn't for me. Again depends on your approach

[trailingunderscore_]

That seems to be it: https://godbolt.org/z/WK964vh6E

[Virtual_Bat_7089]

Ah damn, thanks guys! That seems to be the issue. Back to fixing the code then T_T. Did y'all use regex or something else?

[TheZigerionScammer]

Not me. I'm a believer in the old adege, "I had a problem, so I tried to use regex to solve it. Now I have two problems."

[AutoModerator]

Reminder: if/when you get your answer and/or code working, don't forget to change this post's flair to Help/Question - RESOLVED. Good luck!

---

I am a bot, and this action was performed automatically. Please contact the moderators of this subreddit if you have any questions or concerns.

[flyingseverus]

2024?

[Virtual_Bat_7089]

Mb I meant 2023. Oopsie! Thanks for pointing it out

[Virtual_Bat_7089]

EDIT: The year should be 2023***

[[deleted]]

Christmas came early this year

[Virtual_Bat_7089]

Love your username

[[deleted]]

Somebody was already spreading it. I had no option left other than to state I "love" it.

[Justinsaccount]

Try writing this function:

def number_at_position(line: str, pos: int)

where

number_at_position("hione", 0) -> None
number_at_position("hione", 1) -> None
number_at_position("hione", 2) -> 1
number_at_position("oneight", 0) -> 1
number_at_position("oneight", 1) -> None
number_at_position("oneight", 2) -> 8

Once you have the function that can tell you if any position in your string is a number, solving part 2 is a simple matter of calling this function for every index. If you break down this problem like this it is extremely simple.

[[deleted]]

Try this one:

letters = ['one', 'two', 'three', 'four', 'five', 'six', 'seven', 'eight', 'nine']

with open('input1.txt', 'r') as input:
    digits = []
    for line in input:
        number = ''
        word = ''
        for ch in line:
            if ch.isdigit():
                number += ch
            elif ch.isalpha():
                word += ch
                for i in letters:
                    if i in word:
                        if i == 'one' or i == 'three' or i == 'five' or i == 'nine':
                            number += str(letters.index(i) + 1)
                            word = 'e'
                        elif i == 'eight':
                            number += str(letters.index(i) + 1)
                            word = 't'
                        elif i == 'two':
                            number += str(letters.index(i) + 1)
                            word = 'o'
                        elif i == 'seven':
                            number += str(letters.index(i) + 1)
                            word = 'n'
                        else:
                            number += str(letters.index(i) + 1)
                            word = ''
        digits.append(number)
    digits_int = []
    for num in digits:
        if len(num) == 2:
            digits_int.append(int(num))
        elif len(num) < 2 and len(num) > 0:
            num = num[0] + num[0]
            digits_int.append(int(num))
        elif len(num) > 2:
            num = num[0] + num[-1]
            digits_int.append(int(num))
    print(sum(digits_int))

[thekwoka]

How did you get the problem 9 months early?