How to parse recursive parentheses correctly?









up vote
2
down vote

favorite
1












I need to parse a string that contains some parentheses disposed recursively, but i'm having trouble with determining priority of parentheses.
For exemple, I have the string



$truth = "((A^¬B)->C)";


and I need to return what is between the parentheses. I've already done it with the following regex:



preg_match_all("~((.*?))~", $truth, $str);


But the problem is that it returns what is between the first "(" and the first ")", which is




(A^¬B




Instead of this, i need it to 'know' where the parentheses closes correctly, in order to return




(A^¬B)->C




How can I return this respecting the priority order? Thanks!










share|improve this question























  • You could just make an exclusion group and match anything but parenthesis with [^()]* instead of .*, but you might probably still run into problems depending on the complexity of the expression you're trying to parse, specially if it's malformed. Regular expressions are handy but they don't apply to every parsing problem.
    – Havenard
    Nov 11 at 1:43











  • Regular expressions are not adequate for parsing a language. Try a parser generator. stackoverflow.com/questions/3720362/…
    – Ralph Ritoch
    Nov 11 at 1:46














up vote
2
down vote

favorite
1












I need to parse a string that contains some parentheses disposed recursively, but i'm having trouble with determining priority of parentheses.
For exemple, I have the string



$truth = "((A^¬B)->C)";


and I need to return what is between the parentheses. I've already done it with the following regex:



preg_match_all("~((.*?))~", $truth, $str);


But the problem is that it returns what is between the first "(" and the first ")", which is




(A^¬B




Instead of this, i need it to 'know' where the parentheses closes correctly, in order to return




(A^¬B)->C




How can I return this respecting the priority order? Thanks!










share|improve this question























  • You could just make an exclusion group and match anything but parenthesis with [^()]* instead of .*, but you might probably still run into problems depending on the complexity of the expression you're trying to parse, specially if it's malformed. Regular expressions are handy but they don't apply to every parsing problem.
    – Havenard
    Nov 11 at 1:43











  • Regular expressions are not adequate for parsing a language. Try a parser generator. stackoverflow.com/questions/3720362/…
    – Ralph Ritoch
    Nov 11 at 1:46












up vote
2
down vote

favorite
1









up vote
2
down vote

favorite
1






1





I need to parse a string that contains some parentheses disposed recursively, but i'm having trouble with determining priority of parentheses.
For exemple, I have the string



$truth = "((A^¬B)->C)";


and I need to return what is between the parentheses. I've already done it with the following regex:



preg_match_all("~((.*?))~", $truth, $str);


But the problem is that it returns what is between the first "(" and the first ")", which is




(A^¬B




Instead of this, i need it to 'know' where the parentheses closes correctly, in order to return




(A^¬B)->C




How can I return this respecting the priority order? Thanks!










share|improve this question















I need to parse a string that contains some parentheses disposed recursively, but i'm having trouble with determining priority of parentheses.
For exemple, I have the string



$truth = "((A^¬B)->C)";


and I need to return what is between the parentheses. I've already done it with the following regex:



preg_match_all("~((.*?))~", $truth, $str);


But the problem is that it returns what is between the first "(" and the first ")", which is




(A^¬B




Instead of this, i need it to 'know' where the parentheses closes correctly, in order to return




(A^¬B)->C




How can I return this respecting the priority order? Thanks!







php regex






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited Nov 11 at 1:37









Nick

20.1k51434




20.1k51434










asked Nov 11 at 1:22









Nicholas Ferreira

284




284











  • You could just make an exclusion group and match anything but parenthesis with [^()]* instead of .*, but you might probably still run into problems depending on the complexity of the expression you're trying to parse, specially if it's malformed. Regular expressions are handy but they don't apply to every parsing problem.
    – Havenard
    Nov 11 at 1:43











  • Regular expressions are not adequate for parsing a language. Try a parser generator. stackoverflow.com/questions/3720362/…
    – Ralph Ritoch
    Nov 11 at 1:46
















  • You could just make an exclusion group and match anything but parenthesis with [^()]* instead of .*, but you might probably still run into problems depending on the complexity of the expression you're trying to parse, specially if it's malformed. Regular expressions are handy but they don't apply to every parsing problem.
    – Havenard
    Nov 11 at 1:43











  • Regular expressions are not adequate for parsing a language. Try a parser generator. stackoverflow.com/questions/3720362/…
    – Ralph Ritoch
    Nov 11 at 1:46















You could just make an exclusion group and match anything but parenthesis with [^()]* instead of .*, but you might probably still run into problems depending on the complexity of the expression you're trying to parse, specially if it's malformed. Regular expressions are handy but they don't apply to every parsing problem.
– Havenard
Nov 11 at 1:43





You could just make an exclusion group and match anything but parenthesis with [^()]* instead of .*, but you might probably still run into problems depending on the complexity of the expression you're trying to parse, specially if it's malformed. Regular expressions are handy but they don't apply to every parsing problem.
– Havenard
Nov 11 at 1:43













Regular expressions are not adequate for parsing a language. Try a parser generator. stackoverflow.com/questions/3720362/…
– Ralph Ritoch
Nov 11 at 1:46




Regular expressions are not adequate for parsing a language. Try a parser generator. stackoverflow.com/questions/3720362/…
– Ralph Ritoch
Nov 11 at 1:46












2 Answers
2






active

oldest

votes

















up vote
3
down vote



accepted










The main problem you have right now is the ? non-greedy bit. If you change that to just .+ greedy it will match what you want.



$truth = "((A^¬B)->C)";
preg_match('/(.+)/', $truth, $match);


Try it



Output



(A^¬B)->C


If you want to match the inner pair you can use a recursive subpattern:



$truth = "((A^¬B)->C)";
preg_match('/(([^()]+|(?0)))/', $truth, $match);


Try It online



Output



A^¬B


If you need to go further then that you can make a lexer/parser. I have some examples here:



https://github.com/ArtisticPhoenix/MISC/tree/master/Lexers






share|improve this answer






















  • Thanks! It solved my problem. And thanks to the others too, it will be useful. =D
    – Nicholas Ferreira
    Nov 11 at 2:14










  • Sure I just added my output converter to my website, artisticphoenix.com/2018/11/11/output-converter it uses the same parsing idea but can convert var_export and print_r to usable arrays. Something I have to do a lot on here... lol
    – ArtisticPhoenix
    Nov 11 at 2:31










  • @ArtisticPhoenix I was just thinking I was going to have to write the same tool myself! Thanks for sharing...
    – Nick
    Nov 11 at 3:50










  • Sure, My site is still a work in progress. lol. I don't get a lot of time to work on it unfortunately
    – ArtisticPhoenix
    Nov 11 at 3:53

















up vote
3
down vote













For your sample string, something like this will recursively give you the contents of the parentheses. It works by forcing the parentheses matched to be the outermost pair by using ^[^(]* and [^)]*$ at each end of the regex.



$truth = "((A^¬B)->C)";
while (strpos($truth, '(') !== false)
preg_match("~^[^(]*((.*?))[^)]*$~", $truth, $str);
$truth = $str[1];
echo "$truthn";



Output



(A^¬B)->C 
A^¬B


Note however this will not correctly parse a string such as (A+B)-(C+D). If that could be your scenario, this answer might help.



Demo on 3v4l.org






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',
    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%2f53245032%2fhow-to-parse-recursive-parentheses-correctly%23new-answer', 'question_page');

    );

    Post as a guest















    Required, but never shown

























    2 Answers
    2






    active

    oldest

    votes








    2 Answers
    2






    active

    oldest

    votes









    active

    oldest

    votes






    active

    oldest

    votes








    up vote
    3
    down vote



    accepted










    The main problem you have right now is the ? non-greedy bit. If you change that to just .+ greedy it will match what you want.



    $truth = "((A^¬B)->C)";
    preg_match('/(.+)/', $truth, $match);


    Try it



    Output



    (A^¬B)->C


    If you want to match the inner pair you can use a recursive subpattern:



    $truth = "((A^¬B)->C)";
    preg_match('/(([^()]+|(?0)))/', $truth, $match);


    Try It online



    Output



    A^¬B


    If you need to go further then that you can make a lexer/parser. I have some examples here:



    https://github.com/ArtisticPhoenix/MISC/tree/master/Lexers






    share|improve this answer






















    • Thanks! It solved my problem. And thanks to the others too, it will be useful. =D
      – Nicholas Ferreira
      Nov 11 at 2:14










    • Sure I just added my output converter to my website, artisticphoenix.com/2018/11/11/output-converter it uses the same parsing idea but can convert var_export and print_r to usable arrays. Something I have to do a lot on here... lol
      – ArtisticPhoenix
      Nov 11 at 2:31










    • @ArtisticPhoenix I was just thinking I was going to have to write the same tool myself! Thanks for sharing...
      – Nick
      Nov 11 at 3:50










    • Sure, My site is still a work in progress. lol. I don't get a lot of time to work on it unfortunately
      – ArtisticPhoenix
      Nov 11 at 3:53














    up vote
    3
    down vote



    accepted










    The main problem you have right now is the ? non-greedy bit. If you change that to just .+ greedy it will match what you want.



    $truth = "((A^¬B)->C)";
    preg_match('/(.+)/', $truth, $match);


    Try it



    Output



    (A^¬B)->C


    If you want to match the inner pair you can use a recursive subpattern:



    $truth = "((A^¬B)->C)";
    preg_match('/(([^()]+|(?0)))/', $truth, $match);


    Try It online



    Output



    A^¬B


    If you need to go further then that you can make a lexer/parser. I have some examples here:



    https://github.com/ArtisticPhoenix/MISC/tree/master/Lexers






    share|improve this answer






















    • Thanks! It solved my problem. And thanks to the others too, it will be useful. =D
      – Nicholas Ferreira
      Nov 11 at 2:14










    • Sure I just added my output converter to my website, artisticphoenix.com/2018/11/11/output-converter it uses the same parsing idea but can convert var_export and print_r to usable arrays. Something I have to do a lot on here... lol
      – ArtisticPhoenix
      Nov 11 at 2:31










    • @ArtisticPhoenix I was just thinking I was going to have to write the same tool myself! Thanks for sharing...
      – Nick
      Nov 11 at 3:50










    • Sure, My site is still a work in progress. lol. I don't get a lot of time to work on it unfortunately
      – ArtisticPhoenix
      Nov 11 at 3:53












    up vote
    3
    down vote



    accepted







    up vote
    3
    down vote



    accepted






    The main problem you have right now is the ? non-greedy bit. If you change that to just .+ greedy it will match what you want.



    $truth = "((A^¬B)->C)";
    preg_match('/(.+)/', $truth, $match);


    Try it



    Output



    (A^¬B)->C


    If you want to match the inner pair you can use a recursive subpattern:



    $truth = "((A^¬B)->C)";
    preg_match('/(([^()]+|(?0)))/', $truth, $match);


    Try It online



    Output



    A^¬B


    If you need to go further then that you can make a lexer/parser. I have some examples here:



    https://github.com/ArtisticPhoenix/MISC/tree/master/Lexers






    share|improve this answer














    The main problem you have right now is the ? non-greedy bit. If you change that to just .+ greedy it will match what you want.



    $truth = "((A^¬B)->C)";
    preg_match('/(.+)/', $truth, $match);


    Try it



    Output



    (A^¬B)->C


    If you want to match the inner pair you can use a recursive subpattern:



    $truth = "((A^¬B)->C)";
    preg_match('/(([^()]+|(?0)))/', $truth, $match);


    Try It online



    Output



    A^¬B


    If you need to go further then that you can make a lexer/parser. I have some examples here:



    https://github.com/ArtisticPhoenix/MISC/tree/master/Lexers







    share|improve this answer














    share|improve this answer



    share|improve this answer








    edited Nov 11 at 2:02

























    answered Nov 11 at 1:56









    ArtisticPhoenix

    14.8k11223




    14.8k11223











    • Thanks! It solved my problem. And thanks to the others too, it will be useful. =D
      – Nicholas Ferreira
      Nov 11 at 2:14










    • Sure I just added my output converter to my website, artisticphoenix.com/2018/11/11/output-converter it uses the same parsing idea but can convert var_export and print_r to usable arrays. Something I have to do a lot on here... lol
      – ArtisticPhoenix
      Nov 11 at 2:31










    • @ArtisticPhoenix I was just thinking I was going to have to write the same tool myself! Thanks for sharing...
      – Nick
      Nov 11 at 3:50










    • Sure, My site is still a work in progress. lol. I don't get a lot of time to work on it unfortunately
      – ArtisticPhoenix
      Nov 11 at 3:53
















    • Thanks! It solved my problem. And thanks to the others too, it will be useful. =D
      – Nicholas Ferreira
      Nov 11 at 2:14










    • Sure I just added my output converter to my website, artisticphoenix.com/2018/11/11/output-converter it uses the same parsing idea but can convert var_export and print_r to usable arrays. Something I have to do a lot on here... lol
      – ArtisticPhoenix
      Nov 11 at 2:31










    • @ArtisticPhoenix I was just thinking I was going to have to write the same tool myself! Thanks for sharing...
      – Nick
      Nov 11 at 3:50










    • Sure, My site is still a work in progress. lol. I don't get a lot of time to work on it unfortunately
      – ArtisticPhoenix
      Nov 11 at 3:53















    Thanks! It solved my problem. And thanks to the others too, it will be useful. =D
    – Nicholas Ferreira
    Nov 11 at 2:14




    Thanks! It solved my problem. And thanks to the others too, it will be useful. =D
    – Nicholas Ferreira
    Nov 11 at 2:14












    Sure I just added my output converter to my website, artisticphoenix.com/2018/11/11/output-converter it uses the same parsing idea but can convert var_export and print_r to usable arrays. Something I have to do a lot on here... lol
    – ArtisticPhoenix
    Nov 11 at 2:31




    Sure I just added my output converter to my website, artisticphoenix.com/2018/11/11/output-converter it uses the same parsing idea but can convert var_export and print_r to usable arrays. Something I have to do a lot on here... lol
    – ArtisticPhoenix
    Nov 11 at 2:31












    @ArtisticPhoenix I was just thinking I was going to have to write the same tool myself! Thanks for sharing...
    – Nick
    Nov 11 at 3:50




    @ArtisticPhoenix I was just thinking I was going to have to write the same tool myself! Thanks for sharing...
    – Nick
    Nov 11 at 3:50












    Sure, My site is still a work in progress. lol. I don't get a lot of time to work on it unfortunately
    – ArtisticPhoenix
    Nov 11 at 3:53




    Sure, My site is still a work in progress. lol. I don't get a lot of time to work on it unfortunately
    – ArtisticPhoenix
    Nov 11 at 3:53












    up vote
    3
    down vote













    For your sample string, something like this will recursively give you the contents of the parentheses. It works by forcing the parentheses matched to be the outermost pair by using ^[^(]* and [^)]*$ at each end of the regex.



    $truth = "((A^¬B)->C)";
    while (strpos($truth, '(') !== false)
    preg_match("~^[^(]*((.*?))[^)]*$~", $truth, $str);
    $truth = $str[1];
    echo "$truthn";



    Output



    (A^¬B)->C 
    A^¬B


    Note however this will not correctly parse a string such as (A+B)-(C+D). If that could be your scenario, this answer might help.



    Demo on 3v4l.org






    share|improve this answer


























      up vote
      3
      down vote













      For your sample string, something like this will recursively give you the contents of the parentheses. It works by forcing the parentheses matched to be the outermost pair by using ^[^(]* and [^)]*$ at each end of the regex.



      $truth = "((A^¬B)->C)";
      while (strpos($truth, '(') !== false)
      preg_match("~^[^(]*((.*?))[^)]*$~", $truth, $str);
      $truth = $str[1];
      echo "$truthn";



      Output



      (A^¬B)->C 
      A^¬B


      Note however this will not correctly parse a string such as (A+B)-(C+D). If that could be your scenario, this answer might help.



      Demo on 3v4l.org






      share|improve this answer
























        up vote
        3
        down vote










        up vote
        3
        down vote









        For your sample string, something like this will recursively give you the contents of the parentheses. It works by forcing the parentheses matched to be the outermost pair by using ^[^(]* and [^)]*$ at each end of the regex.



        $truth = "((A^¬B)->C)";
        while (strpos($truth, '(') !== false)
        preg_match("~^[^(]*((.*?))[^)]*$~", $truth, $str);
        $truth = $str[1];
        echo "$truthn";



        Output



        (A^¬B)->C 
        A^¬B


        Note however this will not correctly parse a string such as (A+B)-(C+D). If that could be your scenario, this answer might help.



        Demo on 3v4l.org






        share|improve this answer














        For your sample string, something like this will recursively give you the contents of the parentheses. It works by forcing the parentheses matched to be the outermost pair by using ^[^(]* and [^)]*$ at each end of the regex.



        $truth = "((A^¬B)->C)";
        while (strpos($truth, '(') !== false)
        preg_match("~^[^(]*((.*?))[^)]*$~", $truth, $str);
        $truth = $str[1];
        echo "$truthn";



        Output



        (A^¬B)->C 
        A^¬B


        Note however this will not correctly parse a string such as (A+B)-(C+D). If that could be your scenario, this answer might help.



        Demo on 3v4l.org







        share|improve this answer














        share|improve this answer



        share|improve this answer








        edited Nov 11 at 2:12

























        answered Nov 11 at 1:37









        Nick

        20.1k51434




        20.1k51434



























             

            draft saved


            draft discarded















































             


            draft saved


            draft discarded














            StackExchange.ready(
            function ()
            StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53245032%2fhow-to-parse-recursive-parentheses-correctly%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

            Top Tejano songwriter Luis Silva dead of heart attack at 64

            ReactJS Fetched API data displays live - need Data displayed static

            政党