Remove characters from a string then reverse it

Question

I need a function that takes a characters array and returns the same array but reversed and without the "*" character.

I have tried a lot of codes without success. This was my last attempt (sorry if doesn't have sense, I am learning JS)


function laClaveSecreta(carac){
    let new_str=[];
    for(i=0;i<carac.length;i++){
        if(carac[i]==="*"){
            new_str=new_str.push(carac.replace(/[*\s]/g, ''));
            
        }
        return new_str.reverse();
    }
    
 
}

//Running the function with this parameter

laClaveSecreta( [ "s", "*", "e", "n", "u", "l", " ", "s", "*", "e", " ", "a", "í", "*", "d", " ", "l", "*", "E", "*"] )

//Result I am looking for

"El día es lunes"

it would be easier to convert the array to a string first, then you can use replace on the entire string. carac.reverse().join("").replace(/[*]/g, "") — Mulan
– Mulan, Commented Mar 3, 2021 at 4:17

Mulan · Accepted Answer · 2021-03-03 04:25:20Z

1

It would be easier to convert the array to a string first, then you can use replace on the entire string -

const input =
  [ "s", "*", "e", "n", "u", "l", " ", "s", "*", "e", " ", "a", "í", "*", "d", " ", "l", "*", "E", "*"]

console.log(input.reverse().join("").replace(/[*]/g, ""))

El día es lunes

This isn't going to teach you much though. I assume this is a homework assignment or something. Using built-in functions like reverse, join, and replace are higher-level. You can solve the problem using a very basic for loop -

function laClaveSecreta(carac)
{ let r = ""
  for (const c of carac)
    if (c == "*")
      continue
    else
      r = c + r
  return r
}

const input =
  [ "s", "*", "e", "n", "u", "l", " ", "s", "*", "e", " ", "a", "í", "*", "d", " ", "l", "*", "E", "*"]

console.log(laClaveSecreta(input))

El día es lunes

edited Mar 3, 2021 at 4:25

answered Mar 3, 2021 at 4:20

Mulan

136k35 gold badges240 silver badges276 bronze badges

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

Comments

Tom Elsey · Accepted Answer · 2021-03-03 04:26:57Z

1

Try using let filteredArray = carac.filter( char => char != '*') then return filteredArray.reverse().join("")

answered Mar 3, 2021 at 4:26

Tom Elsey

212 bronze badges

Comments

shet_tayyy · Accepted Answer · 2021-03-03 04:27:22Z

0

I have filtered the array, reversed it and then I joined it without regex. The code looks cleaner and explains the steps.

const laClaveSecreta = (arr) => arr.filter(arrItem => arrItem !== '*').reverse().join('');

laClaveSecreta( [ "s", "*", "e", "n", "u", "l", " ", "s", "*", "e", " ", "a", "í", "*", "d", " ", "l", "*", "E", "*"] )

edited Mar 3, 2021 at 4:27

answered Mar 3, 2021 at 4:22

shet_tayyy

5,79612 gold badges56 silver badges95 bronze badges

2 Comments

Mulan Over a year ago

why bother checking arr.length? filter, reverse and join also work on empty arrays. why bother with variable assignment newStr when you can just write return arr.filter(...)... instead?

shet_tayyy Over a year ago

Just an old habit to make code readable and fail-proof but good suggestions

Mister Jojo · Accepted Answer · 2021-03-03 04:51:30Z

0

you can do that this way...

const laClaveSecreta  = carac =>
  {
  let new_str = '';
  for(i=carac.length;i--;)
    if(carac[i]!=="*") new_str +=carac[i]
  return new_str 
  }

console.log( laClaveSecreta('s*enul s*e aí*d l*E*'))

or

const laClaveSecreta  = carac => carac.reduceRight((a,c)=> (c==="*") ? a : a+c ,'')

console.log( laClaveSecreta([...'s*enul s*e aí*d l*E*']))

edited Mar 3, 2021 at 4:51

answered Mar 3, 2021 at 4:42

Mister Jojo

22.9k6 gold badges28 silver badges45 bronze badges

Comments

score 0 · Accepted Answer · 2021-03-03 04:51:47Z

0

Another approach using Array.prototype.filter().

const input =
  [ "s", "*", "e", "n", "u", "l", " ", "s", "*", "e", " ", "a", "í", "*", "d", " ", "l", "*", "E", "*"]

console.log(input.reverse().filter(item => item !== "*").join(""));

edited Mar 3, 2021 at 4:51

answered Mar 3, 2021 at 4:28

user6386114

Comments

nodeDevcoder · Accepted Answer · 2021-03-03 04:21:19Z

-1

function laClaveSecreta(carac){
  let new_str=[];
  for(i=0;i<carac.length;i++){
    if(carac[i] !=="*"){
        new_str.push(carac[i]);
    }
}
 return new_str.reverse();
}

answered Mar 3, 2021 at 4:21

nodeDevcoder

4973 silver badges14 bronze badges

1 Comment

Amit Verma Over a year ago

While this piece of code may provide a solution to the question, it's better to add context as to why/how it works. This can help future users refer to and eventually apply this knowledge to their own code. You are also likely to have positive feedback/upvotes from users, when the code is explained.

Collectives™ on Stack Overflow

Remove characters from a string then reverse it

6 Answers 6

Comments

Comments

2 Comments

Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

Comments

Comments

2 Comments

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related