Reduce processing time for re.sub() in a nested loop Planned maintenance scheduled April 23, 2019 at 00:00UTC (8:00pm US/Eastern) Announcing the arrival of Valued Associate #679: Cesar Manara Unicorn Meta Zoo #1: Why another podcast?Parse emails and create HTML markup from attachmentsPython code to write simple VPK archive filesReusable, generic save methodMod maker (creates XML files to be used as mods)Finding all non-empty directories and their files on an SFTP server with ParamikoGroup duplicate files (part 3)Console application to search for files that contain a match for a regexParsing million words of list in python regular expression is slowIdentify Duplicate FilesC Recursive Opendir Wrapper to Sort Directories First (ascending/descending)

How to play a character with a disability or mental disorder without being offensive?

Is CEO the "profession" with the most psychopaths?

Why do early math courses focus on the cross sections of a cone and not on other 3D objects?

Trademark violation for app?

Why is it faster to reheat something than it is to cook it?

Why is the AVR GCC compiler using a full `CALL` even though I have set the `-mshort-calls` flag?

ArcGIS Pro Python arcpy.CreatePersonalGDB_management

How does the math work when buying airline miles?

Taylor expansion of ln(1-x)

Hangman Game with C++

Why does the remaining Rebel fleet at the end of Rogue One seem dramatically larger than the one in A New Hope?

How does Python know the values already stored in its memory?

What is "gratricide"?

Why should I vote and accept answers?

Can the Great Weapon Master feat's damage bonus and accuracy penalty apply to attacks from the Spiritual Weapon spell?

Illegal assignment from sObject to Id

Question about debouncing - delay of state change

Find 108 by using 3,4,6

Can anything be seen from the center of the Boötes void? How dark would it be?

Most bit efficient text communication method?

Disembodied hand growing fangs

How much damage would a cupful of neutron star matter do to the Earth?

How to write this math term? with cases it isn't working

If Windows 7 doesn't support WSL, then what does Linux subsystem option mean?



Reduce processing time for re.sub() in a nested loop



Planned maintenance scheduled April 23, 2019 at 00:00UTC (8:00pm US/Eastern)
Announcing the arrival of Valued Associate #679: Cesar Manara
Unicorn Meta Zoo #1: Why another podcast?Parse emails and create HTML markup from attachmentsPython code to write simple VPK archive filesReusable, generic save methodMod maker (creates XML files to be used as mods)Finding all non-empty directories and their files on an SFTP server with ParamikoGroup duplicate files (part 3)Console application to search for files that contain a match for a regexParsing million words of list in python regular expression is slowIdentify Duplicate FilesC Recursive Opendir Wrapper to Sort Directories First (ascending/descending)



.everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty,.everyoneloves__bot-mid-leaderboard:empty margin-bottom:0;








0












$begingroup$


I am preprocessing a set of files where one file is included in another file with an include tag like shown below:



file A



include file B
include file C
contents of file A
include file D
contents of file A


Here i need to replace the include tags with contents of each of the respective files. Just like what a compiler does. I have two functions for that



 def parseContent(self, rdbFilePath, content):
finalContent = self.removeComments(content)
includeFileSearch = re.compile(r'(?P<tag>(s)*include+(s*)"(s*)(?P<file>[a-zA-Z0-9._/]*)")')
for includes in includeFileSearch.finditer(finalContent):
finalContent = re.sub(includes.group('tag'),self.parseIncludes(rdbFilePath, includes.group('file')), finalContent)
return finalContent

def parseIncludes(self, rdbFilePath, file):
path = rdbFilePath + "/" + file
f = open(path)
pathDir = os.path.dirname(path)
includedFileContent = self.parseContent(pathDir, f.read())
return includedFileContent


As you can see, functions parseContent and parseIncludes is called in each other recursively to replace all the include tags in every file. The logic works fine. But it takes a bit long time to execute. Is there any better way to do the same with lesser execution time?









share







New contributor




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







