• lseif
    link
    fedilink
    arrow-up
    2
    ·
    7 months ago

    i personally find this a lot less readable than the switch example. the case keywords at the start of the line quickly signify its meaning, unlike with => after the pattern. though i dont speak for everybody.

    • Doods@infosec.pub
      link
      fedilink
      arrow-up
      1
      arrow-down
      1
      ·
      edit-2
      7 months ago

      How about this one? it more closely mirrors the switch example:

      match suffix {
      'G' | 'g' => mem -= 30,
      'M' | 'm' => mem -= 20,
      'K' | 'k' => mem -= 10,
      _ => {},
      }
      

      How about this other one? it goes as far as cloning the switch example’s indentation:

      match suffix {
      'G' | 'g' => {
      	mem -= 30;
              }
      'M' | 'm' => {
      	mem -= 20;
              }
      'K' | 'k' => {
      	mem -= 10;
              }
      _ => {},
      }
      
      • lseif
        link
        fedilink
        arrow-up
        2
        ·
        7 months ago

        the problem is that, while skimming the start of each lines, nothing about 'G' | 'g' tells me that its a branch. i need to spend more time parsing it. mind you, this may simply be a problem with rust’s syntax, not just ur formatting.