Returning an empty array instead of null

Question

I am wanting to return an empty array. However, it keeps returning null. How can I resolve this in my code

I am wanting the mentions array to return blank on this occasion, but it keeps coming back as null

function getTweetData(tweet) {
  let hashtag = tweet.match(/#\w+/g)
  let atSign = tweet.match(/@\w+/g)

  let tweetObj = {
    tags: hashtag,
    mentions: atSign,
    tagCount: 0,
    mentionCount: 0,
    length: tweet.length
  }

  if (tweet.match(/#/g))
    tweetObj.tagCount++

  if (tweet.match(/@/g))
    tweetObj.mentionCount++

  console.log(tweetObj)
  return tweetObj
}

Test

it('Should increase the count of tags', () => {
    expect(getTweetData('My awesome tweet about #coding')).to.eql({ tags: ['#coding'], mentions: [], tagCount: 1, mentionCount: 0, length: 30 })
  });

Nothing in the code creating tweetObj attempts to create a property called mentions...? Literally, searching for mentions just finds it in the test. — T.J. Crowder
– T.J. Crowder, Commented Jun 23, 2019 at 13:49

Alon Yampolski · Accepted Answer · 2019-06-23 13:51:24Z

5

You can modify:

let atSign = tweet.match(/@\w+/g)

to:

let atSign = tweet.match(/@\w+/g) || []

Adding || [] will assign an empty array to atSign in a case where your regex returns null

answered Jun 23, 2019 at 13:51

Alon Yampolski

8497 silver badges15 bronze badges

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

Comments

Ziv Ben-Or · Accepted Answer · 2019-06-23 13:53:19Z

2

You can do this:

let hashtag = tweet.match(/#\w+/g) || []
let atSign = tweet.match(/@\w+/g)  || []

answered Jun 23, 2019 at 13:53

Ziv Ben-Or

1,1946 silver badges15 bronze badges

Collectives™ on Stack Overflow

Returning an empty array instead of null

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related