Skip to content

Unhealthy code style #18

@chiro-hiro

Description

@chiro-hiro

I've found you are always prevent to use for loop but for loop is better than while loop in case of determined loop steps. It's more reliable.

E.g:

function nonZero (len) {
  var out = Buffer.allocUnsafe(len)
  var i = 0
  var cache = randomBytes(len * 2)
  var cur = 0
  var num
  while (i < len) {
    if (cur === cache.length) {
      cache = randomBytes(len * 2)
      cur = 0
    }
    num = cache[cur++]
    if (num) {
      out[i++] = num
    }
  }
  return out
}

Could be:

function nonZero (len) {
  var out = Buffer.allocUnsafe(len)
  var cache = randomBytes(len * 2)
  var num
  for (  var i = 0, cur = 0; i < len; i++) {
    if (cur === cache.length) {
      cache = randomBytes(len * 2)
      cur = 0
    }
    num = cache[cur++]
    if (num) {
      out[i] = num
    }
  }
  return out
}

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions