정수가 아닌 루프 변수를 사용하는 것이 올바르게 작동하는 것과 같은 경우에도 일반적으로 나쁜 스타일이라는 다른 모든 대답에 동의합니다. 그러나 여기 스타일이 나쁜 또 다른 이유가있는 것 같습니다.

코드는 사용 가능한 선 너비가 정확히 0에서 5.0까지의 0.5의 배수임을 알고 있습니다. 그래야합니까? 그것은 쉽게 변경 될 수있는 사용자 인터페이스 결정처럼 보입니다 (예를 들어, 너비가 커짐에 따라 사용 가능한 너비 사이의 간격이 더 커지기를 원할 수도 있습니다. 0.25, 0.5, 0.75, 1.0, 1.5, 2.0, 2.5, 3.0, 4.0, 5.0 또는 무엇인가).

코드는 사용 가능한 선 너비가 모두 부동 소수점 숫자와 10 진수로 “좋은”표현을 가지고 있음을 “알고”있습니다. 그것은 또한 변할 수도있는 것 같습니다. (어느 시점에서 0.1, 0.2, 0.3, …을 원할 수 있습니다.)

여러분의 코드는 버튼에 넣을 텍스트가 자바 스크립트가 부동 소수점 값으로 바뀌는 것임을 “알고”있습니다. 그것은 또한 변할 수도있는 것 같습니다. (예를 들어 언젠가는 1/3과 같은 너비를 원할 것입니다.이 너비는 0.33333333333333 또는 다른 것으로 표시하지 않을 것입니다. 또는 “1.5”와 일관성을 유지하기 위해 “1”대신 “1.0”을보고 싶을 수도 있습니다. .)

이것들은 모두 하나의 약점의 표현처럼 느껴집니다. 이것은 일종의 레이어 혼합입니다. 이러한 부동 소수점 숫자는 소프트웨어의 내부 논리의 일부입니다. 버튼에 표시되는 텍스트는 사용자 인터페이스의 일부입니다. 여기 코드에서보다 분리되어 있어야합니다. “이 중 어느 것이 강조 표시되어야합니까?”와 같은 개념 사용자 인터페이스 문제이며 아마도 부동 소수점 값에 묶여서는 안됩니다. 그리고 여기서 루프는 실제로 선 너비가 아닌 버튼 위의 루프 입니다. 그렇게하면 정수가 아닌 값을 갖는 루프 변수를 사용하려는 유혹이 사라집니다. 연속적인 정수 또는 for … in / for … of 루프를 사용하는 것입니다.

제 생각에는 정수가 아닌 숫자를 반복하려고 하는 대부분의 경우는 다음과 같습니다. 숫자 문제와 전혀 관련이없는 다른 이유가 있으며 코드가 다르게 구성되어야하는 이유가 있습니다. ( 모든 경우가 아니라 ; 일부 수학적 알고리즘이 정수가 아닌 값에 대한 루프로 가장 깔끔하게 표현 될 수 있다고 상상할 수 있습니다.)

답변

var BUTTONS=11;
var STANDARD_LINE=3;

for(var i=0; i<BUTTONS; i++) {
    button.text = (i*0.5)+'';
    if (i==STANDARD_LINE) {
      button.color='red';
    }
}

function precisionRound(number, precision) {
  let factor = Math.pow(10, precision);
  return Math.round(number * factor) / factor;
}

var maxButtonValue = 5.0;
var buttonSpacing = 0.5;

let countEstimate = precisionRound(maxButtonValue / buttonSpacing, 5);
var buttonCount = Math.floor(countEstimate) + 1;

var highlightPosition = 3;
var highlightColor = 'red';

for (let i=0; i < buttonCount; i++) {
    let buttonValue = i / buttonSpacing;
    button.text = buttonValue.toString();
    if (i == highlightPosition) {
        button.color = highlightColor;
    }
}

var MAX=5.0;
var DIFF=0.5
var STANDARD_LINE=1.5;

for(var i=0; (i*DIFF) < MAX ; i=i+1){
    var val = i * DIFF

    button.text=val+'';

    if(val==STANDARD_LINE){
      button.color='red';
    }
}