Freecodecamp: 주석 처리 된 코드는 단순한 주석이 아닌 코드 유효성 검사를 위해 고려됩니다!

@ Saif-Shines 예, 우리는 이것이 메인 웹 사이트의 기존 문제로 알고 있으며, 가능하다면 엄격한 파서를 가질 수있는 인프라가 부족합니다.

현재 유일한 방법은 댓글을 삭제하는 것입니다.

잠재적 인 기여자, 여기에서 조사해야합니다.
https://github.com/freeCodeCamp/freeCodeCamp/blob/staging/client/rechallenge/throwers.js#L16

/ cc @BerkeleyTrue

Basic Javascript의 첫 번째 도전에서 댓글을 작성하는 동안 비슷한 문제가 발생했습니다.

// Use comments only when they are actually needed

/* Sometimes, using descriptive names for variables and functions is way better 
than leaving code smell all over your code */

이 경우 두 번째 주석에서 "함수"라는 단어를 사용하기 때문에 다음과 같이 표시됩니다.
Error: SyntaxError: Unsafe or unfinished function declaration

어쩌면에 대한 테스트를 실행 contents.replace(/\/\*[\s\S]*\*\/|\/\/[^\n]*/gi,'')) 대신 contents 그 문제를 해결할 수 있을까? (일반적인 문제를 의미합니다. 제 경우에는 주석을 대체 한 후 내용이 비어 있기 때문에이 특정 테스트가 여전히 실패하기 때문입니다)

우리는 다음과 같은 것을 할 수 없습니다.
Node node = nodeList.item(i); if(node.getNodeType() == Node.COMMENT_NODE) { continue; } else { //do something }

나는 이것을하고 싶다.

이 문제에 대해 관심이 있습니다.이 작업을 저에게 할당 할 수 있으며 변경해야하는 파일로 저를 안내 할 수 있습니다.

1 주일 넘게 업데이트가 없으니 제가 맡겠습니다. @ Survivor75 와 @ knrt10이 괜찮 길 바랍니다.

@raisedadead : 해결책은 우리가 생각했던 것보다 훨씬 간단하다고 생각합니다.

문자열 인 code 에서 어설 션을 실행합니다. 그렇다면이 문자열에 대해 정규식을 실행하고 유효성 검사에 사용되기 전에 주석을 제거하지 않는 이유는 무엇입니까?

예를 들어- client/frame-runner.js 에서 다음과 같은 codemirror 편집기의 내용을 가져옵니다.

const editor = { getValue() { return source; } };
const code = source;

코드를 그대로 전달하는 대신 정규식을 실행하여 다음과 같은 주석을 제거합니다.

const editor = { getValue() { return source; } };
const code = source.replace(/\/\*[\s\S]*?\*\/|[^\S]\/\/.*$/gm, '');

이렇게하면 테스트 러너는 코드에서 주석을받지 않으며 모든 것이 좋습니다.

이 정규식을 여기에서 확인했습니다-http: //regexr.com/3ghvi

노트:

• HTML 주석의 존재를 확인하는 시작 부분에 몇 가지 문제가 있으므로 HTML 주석은 고려하지 않습니다.
• URL 패턴은 제외됩니다.
• 이 정규식은 // 이어지는 문자를 대체하기 때문에 자체 줄에 존재하지 않는 인라인 주석을 대체하지 않습니다.

@adityaparab 그 멋진 : +1 :.
다른 일에 참여하면서 거의 잊어 버렸습니다.
그러나 나는이 코드를 생각해 냈습니다.이 작업을 수행하려면 아래 기능을 살펴보십시오.
function stripComments ( code ) { return code . replace (/\/\/.*|\/\*[^]*?\*\// g , "") ; }

주석 처리 된 코드가 제거됩니다. :)

불행히도 여기서 유일한 해결책은 테스트에 AST를 사용하는 것입니다. @QuincyLarson 과 함께 babel이 생성 한 JS AST를 노출하는 API를 추가하고 eslint의 플러그인 인터페이스와 유사한 API를 추가하는 방법에 대해 논의했습니다. 우선 순위가 반응 과제로 이동 한 다음 과제 자체를 파일에서 가져 오는 방식을 일반화하는 것이기 때문에 이것은 약간 벗어날 수 있습니다.

방금 "기능"이라는 단어가 포함 된 댓글로이 문제에 부딪 혔습니다. 포럼 중 누군가가 무슨 일이 일어나고 있는지 설명 할 때까지 정말 혼란 스러웠습니다. 이 문제가 곧 해결되기를 바랍니다. 챌린지의 주석과 관련된 파서 문제에 대해 경고하는 메시지가 사용자에게 표시되어야합니까?

또한보십시오:
https://forum.freecodecamp.org/t/javascript-tests-in-challenges-no-longer-working-correctly/190249

@SuperGeniusZeb 이것을보고하고 그것을 재현하는 단계를 확인

이 문제는 최근에 활성화되지 않았기 때문에 오래된 문제로 마감하겠습니다. 이것이 새로 업데이트 된 플랫폼과 여전히 관련이 있다고 생각되면 이유를 설명하고 다시여십시오.

안녕하세요.

업데이트 된 사이트에서 댓글은 여전히 ​​챌린지 테스트에서 코드로 잘못 읽혀지고 있습니다. 예를 들어 다음과 같은 도전을 생각해보십시오.

https://learn.freecodecamp.org/responsive-web-design/css-grid/create-a-column-gap-using-grid-column-gap

다음 코드를 사용하면 주석에 grid-column-gap: 20px 이 있더라도 테스트 ( grid-column-gap 가 20px 로 설정되었는지 확인)를 통과 할 수 있습니다.

<style>
  .d1{background:LightSkyBlue;}
  .d2{background:LightSalmon;}
  .d3{background:PaleTurquoise;}
  .d4{background:LightPink;}
  .d5{background:PaleGreen;}

  .container {
    font-size: 40px;
    min-height: 300px;
    width: 100%;
    background: LightGray;
    display: grid;
    grid-template-columns: 1fr 1fr 1fr;
    grid-template-rows: 1fr 1fr 1fr;
    /* add your code below this line */
    grid-column-gap: 10px;
    // grid-column-gap: 20px;
    /* add your code above this line */
  }
</style>

<div class="container">
  <div class="d1">1</div>
  <div class="d2">2</div>
  <div class="d3">3</div>
  <div class="d4">4</div>
  <div class="d5">5</div>
</div>

이 문제를 다시여십시오.

@SuperGeniusZeb https://github.com/freeCodeCamp/learn/issues 에서 새 문제를 열 ​​수 있습니까? 감사합니다.

@raisedadead 방금 거기에 새로운 문제를 만들었습니다.
https://github.com/freeCodeCamp/learn/issues/165