Extracting numeric values from a string containing key:value pairsSimplify splitting a String into alpha and numeric partsRegex to remove inline javascript from stringExtraction of numbers from currency stringRemoving nested blocks from a stringExtracting tuples from Excel sheetRegex to obtain float values from a large text fileExtracting JSON from string using regexExtracting an IP address from a hosts file by hostnameExtracting parts of filenames from an arrayExtracting parameters from request URL path components using regexes

Mimic lecturing on blackboard, facing audience

Why is the sun approximated as a black body at ~ 5800 K?

How to preserve electronics (computers, iPads and phones) for hundreds of years

Unable to disable Microsoft Store in domain environment

Purpose of creating non root user

When and why was runway 07/25 at Kai Tak removed?

Is there anyway, I can have two passwords for my wi-fi

SOQL query causes internal Salesforce error

Make a border of symbols in Gimp

Pre-Employment Background Check With Consent For Future Checks

Proving a complicated language is not a CFL

Quoting Keynes in a lecture

Why does the Persian emissary display a string of crowned skulls?

Sigmoid with a slope but no asymptotes?

How to leave product feedback on macOS?

Number Theory: Problem with proofs

Giving feedback to someone without sounding prejudiced

Should I assume I have passed probation?

What does "tick" mean in this sentence?

Consistent Linux device enumeration

Telemetry for feature health

What should be the ideal length of sentences in a blog post for ease of reading?

Alignment of six matrices

If the only attacker is removed from combat, is a creature still counted as having attacked this turn?



Extracting numeric values from a string containing key:value pairs


Simplify splitting a String into alpha and numeric partsRegex to remove inline javascript from stringExtraction of numbers from currency stringRemoving nested blocks from a stringExtracting tuples from Excel sheetRegex to obtain float values from a large text fileExtracting JSON from string using regexExtracting an IP address from a hosts file by hostnameExtracting parts of filenames from an arrayExtracting parameters from request URL path components using regexes













4












$begingroup$


I am writing function numericValues(text: String): List[Int] to extract patterns """([a-z]+)s*:s*(d+)""" and return the list of the numeric values :



numericValues("a123 : 0 abc:123 123:abc xyz:1") // List(123, 1)


I would write numericValues like this:



def numericValues(text: String): List[Int] = 
val regex = """([a-z]+)s*:s*(d+)""".r
regex.findAllIn(text).toList.flatMap s =>
PartialFunction.condOpt(s) case regex(_, num) => num.toInt




I guess the condOpt invocation is redundant and I wonder how to simplify this implementation. Also, I'd appreciate comments on improvements to style and best practice.










share|improve this question









New contributor




Michael is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.







