[java] MongoDB Java 드라이버가 조건부에서 난수 생성기를 사용하는 이유는 무엇입니까?

그 라인의 역사를 조사한 후, 나의 주요 결론은 직장에서 무능한 프로그래밍이 있었다는 것입니다.

1. 그 선은 사실 복잡하다. 일반적인 형태

   a? true : b

   에 대한 boolean a, b간단한

   a || b

2. 주변 부정과 과도한 괄호는 사물을 더욱 복잡하게 만듭니다. De Morgan의 법칙 을 염두에두면 이 코드가 중요하다는 것은 사소한 관찰입니다.

   if (!_ok && Math.random() <= 0.1)
     return res;

3. 이 논리 를 처음 도입 한 커밋 에는

   if (_ok == true) {
     _logger.log( Level.WARNING , "Server seen down: " + _addr, e );
   } else if (Math.random() < 0.1) {
     _logger.log( Level.WARNING , "Server seen down: " + _addr );
   }

   — 무능한 코딩의 또 다른 예이지만 반대 논리를 주목하십시오 . 여기서는 _ok다른 경우 중 하나 또는 10 % 인 경우 이벤트가 기록되는 반면 2의 코드 는 10 %의 시간을 반환 하고 90 %의 시간을 기록합니다. 따라서 나중에 커밋은 명확성뿐만 아니라 정확성 자체를 망쳤습니다.

   나는 당신이 게시 한 코드에서 실제로 저자가 if-then어떻게 원본을 초기 return상태에 필요한 부정으로 어떻게 변형 시켰는지 알 수 있다고 생각 합니다. 그러나 그는 불평등을 뒤집어서 효과적인 “이중 부정”을 엉망으로 만들었습니다.

4. 코딩 스타일 문제는 제쳐두고, 확률 적 로깅은 그 자체로는 상당히 모호한 관행입니다. 특히 로그 항목에는 고유 한 동작이 문서화되어 있지 않기 때문입니다. 서버의 현재 다운 상태와 같은 사실에 대한 설명을 줄이려는 의도는 분명합니다. 적절한 해결책은 10 %의 임의의 관찰을 제외하고 각각의 관찰이 아닌 서버 상태의 변경 사항 만 기록하는 것 입니다. 그렇습니다. 조금만 더 노력하면됩니다.

나는 단지 세 줄의 코드 를 검사하여 축적 된이 무능의 모든 증거가 프로젝트 전체를 공정하게 말하지 않고,이 작업이 최대한 빨리 정리 되기를 바랄 뿐입니다 .

답변

  private int logit = -1;

 if( !ok && (logit = (logit + 1 ) % 10)  == 0 ) { //log error

private synchronized int getLogit() {
   return (logit = (logit + 1 ) % 10);
}

 if( !ok && getLogit() == 0 ) { //log error