C# - String.Split() Remove last item

Question

I have a RTB that is loaded with a file that looks like this:

J6      INT-00113G  227.905  174.994  180  SOIC8    
J3      INT-00113G  227.905  203.244  180  SOIC8     
U13     EXCLUDES    242.210  181.294  180  QFP128    
U3      IC-00276G   236.135  198.644  90   BGA48     
U12     IC-00270G   250.610  201.594  0    SOP8      
J1      INT-00112G  269.665  179.894  180  SOIC16    
J2      INT-00112G  269.665  198.144  180  SOIC16

I want to remove the last column using the string.Split() method.

So far I have:

// Splits the lines in the rich text boxes
string[] lines = richTextBox2.Text.Split('\n');

foreach (var newLine in lines)
{
     newLine.Split(' ');
     line = line[0] + line[1] + line[2] + line[3] + line[4];  #This is the code that does not work.
}

However this does not work... does anyone know the problem and how to do this properly so the file looks like this?:

J6      INT-00113G  227.905  174.994  180      
J3      INT-00113G  227.905  203.244  180       
U13     EXCLUDES    242.210  181.294  180     
U3      IC-00276G   236.135  198.644  90      
U12     IC-00270G   250.610  201.594  0         
J1      INT-00112G  269.665  179.894  180     
J2      INT-00112G  269.665  198.144  180

EDIT: I also think that I need to string.Split(' ') each line that is already split?

That's obviously not going to work. You can't turn a string array into a string. — BoltClock
– BoltClock, Commented Jul 25, 2011 at 22:45

Coeffect · Accepted Answer · 2011-07-25 23:01:08Z

7

This might work (untested):

string[] lines = richTextBox2.Text.Split('\n');
for( int i = 0; i < lines.Length; i ++ )
{
    lines[i] = lines[i].Trim(); //remove white space
    lines[i] = lines[i].substring(0, lines[i].LastIndexOf(' ');
}
string masterLine = String.Join(Environment.NewLine, lines);

edited Jul 25, 2011 at 23:01

answered Jul 25, 2011 at 22:47

Coeffect

8,8822 gold badges29 silver badges42 bronze badges

Sign up to request clarification or add additional context in comments.

11 Comments

iandotkelly Over a year ago

This is the best solution as it doesn't rely on knowing what character to split at - such as the 41 or 43 used in some solutions

Dan Tao Over a year ago

@iandotkelly: You're making some assumptions in declaring this the "best" solution based on the reasons stated. Only the OP knows exactly what behavior he wants. For example, suppose the file format might change such that another column is added and he wants that column to be ignored also. Then this solution would no longer work, but the fixed-width cut-off approach still would. Of course, scenarios illustrating the opposite case could easily be concocted as well. It all depends on exactly what the goal is.

theNoobGuy Over a year ago

@Mannimarco: This does what I need it to! Dan Tao +1 for being correct about the OP knowing what is needed.

Coeffect Over a year ago

@Dan Tao: True about the columns. Removing two columns this way would be a pain. I just hate counting characters =p

Dan Tao Over a year ago

@Colton: In that case then you have some lines with no whitespace characters in them. So the solution above will still not work; it will just give you a blank line in such cases. Is it possible your columns are delimited with tab characters rather than spaces? If so you could try LastIndexOfAny with ' ' and \t.

|

Dan Tao · Accepted Answer · 2011-07-25 22:48:51Z

3

This is a fixed-width layout, so you can accomplish what you want simply by cutting off all the content to the right of the fifth column:

string[] lines = File.ReadAllLines(path);
for (int i = 0; i < lines.Length; ++i)
{
    lines[i] = lines[i].Substring(0, 43);
}

Example input file:

J6      INT-00113G  227.905  174.994  180  SOIC8    
J3      INT-00113G  227.905  203.244  180  SOIC8     
U13     EXCLUDES    242.210  181.294  180  QFP128    
U3      IC-00276G   236.135  198.644  90   BGA48     
U12     IC-00270G   250.610  201.594  0    SOP8      
J1      INT-00112G  269.665  179.894  180  SOIC16    
J2      INT-00112G  269.665  198.144  180  SOIC16

Output:

J6      INT-00113G  227.905  174.994  180  
J3      INT-00113G  227.905  203.244  180  
U13     EXCLUDES    242.210  181.294  180  
U3      IC-00276G   236.135  198.644  90   
U12     IC-00270G   250.610  201.594  0    
J1      INT-00112G  269.665  179.894  180  
J2      INT-00112G  269.665  198.144  180

answered Jul 25, 2011 at 22:48

Dan Tao

129k57 gold badges309 silver badges451 bronze badges

3 Comments

theNoobGuy Over a year ago

If it is not a fixed width how can I go about this?

Dan Tao Over a year ago

@Colton: Mannimarco's answer ought to help you out in that case. (However, if it's not a fixed-width layout then why did you supply us with fictional input?)

theNoobGuy Over a year ago

This works, but not guarenteed for an unfixed width. Thank you though.

