jhipster / jhipster/prettier-java
[2.10.0] Wrongly removes parentheses
- Lingua principale
- Java
- Stelle
- 1.2k
- Fork
- 120
- Merge medio
- 1g 14h
- PR unite (30g)
- 27
Descrizione
**Prettier-Java 2.10.0**
```sh
--print-width 80
```
**Input:**
```java
return a != null && (b == c);
(minimumRelevanceScore * 2) - 1;
```
**Output:**
```java
return a != null && b == c;
minimumRelevanceScore * 2 - 1;
```
**Expected behavior:**
Unchanged.
The second line is not technically incorrect but I personally don't like parentheses removed in such expressions. There is a long-standing issue in Prettier for JS for the [same thing](https://github.com/prettier/prettier/issues/187). The first one is wrong because it changes the meaning of the program.
P.S. I now see that Prettier for JS does the same thing. Horrible. And maybe it's because of the same issue. I would like Prettier for Java to be better than that! 😄
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Riproduci il problema con gli snippet Java e --print-width 80, quindi segui la logica del formatter responsabile della rimozione delle parentesi. Il lavoro è completato quando la prima espressione conserva le parentesi, in modo che il suo significato rimanga invariato, mentre il comportamento di formattazione segnalato è coperto da un controllo di regressione.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- java
- Ambito
- tooling
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Tranquilla
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 52/100