Parse xml with empty valued attribute

Question

Have and input of this format:

<table>
<tbody>
    <tr bgcolor='#999999'>
        <td nowrap width='1%'>
        </td>
        <td nowrap width='3%' align='center'>
            <font style='font-size: 8pt'> System ID </font>
        </td>
        <td nowrap width='5%' align='center'>

In order to remove nowrap attribute , was earlier using this code:

    if (deletedString == null)
    {
        return exportedTable;
    }

    int tagPos = 0;
    String resultTable = exportedTable;
    while (resultTable.indexOf(deletedString) != -1)
    {
        tagPos = resultTable.indexOf(deletedString, tagPos);
        String beforTag = resultTable.substring(0, tagPos);
        String afterTag = resultTable.substring(tagPos + deletedString.length());

        resultTable = beforTag + afterTag;
    }
    return resultTable;

deletedString is nowrap, and input is exportedTable. But this is causing Performance issues. Is there any better way to do it?

Community · Accepted Answer · 2017-05-23 12:05:55Z

1

My recommendation: StringUtils.remove(source, substring) will remove all instances of the substring from the source string. This answer benchmarked this method and found it to be five times faster than a few alternatives.

Alternatively, use a StringBuilder to aggregate your substrings - every time you concatenate two strings you're creating a new string, whereas StringBuilder is mutable and doesn't need to create a new copy on an update.

edited May 23, 2017 at 12:05

CommunityBot

11 silver badge

answered Jun 18, 2015 at 13:55

Zim-Zam O'Pootertoot

18.2k4 gold badges45 silver badges71 bronze badges

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

1 Comment

hm5 Over a year ago

I will try with the first approach as u suggested. For the second one, actually the performance lag is more due to the no of iterations of while loop, than the String concatenation part.

blaqksilhouette · Accepted Answer · 2015-06-18 13:52:15Z

0

You could create a xmlstreamreader and have a while loop that parses through the xml as long as the streamreader.hasNext().

Format:

//Create stream reader
//Position at beginning of document

//While the stream reader has next (can see next line)
//perform action

answered Jun 18, 2015 at 13:52

blaqksilhouette

19 bronze badges

Collectives™ on Stack Overflow

Parse xml with empty valued attribute

2 Answers 2

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related