Beefy Boxes and Bandwidth Generously Provided by pair Networks
Problems? Is your data what you think it is?
 
PerlMonks  

Re^5: Regex: Identifying comments

by remiah (Hermit)
on Aug 31, 2012 at 05:37 UTC ( #990924=note: print w/ replies, xml ) Need Help??


in reply to Re^4: Regex: Identifying comments
in thread Regex: Identifying comments

umm..
No.

You will not see my regex fault with your examples. My regex will stumble with this sql.

update set bar = bar - 1 ; -- subtraction symbol may disappear.

I expected to see SQL parser solution in this thread, like this

my $p = SQLParser->new(type=>'mysql', sql=>$sql) or die SQLParser->error(); $p->prettyprint(1); $p->without_comment(1); print $p->sql;
At first I looked SQL::Parser. It seems quite near for such tasks, but I couldn't find good solution to rip off comments. Do you know such module?


Comment on Re^5: Regex: Identifying comments
Select or Download Code
Re^6: Regex: Identifying comments
by pvaldes (Chaplain) on Aug 31, 2012 at 15:32 UTC
    use strict; use warnings; while (<DATA>) { chomp; next if /^\s*--/; print $_,"\n" if !/--/; elsif (/--/ && !/--(.*?);/){ s/--(.*?)$//; print $_,"\n"} elsif (/--(.*?);\s*--(.*?)$/){ s/\s*--$2//; print $_,"\n"} else { print $_,"\n"} } __DATA__ select 'text' from foo; -- comment select '--Not comment' from foo; --But this is select q from z; -- as is this select '--Not this' + '--either' from foo; select 'qaws' + make from "a"; -- comment with 'a' quote select 'a' from 'b' with 'c' -- comment with 'a --' comment -- test comment (add1) select 'text\'s' from foo --escaped ... (add2) select 'text\'s' from foo --escaped' ... (add3) update set bar = bar - 1 ; -- subtraction symbol preserved create table ( -- fo field text, -- fufufu field int)

      I think you need to revisit this...

      syntax error at 991028.pl line 7, near "elsif" syntax error at 991028.pl line 10, near "elsif" syntax error at 991028.pl line 12, near ""\n"}" Execution of 991028.pl aborted due to compilation errors.
        first elsif should probably be "if" only

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://990924]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others drinking their drinks and smoking their pipes about the Monastery: (12)
As of 2014-10-21 09:58 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    For retirement, I am banking on:










    Results (99 votes), past polls