ReplaceAll for big comma separated string

Question

I have used the replaceAll method of Javascript to replace the occurrences of character code " to Double Quotes in a big comma separated String. I really worry whether it will cause any issues or performance issues when deal with large amount of data.

Data will be like below:-

"TEST,TEST,&#34;TEST&#34;,TEST,
TEST,TEST,TEST,TEST,
&#34;TEST&#34;,TEST,TEST,TEST,
.
.
.Many Rows
.
&#34;TEST&#34;,TEST,TEST,TEST"

My JavaScript code:-

data = data.replaceAll('&#34;','"');

Please guide me if there is a better way to do this or this is really good approach?

Don't worry about performance before you've done any measurements. — evolutionxbox, Sep 07 '21 at 13:20
Your approach seems fine to me. Do you have any other requirements? — Tim Biegeleisen, Sep 07 '21 at 13:22
Thank you for your comments. I have done testing up to 50k rows and looks fine. However, I want to ensure this is good or is there any better approaches. — Ramachandran Periyasamy, Sep 07 '21 at 13:22
@TimBiegeleisen Thanks... I have to render this data as table in the page, already completed that part. — Ramachandran Periyasamy, Sep 07 '21 at 13:26
"Painting" a table _"with up to 50k rows"_ will take sooo much more time compared to a simple `.replaceAll()` — Andreas, Sep 07 '21 at 13:27
@Andreas I have used pivottable.js so it will not display information all at a time but when user drag and drop a column. — Ramachandran Periyasamy, Sep 07 '21 at 13:30
My basic test shows that doing a `replaceAll` on 50K rows takes 4ms. 150K took 12ms. — evolutionxbox, Sep 07 '21 at 13:44

score 0 · Answer 1 · answered Sep 21 '21 at 09:10

0

ReplaceAll function was deprecated and not supported for IE browser.

I have used regEx with replace function to do this and working fine.

if(data  && data.indexOf('&#34;') >= 0)
     data = data.replace(/&#34;/g,'"');

answered Sep 21 '21 at 09:10

Ramachandran Periyasamy

69
8

ReplaceAll for big comma separated string

1 Answers1