jhipster / jhipster/prettier-java

[2.10.0] Wrongly removes parentheses

Aperta
#948 4 commenti 0 reazioni 0 assegnatari Vedi su GitHub
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.