browserify / browserify/static-eval

Differences in `ReturnStatement` evaluations

Aperta
#39 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
JavaScript
Stelle
177
Fork
27
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Why does the following work:
```js
[1, 2].map(function(x) { return x + 1 })
```
But not the following:
```js
[{a: 1}, {a: 2}].map(function(x) { return x["a"] })
```

Since `.map()` does not seem to be an issue, and member expressions of course aren’t, I’m not sure why this fails. The actual error is `TypeError: Cannot read properties of null (reading 'a')`

The inner function ASTs are respectively:
```js
ReturnStatement {
type: 'ReturnStatement',
argument: BinaryExpression {
type: 'BinaryExpression',
operator: '+',
left: Identifier { type: 'Identifier', name: 'x' },
right: Literal { type: 'Literal', value: 1, raw: '1' }
}
}
```
And:
```js
ReturnStatement {
type: 'ReturnStatement',
argument: ComputedMemberExpression {
type: 'MemberExpression',
computed: true,
object: Identifier { type: 'Identifier', name: 'x' },
property: Literal { type: 'Literal', value: 'a', raw: '"a"' }
}
}
```

Statically evaluating a MemberExpression is not harder than a BinaryExpression, is it?

After looking at the code, it seems to me that in both cases the `x` statement is statically evaluated to `null`. In the `BinaryExpression` code, the binary evaluation returns `null + 1` which is `1`, and you then allow and execute the function.

However since `null["a"]` raises an error, the function is not allowed and executed. However `x["a"]` is knowable statically (it is either the property or `undefined` if it’s absent − if you call `evaluate('x["b"]', {x: {a: 1}})`, you get `undefined`, not an error), so the function should execute in this case as well.

There is a simple workaround which is to return `(x || {a: ""})["a"]` instead of `x["a"]`, this avoids throwing an error by ensuring there is a fallback value to `x.a` if `x` is undefined.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Inizia con il percorso di valutazione di ReturnStatement e confronta il modo in cui vengono gestiti gli argomenti di BinaryExpression e MemberExpression. Riproduci entrambi i callback di map e l’esempio evaluate('x[\"b\"]', {x: {a: 1}}), quindi verifica che l’accesso a un membro noto staticamente non rifiuti erroneamente il callback, mentre l’accesso effettivo a null continui a comportarsi correttamente.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
javascript
Ambito
tooling
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.