$endgroup$
















    4












    $begingroup$


    I am writing function numericValues(text: String): List[Int] to extract patterns """([a-z]+)s*:s*(d+)""" and return the list of the numeric values :



    numericValues("a123 : 0 abc:123 123:abc xyz:1") // List(123, 1)


    I would write numericValues like this:



    def numericValues(text: String): List[Int] = 
    val regex = """([a-z]+)s*:s*(d+)""".r
    regex.findAllIn(text).toList.flatMap s =>
    PartialFunction.condOpt(s) case regex(_, num) => num.toInt




    I guess the condOpt invocation is redundant and I wonder how to simplify this implementation. Also, I'd appreciate comments on improvements to style and best practice.










    share|improve this question









    New contributor




    Michael is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
    Check out our Code of Conduct.







    $endgroup$














      4












      4








      4





      $begingroup$


      I am writing function numericValues(text: String): List[Int] to extract patterns """([a-z]+)s*:s*(d+)""" and return the list of the numeric values :



      numericValues("a123 : 0 abc:123 123:abc xyz:1") // List(123, 1)


      I would write numericValues like this:



      def numericValues(text: String): List[Int] = 
      val regex = """([a-z]+)s*:s*(d+)""".r
      regex.findAllIn(text).toList.flatMap s =>
      PartialFunction.condOpt(s) case regex(_, num) => num.toInt




      I guess the condOpt invocation is redundant and I wonder how to simplify this implementation. Also, I'd appreciate comments on improvements to style and best practice.










      share|improve this question









      New contributor




      Michael is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.







      $endgroup$




      I am writing function numericValues(text: String): List[Int] to extract patterns """([a-z]+)s*:s*(d+)""" and return the list of the numeric values :



      numericValues("a123 : 0 abc:123 123:abc xyz:1") // List(123, 1)


      I would write numericValues like this:



      def numericValues(text: String): List[Int] = 
      val regex = """([a-z]+)s*:s*(d+)""".r
      regex.findAllIn(text).toList.flatMap s =>
      PartialFunction.condOpt(s) case regex(_, num) => num.toInt




      I guess the condOpt invocation is redundant and I wonder how to simplify this implementation. Also, I'd appreciate comments on improvements to style and best practice.







      regex scala






      share|improve this question









      New contributor




      Michael is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.











      share|improve this question









      New contributor




      Michael is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.









      share|improve this question




      share|improve this question








      edited 5 mins ago







      Michael













      New contributor




      Michael is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.









      asked 11 hours ago









      MichaelMichael

      1235




      1235




      New contributor




      Michael is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.





      New contributor





      Michael is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.






      Michael is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.




















          1 Answer
          1






          active

          oldest

          votes


















          6












          $begingroup$

          You usually want a zero-width word boundary b on either end of the regex, to avoid matching things like 1a:1a.



          There's no need to capture [a-z]+ since you are throwing it away.



          You can use lookbehind (?<=…) assertions to require a match to be preceded by whatever, without including the whatever in the match result. This means no need for capturing parenthesis, only the integer is included in the match, and the final map is simply _.toInt. Variable-width lookbehind was introduced in Java 9; older versions have only fixed-width lookbehind.



          Finally, removing the variable makes braces unnecessary.



          def numericValues(text: String): List[Int] = """(?<=b[a-z]+s*:s*)d+b""".r.findAllIn(text).toList.map(_.toInt)


          With only fixed-width lookbehind, you could postprocess the matches to remove non-numerics:



          ….map(_.replaceAll("[^0-9]", "").toInt)


          Or, more idiomatically and less hacky, just capture the digits and extract the capture groups:



          """b[a-z]+s*:s*(d+)b""".r.findAllIn(text).matchData.toList.map(_.group(1).toInt)





          share|improve this answer











          $endgroup$












          • $begingroup$
            Thanks a lot. What if I have only fixed-width lookbehind ?
            $endgroup$
            – Michael
            8 hours ago






          • 1




            $begingroup$
            then you can't use variable-width lookbehind :) See edit.
            $endgroup$
            – Oh My Goodness
            7 hours ago










          Your Answer





          StackExchange.ifUsing("editor", function ()
          return StackExchange.using("mathjaxEditing", function ()
          StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix)
          StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
          );
          );
          , "mathjax-editing");

          StackExchange.ifUsing("editor", function ()
          StackExchange.using("externalEditor", function ()
          StackExchange.using("snippets", function ()
          StackExchange.snippets.init();
          );
          );
          , "code-snippets");

          StackExchange.ready(function()
          var channelOptions =
          tags: "".split(" "),
          id: "196"
          ;
          initTagRenderer("".split(" "), "".split(" "), channelOptions);

          StackExchange.using("externalEditor", function()
          // Have to fire editor after snippets, if snippets enabled
          if (StackExchange.settings.snippets.snippetsEnabled)
          StackExchange.using("snippets", function()
          createEditor();
          );

          else
          createEditor();

          );

          function createEditor()
          StackExchange.prepareEditor(
          heartbeatType: 'answer',
          autoActivateHeartbeat: false,
          convertImagesToLinks: false,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: null,
          bindNavPrevention: true,
          postfix: "",
          imageUploader:
          brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
          contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
          allowUrls: true
          ,
          onDemand: true,
          discardSelector: ".discard-answer"
          ,immediatelyShowMarkdownHelp:true
          );



          );






          Michael is a new contributor. Be nice, and check out our Code of Conduct.









          draft saved

          draft discarded


















          StackExchange.ready(
          function ()
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f215831%2fextracting-numeric-values-from-a-string-containing-keyvalue-pairs%23new-answer', 'question_page');

          );

          Post as a guest















          Required, but never shown

























          1 Answer
          1






          active

          oldest

          votes








          1 Answer
          1






          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes









          6












          $begingroup$

          You usually want a zero-width word boundary b on either end of the regex, to avoid matching things like 1a:1a.



          There's no need to capture [a-z]+ since you are throwing it away.



          You can use lookbehind (?<=…) assertions to require a match to be preceded by whatever, without including the whatever in the match result. This means no need for capturing parenthesis, only the integer is included in the match, and the final map is simply _.toInt. Variable-width lookbehind was introduced in Java 9; older versions have only fixed-width lookbehind.



          Finally, removing the variable makes braces unnecessary.



          def numericValues(text: String): List[Int] = """(?<=b[a-z]+s*:s*)d+b""".r.findAllIn(text).toList.map(_.toInt)


          With only fixed-width lookbehind, you could postprocess the matches to remove non-numerics:



          ….map(_.replaceAll("[^0-9]", "").toInt)


          Or, more idiomatically and less hacky, just capture the digits and extract the capture groups:



          """b[a-z]+s*:s*(d+)b""".r.findAllIn(text).matchData.toList.map(_.group(1).toInt)





          share|improve this answer











          $endgroup$












          • $begingroup$
            Thanks a lot. What if I have only fixed-width lookbehind ?
            $endgroup$
            – Michael
            8 hours ago






          • 1




            $begingroup$
            then you can't use variable-width lookbehind :) See edit.
            $endgroup$
            – Oh My Goodness
            7 hours ago















          6












          $begingroup$

          You usually want a zero-width word boundary b on either end of the regex, to avoid matching things like 1a:1a.



          There's no need to capture [a-z]+ since you are throwing it away.



          You can use lookbehind (?<=…) assertions to require a match to be preceded by whatever, without including the whatever in the match result. This means no need for capturing parenthesis, only the integer is included in the match, and the final map is simply _.toInt. Variable-width lookbehind was introduced in Java 9; older versions have only fixed-width lookbehind.



          Finally, removing the variable makes braces unnecessary.



          def numericValues(text: String): List[Int] = """(?<=b[a-z]+s*:s*)d+b""".r.findAllIn(text).toList.map(_.toInt)


          With only fixed-width lookbehind, you could postprocess the matches to remove non-numerics:



          ….map(_.replaceAll("[^0-9]", "").toInt)


          Or, more idiomatically and less hacky, just capture the digits and extract the capture groups:



          """b[a-z]+s*:s*(d+)b""".r.findAllIn(text).matchData.toList.map(_.group(1).toInt)





          share|improve this answer











          $endgroup$












          • $begingroup$
            Thanks a lot. What if I have only fixed-width lookbehind ?
            $endgroup$
            – Michael
            8 hours ago






          • 1




            $begingroup$
            then you can't use variable-width lookbehind :) See edit.
            $endgroup$
            – Oh My Goodness
            7 hours ago













          6












          6








          6





          $begingroup$

          You usually want a zero-width word boundary b on either end of the regex, to avoid matching things like 1a:1a.



          There's no need to capture [a-z]+ since you are throwing it away.



          You can use lookbehind (?<=…) assertions to require a match to be preceded by whatever, without including the whatever in the match result. This means no need for capturing parenthesis, only the integer is included in the match, and the final map is simply _.toInt. Variable-width lookbehind was introduced in Java 9; older versions have only fixed-width lookbehind.



          Finally, removing the variable makes braces unnecessary.



          def numericValues(text: String): List[Int] = """(?<=b[a-z]+s*:s*)d+b""".r.findAllIn(text).toList.map(_.toInt)


          With only fixed-width lookbehind, you could postprocess the matches to remove non-numerics:



          ….map(_.replaceAll("[^0-9]", "").toInt)


          Or, more idiomatically and less hacky, just capture the digits and extract the capture groups:



          """b[a-z]+s*:s*(d+)b""".r.findAllIn(text).matchData.toList.map(_.group(1).toInt)





          share|improve this answer











          $endgroup$



          You usually want a zero-width word boundary b on either end of the regex, to avoid matching things like 1a:1a.



          There's no need to capture [a-z]+ since you are throwing it away.



          You can use lookbehind (?<=…) assertions to require a match to be preceded by whatever, without including the whatever in the match result. This means no need for capturing parenthesis, only the integer is included in the match, and the final map is simply _.toInt. Variable-width lookbehind was introduced in Java 9; older versions have only fixed-width lookbehind.



          Finally, removing the variable makes braces unnecessary.



          def numericValues(text: String): List[Int] = """(?<=b[a-z]+s*:s*)d+b""".r.findAllIn(text).toList.map(_.toInt)


          With only fixed-width lookbehind, you could postprocess the matches to remove non-numerics:



          ….map(_.replaceAll("[^0-9]", "").toInt)


          Or, more idiomatically and less hacky, just capture the digits and extract the capture groups:



          """b[a-z]+s*:s*(d+)b""".r.findAllIn(text).matchData.toList.map(_.group(1).toInt)






          share|improve this answer














          share|improve this answer



          share|improve this answer








          edited 7 hours ago

























          answered 9 hours ago









          Oh My GoodnessOh My Goodness

          1,554314




          1,554314











          • $begingroup$
            Thanks a lot. What if I have only fixed-width lookbehind ?
            $endgroup$
            – Michael
            8 hours ago






          • 1




            $begingroup$
            then you can't use variable-width lookbehind :) See edit.
            $endgroup$
            – Oh My Goodness
            7 hours ago
















          • $begingroup$
            Thanks a lot. What if I have only fixed-width lookbehind ?
            $endgroup$
            – Michael
            8 hours ago






          • 1




            $begingroup$
            then you can't use variable-width lookbehind :) See edit.
            $endgroup$
            – Oh My Goodness
            7 hours ago















          $begingroup$
          Thanks a lot. What if I have only fixed-width lookbehind ?
          $endgroup$
          – Michael
          8 hours ago




          $begingroup$
          Thanks a lot. What if I have only fixed-width lookbehind ?
          $endgroup$
          – Michael
          8 hours ago




          1




          1




          $begingroup$
          then you can't use variable-width lookbehind :) See edit.
          $endgroup$
          – Oh My Goodness
          7 hours ago




          $begingroup$
          then you can't use variable-width lookbehind :) See edit.
          $endgroup$
          – Oh My Goodness
          7 hours ago










          Michael is a new contributor. Be nice, and check out our Code of Conduct.









          draft saved

          draft discarded


















          Michael is a new contributor. Be nice, and check out our Code of Conduct.












          Michael is a new contributor. Be nice, and check out our Code of Conduct.











          Michael is a new contributor. Be nice, and check out our Code of Conduct.














          Thanks for contributing an answer to Code Review Stack Exchange!


          • Please be sure to answer the question. Provide details and share your research!

          But avoid


          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.

          Use MathJax to format equations. MathJax reference.


          To learn more, see our tips on writing great answers.




          draft saved


          draft discarded














          StackExchange.ready(
          function ()
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f215831%2fextracting-numeric-values-from-a-string-containing-keyvalue-pairs%23new-answer', 'question_page');

          );

          Post as a guest















          Required, but never shown





















































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown

































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown







          Popular posts from this blog

          瀋陽號驅逐艦 目录 接收與服役 配置反潛直升機 武進三型性能升級 歷史 除役 參考資料 外部連結 导航菜单Taiwan Air Power海疆老兵-陽字號驅逐艦沿革World Navies Today: Taiwan (Republic of China)DD-839 USS POWER

          波兰旗帜列表 目录 国旗 军旗 其他制服部门旗帜 特别国家机构船只 参考文献 外部链接 导航菜单Polskie flagi, chorągwie, bandery... [波兰旗帜、条幅、船旗等]原始内容Ustawa z dnia 31 stycznia 1980 r. o godle, barwach i hymnie Rzeczypospolitej Polskiej oraz o pieczęciach państwowychZarządzenie Ministra Obrony Narodowej z dnia 14 grudnia 2005 r. zmieniające zarządzenie w sprawie szczegółowych zasad używania znaków Sił Zbrojnych Rzeczypospolitej Polskiej oraz ustalenia innych znaków używanych w Siłach Zbrojnych Rzeczypospolitej PolskiejZarządzenie Ministra Obrony Narodowej z dnia 29 stycznia 1996 r. w sprawie szczegółowych zasad używania znaków Sił Zbrojnych Rzeczypospolitej Polskiej oraz ustalenia innych znaków używanych w Siłach Zbrojnych Rzeczypospolitej PolskiejUstawa z dnia 19 lutego 1993 r. o znakach Sił Zbrojnych Rzeczypospolitej PolskiejHistoria Marynarki Wojennej RP [波兰海军史]Rozporządzenie Ministra Spraw Wewnętrznych i Administracji z dnia 12 kwietnia 2002 r. w sprawie wzoru flagi oraz oznakowania jednostek pływających i statków powietrznych Straży GranicznejRozporządzenie Ministra Spraw Wewnętrznych i Administracji z dnia 18 kwietnia 2005 r. w sprawie wzoru flagi oraz oznakowania jednostek pływających i statków powietrznych PolicjiRozporządzenie Ministra Infrastruktury z dnia 21 października 2005 r. w sprawie wzorów flag dla statków morskich na oznaczenie pełnionej specjalnej służby państwowej oraz okoliczności i warunków ich podnoszenia波兰旗帜波兰

          Indenting and Dedenting ASP code with Python