Processing file inclusion directives using mutual recursion 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)
Can the Great Weapon Master feat's damage bonus and accuracy penalty apply to attacks from the Spiritual Weapon spell?
Crossing US/Canada Border for less than 24 hours
Putting class ranking in CV, but against dept guidelines
How could we fake a moon landing now?
Most bit efficient text communication method?
Is it possible for SQL statements to execute concurrently within a single session in SQL Server?
Denied boarding although I have proper visa and documentation. To whom should I make a complaint?
Chinese Seal on silk painting - what does it mean?
Why do we need to use the builder design pattern when we can do the same thing with setters?
Maximum summed subsequences with non-adjacent items
How do I use the new nonlinear finite element in Mathematica 12 for this equation?
Effects on objects due to a brief relocation of massive amounts of mass
Is there any word for a place full of confusion?
Morning, Afternoon, Night Kanji
What is the topology associated with the algebras for the ultrafilter monad?
Why should I vote and accept answers?
What initially awakened the Balrog?
Why do we bend a book to keep it straight?
Performance gap between vector<bool> and array
What is a fractional matching?
What is this clumpy 20-30cm high yellow-flowered plant?
Drawing without replacement: why is the order of draw irrelevant?
Can anything be seen from the center of the Boötes void? How dark would it be?
Dating a Former Employee
Processing file inclusion directives using mutual recursion
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;
$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 call 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 performance recursion regex file
edited 7 mins ago
200_success
131k17157422
asked 16 mins ago
Hari KrishnanHari Krishnan
1062
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$
add a comment |
$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 call 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 performance recursion regex file
edited 7 mins ago
200_success
131k17157422
asked 16 mins ago
Hari KrishnanHari Krishnan
1062
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$
$begingroup$
Are you parsing XML with regex? You need to read this
$endgroup$
– AJNeufeld
7 mins ago
add a comment |
$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 call 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 performance recursion regex file
edited 7 mins ago
200_success
131k17157422
asked 16 mins ago
Hari KrishnanHari Krishnan
1062
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 call 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 performance recursion regex file
python performance recursion regex file
edited 7 mins ago
200_success
131k17157422
asked 16 mins ago
Hari KrishnanHari Krishnan
1062
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.
edited 7 mins ago
200_success
131k17157422
asked 16 mins ago
Hari KrishnanHari Krishnan
1062
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.
edited 7 mins ago
200_success
131k17157422
edited 7 mins ago
200_success
131k17157422
edited 7 mins ago
200_success
131k17157422
131k17157422
asked 16 mins ago
Hari KrishnanHari Krishnan
1062
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 16 mins ago
Hari KrishnanHari Krishnan
1062
asked 16 mins ago
Hari KrishnanHari Krishnan
1062
1062
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.
$begingroup$
Are you parsing XML with regex? You need to read this
$endgroup$
– AJNeufeld
7 mins ago
add a comment |
$begingroup$
Are you parsing XML with regex? You need to read this
$endgroup$
– AJNeufeld
7 mins ago
$begingroup$
Are you parsing XML with regex? You need to read this
$endgroup$
– AJNeufeld
7 mins ago
$begingroup$
Are you parsing XML with regex? You need to read this
$endgroup$
– AJNeufeld
7 mins ago
add a comment |
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.
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
var $window = $(window),
onScroll = function(e)
var $elem = $('.new-login-left'),
docViewTop = $window.scrollTop(),
docViewBottom = docViewTop + $window.height(),
elemTop = $elem.offset().top,
elemBottom = elemTop + $elem.height();
if ((docViewTop elemBottom))
StackExchange.using('gps', function() StackExchange.gps.track('embedded_signup_form.view', location: 'question_page' ); );
$window.unbind('scroll', onScroll);
;
$window.on('scroll', onScroll);
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f217712%2fprocessing-file-inclusion-directives-using-mutual-recursion%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.
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.
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
var $window = $(window),
onScroll = function(e)
var $elem = $('.new-login-left'),
docViewTop = $window.scrollTop(),
docViewBottom = docViewTop + $window.height(),
elemTop = $elem.offset().top,
elemBottom = elemTop + $elem.height();
if ((docViewTop elemBottom))
StackExchange.using('gps', function() StackExchange.gps.track('embedded_signup_form.view', location: 'question_page' ); );
$window.unbind('scroll', onScroll);
;
$window.on('scroll', onScroll);
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f217712%2fprocessing-file-inclusion-directives-using-mutual-recursion%23new-answer', 'question_page');
);
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
var $window = $(window),
onScroll = function(e)
var $elem = $('.new-login-left'),
docViewTop = $window.scrollTop(),
docViewBottom = docViewTop + $window.height(),
elemTop = $elem.offset().top,
elemBottom = elemTop + $elem.height();
if ((docViewTop elemBottom))
StackExchange.using('gps', function() StackExchange.gps.track('embedded_signup_form.view', location: 'question_page' ); );
$window.unbind('scroll', onScroll);
;
$window.on('scroll', onScroll);
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
var $window = $(window),
onScroll = function(e)
var $elem = $('.new-login-left'),
docViewTop = $window.scrollTop(),
docViewBottom = docViewTop + $window.height(),
elemTop = $elem.offset().top,
elemBottom = elemTop + $elem.height();
if ((docViewTop elemBottom))
StackExchange.using('gps', function() StackExchange.gps.track('embedded_signup_form.view', location: 'question_page' ); );
$window.unbind('scroll', onScroll);
;
$window.on('scroll', onScroll);
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
var $window = $(window),
onScroll = function(e)
var $elem = $('.new-login-left'),
docViewTop = $window.scrollTop(),
docViewBottom = docViewTop + $window.height(),
elemTop = $elem.offset().top,
elemBottom = elemTop + $elem.height();
if ((docViewTop elemBottom))
StackExchange.using('gps', function() StackExchange.gps.track('embedded_signup_form.view', location: 'question_page' ); );
$window.unbind('scroll', onScroll);
;
$window.on('scroll', onScroll);
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
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
$begingroup$
Are you parsing XML with regex? You need to read this
$endgroup$
– AJNeufeld
7 mins ago