Day 9: Disk Fragmenter

Megathread guidelines

  • Keep top level comments as only solutions, if you want to say something other than a solution put it in a new post. (replies to comments can be whatever)
  • You can send code in code blocks by using three backticks, the code, and then three backticks or use something such as https://topaz.github.io/paste/ if you prefer sending it through a URL

FAQ

  • hades@lemm.ee
    link
    fedilink
    arrow-up
    3
    ·
    3 days ago

    This part looks suspicious:

        for f in range(len(free) - 2):
            disk_map[free[f]] = disk_map.pop(max(disk_map.keys()))
    

    You’re always moving exactly len(free) - 2 blocks, but that doesn’t sound to be correct in all cases. If you consider the following input: 191, you only need to move one block, and not seven.

    • urquell@lemm.ee
      link
      fedilink
      arrow-up
      1
      ·
      3 days ago

      I’m always moving one (file)part at a time, so that should be fine… I think.