BrokenGlass · Accepted Answer · 2011-07-25 22:52:12Z

1

You are not saying what doesn't work..using the Lines property you could do something like this:

richTextBox2.Lines = richTextBox2.Lines
                                 .Select( l => string.Join(" ", l.Split(' ')
                                                     .Take(5)))
                                 .ToArray();

This would only work of course if the space only occurs as separator between columns.

edited Jul 25, 2011 at 22:52

answered Jul 25, 2011 at 22:44

BrokenGlass

161k28 gold badges297 silver badges339 bronze badges

Comments

Steve Sheldon · Accepted Answer · 2011-07-25 23:09:24Z

This ones fairly easy to read ....

        string data = "J6      INT-00113G  227.905  174.994  180  SOIC8\r\nJ3      INT-00113G  227.905  203.244  180  SOIC8\r\nU13     EXCLUDES    242.210  181.294  180  QFP128\r\nU3      IC-00276G   236.135  198.644  90   BGA48\r\nU12     IC-00270G   250.610  201.594  0    SOP8\r\nJ1      INT-00112G  269.665  179.894  180  SOIC16\r\nJ2      INT-00112G  269.665  198.144  180  SOIC16\r\n";

        // Split on new line
        string[] lines = data.Split(new string[] {"\r\n"}, int.MaxValue, StringSplitOptions.RemoveEmptyEntries);

        StringBuilder result = new StringBuilder();

        foreach (string line in lines)
        {
            // Find the last space in the line
            int lastSpace = line.LastIndexOf(' ');

            // delete the end of the string from the last space
            string newLine = line.Remove(lastSpace);

            // rebuild string using stringBuilder
            result.AppendLine(newLine);
        }

        Console.WriteLine("Old List:");
        Console.Write(data);

        Console.WriteLine("New List:");
        Console.Write(result);
    }

This one is probably close to O(n):

        StringBuilder result = new StringBuilder();
        string data = "J6      INT-00113G  227.905  174.994  180  SOIC8\r\nJ3      INT-00113G  227.905  203.244  180  SOIC8\r\nU13     EXCLUDES    242.210  181.294  180  QFP128\r\nU3      IC-00276G   236.135  198.644  90   BGA48\r\nU12     IC-00270G   250.610  201.594  0    SOP8\r\nJ1      INT-00112G  269.665  179.894  180  SOIC16\r\nJ2      INT-00112G  269.665  198.144  180  SOIC16\r\n";

        // Split on new line
        int startchar = 0;
        int lastspace = 0;

        for(int i = 0; i < data.Length - 1; i++)
        {
            char current = data[i];

            if (current == ' ')
            {
                // remember last space
                lastspace = i;
            }
            else if (current == '\n')
            {
                result.AppendLine(data.Substring(startchar, lastspace - startchar));

                if(i != data.Length - 1)
                {
                    startchar = i + 1;
                    lastspace = startchar;
                }
            }
        }


        Console.Write(result.ToString());

Jonathan Wood · Accepted Answer · 2011-07-25 22:45:22Z

0

You're the only one that "knows the problem". Unfortunately, you decided not to tell us what it is. (Strange.)

What I would do is parse each line as you are, and then use LastIndexOf() to find the last space, and just trim the string there. Optionally, you could pass the result to Trim() to remove any trailing spaces.

answered Jul 25, 2011 at 22:45

Jonathan Wood

68.1k86 gold badges309 silver badges542 bronze badges

1 Comment

theNoobGuy Over a year ago

Now that I look at my code posted, I am splitting each line in the string. So every string will look similar to this: J1 INT-00112G 269.665 179.894 180 SOIC16. However, I need to then split that line by something like .Split(' ') so it splits it by spaces to get new strings.. I will post updated code above... But pretty much I can't do: line[0] + line[1]...;

shf301 · Accepted Answer · 2011-07-25 22:46:39Z

0

Assuming that your lines are always the same width then:

string parsed = richTextBox2.Text.Split('\n').
         Select(l => l.Substring(41) + Environment.NewLine);

answered Jul 25, 2011 at 22:46

shf301

31.5k2 gold badges57 silver badges88 bronze badges

Comments

zeFrenchy · Accepted Answer · 2011-07-25 22:49:21Z

0

You should probably read your file line by line in the first place using readline() then split each line with split.

answered Jul 25, 2011 at 22:49

zeFrenchy

6,5911 gold badge29 silver badges36 bronze badges

Comments

Tejas · Accepted Answer · 2015-12-16 06:51:00Z

0

var myString = "I want to remove the last item";
var mySplitResult = myString.split(" ");
var lastitem =  mySplitResult[mySplitResult.length-1]

answered Dec 16, 2015 at 6:51

Tejas

4675 silver badges13 bronze badges

Collectives™ on Stack Overflow

C# - String.Split() Remove last item

8 Answers 8

11 Comments

3 Comments

Comments

Comments

1 Comment

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

8 Answers 8

11 Comments

3 Comments

Comments

Comments

1 Comment

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related