if 조건문에 변수 할당, 모범 사례? [닫은]

1 년 전에 Java와 같은 고전적인 OO 언어에서 JavaScript로 옮겼습니다. 다음 코드는 Java에서 확실히 권장되지 않거나 올바르지 않습니다.

if(dayNumber = getClickedDayNumber(dayInfo))
{
    alert("day number found : " + dayNumber);
}
function getClickedDayNumber(dayInfo)
{
    dayNumber = dayInfo.indexOf("fc-day");
    if(dayNumber != -1) //substring found
    {
        //normally any calendar month consists of "40" days, so this will definitely pick up its day number.
        return parseInt(dayInfo.substring(dayNumber+6, dayNumber+8));
    }
    else return false;
}

기본적으로 방금 if 조건 문의 값에 변수를 할당하고 할당 된 값이 부울 인 것처럼 즉시 확인할 수 있다는 것을 알았습니다.

더 안전한 내기를 위해 일반적으로 코드를 두 줄로 분리하고 먼저 할당 한 다음 변수를 확인합니다.하지만 이제는 이것을 발견 했으므로 경험이 많은 JavaScript 개발자의 눈에 좋은 방법인지 아닌지 궁금합니다.

나는 그것을 추천하지 않을 것입니다. 문제는 값을 비교하려고하지만 또는 =대신 단일을 사용하는 일반적인 오류처럼 보입니다 . 예를 들어 다음과 같은 경우 :=====

if (value = someFunction()) {
    ...
}

그게 의도 한 것인지, 아니면 이렇게 작성하려고했는지 알 수 없습니다.

if (value == someFunction()) {
    ...
}

실제로 할당을 수행하려면 명시 적 비교도 수행하는 것이 좋습니다.

if ((value = someFunction()) === <whatever truthy value you are expecting>) {
    ...
}

좋은 습관이 아니라는 증거는 없습니다. 예, 실수처럼 보일 수 있지만 현명한 댓글로 쉽게 해결할 수 있습니다. 예를 들어 :

if (x = processorIntensiveFunction()) { // declaration inside if intended
    alert(x);
}

이 함수가 다음과 같이 두 번째로 실행되도록 허용해야하는 이유 :

alert(processorIntensiveFunction());

첫 번째 버전이 나빠 보이기 때문에? 나는 그 논리에 동의 할 수 없습니다.

나는 그것을 여러 번했다. JavaScript 경고를 우회하기 위해 두 개의 괄호를 추가합니다.

if ((result = get_something())) { }

그것을 피해야합니다. 정말로 그것을 사용하고 싶다면 그 위에 무엇을하고 있는지에 대한 주석을 작성하십시오.

Java에서도이 작업을 수행 할 수 있습니다. 그리고 그것은 좋은 습관이 아닙니다. :)

(그리고 ===형식화 된 동등성을 위해 자바 스크립트에서 사용하십시오 . JS에 대한 Crockford의 The Good Parts 책을 읽어보십시오.)

while루프를 사용 하는 경우가 하나 있습니다 .
파일을 읽을 때 일반적으로 다음과 같이합니다.

void readFile(String pathToFile) {
    // Create a FileInputStream object
    FileInputStream fileIn = null;
    try {
        // Create the FileInputStream
        fileIn = new FileInputStream(pathToFile);
        // Create a variable to store the current line's text in
        String currentLine;
        // While the file has lines left, read the next line,
        // store it in the variable and do whatever is in the loop
        while((currentLine = in.readLine()) != null) {
            // Print out the current line in the console
            // (you can do whatever you want with the line. this is just an example)
            System.out.println(currentLine);
        }
    } catch(IOException e) {
        // Handle exception
    } finally {
        try {
            // Close the FileInputStream
            fileIn.close();
        } catch(IOException e) {
            // Handle exception
        }
    }
}

while9 행 에서 -loop를 보십시오. 거기에서 새로운 행을 읽고 변수에 저장 한 다음 루프의 내용을 실행합니다. 나는 이것이-문이 아니라는 것을 알고 if있지만 while 루프도 귀하의 질문에 포함될 수 있다고 생각합니다.

