Capturing repeating groups in GO












2















I'm trying to create a function that can parse strings which consist of an uppercase word followed by zero or more arguments which are encapsulated in double quotes.



For example, each of the following lines:



COPY "filename one" "filename two"
REMOVE "filename"
LIST "x" "y" "z"
DISCONNECT


The result should be a string (the command) followed by a string (the arguments inside the quotes). I created the following regular expression:



re1, _ := regexp.Compile(`([A-Z]+)(?: "([^"]+)")*`)
results := re1.FindAllStringSubmatch(input, -1)


However, no-matter what I try, only the last argument gets captured.



An example of my problem: https://play.golang.org/p/W1rE1X4SWf5



"arg1" is not captured in this example. What am I missing?










share|improve this question

























  • Could a command (without considering args) contain white space?

    – Ehsan.Saradar
    Nov 22 '18 at 15:57
















2















I'm trying to create a function that can parse strings which consist of an uppercase word followed by zero or more arguments which are encapsulated in double quotes.



For example, each of the following lines:



COPY "filename one" "filename two"
REMOVE "filename"
LIST "x" "y" "z"
DISCONNECT


The result should be a string (the command) followed by a string (the arguments inside the quotes). I created the following regular expression:



re1, _ := regexp.Compile(`([A-Z]+)(?: "([^"]+)")*`)
results := re1.FindAllStringSubmatch(input, -1)


However, no-matter what I try, only the last argument gets captured.



An example of my problem: https://play.golang.org/p/W1rE1X4SWf5



"arg1" is not captured in this example. What am I missing?










share|improve this question

























  • Could a command (without considering args) contain white space?

    – Ehsan.Saradar
    Nov 22 '18 at 15:57














2












2








2








I'm trying to create a function that can parse strings which consist of an uppercase word followed by zero or more arguments which are encapsulated in double quotes.



For example, each of the following lines:



COPY "filename one" "filename two"
REMOVE "filename"
LIST "x" "y" "z"
DISCONNECT


The result should be a string (the command) followed by a string (the arguments inside the quotes). I created the following regular expression:



re1, _ := regexp.Compile(`([A-Z]+)(?: "([^"]+)")*`)
results := re1.FindAllStringSubmatch(input, -1)


However, no-matter what I try, only the last argument gets captured.



An example of my problem: https://play.golang.org/p/W1rE1X4SWf5



"arg1" is not captured in this example. What am I missing?










share|improve this question
















I'm trying to create a function that can parse strings which consist of an uppercase word followed by zero or more arguments which are encapsulated in double quotes.



For example, each of the following lines:



COPY "filename one" "filename two"
REMOVE "filename"
LIST "x" "y" "z"
DISCONNECT


The result should be a string (the command) followed by a string (the arguments inside the quotes). I created the following regular expression:



re1, _ := regexp.Compile(`([A-Z]+)(?: "([^"]+)")*`)
results := re1.FindAllStringSubmatch(input, -1)


However, no-matter what I try, only the last argument gets captured.



An example of my problem: https://play.golang.org/p/W1rE1X4SWf5



"arg1" is not captured in this example. What am I missing?







regex go






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited Nov 22 '18 at 16:21









mrzasa

9,675104078




9,675104078










asked Nov 22 '18 at 15:40









XatooXatoo

2,2662144




2,2662144













  • Could a command (without considering args) contain white space?

    – Ehsan.Saradar
    Nov 22 '18 at 15:57



















  • Could a command (without considering args) contain white space?

    – Ehsan.Saradar
    Nov 22 '18 at 15:57

















Could a command (without considering args) contain white space?

– Ehsan.Saradar
Nov 22 '18 at 15:57





Could a command (without considering args) contain white space?

– Ehsan.Saradar
Nov 22 '18 at 15:57












4 Answers
4






active

oldest

votes


















2














If your commands are well defined, e.i. command names are always upper-case and arguments are always after the command then a looser regex might just fit your use case:



re1, _ := regexp.Compile(`([A-Z]+)|(?: "([^"]+)")`)
results := re1.FindAllStringSubmatch(`COMMAND "arg1" "arg2" "arg3"`, -1)

fmt.Println("Command:", results[0][1])
for _, arg := range results[1:] {
fmt.Println("Arg:", arg[2])
}


Playground






share|improve this answer


























  • Awesome, this implementation works for me!

    – Xatoo
    Nov 22 '18 at 17:00











  • Probably, regexp.Compile(`^([A-Z]+)| "([^"]+)"`) will be more precise as it matches the command at the start of string only.

    – Wiktor Stribiżew
    Nov 22 '18 at 17:25





















