Remove Everything After Certain Character Within String Using JavaScript

Question

EDIT: My code does work - I just had a typo when I was console.logging. It uses the same technique already found in the answer here.

I have a function that should remove everything after the comma in the string:

function shortenToDate(longDate) {

  let newDate = longDate.substring(0, longDate.indexOf(","));

  return newDate;

}

^ You just need to take a chunk out of the string from the 0 index to the indexOf() the first instance of whatever character you wish to remove everything after.

I had also tried:

function shortenToDate(longDate) {

  return longDate.substring(longDate.indexOf(0, ","));

}

console.log(shortenToDate(shortenToDate("Friday May 2, 9am")));

Which didn't have any effect. It just returned Friday May 2, 9am.

Regarding your second snippet: "And that didn't return anything"... it clearly does: jsfiddle.net/wxprm41z — Gerardo Furtado
– Gerardo Furtado, Commented Apr 15, 2019 at 2:33
Here is an image to show you (answering to your now deleted comment): imgur.com/UZTF6Be. — Gerardo Furtado
– Gerardo Furtado, Commented Apr 15, 2019 at 2:39
So then my second guess was actually working. I don't know why it wasn't logging anything to the console before. — HappyHands31
– HappyHands31, Commented Apr 15, 2019 at 2:42
Yes, it was working. So, at the end, your question was unnecessary! What strikes me the most is that 3 high-rep users saw your question, saw that you had a working code (I hope so...) but, instead of voting to close and leave you a comment, decided to write an answer. — Gerardo Furtado
– Gerardo Furtado, Commented Apr 15, 2019 at 3:14
@GerardoFurtado I'm going to go ahead and delete this question. The question that I linked already has the correct answer with (0, longDate.indexOf(",")). — HappyHands31
– HappyHands31, Commented Apr 15, 2019 at 14:34

Code Maniac · Accepted Answer · 2019-08-26 11:22:30Z

4

You can simply use split and take the 0^th index

const shortenToDate = longDate => longDate.split(',',1)[0];
console.log(shortenToDate("Friday May 2, 9am"))

Problems

In the first snippet you're using

longDate.substring(longDate.indexOf(","), longDate.length -1);

but you want from 0^th index

const  shortenToDate = longDate => longDate.substring(0,longDate.indexOf(","));

console.log(shortenToDate("Friday May 2, 9am"))

edited Aug 26, 2019 at 11:22

answered Apr 15, 2019 at 2:33

Code Maniac

37.9k5 gold badges44 silver badges65 bronze badges

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

1 Comment

HappyHands31 Over a year ago

That is what I was doing in my second snippet. I don't know why it wasn't logging anything to the console. That code is correct, then.

A l w a y s S u n n y · Accepted Answer · 2019-04-15 02:36:27Z

2

How about that with String.prototype.split() and Array.prototype.shift()?

function shortenToDate(longDate) {
  let newDate = longDate.split(',');
  return newDate.shift();
}

console.log(shortenToDate("Friday May 2, 9am"))

answered Apr 15, 2019 at 2:36

A l w a y s S u n n y

38.8k9 gold badges68 silver badges120 bronze badges

Comments

CertainPerformance · Accepted Answer · 2019-04-15 02:31:48Z

0

It would probably be easier to use a regular expression - match a comma, followed by any characters, and replace with the empty string:

const shortenToDate = longDate => longDate.replace(/,.*/, '');
console.log(shortenToDate("Friday May 2, 9am"))

answered Apr 15, 2019 at 2:31

CertainPerformance

373k55 gold badges354 silver badges359 bronze badges

1 Comment

CertainPerformance Over a year ago

Why the downvote, did I make a mistake somewhere? Please let me know, I'll be happy to fix it!

Collectives™ on Stack Overflow

Remove Everything After Certain Character Within String Using JavaScript

3 Answers 3

1 Comment

Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related