Extract only Capital letters from a string - javascript

Question

 function onlyCapitalLetters (str)  

 { 

let newStr = "";

for (let i = 0; i < str.length; i ++) {
    if (str[i].includes("ABCDEFGHIJKLMNOPQRSTUVWXYZ")) {
        newStr += str[i];
       
    }
}
return newStr;
}

onlyCapitalLetters("AMazing"); // should get AM

Hi, I'm trying to write a function, that will return a new string with only capital letters. When I try to run this function, I don't see any output. Please help!!!

The function you have written will require each letter to include the entire alphabet — Nick, Dec 22 '20 at 06:19
`str[i].includes("ABCDEFGHIJKLMNOPQRSTUVWXYZ")` should be `"ABCDEFGHIJKLMNOPQRSTUVWXYZ".includes(str[i])` — Hao Wu, Dec 22 '20 at 06:21

score 5 · Answer 1 · answered Dec 22 '20 at 06:26

5

In practice, you would probably use a regex approach here:

function onlyCapitalLetters (str) {
    return str.replace(/[^A-Z]+/g, "");
}

console.log(onlyCapitalLetters("AMazing")); // should get AM

answered Dec 22 '20 at 06:26

Tim Biegeleisen

502,043
27
286
360

score 2 · Answer 2 · answered Dec 22 '20 at 06:21

Include requires everything within to be included in the provided string. Use regex instead

function onlyCapitalLetters (str) { 
  let newStr = "";

  for (let i = 0; i < str.length; i++) {
      if (str[i].match(/[A-Z]/)) {
          newStr += str[i];
      }
   }
   return newStr;
}


console.log(onlyCapitalLetters("AMazing")); // should get AM

You could one line this function like this

const capital = (str) => str.split('').filter(a => a.match(/[A-Z]/)).join('')

console.log(capital("AMazinG"))

score 0 · Answer 3 · answered Dec 22 '20 at 06:25

The string of letters (uppercaseLetters) should include the current letter (str[i]), and not the other way around:

function onlyCapitalLetters(str) {
  let newStr = "";
  
  const uppercaseLetters = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";

  for (let i = 0; i < str.length; i++) {
    if (uppercaseLetters.includes(str[i])) {
      newStr += str[i];
    }
  }
  
  return newStr;
}

console.log(onlyCapitalLetters("AMazing")); // should get AM

A better option would be to use String.match() to find all uppercase letters. The match method returns an array of letters found, or null if none found, so we'll need to use an empty array as a fallback. Join the resulting array with an empty string.

function onlyCapitalLetters(str) {
  return (str.match(/[A-Z]/g) || []).join('');
}

console.log(onlyCapitalLetters("AMazing")); // should get AM

score 0 · Answer 4 · answered Dec 22 '20 at 06:26

String.Includes() checks for all the characters in the string.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/includes

You can also use ASCII code for the Characters to verify if it is a capital letter.

function onlyCapitalLetters (str) { 
 let newStr = "";

    for (let i = 0; i < str.length; i++) {
      if (str[i].charCodeAt(0) >= 65 && str[i].charCodeAt(0) <= 90) {
          newStr += str[i];
       }
    }
  return newStr;
}

console.log(onlyCapitalLetters("AMazing"));

Extract only Capital letters from a string - javascript

4 Answers4

Related