Stuck on c# loop with split

Question

1.00/5 (1 vote)

See more:

I have a test string "1,2,3,4,5,6,7,8,9,10,11,12,13"
I want the output to be
123
456
789
101112
13

I am not sure of the syntax for slpit. Can someone help?

string strMyTestMessage = "1,2,3,4,5,6,7,8,9,10,11,12,13";                        
string[] splitted = strMyTestMessage.Split(separator: new[] { "," }, count: 13, options: StringSplitOptions.None);
                                                
string strCountTest;

for (int i = 0; i < splitted.Length; i++)
{                            
if (i / 3 == 1)
 strCountTest = splitted[i].ToString() + "\n";                            
else
 strCountTest = splitted[i].ToString();

onsole.WriteLine(strCountTest);
								
}

Posted 6-Oct-14 9:25am

Member 10683143

Add a Solution

Comments

Mehdi Gholam 6-Oct-14 15:40pm

...and what is the rule for such a "split", every 3 commas?

Member 10683143 6-Oct-14 15:42pm

yeah. That is what I was trying the i /3. Every 3 commas split the new line.

Sergey Alexandrovich Kryukov 6-Oct-14 15:44pm

Why having this string in first place? (If this just for testing, I would understand, but still think about it.)
One simple advice: use the debugger (after to you answer the Mehdi's question). The problem is way too simple.
—SA

Member 10683143 6-Oct-14 15:45pm

This was for testing. The real string is being built from database values.

Sergey Alexandrovich Kryukov 6-Oct-14 15:48pm

That would mean wrong database schema design. When you need numeric data, use numeric types, when you need list data, use relations. Use normal forms and other good design stuff.
—SA

Member 10683143 6-Oct-14 15:51pm

3rd party system. They designed most of it with char or varchar. Not my design. Absolutely sucks.

Sergey Alexandrovich Kryukov 6-Oct-14 15:55pm

I understand. It's good that you understand it. Well, the problem you formulated would be a really small bit of suffering. Just use the debugger. I cannot see any apparent problem.
—SA

CPallini 6-Oct-14 15:46pm

Why aren't you sure? Isn't the documentation available to you? Haven't you a compiler?
(It looks right)

3 solutions

Solution 2

try this

C#

string strMyTestMessage = "1,2,3,4,5,6,7,8,9,10,11,12,13";
string[] strArray = strMyTestMessage.Split(',');
int iterations = strArray.Length;
for (int i = 0; i <= iterations; )
{
   Console.WriteLine(strArray[i] + ((i + 1) < iterations ? strArray[i + 1] : "") + ((i + 2) < iterations ? strArray[i + 2] : ""));

   i = i + 3;
}

good luck ;-)

Posted 6-Oct-14 17:48pm

george4986

Solution 3

I accepted CPallini322 solution because it was the if (i % 3 == 2) that lead me to my own solution using string builder. Sorry for the delayed response I was on vacation.

string strMyTestMessage = "1,2,3,4,5,6,7,8,9,10,11,12,13";
string[] splitted = strMyTestMessage.Split(separator: new[] { "," }, count: 13, options: StringSplitOptions.None);


            StringBuilder sb = new StringBuilder();
            string strOutput;

            for (int i = 0; i < 13; i++)
            {
                               
                strOutput = splitted[i].ToString();

                if (i % 3 == 2)
                {
                    sb.Append(strOutput).Append("\n");
                    strOutput = "";
                }
                else
                {
                    sb.Append(strOutput);
                    strOutput = "";
                }
                                           

            }
            Console.WriteLine(sb.ToString());

Posted 13-Oct-14 15:46pm

Member 10683143

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

CPallini · Accepted Answer · 2014-10-06T09:56:00

The split options look correct. However, int order to obtain the expected output you should modify your loop this way:

C#

for (int i = 0; i < splitted.Length; i++)
{
  if (i % 3 == 2)
    strCountTest = splitted[i].ToString() + "\n";
  else
    strCountTest = splitted[i].ToString();

  Console.Write(strCountTest);

}