Develop Reference

I want to display one item per column in a table with a c# splitstring

I display the data coming from an excel table and I separate them with a splitstring and I want each item to be displayed in cells separated horizontally rather than separated vertically
here is my code to display the data :
private void btnExcelReader_Click(object sender, EventArgs e)
{
long lastRow = excelRange.Rows.Count;
long lastCol = excelRange.Columns.Count;
int rowTotal = 0;
for (int i = 1; i < lastRow; i++)
{
if (excelRange.Cells[i, 1].Value2 != null)
{
rowTotal++;
}
}
for (int i = 1; i < rowTotal; i += 4)
{
char[] separator = new char[] { ' ' };
string current = excelRange.Cells[i, 1].Value2;
string[] splitCurrent = current.Split(separator,
StringSplitOptions.RemoveEmptyEntries);
for (int j = 0; j < splitCurrent.Length; j++)
{
lvName.Items.Add(splitCurrent[j]);
}
}
lvName.AutoResizeColumns(ColumnHeaderAutoResizeStyle.ColumnContent);
lvName.AutoResizeColumns(ColumnHeaderAutoResizeStyle.HeaderSize);
}
I try with columns.add, i create list and after that put my item in subitems.. nothing work.
Here is the image of the result in my ListView :

c# StringSplitOptions.RemoveEmptyEntries not work

I want to split the namelist with spaces and remove the spaces but the space is not deleted.Also, random numbers always sound the same like in the photo.
form1.cs button click codes
string[] cells;
char[] ayraclar = { ' ' };
foreach (string row in createdMailList.Items)
{
cells = row.Split(ayraclar, StringSplitOptions.RemoveEmptyEntries);
string[] isimler = new string[cells.Length];
for (int i = 0; i < cells.Length; i++)
{
string[] chars = new string[cells[i].Length];
isimler[i] = cells[i].ToString();
listBox1.Items.Add(helper.anlamliMail(isimler));
}
}
helper.cs class codes
public static string anlamliMail(string[] items)
{
string kelime ="";
var rnd = new Random();
foreach (var row in items)
{
if (!string.IsNullOrEmpty(row))
{
kelime = items[rnd.Next(0, items.Length)] + rnd.Next(0, 9999);
}
}
return kelime + "#yahoo.com";
}
createdMailList content
result :
I split the list of names with spaces, but the space is coming. that's why there are only numbers of mails

