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

          名間水力發電廠 目录 沿革 設施 鄰近設施 註釋 外部連結 导航菜单23°50′10″N 120°42′41″E / 23.83611°N 120.71139°E / 23.83611; 120.7113923°50′10″N 120°42′41″E / 23.83611°N 120.71139°E / 23.83611; 120.71139計畫概要原始内容臺灣第一座BOT 模式開發的水力發電廠-名間水力電廠名間水力發電廠 水利署首件BOT案原始内容《小檔案》名間電廠 首座BOT水力發電廠原始内容名間電廠BOT - 經濟部水利署中區水資源局

          格濟夫卡 參考資料 导航菜单51°3′40″N 34°2′21″E / 51.06111°N 34.03917°E / 51.06111; 34.03917ГезівкаПогода в селі 编辑或修订