C# Function that generates strings according to input

hi sa...

input string: botle
Substitute string: **1*2

use trim function for input string and substitute string...

like s1=s1.Trim(); and s2=s2.Trim() it'll remove all the leading and trailing spaces if comes by accident.

I think the last post was working for you.

Cheers coding

hi again,

I changes to the following part of the code:

                temp = source;
                for (int j = 0; j < Tokens[i].Length; j++)
                {
                    string token = Tokens[i];
                   
                    if (sourceSubst.IndexOf(token[j]) == loc[j])
                    {
                        temp = temp.Remove(loc[j], 1);
                        temp = temp.Insert(loc[j], token[j].ToString());
                    }
                    else
                    {
                        temp = temp.Remove(sourceSubst.IndexOf(token[j]), 1);
                        temp = temp.Insert(sourceSubst.IndexOf(token[j]), token[j].ToString());
                    }

                }

and it seems to be working fine.... can you please confirm please??

Thanks again,

since you used 'AxxoSt' as a testing string, you did not captured the following error:

When I tried with the following I got the following result:

input string: botle
Substitute string: **1*2
============Output============
bo1l2 -> correct
bole2 -> should be botl2
bo1le -> correct

Thanks,

I am seeing the code and the problem is in the following part:

                for (int j = 0; j < Tokens[i].Length; j++)
                {
                    string token = Tokens[i]; //get char to change
                    temp = temp.Remove(loc[j], 1);//remove it from orig string
                    temp = temp.Insert(loc[j], token[j].ToString()); //replace it with the new char

                }
because when you are replacing one character, it is always taking the first position.

Note that s2 can have identical characters at different possitions (eg: **a*ac)

ok i got your problem ... have to work on it again will soon post the solution here

Hi Amit,

I did not understood your last post, maybe I was not clear :)

When I run the program after the fixes you gave me the results are not good.
The code should give the following results:
but they should be as:
bo1tom
bott2m
bo1t2m

but you gave me the following:
bo1t2m -> correct
bo2tom -> '2' should be placed instead of the 'o'
bo1tom -> correct

thanks again

hi dear,

you did'nt mark my answer as i have give you as much help as i can... and you can customize the code according to your requirement.

Hi Amit,

Thanks for your help .. it is really appreciated.
But there is still a problem:

The results I am getting now are the following:
bo1t2m
bo2tom
bo1tom

but they should be as:
bo1tom
bott2m
bo1t2m

Change the following line in InsertSymbol() function

temp =temp.Insert(loc[j]-1, token[j].ToString());

to 

temp =temp.Insert(loc[j], token[j].ToString());It was another problem i left with

hi friend,

yes you are right i found the bug. it was because of string.Replace funtion inside the insertSymbol function. well i have fixed it and now its working great.

        public static void InsertSymbol(string source, int[] loc, List Tokens)
        {
            string temp = "";
            for (int i = 0; i < Tokens.Count; i++)
            {
                temp = source;
                for(int j=0;j                {
                    string token = Tokens[i];
                    temp = temp.Remove(loc[j], 1);
                    temp =temp.Insert(loc[j]-1, token[j].ToString());
                    
                }
                Output.Add(temp);
            }
        }

Now please mark my answer.

Hi Amit,

Thanks for your code.... it looks really cool, but I think that somewhere there is a small bug :(

Consider the following inputs:
            string s1 = "bottom";
            string s2 = "**1*2*";

The answer I am getting for your code is the following:
b2112m
bo22om
bo11om

but it should be as follows:
bo1tom
bott2m
bo1t2m

what do you think?

Hi,
 

Try this,

 public static string[] ArrayOfString(string string1, string string2)
        {
            return new string[] { string1, string2 };
        }

Regards,
raaj