Else if statment

Else if statment

• ChSchmidt

  ok...thanks
  
  

• jrboddie

  Equals is not preferable, Always prefer String.Compare whiel comparing 2 strings, Its fast because it is evaluated on low level numeric values rather that == or Equals()... Its recommended my .Net Performance Articles on String usage.

  Best Regards,

  
  
  

• sreejith s

  Here is some page about this violation of ToLower and Compare methods:
  http://msdnwiki.microsoft.com/en-us/mtpswiki/c7d0f62f-32a1-4d63-a835-c0a398377436.aspx

  Problem is not about Equals() because this method should be used. But should be used only when we know the case of string characters. If because we don't know the case of string, we need to do some ToLower or ToUpper operation and then have Equal or == opeation, then is prefered Compare method. Maybe It's easyer to alway use Compare.

  
  
  

• R.Tutus

  ahmedilyas wrote:

  does the same thing pretty much

  Yes it does the same thing but not with the performance offered by String.Compare(). Write a test to compare 20000 strings from both methods, you'll see the difference.

  Cheers ;-)

  
  
  

• quix ltd

  ok but re-read -....i was trying to make it all same case so its not case sensitive....
  
  

• Sunergy

  i guess u should remove ; after both if and else if statements.
  
  
  

• Grotius

  when comparing text, always use the Equals() method, not the operator =

   

  overall your code should be :

  if (lcarrier.Text.ToLower().Trim().Equals("altell"))
  {
                  endadd.Text = "@message.alltel.com";
  }
  else if (lcarrier.Text.ToLower().Trim().Equals("at&t"))
  {
                  endadd.Text = "@mmode.com";
  }

   

  the text is trimmed (removes spaces) and is lowered to be 1 lower case and make sure its all equal and consistant

  
  
  

• Muhsin Zahid Uğur

  Hello Remove ';' from if's and else's

  if (lcarrier.Text == "Altell") ; //What it is doing Here
  {
  endadd.Text == "@message.alltel.com";
  }
  else if (lcarrier.Text == "AT&T") ; // Again
  {
  endadd.Text == ("@mmode.com");
  }[/code]
  

  Remove them and I hope it'll be perfect now .

  Cheers

  
  
  

• saarar

  well this is what i've been told and what many of my colleagues use...I prefer to use Equals() for string
  
  

• cmwith

  >> if (lcarrier.Text == "Altell") ; //What it is doing Here

  The syntax for an if() statement is:

  if (condition) statement ;

  this is normally written as

  if (condition)
  statement ;

  just to make it easier to read. Then to group many statement into one, they are wrapped in braces:

  if (condition)
  {
  statement ;
  statement ;
  statement ;
  }

  which is why these's no semi-colon at the end of the if() line.

  
  
  

• MrZap

  ahmedilyas wrote:

  ok but re-read -....i was trying to make it all same case so its not case sensitive....

  
  
  Indeed - by lowercasing every string! The last parameter of String.Compare() being "true" tells it to ignore case. Much MUCH faster than lowercasing and returning a new copy of each string.
  
  

• Antonio Neeves

  >> when comparing text, always use the Equals() method, not the operator =

  The is absolutely no need to prefer Equals over operator=. String.operator= merely calls String.Equals. Your statement is true for reference types which do not implement value semantic (string does). It is also is true for all classes in Java (but we're talk about C# here)

  
  
  

• jatwood

  You shouldn't use ToLower() and Equals(), you should use String.Compare(str1, str2, true)
  
  
  

• Kouroshd

  does the same thing pretty much
  
  
• Else if statment