fix (frameworks/react): small adjusts

santanche · Sep 4, 2024 · 70e6989 · 70e6989
1 parent bff9a0c
commit 70e6989
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 18 deletions.
diff --git a/frameworks/react/6-2-hook-state-object/src/MedicationItem.jsx b/frameworks/react/6-2-hook-state-object/src/MedicationItem.jsx
@@ -1,5 +1,9 @@
 import { useState } from 'react'
 
+function incrementMedication(medication, setMedication) {
+  setMedication({...medication, quantity: medication.quantity + 1})
+}
+
 export default function MedicationItem() {
   const [medication, setMedication] = useState({
     name: 'Velocirest',
@@ -9,9 +13,9 @@ export default function MedicationItem() {
     unity: 'mg',
     quantity: 1,
     frequency: 'day'
-    })
+  })
 
-  const { name, description, image, dose, unity, quantity, frequency } = medication;
+  const { name, description, image, dose, unity, quantity, frequency } = medication
 
   return (
     <div style={{ width: '300px', background: 'lightgray' }}>
@@ -20,7 +24,7 @@ export default function MedicationItem() {
       <p>{description}</p>
       <p><b>dose:</b> {dose} {unity}</p>
       <p><b>frequency:</b> {quantity} / {frequency}</p>
-      <button type="button" onClick={() => setMedication({ ...medication, quantity: quantity + 1 })}>
+      <button type="button" onClick={() => incrementMedication(medication, setMedication)}>
         Increase quantity
       </button>
     </div>

diff --git a/frameworks/react/6-3-hook-effect-object/src/MedicationItem.jsx b/frameworks/react/6-3-hook-effect-object/src/MedicationItem.jsx
@@ -1,5 +1,9 @@
 import { useState, useEffect } from 'react'
 
+function incrementMedication(medication, setMedication) {
+  setMedication({...medication, quantity: medication.quantity + 1})
+}
+
 function calculateWeekly(dose, quantity, frequency) {
   const frequencyTimes = {
     '8 hours': 21,
@@ -22,12 +26,6 @@ export default function MedicationItem() {
     })
   const [weeklyDose, setWeeklyDose] = useState(0)
 
-  const frequencyTimes = {
-    '8 hours': 21,
-    'day': 7,
-    'week': 1
-  }
-
   useEffect(() => {
     setWeeklyDose(calculateWeekly(medication.dose, medication.quantity, medication.frequency))
   }, [medication])
@@ -42,7 +40,7 @@ export default function MedicationItem() {
       <p><b>dose:</b> {dose} {unity}</p>
       <p><b>frequency:</b> {quantity} / {frequency}</p>
       <p><b>weekly dose:</b> {weeklyDose} {unity}</p>
-      <button type="button" onClick={() => setMedication({ ...medication, quantity: quantity + 1 })}>
+      <button type="button" onClick={() => incrementMedication(medication, setMedication)}>
         Increase quantity
       </button>
     </div>

diff --git a/full-stack/5-react-fastapi/README.md b/full-stack/5-react-fastapi/README.md