Skip to main content

string manipulation - How to write a function to remove comments from a .m source file preserving formatting such as line wrapping reasonably?


How to write a function to remove comments from a .m source file preserving formatting such as line wrapping reasonably?


This means if I have


foo;
(* comment 1 *)
bar[
baz (* comment 2 *)

(* comment 3 *)
];

I'd ideally end up getting


foo;
bar[
baz
];

Answer



Here is an alternative "first principle" approach, which does not use string patterns as a main tool, but instead makes use of the fact that comments have a simple structure and can only be escaped when they appear inside strings. Therefore, we can write a very simple parser which only parses strings and comments. Here is the tokenizer:



ClearAll[expr, parse, string, comments,tokenize, $commentPattern, $stringPattern];

tokenize[s_String] := StringSplit[s, t : "\\\"" | "\"" | "(*" | "*)" :> t];

Here are auxiliary patterns we will need:


$stringPattern = 
PatternSequence["\"", middle___, "\""] /; ! MemberQ[{middle}, "\""];

$commentPattern =
PatternSequence["(*", middle___, "*)"] /;

Count[{middle}, "(*"] == Count[{middle}, "*)"];

Here is the parser:


parse[left___, s : $stringPattern, right___] :=
expr[parse[left], string[s], parse[right]];

parse[left___, c : $commentPattern, right___] :=
expr[parse[left], comments[c], parse[right]];

parse[{tokens___}] := parse[tokens];

parse[tokens___] := expr[tokens];

The heads expr, string and comments are inert heads.


Finally, here is the function to remove comments from a string of code:


ClearAll[removeComments];
removeComments[s_String] :=
StringJoin[
DeleteCases[parse[tokenize@s], _comments, Infinity] /.
expr | string -> Sequence
]


Applying this to the initial string of code str as removeComments[str] returns the expected answer.


This parser won't be particularly fast. The reason I like this approach is that it does not rely on some external things such as specific forms of Import, so it will only be wrong if the principles are wrong (e.g. I missed some other forms where comments can be escaped, etc), but I consider string manipulations rather fragile for parsing purposes generally. Interestingly, this seems to be one of the simplest problems I know of which illustrates that regexps are not sufficient to parse code representing recursive (nested) expressions / statements.


Comments

Popular posts from this blog

functions - Get leading series expansion term?

Given a function f[x] , I would like to have a function leadingSeries that returns just the leading term in the series around x=0 . For example: leadingSeries[(1/x + 2)/(4 + 1/x^2 + x)] x and leadingSeries[(1/x + 2 + (1 - 1/x^3)/4)/(4 + x)] -(1/(16 x^3)) Is there such a function in Mathematica? Or maybe one can implement it efficiently? EDIT I finally went with the following implementation, based on Carl Woll 's answer: lds[ex_,x_]:=( (ex/.x->(x+O[x]^2))/.SeriesData[U_,Z_,L_List,Mi_,Ma_,De_]:>SeriesData[U,Z,{L[[1]]},Mi,Mi+1,De]//Quiet//Normal) The advantage is, that this one also properly works with functions whose leading term is a constant: lds[Exp[x],x] 1 Answer Update 1 Updated to eliminate SeriesData and to not return additional terms Perhaps you could use: leadingSeries[expr_, x_] := Normal[expr /. x->(x+O[x]^2) /. a_List :> Take[a, 1]] Then for your examples: leadingSeries[(1/x + 2)/(4 + 1/x^2 + x), x] leadingSeries[Exp[x], x] leadingSeries[(1/x + 2 + (1 - 1/x...

mathematical optimization - Minimizing using indices, error: Part::pkspec1: The expression cannot be used as a part specification

I want to use Minimize where the variables to minimize are indices pointing into an array. Here a MWE that hopefully shows what my problem is. vars = u@# & /@ Range[3]; cons = Flatten@ { Table[(u[j] != #) & /@ vars[[j + 1 ;; -1]], {j, 1, 3 - 1}], 1 vec1 = {1, 2, 3}; vec2 = {1, 2, 3}; Minimize[{Total@((vec1[[#]] - vec2[[u[#]]])^2 & /@ Range[1, 3]), cons}, vars, Integers] The error I get: Part::pkspec1: The expression u[1] cannot be used as a part specification. >> Answer Ok, it seems that one can get around Mathematica trying to evaluate vec2[[u[1]]] too early by using the function Indexed[vec2,u[1]] . The working MWE would then look like the following: vars = u@# & /@ Range[3]; cons = Flatten@{ Table[(u[j] != #) & /@ vars[[j + 1 ;; -1]], {j, 1, 3 - 1}], 1 vec1 = {1, 2, 3}; vec2 = {1, 2, 3}; NMinimize[ {Total@((vec1[[#]] - Indexed[vec2, u[#]])^2 & /@ R...

What is and isn't a valid variable specification for Manipulate?

I have an expression whose terms have arguments (representing subscripts), like this: myExpr = A[0] + V[1,T] I would like to put it inside a Manipulate to see its value as I move around the parameters. (The goal is eventually to plot it wrt one of the variables inside.) However, Mathematica complains when I set V[1,T] as a manipulated variable: Manipulate[Evaluate[myExpr], {A[0], 0, 1}, {V[1, T], 0, 1}] (*Manipulate::vsform: Manipulate argument {V[1,T],0,1} does not have the correct form for a variable specification. >> *) As a workaround, if I get rid of the symbol T inside the argument, it works fine: Manipulate[ Evaluate[myExpr /. T -> 15], {A[0], 0, 1}, {V[1, 15], 0, 1}] Why this behavior? Can anyone point me to the documentation that says what counts as a valid variable? And is there a way to get Manpiulate to accept an expression with a symbolic argument as a variable? Investigations I've done so far: I tried using variableQ from this answer , but it says V[1...