Re: a regex to split this...

You need to use the non-capturing pattern (?:foo) for the alternation unless you want it to be included in the results.

(undef, $userid) = split(qr{(?:/|\\)}, $ENV{'REMOTE_USER'});
[download]

Update
You actually don't need the grouping at all, and could just use alternation without parens, or could use a character class. These all work the same:

# Alternation with non-capturing clustering (?:foo)
(undef, $userid) = split(qr{(?:/|\\)}, $ENV{'REMOTE_USER'});

# Alternation without clustering
(undef, $userid) = split(qr{\\|/}, $ENV{'REMOTE_USER'});

# Character class including '\' and '/'
(undef, $userid) = split(qr{[\\/]}, $ENV{'REMOTE_USER'});
[download]

Comment on Re: a regex to split this... Select or Download Code

Replies are listed 'Best First'.
Re^2: a regex to split this... by Leviathan (Scribe) on Aug 09, 2006 at 12:11 UTC
You don't need to do that, because the delimiter is included in the result if the pattern has parenthese. `print join ":", split "-", "1-2-3-4"; # as opposed to: print join ":", split "(-)", "1-2-3-4";` [download] -- Leviathan.	[reply] [d/l]
Re^3: a regex to split this... by imp (Priest) on Aug 09, 2006 at 12:42 UTC
I initially reacted to this line in the OP: `$username =~ /(cyc(\\|\/)(.*?)/;` [download] Which had several problems. Does not assign the resulting match Should use non-capturing clustering or a character class for the '/' or '\' Has an extra opening paren Uses non-greedy match to select the username These would work: `($username) = $ENV{REMOTE_USER} =~ /cyc(?:\\\|\/)(.+)/; ($username) = $ENV{REMOTE_USER} =~ /cyc[\/\\](.+)/; # And to cleanup a little ($username) = $ENV{REMOTE_USER} =~ m{cyc(?:\\\|/)(.+)}; ($username) = $ENV{REMOTE_USER} =~ m{cyc[/\\](.+)};` [download] But not having coffee yet I corrected the split version instead. It's also worth nothing that `split("\\\|/","a\|/b")` does not work as one might think. `perl -e 'my ($a,$b) = split("\\\|/","a/b"); print "a:$a\nb:$b\n"' a:a/b b:` [download] This happens because that pattern is actually looking for a literal '\|/', as shown here: `perl -MO=Deparse -e 'my ($a,$b) = split("\\\|/","a/b");' my($a, $b) = split(m[\\|/], 'a/b', 3); perl -e 'my ($a,$b) = split("\\\|/","a\|/b"); print "a:$a\nb:$b\n"' a:a b:b` [download]	[reply] [d/l] [select]