1














When you try to capture repeated match, only the last one is captured.
I'd try to do it in two steps: first split commmand and arguments, then parse the arguments.



Splitting to command and arguments can be done with ([A-Z]+)((?: "[^"]+")*) (demo):





  • ([A-Z]+) in first group, you get the command


  • ((?: "[^"]+")*) in the second group, you'll get arguments in quotes, separated by spaces


Then you can use FindAllString with "([^"]+)" to extract arguments (demo).






share|improve this answer































    1














    I think this may solve your problem



    re1, _ := regexp.Compile(`([A-Z]+)(?: *)`)
    commandText:=`COPY "filename one" "filename two"`
    if re1.Match(byte(commandText)){
    index:=re1.FindIndex(byte(commandText))[1]
    commandArgs:=commandText[index:]
    commandArgsRegex,_:=regexp.Compile(`"([^"]+)"`)
    fmt.Println("Command= " , commandText[0:index])
    for i,arg:=range commandArgsRegex.FindAllString(commandArgs,-1){
    fmt.Println("args ", i,"= " , arg)
    }
    }else{
    fmt.Println("Failed")
    }





    share|improve this answer































      0














      Add an extra capture group. If you make it optional extra data will be empty but the match will work



      re1, _ := regexp.Compile(`^([A-Z]+)(s"[^"]+")(s"[^"]+")?(s"[^"]+")?$`)


      Add more (s"[^"]+")? expressions up to the maximum you need. I put in two as there is an expression with 3 parameters in your examples






      share|improve this answer























        Your Answer






        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: "1"
        };
        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: true,
        noModals: true,
        showLowRepImageUploadWarning: true,
        reputationToPostImages: 10,
        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
        });


        }
        });














        draft saved

        draft discarded


















        StackExchange.ready(
        function () {
        StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53434310%2fcapturing-repeating-groups-in-go%23new-answer', 'question_page');
        }
        );

        Post as a guest















        Required, but never shown

























        4 Answers
        4






        active

        oldest

        votes








        4 Answers
        4






        active

        oldest

        votes









        active

        oldest

        votes






        active

        oldest

        votes









        2














        If your commands are well defined, e.i. command names are always upper-case and arguments are always after the command then a looser regex might just fit your use case:



        re1, _ := regexp.Compile(`([A-Z]+)|(?: "([^"]+)")`)
        results := re1.FindAllStringSubmatch(`COMMAND "arg1" "arg2" "arg3"`, -1)

        fmt.Println("Command:", results[0][1])
        for _, arg := range results[1:] {
        fmt.Println("Arg:", arg[2])
        }


        Playground






        share|improve this answer


























        • Awesome, this implementation works for me!

          – Xatoo
          Nov 22 '18 at 17:00











        • Probably, regexp.Compile(`^([A-Z]+)| "([^"]+)"`) will be more precise as it matches the command at the start of string only.

          – Wiktor Stribiżew
          Nov 22 '18 at 17:25


















        2














        If your commands are well defined, e.i. command names are always upper-case and arguments are always after the command then a looser regex might just fit your use case:



        re1, _ := regexp.Compile(`([A-Z]+)|(?: "([^"]+)")`)
        results := re1.FindAllStringSubmatch(`COMMAND "arg1" "arg2" "arg3"`, -1)

        fmt.Println("Command:", results[0][1])
        for _, arg := range results[1:] {
        fmt.Println("Arg:", arg[2])
        }


        Playground






        share|improve this answer


























        • Awesome, this implementation works for me!

          – Xatoo
          Nov 22 '18 at 17:00











        • Probably, regexp.Compile(`^([A-Z]+)| "([^"]+)"`) will be more precise as it matches the command at the start of string only.

          – Wiktor Stribiżew
          Nov 22 '18 at 17:25
















        2












        2








        2







        If your commands are well defined, e.i. command names are always upper-case and arguments are always after the command then a looser regex might just fit your use case:



        re1, _ := regexp.Compile(`([A-Z]+)|(?: "([^"]+)")`)
        results := re1.FindAllStringSubmatch(`COMMAND "arg1" "arg2" "arg3"`, -1)

        fmt.Println("Command:", results[0][1])
        for _, arg := range results[1:] {
        fmt.Println("Arg:", arg[2])
        }


        Playground






        share|improve this answer















        If your commands are well defined, e.i. command names are always upper-case and arguments are always after the command then a looser regex might just fit your use case:



        re1, _ := regexp.Compile(`([A-Z]+)|(?: "([^"]+)")`)
        results := re1.FindAllStringSubmatch(`COMMAND "arg1" "arg2" "arg3"`, -1)

        fmt.Println("Command:", results[0][1])
        for _, arg := range results[1:] {
        fmt.Println("Arg:", arg[2])
        }


        Playground







        share|improve this answer














        share|improve this answer



        share|improve this answer








        edited Nov 22 '18 at 16:23

























        answered Nov 22 '18 at 16:15









        ssemillassemilla

        3,087424




        3,087424













        • Awesome, this implementation works for me!

          – Xatoo
          Nov 22 '18 at 17:00











        • Probably, regexp.Compile(`^([A-Z]+)| "([^"]+)"`) will be more precise as it matches the command at the start of string only.

          – Wiktor Stribiżew
          Nov 22 '18 at 17:25





















        • Awesome, this implementation works for me!

          – Xatoo
          Nov 22 '18 at 17:00











        • Probably, regexp.Compile(`^([A-Z]+)| "([^"]+)"`) will be more precise as it matches the command at the start of string only.

          – Wiktor Stribiżew
          Nov 22 '18 at 17:25



















        Awesome, this implementation works for me!

        – Xatoo
        Nov 22 '18 at 17:00





        Awesome, this implementation works for me!

        – Xatoo
        Nov 22 '18 at 17:00













        Probably, regexp.Compile(`^([A-Z]+)| "([^"]+)"`) will be more precise as it matches the command at the start of string only.

        – Wiktor Stribiżew
        Nov 22 '18 at 17:25







        Probably, regexp.Compile(`^([A-Z]+)| "([^"]+)"`) will be more precise as it matches the command at the start of string only.

        – Wiktor Stribiżew
        Nov 22 '18 at 17:25















        1














        When you try to capture repeated match, only the last one is captured.
        I'd try to do it in two steps: first split commmand and arguments, then parse the arguments.



        Splitting to command and arguments can be done with ([A-Z]+)((?: "[^"]+")*) (demo):





        • ([A-Z]+) in first group, you get the command


        • ((?: "[^"]+")*) in the second group, you'll get arguments in quotes, separated by spaces


        Then you can use FindAllString with "([^"]+)" to extract arguments (demo).






        share|improve this answer




























          1














          When you try to capture repeated match, only the last one is captured.
          I'd try to do it in two steps: first split commmand and arguments, then parse the arguments.



          Splitting to command and arguments can be done with ([A-Z]+)((?: "[^"]+")*) (demo):





          • ([A-Z]+) in first group, you get the command


          • ((?: "[^"]+")*) in the second group, you'll get arguments in quotes, separated by spaces


          Then you can use FindAllString with "([^"]+)" to extract arguments (demo).






          share|improve this answer


























            1












            1








            1







            When you try to capture repeated match, only the last one is captured.
            I'd try to do it in two steps: first split commmand and arguments, then parse the arguments.



            Splitting to command and arguments can be done with ([A-Z]+)((?: "[^"]+")*) (demo):





            • ([A-Z]+) in first group, you get the command


            • ((?: "[^"]+")*) in the second group, you'll get arguments in quotes, separated by spaces


            Then you can use FindAllString with "([^"]+)" to extract arguments (demo).






            share|improve this answer













            When you try to capture repeated match, only the last one is captured.
            I'd try to do it in two steps: first split commmand and arguments, then parse the arguments.



            Splitting to command and arguments can be done with ([A-Z]+)((?: "[^"]+")*) (demo):





            • ([A-Z]+) in first group, you get the command


            • ((?: "[^"]+")*) in the second group, you'll get arguments in quotes, separated by spaces


            Then you can use FindAllString with "([^"]+)" to extract arguments (demo).







            share|improve this answer












            share|improve this answer



            share|improve this answer










            answered Nov 22 '18 at 16:05









            mrzasamrzasa

            9,675104078




            9,675104078























                1














                I think this may solve your problem



                re1, _ := regexp.Compile(`([A-Z]+)(?: *)`)
                commandText:=`COPY "filename one" "filename two"`
                if re1.Match(byte(commandText)){
                index:=re1.FindIndex(byte(commandText))[1]
                commandArgs:=commandText[index:]
                commandArgsRegex,_:=regexp.Compile(`"([^"]+)"`)
                fmt.Println("Command= " , commandText[0:index])
                for i,arg:=range commandArgsRegex.FindAllString(commandArgs,-1){
                fmt.Println("args ", i,"= " , arg)
                }
                }else{
                fmt.Println("Failed")
                }





                share|improve this answer




























                  1














                  I think this may solve your problem



                  re1, _ := regexp.Compile(`([A-Z]+)(?: *)`)
                  commandText:=`COPY "filename one" "filename two"`
                  if re1.Match(byte(commandText)){
                  index:=re1.FindIndex(byte(commandText))[1]
                  commandArgs:=commandText[index:]
                  commandArgsRegex,_:=regexp.Compile(`"([^"]+)"`)
                  fmt.Println("Command= " , commandText[0:index])
                  for i,arg:=range commandArgsRegex.FindAllString(commandArgs,-1){
                  fmt.Println("args ", i,"= " , arg)
                  }
                  }else{
                  fmt.Println("Failed")
                  }





                  share|improve this answer


























                    1












                    1








                    1







                    I think this may solve your problem



                    re1, _ := regexp.Compile(`([A-Z]+)(?: *)`)
                    commandText:=`COPY "filename one" "filename two"`
                    if re1.Match(byte(commandText)){
                    index:=re1.FindIndex(byte(commandText))[1]
                    commandArgs:=commandText[index:]
                    commandArgsRegex,_:=regexp.Compile(`"([^"]+)"`)
                    fmt.Println("Command= " , commandText[0:index])
                    for i,arg:=range commandArgsRegex.FindAllString(commandArgs,-1){
                    fmt.Println("args ", i,"= " , arg)
                    }
                    }else{
                    fmt.Println("Failed")
                    }





                    share|improve this answer













                    I think this may solve your problem



                    re1, _ := regexp.Compile(`([A-Z]+)(?: *)`)
                    commandText:=`COPY "filename one" "filename two"`
                    if re1.Match(byte(commandText)){
                    index:=re1.FindIndex(byte(commandText))[1]
                    commandArgs:=commandText[index:]
                    commandArgsRegex,_:=regexp.Compile(`"([^"]+)"`)
                    fmt.Println("Command= " , commandText[0:index])
                    for i,arg:=range commandArgsRegex.FindAllString(commandArgs,-1){
                    fmt.Println("args ", i,"= " , arg)
                    }
                    }else{
                    fmt.Println("Failed")
                    }






                    share|improve this answer












                    share|improve this answer



                    share|improve this answer










                    answered Nov 22 '18 at 16:10









                    Ehsan.SaradarEhsan.Saradar

                    45337




                    45337























                        0














                        Add an extra capture group. If you make it optional extra data will be empty but the match will work



                        re1, _ := regexp.Compile(`^([A-Z]+)(s"[^"]+")(s"[^"]+")?(s"[^"]+")?$`)


                        Add more (s"[^"]+")? expressions up to the maximum you need. I put in two as there is an expression with 3 parameters in your examples






                        share|improve this answer




























                          0














                          Add an extra capture group. If you make it optional extra data will be empty but the match will work



                          re1, _ := regexp.Compile(`^([A-Z]+)(s"[^"]+")(s"[^"]+")?(s"[^"]+")?$`)


                          Add more (s"[^"]+")? expressions up to the maximum you need. I put in two as there is an expression with 3 parameters in your examples






                          share|improve this answer


























                            0












                            0








                            0







                            Add an extra capture group. If you make it optional extra data will be empty but the match will work



                            re1, _ := regexp.Compile(`^([A-Z]+)(s"[^"]+")(s"[^"]+")?(s"[^"]+")?$`)


                            Add more (s"[^"]+")? expressions up to the maximum you need. I put in two as there is an expression with 3 parameters in your examples






                            share|improve this answer













                            Add an extra capture group. If you make it optional extra data will be empty but the match will work



                            re1, _ := regexp.Compile(`^([A-Z]+)(s"[^"]+")(s"[^"]+")?(s"[^"]+")?$`)


                            Add more (s"[^"]+")? expressions up to the maximum you need. I put in two as there is an expression with 3 parameters in your examples







                            share|improve this answer












                            share|improve this answer



                            share|improve this answer










                            answered Nov 22 '18 at 16:25









                            VorsprungVorsprung

                            22.6k32044




                            22.6k32044






























                                draft saved

                                draft discarded




















































                                Thanks for contributing an answer to Stack Overflow!


                                • 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.


                                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%2fstackoverflow.com%2fquestions%2f53434310%2fcapturing-repeating-groups-in-go%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

                                404 Error Contact Form 7 ajax form submitting

                                How to know if a Active Directory user can login interactively

                                TypeError: fit_transform() missing 1 required positional argument: 'X'