Regarding replacing regexp

Regarding replacing regexp

[shirish]

[-- Attachment #1: Type: text/plain, Size: 459 bytes --]

Hi,

I am in the process of learning how to use emacs. I know about the
replace-regexp but I am unable to create a regular expression to do the
below.
Any help is much appreciated.

public static final String SUCCESS //Successmesg
public static final String FAILURE  //failuremessage

I need to replace that with

public static final String SUCCESS = "SUCCESS";//Successmesg
public static final String FAILURE  = "FAILURE"; //failuremessage

Thanks,
Shirish.

[-- Attachment #2: Type: text/html, Size: 770 bytes --]

Re: Regarding replacing regexp

[Deniz Dogan]

On 2012-05-10 11:37, shirish wrote:
> Hi,
>
> I am in the process of learning how to use emacs. I know about the
> replace-regexp but I am unable to create a regular expression to do the
> below.
> Any help is much appreciated.
>
> public static final String SUCCESS //Successmesg
> public static final String FAILURE  //failuremessage
>
> I need to replace that with
>
> public static final String SUCCESS = "SUCCESS";//Successmesg
> public static final String FAILURE  = "FAILURE"; //failuremessage
>
> Thanks,
> Shirish.
>
>

There are a lot of ways to do it.  One could be:

String \([^ ]+\)

and replace it with:

String \1 = "\1"

I would personally use keyboard macros for it.

I hope that helps,
Deniz

Re: Regarding replacing regexp

[XeCycle]

[-- Attachment #1: Type: text/plain, Size: 1148 bytes --]

Deniz Dogan <deniz@dogan.se> writes:

> On 2012-05-10 11:37, shirish wrote:
>> Hi,
>>
>> I am in the process of learning how to use emacs. I know about the
>> replace-regexp but I am unable to create a regular expression to do the
>> below.
>> Any help is much appreciated.
>>
>> public static final String SUCCESS //Successmesg
>> public static final String FAILURE  //failuremessage
>>
>> I need to replace that with
>>
>> public static final String SUCCESS = "SUCCESS";//Successmesg
>> public static final String FAILURE  = "FAILURE"; //failuremessage
>>
>> Thanks,
>> Shirish.
>>
>>
>
> There are a lot of ways to do it.  One could be:
>
> String \([^ ]+\)
>
> and replace it with:
>
> String \1 = "\1"

This won't do very well.

Try:

SUCCESS\|FAILURE => \& = "\&";

I may be wrong about those escapes, try it.

> I would personally use keyboard macros for it.

Yes, This would be fine, too.  And usually easier than regexps.

-- 
Carl Lei (XeCycle)
Department of Physics, Shanghai Jiao Tong University
OpenPGP public key: 7795E591
Fingerprint: 1FB6 7F1F D45D F681 C845 27F7 8D71 8EC4 7795 E591

[-- Attachment #2: Type: application/pgp-signature, Size: 489 bytes --]

Re: Regarding replacing regexp

[Barry Margolin]

In article <mailman.942.1336705240.855.help-gnu-emacs@gnu.org>,
 XeCycle <XeCycle@Gmail.com> wrote:

> Deniz Dogan <deniz@dogan.se> writes:
> 
> > On 2012-05-10 11:37, shirish wrote:
> >> Hi,
> >>
> >> I am in the process of learning how to use emacs. I know about the
> >> replace-regexp but I am unable to create a regular expression to do the
> >> below.
> >> Any help is much appreciated.
> >>
> >> public static final String SUCCESS //Successmesg
> >> public static final String FAILURE  //failuremessage
> >>
> >> I need to replace that with
> >>
> >> public static final String SUCCESS = "SUCCESS";//Successmesg
> >> public static final String FAILURE  = "FAILURE"; //failuremessage
> >>
> >> Thanks,
> >> Shirish.
> >>
> >>
> >
> > There are a lot of ways to do it.  One could be:
> >
> > String \([^ ]+\)
> >
> > and replace it with:
> >
> > String \1 = "\1"
> 
> This won't do very well.

Except that he forgot the semicolon, what's wrong about it?

> 
> Try:
> 
> SUCCESS\|FAILURE => \& = "\&";
> 
> I may be wrong about those escapes, try it.

You're assuming those are the only names he has to replace.  The first 
responder assumed that they were just examples of a common pattern.

Plus, your version will replace them in other contexts, e.g. the program 
is likely to contain something like:

if (...) return SUCCESS;

but you wouldn't want to change this with the replacement.

-- 
Barry Margolin, barmar@alum.mit.edu
Arlington, MA
*** PLEASE post questions in newsgroups, not directly to me ***

Re: Regarding replacing regexp

[XeCycle]

[-- Attachment #1: Type: text/plain, Size: 1463 bytes --]

Barry Margolin <barmar@alum.mit.edu> writes:

[...]

>> > There are a lot of ways to do it.  One could be:
>> >
>> > String \([^ ]+\)
>> >
>> > and replace it with:
>> >
>> > String \1 = "\1"
>> 
>> This won't do very well.
>
> Except that he forgot the semicolon, what's wrong about it?

May match something more, of course; mine has the same problem.

>> 
>> Try:
>> 
>> SUCCESS\|FAILURE => \& = "\&";
>> 
>> I may be wrong about those escapes, try it.
>
> You're assuming those are the only names he has to replace.  The first 
> responder assumed that they were just examples of a common pattern.

Huh?  Don't know what the OP want, though.

> Plus, your version will replace them in other contexts, e.g. the program 
> is likely to contain something like:
>
> if (...) return SUCCESS;
>
> but you wouldn't want to change this with the replacement.

That's a common problem about such replacements.  Similarly one
may want to define a String object somewhere else, Deniz's
solution will fail, too.  Query replace to the rescue.  It's up
to the OP to decide which one will be better, or combine them to
remove more mismatches:

String \(SUCCESS\|FAILURE\)

Of course, those tools are rarely accurate in such things like
code refactoring.

-- 
Carl Lei (XeCycle)
Department of Physics, Shanghai Jiao Tong University
OpenPGP public key: 7795E591
Fingerprint: 1FB6 7F1F D45D F681 C845 27F7 8D71 8EC4 7795 E591

[-- Attachment #2: Type: application/pgp-signature, Size: 489 bytes --]

Re: Regarding replacing regexp

[shirish]

[-- Attachment #1: Type: text/plain, Size: 842 bytes --]

Thanks Demiz.


On Thu, May 10, 2012 at 9:56 PM, Deniz Dogan <deniz@dogan.se> wrote:

> On 2012-05-10 11:37, shirish wrote:
>
>> Hi,
>>
>> I am in the process of learning how to use emacs. I know about the
>> replace-regexp but I am unable to create a regular expression to do the
>> below.
>> Any help is much appreciated.
>>
>> public static final String SUCCESS //Successmesg
>> public static final String FAILURE  //failuremessage
>>
>> I need to replace that with
>>
>> public static final String SUCCESS = "SUCCESS";//Successmesg
>> public static final String FAILURE  = "FAILURE"; //failuremessage
>>
>> Thanks,
>> Shirish.
>>
>>
>>
> There are a lot of ways to do it.  One could be:
>
> String \([^ ]+\)
>
> and replace it with:
>
> String \1 = "\1"
>
> I would personally use keyboard macros for it.
>
> I hope that helps,
> Deniz
>

[-- Attachment #2: Type: text/html, Size: 1445 bytes --]

Re: Regarding replacing regexp

[Peter Dyballa]

Am 10.05.2012 um 11:37 schrieb shirish:

> public static final String SUCCESS //Successmesg
> public static final String FAILURE  //failuremessage
> 
> I need to replace that with
> 
> public static final String SUCCESS = "SUCCESS";//Successmesg
> public static final String FAILURE  = "FAILURE"; //failuremessage

You could try:

	\(final \w+ \)\([A-Z]+\) -> \1\2 = "\2";

The text "final <some word> " followed by a sequence of upper-case letters (\([A-Z]+\)) is found and saved in registers and substituted with the first finding followed by '=' and the found sequence of upper-case letters in double quotation marks followed by ';'. The remainder of the original line stays unchanged.

--
Greetings

  Pete

One cannot live by television, video games, top ten CDs, and dumb movies alone.
				– Amiri Baraka, 1999

Re: Regarding replacing regexp

[shirish]

[-- Attachment #1: Type: text/plain, Size: 1031 bytes --]

Thanks Peter


On Fri, May 11, 2012 at 2:09 PM, Peter Dyballa <Peter_Dyballa@web.de> wrote:

>
> Am 10.05.2012 um 11:37 schrieb shirish:
>
> > public static final String SUCCESS //Successmesg
> > public static final String FAILURE  //failuremessage
> >
> > I need to replace that with
> >
> > public static final String SUCCESS = "SUCCESS";//Successmesg
> > public static final String FAILURE  = "FAILURE"; //failuremessage
>
> You could try:
>
>        \(final \w+ \)\([A-Z]+\) -> \1\2 = "\2";
>
> The text "final <some word> " followed by a sequence of upper-case letters
> (\([A-Z]+\)) is found and saved in registers and substituted with the first
> finding followed by '=' and the found sequence of upper-case letters in
> double quotation marks followed by ';'. The remainder of the original line
> stays unchanged.
>
> --
> Greetings
>
>  Pete
>
> One cannot live by television, video games, top ten CDs, and dumb movies
> alone.
>                                – Amiri Baraka, 1999
>
>

[-- Attachment #2: Type: text/html, Size: 1506 bytes --]