이에 대한 이유는를 사용하는 경우이다 FileInputStream, 당신이 전화를 할 때마다 FileInputStream.readLine(), 당신이 단지와 루프에서 호출했을 그렇다면, 파일의 다음 라인을 읽어 fileIn.readLine() != null변수를 할당하는 대신 호출하지 않고 (currentLine = fileIn.readLine()) != null, 다음에서 호출 루프 내부에서도 두 번째 줄만 얻을 수 있습니다.

당신이 이해하고 행운을 빕니다!

Java의 if 문 내에서도 할당을 수행 할 수 있습니다. 좋은 예는 무언가를 읽고 쓰는 것입니다.

http://www.exampledepot.com/egs/java.io/CopyFile.html?l=new

코드:

// Copies src file to dst file.
// If the dst file does not exist, it is created
void copy(File src, File dst) throws IOException 
{
    InputStream in = new FileInputStream(src);
    OutputStream out = new FileOutputStream(dst);

    // Transfer bytes from in to out
    byte[] buf = new byte[1024];
    int len;
    while ((len = in.read(buf)) > 0) {
        out.write(buf, 0, len);
    }
    in.close();
    out.close();
}

Martin Fowlers의 책 리팩토링 을 참조하면 기존 코드의 디자인을 개선 할 수 있습니다 ! 그런 다음 좋은 연습이 될 몇 가지 경우가 있습니다. 함수 또는 메서드 호출을 사용하여 케이스를 주장하는 길고 복잡한 조건 :

"자극

프로그램에서 가장 일반적인 복잡성 영역 중 하나는 복잡한 조건부 논리에 있습니다. 조건을 테스트하고 다양한 조건에 따라 다양한 작업을 수행하는 코드를 작성하면 매우 긴 방법으로 빠르게 끝납니다. 방법의 길이는 그 자체로 읽기 어렵게 만드는 요소이지만 조건은 어려움을 증가시킵니다. 문제는 일반적으로 조건 검사와 작업의 코드가 어떤 일이 발생하는지 알려주지 만 그 이유를 쉽게 모호하게 할 수 있다는 사실에 있습니다.

큰 코드 블록과 마찬가지로, 코드 블록을 분해하고 해당 코드 블록의 의도를 따서 명명 된 메서드 호출로 코드 청크를 대체하여 의도를 더 명확하게 만들 수 있습니다. > 조건을 사용하면 조건부 부분과 각 대안에 대해이 작업을 수행하여 추가 혜택을받을 수 있습니다. 이렇게하면 상태를 강조하고> 당신이 무엇을 기준으로 삼고 있는지 명확하게 알 수 있습니다. 분기 이유도 강조합니다. "

그리고 네 그의 대답은 Java 구현에도 유효합니다. 예제에서는 변수에 조건부 함수를 할당하지 않습니다.

좋은 습관이 아닙니다. 곧 그것에 대해 혼란 스러울 것입니다. 일반적인 오류 인 "="및 "=="연산자를 잘못 사용하는 것과 비슷해 보입니다.

두 줄의 코드로 나누어야합니다. 코드를 더 명확하게 만드는 데 도움이 될뿐만 아니라 나중에 쉽게 리팩토링 할 수 있습니다. IF 조건을 변경한다고 상상해보십시오. 실수로 줄을 제거하고 변수에 더 이상 할당 된 값을 얻지 못할 수 있습니다.

나는 이것을 좀 더 구식 C 스타일이라고 생각한다. JavaScript에서는 실제로 좋은 습관이 아니므로 피해야합니다.

다음과 같이 할 수 있습니다.

if (value = /* sic */ some_function()){
  use_value(value)
}

나는 golang에서 왔는데, 다음과 같은 것을 보는 것이 일반적입니다.

if (err := doSomething(); err != nil) {
    return nil, err
}

된 err것과 범위가 if블록 만. 따라서 여기에 내가 es6에서하고있는 작업이 있는데, 꽤보기 흉해 보이지만 내 엄격한 eslint 규칙이 윙윙 거리게하지 않고 똑같이 달성합니다.

{
  const err = doSomething()
  if (err != null) {
    return (null, err)
  }
}

추가 중괄호는 새로운 "어휘 범위"를 정의합니까? 즉 const,을 err사용할 수 있으며 외부 블록에서는 사용할 수 없습니다.