Tutorial page "Lifting State Up" demo can give incorrect boiling verdict
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 11.8k
- Forks
- 7.9k
- Ø Merge
- 1 T. 11 Std.
- Gemergte PRs (30 T.)
- 11
Beschreibung
Because the Boiling verdict uses the temperature in celsius , if the user puts in fahrenheit 211.9999 this will incorrectly report that it will boil. This is because the conversion to celsius is limited to 3 digits and gets rounded up to 100.

A fix for this is to change BoilingVerdict to accept scale and temperature and the code to do a slightly different check for each:
function BoilingVerdict(props) {
if (props.scale === 'c' ? props.temperature >= 100 : props.temperature >= 212) {
return <p>The water would boil.</p>;
}
return <p>The water would not boil.</p>;
}
<BoilingVerdict
scale={this.state.scale}
temperature={this.state.temperature}
/>

I know that I could do a fork and pull request but this is about as far as my interest in this subject goes so anyone else is free to fix this admittedly tiny issue.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie mit der Tutorial-Seite „Lifting State Up“ und suchen Sie die Komponente BoilingVerdict sowie deren Verwendung. Überprüfen Sie das Ergebnis für Fahrenheit-Werte knapp unter 212, aktualisieren Sie dann die Komponente und bestätigen Sie, dass 211.9999 nicht als siedend gemeldet wird, während Schwellenwerte dies weiterhin tun.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, react
- Bereich
- documentation
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100