• dns
    link
    fedilink
    arrow-up
    10
    ·
    edit-2
    1 year ago

    My solution was worse than most: replace one -> one1one You are only going to do the replace all for each number and if the “e” is also in eight it is still there for the next set of replace.

    A better quick and dirty solution from Mastodon was to just add the common character first: twone -> twoone

      • AnarchistArtificer@lemmy.world
        link
        fedilink
        English
        arrow-up
        1
        ·
        1 year ago

        I have a friend who says that “whatever works is elegant” and solutions like OP’s is why I simultaneously love and hate that phrase.

        • sus@programming.dev
          link
          fedilink
          arrow-up
          2
          ·
          edit-2
          1 year ago

          Behold, elegance:

          digits = { "one": [1,2], "two": [2,2], "three": [3,4], "four": [4,3], "five": [5,3], "six": [6,2], "seven": [7,4], "eight": [8,4], "nine": [9,3], "1": [1,1], "2": [2,1], "3": [3,1], "4": [4,1], "5": [5,1], "6": [6,1], "7": [7,1], "8": [8,1], "9": [9,1] }

          and what comes afterwards is even more elegant, for it works!

    • damium@programming.dev
      link
      fedilink
      English
      arrow-up
      1
      ·
      1 year ago

      You can also use o1e as there are never more than a single shared character. It also doesn’t change the string size so it can be done in place. Still an ugly hack of a solution.