you have to create a random object once only , not each time when you call anlamliMail
var rnd = new Random();
for (int i = 0; i < cells.Length; i++)
isimler[i] = cells[i].ToString();
foreach (var item in isimler)
listBox1.Items.Add(helper.anlamliMail(rnd,isimler));
and fix anlamliMai
public static string anlamliMail(Random rnd, string[] items)
{
string kelime ="";
do
{
kelime = items[rnd.Next(0, items.Length);
}
while (! string.IsNullOrEmpty(kelime));
return kelime + rnd.Next(0, 9999).ToString() + "#yahoo.com";
}

Replacing words input by the user within a sentence and then outputting the sentence but with the censored/replaced words

I am aiming to replace words within an array (input by the user) that appear in a sentence. I want to then output this sentence but with the replace version which would be either ***** or for example censor would become c****r.
static void CensorSelectedWords(string _sentence, int _wordCount)
{
string[] words = new string[_wordCount];
string[] splitSentence = new string[_wordCount];
for (int i = 0; i < words.Length; i++)
{
Console.WriteLine("Type in the words you wish to be censored: ");
words[i] = Console.ReadLine();
splitSentence = _sentence.Split(' ', '.', '?', ',', '!');
for (int j = 0; j < splitSentence.Length; j++)
if (splitSentence[j] == words[i])
splitSentence[j] = words[i].Replace(words[i], "*");
}
Console.WriteLine(splitSentence);
}

You can replace every word within _sentence like this:
static void CensorSelectedWords(string _sentence, int _wordCount)
{
string astr;
string[] words = new string[_wordCount];
string[] splitSentence = new string[_wordCount];
for (int i = 0; i < words.Length; i++)
{
Console.WriteLine("Type in the words you wish to be censored: ");
words[i] = Console.ReadLine();
astr = "";
for (int j = 0; j < words[I].Length; j++)
astr += "*";
_sentence = _sentence.Replace(word[i], astr);
}
Console.WriteLine(_sentence);
}

I'm slightly confused at what the output should be, but here's one way to do it. I've extracted the functionality of replacing a word with censored text into separate function so it can be modified independently.
The rest is very simple - get a sentence, get a list of censored words, then replace all the words with their censored counterparts in the sentence and display it:
public static string HideAllButFirstAndLast(string word)
{
if (word == null) return null;
if (word.Length < 4) return new string('*', word.Length);
return word[0] + new string('*', word.Length - 2) + word[word.Length - 1];
}
private static void Main()
{
Console.WriteLine("Enter a sentence: ");
var sentence = Console.ReadLine();
Console.WriteLine("Enter a comma-separated list of censored words:");
var censoredWords = Console.ReadLine()
.Split(new[] {','}, StringSplitOptions.RemoveEmptyEntries)
.Select(word => word.Trim())
.ToList();
// Split sentence into words, replace words as needed, and join the words again
sentence = string.Join(" ", sentence.Split().Select(word =>
censoredWords.Contains(word, StringComparer.OrdinalIgnoreCase)
? HideAllButFirstAndLast(word)
: word));
Console.WriteLine(sentence);
GetKeyFromUser("\n\nDone! Press any key to exit...");
}
Output

Split string into array of words

I want to split a string into an array of words without using string.Split. I tried already this code and it is working but cant assign the result into the array
string str = "Hello, how are you?";
string tmp = "";
int word_counter = 0;
for (int i = 0; i < str.Length; i++)
{
if (str[i] == ' ')
{
word_counter++;
}
}
string[] words = new string[word_counter+1];
for (int i = 0; i < str.Length; i++)
{
if (str[i] != ' ')
{
tmp = tmp + str[i];
continue;
}
// here is the problem, i cant assign every tmp in the array
for (int j = 0; j < words.Length; j++)
{
words[j] = tmp;
}
tmp = "";
}

You just need a kind of index pointer to put up your item one by one to the array:
string str = "Hello, how are you?";
string tmp = "";
int word_counter = 0;
for (int i = 0; i < str.Length; i++) {
if (str[i] == ' ') {
word_counter++;
}
}
string[] words = new string[word_counter + 1];
int currentWordNo = 0; //at this index pointer
for (int i = 0; i < str.Length; i++) {
if (str[i] != ' ') {
tmp = tmp + str[i];
continue;
}
words[currentWordNo++] = tmp; //change your loop to this
tmp = "";
}
words[currentWordNo++] = tmp; //do this for the last assignment
In my example the index pointer is named currentWordNo

Try using regular expressions, like this:
string str = "Hello, how are you?";
// words == ["Hello", "how", "are", "you"]
string[] words = Regex.Matches(str, "\\w+")
.OfType<Match>()
.Select(m => m.Value)
.ToArray();
String.Split is not a good option since there are too many characters to split on: ' ' (space), '.', ',', ';', '!' etc.
Word is not just a stuff between spaces, there are punctuations to consider, non-breaking spaces etc. Have a look at the input like this:
string str = "Bad(very bad) input to test. . ."
Note
Absence of space after "Bad"
Non-breaking space
Addition spaces after full stops
And the right output should be
["Bad", "very", "bad", "input", "to", "test"]

You can also use a List to create your words list:
string str = "Hello, how are you?";
string tmp = "";
List<string> ListOfWords = new List<string>();
int j = 0;
for (int i = 0; i < str.Length; i++)
{
if (str[i] != ' ')
{
tmp = tmp + str[i];
continue;
}
// here is the problem, i cant assign every tmp in the array
ListOfWords.Add(tmp);
tmp = "";
}
ListOfWords.Add(tmp);
In this way you can avoid to count the number of word and the code is more simple. Use ListOfWord[x] to read any word

interview question - how to loop through an array with a different starting point?

Let's say you have the string "This is a test"
I pass it to method zee, like ("This is a test", 1)
and want "test This is a";
I pass it to method zee, like ("This is a test", 2)
and want "a test This is";
the number can exceed the total words in variable. If it does it should loop around.
I started with....
public static string zee(string origString, int i)
{
StringBuilder sb = new StringBuilder();
ArrayList list = new ArrayList();
list.AddRange(origString.Split(' '));
// not sure here -
for (int c = i; c < (list.Count + i); c++)
{
sb.AppendFormat("{0} ", list[c]);
}
return sb.ToString();
}

for(int j=0; j < list.length; j++){
int idx = (j + i) % list.length;
sb.AppendFormat("{0} " , list[idx]);
}
Mostly like Brent Arias's solution, but I think a for loop is more readable, less likely to go infinite.
public static string zee(string origString, int i)
{
StringBuilder sb = new StringBuilder();
List<string> list = new List<string>();
list.AddRange(origString.Split(' '));
for (int j = 0; j < list.Count; j++)
{
int idx = (j + i) % list.Count;
sb.AppendFormat("{0} ", list[idx]);
}
return sb.ToString();
}

This is how I'd solve it.
private static string f(string s, int start)
{
var arr=s.Split(' ');
start %= arr.Length;
var res=arr.Skip(arr.Length - start).ToList();
res.AddRange(arr.Take(arr.Length - start));
return string.Join(" ", res);
}
I tried writing a one liner with linq but I don't see how to combine 2 lists. Union and Join aren't what I need.

This is how I'd solve it using strings.
public static string zee(string origString, int i)
{
string[] splitStr = origString.Split(' ');
string newStr = "";
// Not sure what you meant by wrap around but this should
// do the trick.
i %= splitStr.Length;
for (int j = (splitStr.Length - i); j < splitStr.Length; j++)
newStr += splitStr[j] + " "; // Add spaces taken by split :(
for (int j = 0; j < (splitStr.Length - i); j++)
newStr += splitStr[j] + " ";
return
newStr;
}

Here's an abomination trying to cram as much into one line as possible:
static string zee(string sentence, int wordCount)
{
var words = sentence.Split(' ');
return string.Join(" ", new[] { words.Skip(words.Count() - wordCount), words.Take(words.Count() - wordCount) }.SelectMany(w => w).ToArray());
}

I havn't tried it, but I think this would do it:
i %= list.Length;
int index = i;
do {
index %= list.Length;
sb.AppendFormat("{0} ", list[index]);
while (++index != i);

static string rearrange(string phase,int index)
{
string[] words = phase.Split(' ');
string[] newwords = new string[words.Length];
int pointer = index;
for (int i = 0; i < words.Length;i++ )
{
if(pointer>=words.Length)
{
pointer = 0;
}
newwords[i] = words[pointer];
pointer++;
}
return string.Join(" ", newwords);
}

Sounds like a homework question to me, but here is an efficient use of the .Net framework:
private static string [] SplitWords(string s, int startWord)
{
string[] words = s.Split(' ');
List<string> output = new List<string>();
output.AddRange(words.Skip(startWord).ToArray());
output.AddRange(words.Take(startWord).ToArray());
return output.ToArray();
}
There is absolutely no error checking in this function so you will have to modify it for production code but you get the idea.

public string SetStart(int startAt)
{
const string sentence = "this is a test so it is";
var words = sentence.Split(' ');
var x = (startAt > words.Count()) ? startAt%words.Count() : startAt;
return string.Join(" ", words.Skip(x).Concat(words.Take(x)));
}