How to replace a specific string using method?

Question

I have problem with my code , I want to replace specific string to a new one but it doesn't work

public void InsertYahoo(TextBox sender)
{
    if (IsGmail(sender))
    {
        ReplaceGmail(sender);
    }
    else if(IsYahoo(sender))
    {
        return;
    }
    else
    {
        sender.Text +="@yahoo.com";
    }
}

public bool IsYahoo(TextBox sender)
{
    if (sender.Text.Contains("@yahoo.com")
    {
        return true;
    }
    else
    { 
        return false;
    }
}

public bool IsGmail(TextBox sender)
{ 
    if (sender.Text.Contains("@gmail.com") 
    {
        return true;
    }
    else
    { 
        return false;
    }
}

public void ReplaceGmail(TextBox sender)
{
    sender.Text.Replace("@gmail.com, "@yahoo.com");
}

This code what i tried , so any suggestions? Also I tried to get the index of @gmail.com and remove it but it did not work neither

Strings are immutable. Replace doesn't change the source string. It returns a new string with the changes requested. — Steve
– Steve, Commented Aug 9, 2019 at 12:28
Yes this what happened to me , so is there another way to do what i want? — Ebrahim ElSayed
– Ebrahim ElSayed, Commented Aug 9, 2019 at 12:32
Re-read the last sentence in my comment. The answer is obvious. — Steve
– Steve, Commented Aug 9, 2019 at 12:33
Also, I don't think that it is correct to add always the @yahoo.com to your source. If you have an input like [email protected] the code above transforms the string in [email protected]@yahoo.com — Steve
– Steve, Commented Aug 9, 2019 at 12:39

Tim Schmelter · Accepted Answer · 2019-08-09 13:02:47Z

3

Strings are immutable, so every method in the String class does not modify the current instance but returns a new one. You have to assign this to the original variable:

sender.Text = sender.Text.Replace("@gmail.com,"@yahoo.com");

If you are interested in why strings are immutable: Why .NET String is immutable?

edited Aug 9, 2019 at 13:02

answered Aug 9, 2019 at 12:29

Tim Schmelter

462k79 gold badges719 silver badges980 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Dmitrii Bychenko · Accepted Answer · 2019-08-09 12:33:47Z

Something like this:

//DONE: we should check for null
//DONE: it's Yahoo if it ends on @yahoo.com (not contains)
public static bool IsYahoo(TextBox sender) =>
  sender != null && 
  sender.Text.TrimEnd().EndsWith("@yahoo.com", StringComparison.OrdinalIgnoreCase);

public static bool IsGmail(TextBox sender) =>
  sender != null && 
  sender.Text.TrimEnd().EndsWith("@gmail.com", StringComparison.OrdinalIgnoreCase);

public static void InsertYahoo(TextBox sender) {
  if (null == sender)
    throw new ArgumentNullException(nameof(sender));

  if (IsYahoo(sender))
    return;

  // Uncomment, In case you want to change gmail only
  //if (!IsGmail(sender)) 
  //  return;

  // If we have an eMail like bla-bla-bla@somewhere
  int p = sender.Text.LastIndexOf('@');

  // ... we change somewhere to yahoo.com
  if (p > 0)
    sender.Text = sender.Text.Substring(0, p) + "@yahoo.com";


}

Collectives™ on Stack Overflow

How to replace a specific string using method?

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related