>>Thanks, Greg. I'll review the code this weekend.
>>
>
>Here's version two of the ChrTran with speed improvement
>String.Length is a call to a method get_Length()
>
>if we loop backwards then the call is only made once instead of every iteration
>The to.Length is also called once and stored in a variable
>
>

>public static partial class ExtensionMethods
>	{
>
>		public static string ChrTran(this string s, string from, string to)
>		{
>			var sb = new StringBuilder(s, s.Length);
>
>			int i, j, toLen = to.Length;
>
>			for (i = sb.Length; --i >= 0; )
>			{
>				if ((j = from.IndexOf(sb[i])) >= 0)
>				{
>					if (j < toLen)
>					{
>						sb[i] = to[j];
>					}
>					else
>					{
>						sb.Remove(i, 1);
>					}
>				}
>			}
>
>			return sb.ToString();
>		}
>
>	}
>

I was thinking we need to use multiple replace commands. Do you think this is the best implementation? I want to add this to our extension methods as I need this function also.

Thanks again.

UPDATE. I'm reviewing the whole thread now, as I may add some goodies from it into our extension methods. I just added between and I am looking at chrtran.

Thanks again.