使用JavaScript删除文本,在字符串中的注释标记后的空白-在字符串中创建新行

时间:2019-04-05 19:44:45

标签: javascript arrays string newline markers

我正在尝试解决this CodeWars的挑战:

  

完成解决方案,以便剥离所有传入的注释标记后面的所有文本。也应去除行尾的任何空格。

     

给出以下输入字符串:

apples, pears # and bananas
grapes
bananas !apples
     

预期输出为:

apples, pears
grapes
bananas

到目前为止,我已经尝试过:

function solution(input, markers) {

  let string = input.split();
  let newString = " ";

  for (let i = 0; i < string.length; i++) {

    let words = string[i];
    //console.log(words);

    if (words.includes(markers || "/n")) {
      //go to the next line and keep building newString
    }
    newString += words;
  }
  return newString.toString();
}

这将返回apples,pears#andbananas/ngrapes/nbananas!apples,因为如您所见,当存在一个标记或存在/n时,我不知道如何在字符串中创建新行。

我尝试过

if (words.includes(markers || "/n")) {
  //go to the next line and keep building newString
  newString += "\n";
}

if (words.includes(markers || "/n")) {
  //go to the next line and keep building newString
  words + "\n";
}

但是这些都不起作用。

1 个答案:

答案 0 :(得分:4)

遇到编码挑战的站点通常具有级别(例如CodeWars)。在这种情况下,我建议您在较简单的级别上坚持更长的时间,直到您真正熟练地解决它们为止。

还要检查其他人提交的解决方案:从中学到很多。

之所以这样说,是因为您的代码中有很多错误,似乎您将从更简单的级别中受益更长的时间,而不仅仅是从此处获取并发布解决方案,而受益匪浅。

您对代码的一些评论:

  • 您用空格初始化newString。那是一个错误的开始。该空间不保证在那里存在。您只能从输入中获取字符。它应该是一个空字符串。
  • 换行符不是"/n",而是"\n"
  • input.split()将字符串转换为字符数组。如果您的目标是使通过索引访问字符成为可能,请意识到您也可以使用字符串:input[i]为您提供该偏移量的字符。
  • 变量名很重要。给变量string命名不是很有帮助。 words也不是,实际上它只包含一个字符。因此character将是一个更好的选择。
  • includes需要一个字符串作为参数,但是您传递了markers|| "/n"没有附加值,因为markers是真实值,因此||会在那里停下来(短路评估)。由于markers是一个数组,而不是字符串,因此includes将该值转换为逗号分隔的字符串。显然,该字符串不太可能在您的输入中出现。您需要分别测试每个标记字符,并检查换行符。
  • 您的if语句的主体为空(主要尝试中)。这没有用。也许您在寻找continue;,它将跳过循环的其余部分并继续进行下一次迭代。
  • 没有规定可以跳过跟随标记字符的字符。
  • 您无法消除标记字符前的空格。
  • newString是一个字符串,因此无需调用newString.toString();

尝试保持您的想法,这里的代码已更正:

function solution(input, markers) {
  let newString = "";
  for (let i = 0; i < input.length; i++) {
    let character = input[i];
    if (markers.includes(character)) {
        // move i to just before the end of the current line
        i = input.indexOf("\n", i)-1;
        // Remove the white space that we already added at the end
        newString = newString.trimRight();
        // If no newline character at end of last line: break
        if (i < 0) break;
        // Skip rest of this iteration
        continue;
    }
    newString += input[i];    
  }
  return newString;
}

但是有更简单的方法可以做到这一点。例如,首先将输入分成几行。

这是我发布的解决方案:

const solution = (input, markers) =>
    input.split("\n").map(line => 
        markers.reduce((line, marker) => 
            line.split(marker, 1)[0].trimRight(), line)).join("\n");