$endgroup$


















    0












    $begingroup$


    I am preprocessing a set of files where one file is included in another file with an include tag like shown below:



    file A



    include file B
    include file C
    contents of file A
    include file D
    contents of file A


    Here i need to replace the include tags with contents of each of the respective files. Just like what a compiler does. I have two functions for that



     def parseContent(self, rdbFilePath, content):
    finalContent = self.removeComments(content)
    includeFileSearch = re.compile(r'(?P<tag>(s)*include+(s*)"(s*)(?P<file>[a-zA-Z0-9._/]*)")')
    for includes in includeFileSearch.finditer(finalContent):
    finalContent = re.sub(includes.group('tag'),self.parseIncludes(rdbFilePath, includes.group('file')), finalContent)
    return finalContent

    def parseIncludes(self, rdbFilePath, file):
    path = rdbFilePath + "/" + file
    f = open(path)
    pathDir = os.path.dirname(path)
    includedFileContent = self.parseContent(pathDir, f.read())
    return includedFileContent


    As you can see, functions parseContent and parseIncludes is called in each other recursively to replace all the include tags in every file. The logic works fine. But it takes a bit long time to execute. Is there any better way to do the same with lesser execution time?









    share







    New contributor




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







    $endgroup$














      0












      0








      0





      $begingroup$


      I am preprocessing a set of files where one file is included in another file with an include tag like shown below:



      file A



      include file B
      include file C
      contents of file A
      include file D
      contents of file A


      Here i need to replace the include tags with contents of each of the respective files. Just like what a compiler does. I have two functions for that



       def parseContent(self, rdbFilePath, content):
      finalContent = self.removeComments(content)
      includeFileSearch = re.compile(r'(?P<tag>(s)*include+(s*)"(s*)(?P<file>[a-zA-Z0-9._/]*)")')
      for includes in includeFileSearch.finditer(finalContent):
      finalContent = re.sub(includes.group('tag'),self.parseIncludes(rdbFilePath, includes.group('file')), finalContent)
      return finalContent

      def parseIncludes(self, rdbFilePath, file):
      path = rdbFilePath + "/" + file
      f = open(path)
      pathDir = os.path.dirname(path)
      includedFileContent = self.parseContent(pathDir, f.read())
      return includedFileContent


      As you can see, functions parseContent and parseIncludes is called in each other recursively to replace all the include tags in every file. The logic works fine. But it takes a bit long time to execute. Is there any better way to do the same with lesser execution time?









      share







      New contributor




      Hari Krishnan 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 preprocessing a set of files where one file is included in another file with an include tag like shown below:



      file A



      include file B
      include file C
      contents of file A
      include file D
      contents of file A


      Here i need to replace the include tags with contents of each of the respective files. Just like what a compiler does. I have two functions for that



       def parseContent(self, rdbFilePath, content):
      finalContent = self.removeComments(content)
      includeFileSearch = re.compile(r'(?P<tag>(s)*include+(s*)"(s*)(?P<file>[a-zA-Z0-9._/]*)")')
      for includes in includeFileSearch.finditer(finalContent):
      finalContent = re.sub(includes.group('tag'),self.parseIncludes(rdbFilePath, includes.group('file')), finalContent)
      return finalContent

      def parseIncludes(self, rdbFilePath, file):
      path = rdbFilePath + "/" + file
      f = open(path)
      pathDir = os.path.dirname(path)
      includedFileContent = self.parseContent(pathDir, f.read())
      return includedFileContent


      As you can see, functions parseContent and parseIncludes is called in each other recursively to replace all the include tags in every file. The logic works fine. But it takes a bit long time to execute. Is there any better way to do the same with lesser execution time?







      python recursion regex





      share







      New contributor




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










      share







      New contributor




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








      share



      share






      New contributor




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









      asked 6 mins ago









      Hari KrishnanHari Krishnan

      1011




      1011




      New contributor




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





      New contributor





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






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




















          0






          active

          oldest

          votes












          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: "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
          );



          );






          Hari Krishnan 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%2f217712%2freduce-processing-time-for-re-sub-in-a-nested-loop%23new-answer', 'question_page');

          );

          Post as a guest















          Required, but never shown

























          0






          active

          oldest

          votes








          0






          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes








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









          draft saved

          draft discarded


















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












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











          Hari Krishnan 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%2f217712%2freduce-processing-time-for-re-sub-in-a-nested-loop%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 - 經濟部水利署中區水資源局

          Is my guitar’s action too high? Announcing the arrival of Valued Associate #679: Cesar Manara Planned maintenance scheduled April 23, 2019 at 23:30 UTC (7:30pm US/Eastern)Strings too stiff on a recently purchased acoustic guitar | Cort AD880CEIs the action of my guitar really high?Μy little finger is too weak to play guitarWith guitar, how long should I give my fingers to strengthen / callous?When playing a fret the guitar sounds mutedPlaying (Barre) chords up the guitar neckI think my guitar strings are wound too tight and I can't play barre chordsF barre chord on an SG guitarHow to find to the right strings of a barre chord by feel?High action on higher fret on my steel acoustic guitar

          香港授勳及嘉獎制度 目录 勳章及獎狀類別 嘉獎等級 授勳及嘉獎提名 統計數字 多次獲頒勳章或獎狀的人士 爭議 褫奪機制 参考文献 外部連結 参见 导航菜单統計數字一九九七年七月二日(星期三)香港特別行政區的授勳制度六七暴動領袖獲大紫荊勳章 董建華被斥為肯定殺人放火董建華授勳楊光 議員窮追猛打蘋論:顛倒是非黑白的大紫荊董讚楊光有貢獻避談暴動董拒答授勳楊光原因撤除勳銜撤除勳銜撤除勳銜特首掌「搣柴」生殺權行為失當罪 隨時「搣柴」失長糧政府刊憲 許仕仁郭炳江遭「搣柴」去年中終極上訴失敗 許仕仁郭炳江撤勳章太平紳士猛料阿Sir講古—— 「搣柴」有故一九九八年授勳名單一九九九年授勳名單二○○三年授勳名單二○○八年授勳名單二○○七年授勳名單政府總部禮賓處 - 授勳及嘉獎香港特別行政區勳章綬帶一覽(PDF)(非官方)