Skip to content

Confusing inspection for sparse arrays #11570

Closed
Closed
Confusing inspection for sparse arrays#11570
Labels
utilIssues and PRs related to the built-in util module.

Description

@TimothyGu
TimothyGu
opened on Feb 26, 2017
  • Version: all supported versions (oldest tested branch is v4.x (4.8.0))
  • Platform: all
  • Subsystem: util

In the node REPL, sparse elements of an array are shown as "empty":

> new Array(3)
[ , ,  ]

However, when one copy the resulting [ , , ] and reenter it into the prompt, a different array is shown, thus confusing the developer:

> [ , ,  ]
[ ,  ]

How the Chrome inspector deals with this situation is explicitly make the inspected result not compliant JS:

> new Array(3)
[undefined × 3]
> [undefined, , undefined] // to differentiate between a hole and `undefined`
[undefined, undefined × 1, undefined]

Firefox is more upfront:

> new Array(3)
Array [ <3 empty slots> ]
> [undefined, , undefined] // to differentiate between a hole and `undefined`
Array [ undefined, <1 empty slot>, undefined ]

Activity

  1. TimothyGu
    added
    utilIssues and PRs related to the built-in util module.
    on Feb 26, 2017

  2. bnoordhuis commented on Feb 27, 2017

    @bnoordhuis
    bnoordhuis
    on Feb 27, 2017
    Member

    I agree it's confusing but I think it's correct. Due to the trailing comma, [ , , ] is actually a two-element array, not three:

    > new Array(3).length
3
> [ , , ].length
2

  3. TimothyGu commented on Feb 27, 2017

    @TimothyGu
    TimothyGu
    on Feb 27, 2017
    MemberAuthor

    Yeah, the REPL is doing its job alright. It's just the inspection is a bit counter-intuitive.

    Personally I'm leaning more towards Firefox's approach. It's clear and unambiguous, and I'm worried the Unicode × character used in Chrome's version might not be friendly towards Windows terminals. Substitutes like the letter X might be confusing in themselves.

  4. aqrln commented on Feb 27, 2017

    @aqrln
    aqrln
    on Feb 27, 2017
    Contributor

    @TimothyGu @bnoordhuis what about something like this?

    > new Array(3)
[ [3 empty] ]
> [1, 2, , , , 3, 4, 5]
[ 1, 2, [3 empty], 3, 4, 5 ]
> [1, 2, , , , 3, 4, 5, , 2]
[ 1, 2, [3 empty], 3, 4, 5, [1 empty], 2 ]
    Screenshot

  5. TimothyGu commented on Feb 27, 2017

    @TimothyGu
    TimothyGu
    on Feb 27, 2017
    MemberAuthor

    I'd be okay with that. Personally I'd prefer the [3 empty] to have the same color as undefined just to be consistent, though that's bikeshedding.

  6. aqrln commented on Feb 27, 2017

    @aqrln
    aqrln
    on Feb 27, 2017
    Contributor

    @TimothyGu IMO it's more appropriate to mark it with special style, but ¯\_(ツ)_/¯

  7. aqrln
    added a commit that references this issue on Mar 7, 2017

    util: change sparse arrays inspection format

    fb4fe0d
  8. addaleax
    closed this as completedin ec2f098on Mar 10, 2017
  9. jungx098
    added a commit that references this issue on Mar 21, 2017

    util: change sparse arrays inspection format

    80fa6bc

  10. 36 remaining items

  11. victorgomes
    added a commit that references this issue on Jun 3, 2024

    [nodejs#11570] Disable Proxy test in parallel/test-v8-serdes.js

    75b9de3
  12. pthier
    added a commit that references this issue on Jul 15, 2024

    [nodejs#11570] Disable Proxy test in parallel/test-v8-serdes.js

    eaa2371
  13. victorgomes
    added a commit that references this issue on Aug 22, 2024

    [nodejs#11570] Disable Proxy test in parallel/test-v8-serdes.js

    aa645f8
  14. pthier
    added a commit that references this issue on Oct 2, 2024

    [nodejs#11570] Disable Proxy test in parallel/test-v8-serdes.js

    5cce325
  15. victorgomes
    added a commit that references this issue on Nov 18, 2024

    [nodejs#11570] Disable Proxy test in parallel/test-v8-serdes.js

    81dde33
  16. victorgomes
    added a commit that references this issue on Dec 18, 2024

    [nodejs#11570] Disable Proxy test in parallel/test-v8-serdes.js

    c4002ba
  17. victorgomes
    added a commit that references this issue on Jan 20, 2025

    [nodejs#11570] Disable Proxy test in parallel/test-v8-serdes.js

    fd88e41
  18. pthier
    added a commit that references this issue on Mar 17, 2025

    [nodejs#11570] Disable Proxy test in parallel/test-v8-serdes.js

    f2b4c5f
  19. pthier
    added a commit that references this issue on Jun 27, 2025

    [nodejs#11570] Disable Proxy test in parallel/test-v8-serdes.js

    bb843d2
  20. pthier
    added a commit that references this issue on Aug 21, 2025

    [nodejs#11570] Disable Proxy test in parallel/test-v8-serdes.js

    fe45941
  21. victorgomes
    added a commit that references this issue on Sep 26, 2025

    [nodejs#11570] Disable Proxy test in parallel/test-v8-serdes.js

    3abbba8
  22. pthier
    added a commit that references this issue on Nov 21, 2025

    [nodejs#11570] Disable Proxy test in parallel/test-v8-serdes.js

    2f786c6
  23. victorgomes
    added a commit that references this issue on Feb 24, 2026

    [nodejs#11570] Disable Proxy test in parallel/test-v8-serdes.js

    e940ad3
  24. pthier
    added a commit that references this issue on Apr 16, 2026

    [nodejs#11570] Disable Proxy test in parallel/test-v8-serdes.js

    41490fc
  25. sjarmak
    added a commit that references this issue on Jul 27, 2026

    util: change sparse arrays inspection format

    ec2f098
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    utilIssues and PRs related to the built-in util module.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions