From 9759b6fc81a72e698b7d004d90970a85d9b64cad Mon Sep 17 00:00:00 2001
From: Balint Morasz <balint@morasz.hu>
Date: Thu, 27 Jul 2023 00:08:58 +0200
Subject: [PATCH] refactor: split page into smaller components part 3

everything is nice and small now
---
 src/components/FajSelector.tsx     |  63 ++++++++++--------
 src/components/OsztalySelector.tsx |  73 +++++++++++++++++++++
 src/pages/CreateCharacter.tsx      | 100 +++++++----------------------
 3 files changed, 134 insertions(+), 102 deletions(-)
 create mode 100644 src/components/OsztalySelector.tsx

diff --git a/src/components/FajSelector.tsx b/src/components/FajSelector.tsx
index 028f1c9..f1ba527 100644
--- a/src/components/FajSelector.tsx
+++ b/src/components/FajSelector.tsx
@@ -1,31 +1,44 @@
 import React from "react";
-import {Faj, FajLabel} from "../domain-models/faj";
+import {Faj, FajDescription, FajLabel, FajSpecials} from "../domain-models/faj";
 import {UseFormRegisterReturn} from "react-hook-form";
 
-function FajSelector(props: {register: () => UseFormRegisterReturn}) {
-    const {register} = props
-    return <div className='row m-2'>
-        <label className='col-lg-1 col-sm-2 col-form-label'>Faj</label>
-        <select className='col form-select' defaultValue={Faj.Ember} {...register()}>
-            <option key={Faj.Ember} value={Faj.Ember}>{FajLabel(Faj.Ember)}</option>
-            <optgroup label='más emberi népek'>
-                <option key={Faj.Amazon} value={Faj.Amazon}>{FajLabel(Faj.Amazon)}</option>
-                <option key={Faj.Birodalmi} value={Faj.Birodalmi}>{FajLabel(Faj.Birodalmi)}</option>
-                <option key={Faj.Eszaki} value={Faj.Eszaki}>{FajLabel(Faj.Eszaki)}</option>
-                <option key={Faj.Etuniai} value={Faj.Etuniai}>{FajLabel(Faj.Etuniai)}</option>
-                <option key={Faj.Osember} value={Faj.Osember}>{FajLabel(Faj.Osember)}</option>
-            </optgroup>
-            <optgroup label='nem emberi fajok'>
-                <option key={Faj.Elf} value={Faj.Elf}>{FajLabel(Faj.Elf)}</option>
-                <option key={Faj.Felelf} value={Faj.Felelf}>{FajLabel(Faj.Felelf)}</option>
-                <option key={Faj.Felork} value={Faj.Felork}>{FajLabel(Faj.Felork)}</option>
-                <option key={Faj.Felszerzet}
-                        value={Faj.Felszerzet}>{FajLabel(Faj.Felszerzet)}</option>
-                <option key={Faj.Gnom} value={Faj.Gnom}>{FajLabel(Faj.Gnom)}</option>
-                <option key={Faj.Torpe} value={Faj.Torpe}>{FajLabel(Faj.Torpe)}</option>
-            </optgroup>
-        </select>
-    </div>
+function FajSelector(props: { register: () => UseFormRegisterReturn, currentFaj: () => Faj }) {
+    const {register, currentFaj} = props
+    return <>
+        <div className='row m-2'>
+            <label className='col-lg-1 col-sm-2 col-form-label'>Faj</label>
+            <select className='col form-select' defaultValue={Faj.Ember} {...register()}>
+                <option key={Faj.Ember} value={Faj.Ember}>{FajLabel(Faj.Ember)}</option>
+                <optgroup label='más emberi népek'>
+                    <option key={Faj.Amazon} value={Faj.Amazon}>{FajLabel(Faj.Amazon)}</option>
+                    <option key={Faj.Birodalmi} value={Faj.Birodalmi}>{FajLabel(Faj.Birodalmi)}</option>
+                    <option key={Faj.Eszaki} value={Faj.Eszaki}>{FajLabel(Faj.Eszaki)}</option>
+                    <option key={Faj.Etuniai} value={Faj.Etuniai}>{FajLabel(Faj.Etuniai)}</option>
+                    <option key={Faj.Osember} value={Faj.Osember}>{FajLabel(Faj.Osember)}</option>
+                </optgroup>
+                <optgroup label='nem emberi fajok'>
+                    <option key={Faj.Elf} value={Faj.Elf}>{FajLabel(Faj.Elf)}</option>
+                    <option key={Faj.Felelf} value={Faj.Felelf}>{FajLabel(Faj.Felelf)}</option>
+                    <option key={Faj.Felork} value={Faj.Felork}>{FajLabel(Faj.Felork)}</option>
+                    <option key={Faj.Felszerzet}
+                            value={Faj.Felszerzet}>{FajLabel(Faj.Felszerzet)}</option>
+                    <option key={Faj.Gnom} value={Faj.Gnom}>{FajLabel(Faj.Gnom)}</option>
+                    <option key={Faj.Torpe} value={Faj.Torpe}>{FajLabel(Faj.Torpe)}</option>
+                </optgroup>
+            </select>
+        </div>
+        <div className='row m-2'>
+            <label className='col-lg-1 col-sm-2 form-label'></label>
+            <div className='col'>
+                <p className='row pt-2'>
+                    {FajDescription(currentFaj())}
+                </p>
+                <ul>
+                    {FajSpecials(currentFaj()).map(((special, i) => (<li key={currentFaj() + i}>{special}</li>)))}
+                </ul>
+            </div>
+        </div>
+    </>
 }
 
 export default FajSelector
\ No newline at end of file
diff --git a/src/components/OsztalySelector.tsx b/src/components/OsztalySelector.tsx
new file mode 100644
index 0000000..bbce2ba
--- /dev/null
+++ b/src/components/OsztalySelector.tsx
@@ -0,0 +1,73 @@
+import React from "react";
+import {
+    Osztaly,
+    OsztalyDescription,
+    OsztalyLabel,
+    OsztalyProperties,
+    OsztalySpecialSkills
+} from "../domain-models/osztaly";
+import {Faj} from "../domain-models/faj";
+import {UseFormRegisterReturn} from "react-hook-form";
+
+function OsztalySelector(props: {currentFaj: () => Faj, currentOsztaly: () => Osztaly, register: () => UseFormRegisterReturn}) {
+    const {currentFaj, currentOsztaly, register} = props;
+
+    return <>
+        <div className='row m-2'>
+            <label className='col-lg-1 col-sm-2 col-form-label'>Osztály</label>
+            <select className="col form-select" defaultValue={Osztaly.Harcos} {...register()}>
+                <optgroup label='Harcos'>
+                    <option value={Osztaly.Harcos}>{OsztalyLabel(Osztaly.Harcos)}</option>
+                    {currentFaj() === Faj.Amazon && (
+                        <option value={Osztaly.Amazon}>{OsztalyLabel(Osztaly.Amazon)}</option>) }
+                    <option value={Osztaly.Barbar}>{OsztalyLabel(Osztaly.Barbar)}</option>
+                    <option value={Osztaly.Ijasz}>{OsztalyLabel(Osztaly.Ijasz)}</option>
+                    <option value={Osztaly.Kaloz}>{OsztalyLabel(Osztaly.Kaloz)}</option>
+                </optgroup>
+                <option value={Osztaly.Pap}>{OsztalyLabel(Osztaly.Pap)}</option>
+                {currentFaj() !== Faj.Amazon && (<option value={Osztaly.Tolvaj}>{OsztalyLabel(Osztaly.Tolvaj)}</option>)}
+                {![Faj.Osember, Faj.Amazon].includes(currentFaj()) && (
+                    <optgroup label='Varázsló'>
+                        <option value={Osztaly.Varazslo}>{OsztalyLabel(Osztaly.Varazslo)}</option>
+                        <option value={Osztaly.Illuzionista}>{OsztalyLabel(Osztaly.Illuzionista)}</option>
+                    </optgroup>
+                )}
+            </select>
+        </div>
+        <div className='row m-2'>
+            <label className='col-lg-1 col-sm-2 form-label'></label>
+            <div className='col' >
+                <p className='row pt-2'>
+                    {OsztalyDescription(currentOsztaly())}
+                </p>
+                <ul>
+                    {OsztalyProperties(currentOsztaly()).map(((special, i) => (<li key={currentOsztaly() + i}>{special}</li>)))}
+                </ul>
+                <p className='d-inline-flex gap-1'>
+                    {OsztalySpecialSkills(currentOsztaly()).map(skill => (
+                        <button
+                            key={'btn' + skill.Id}
+                            className='btn btn-outline-dark'
+                            type='button'
+                            data-bs-toggle="collapse"
+                            data-bs-target={"#collapse-"+skill.Id}
+                            aria-expanded="false"
+                            aria-controls="collapseExample"
+                        >
+                            {skill.Name}
+                        </button>
+                    ))}
+                </p>
+                {OsztalySpecialSkills(currentOsztaly()).map(skill => (
+                    <div key={'collapse'+skill.Id} className="collapse" id={'collapse-' + skill.Id}>
+                        <div className="card card-body">
+                            {skill.Description}
+                        </div>
+                    </div>
+                ))}
+            </div>
+        </div>
+    </>
+}
+
+export default OsztalySelector;
\ No newline at end of file
diff --git a/src/pages/CreateCharacter.tsx b/src/pages/CreateCharacter.tsx
index a6f9a66..7bd373e 100644
--- a/src/pages/CreateCharacter.tsx
+++ b/src/pages/CreateCharacter.tsx
@@ -1,16 +1,14 @@
 import React from 'react';
 import './CreateCharacter.css'
 import {useForm} from "react-hook-form";
-import {Faj, FajDescription, FajLabel, FajSpecials} from "../domain-models/faj";
-import {
-    Osztaly,
-    OsztalyDescription,
-    OsztalyLabel,
-    OsztalyProperties,
-    OsztalySpecialSkills
-} from "../domain-models/osztaly"
+import {Faj} from "../domain-models/faj";
+import {Osztaly} from "../domain-models/osztaly"
 import FajSelector from "../components/FajSelector";
 import Tulajdonsagok from "../components/Tulajdonsagok";
+import OsztalySelector from "../components/OsztalySelector";
+
+const FAJ_FIELD_NAME = 'faj'
+const OSZTALY_FIELD_NAME = 'osztaly'
 
 function CreateCharacterPage() {
     const {
@@ -22,9 +20,9 @@ function CreateCharacterPage() {
     } = useForm()
     const sendForm = (data: any) => console.log(data)
 
-    const currentFaj = () => watch().faj || Faj.Ember
+    const currentFaj = () => watch(FAJ_FIELD_NAME, Faj.Ember)
 
-    const currentOsztaly = () => watch().osztaly || Osztaly.Harcos
+    const currentOsztaly = () => watch(OSZTALY_FIELD_NAME, Osztaly.Harcos)
 
     return (
         <div className='row'>
@@ -42,78 +40,26 @@ function CreateCharacterPage() {
                                defaultValue='Névtelen Kalandozó' {...register('nev', {required: true})} />
                         {errors.nev && <span className='form-field-error'>A karaktered nem mászkálhat névtelenül a világban!</span>}
                     </div>
-                    <FajSelector register={() => register('faj', {required: true})} />
-                    <div className='row m-2'>
-                        <label className='col-lg-1 col-sm-2 form-label'></label>
-                        <div className='col'>
-                            <p className='row pt-2'>
-                                {FajDescription(currentFaj())}
-                            </p>
-                            <ul>
-                                {FajSpecials(currentFaj()).map(((special, i) => (<li key={currentFaj()+i}>{special}</li>)))}
-                            </ul>
-                        </div>
-                    </div>
+                    <FajSelector
+                        register={() => register(FAJ_FIELD_NAME, {required: true})}
+                        currentFaj={currentFaj}
+                    />
                     <hr/>
-                    <Tulajdonsagok currentFaj={currentFaj} watch={watch} setValue={setValue} register={register} />
+                    <Tulajdonsagok
+                        currentFaj={currentFaj}
+                        watch={watch}
+                        setValue={setValue}
+                        register={register}
+                    />
                     <hr />
                     <div className='row'>
                         <h5 className='col-lg-2 col-sm-4 align-self-center'>Tanult</h5>
                     </div>
-                    <div className='row m-2'>
-                        <label className='col-lg-1 col-sm-2 col-form-label'>Osztály</label>
-                        <select className="col form-select" defaultValue={Osztaly.Harcos} {...register('osztaly', {required: true})}>
-                            <optgroup label='Harcos'>
-                                <option value={Osztaly.Harcos}>{OsztalyLabel(Osztaly.Harcos)}</option>
-                                {currentFaj() === Faj.Amazon && (
-                                    <option value={Osztaly.Amazon}>{OsztalyLabel(Osztaly.Amazon)}</option>) }
-                                <option value={Osztaly.Barbar}>{OsztalyLabel(Osztaly.Barbar)}</option>
-                                <option value={Osztaly.Ijasz}>{OsztalyLabel(Osztaly.Ijasz)}</option>
-                                <option value={Osztaly.Kaloz}>{OsztalyLabel(Osztaly.Kaloz)}</option>
-                            </optgroup>
-                            <option value={Osztaly.Pap}>{OsztalyLabel(Osztaly.Pap)}</option>
-                            {currentFaj() !== Faj.Amazon && (<option value={Osztaly.Tolvaj}>{OsztalyLabel(Osztaly.Tolvaj)}</option>)}
-                            {![Faj.Osember, Faj.Amazon].includes(currentFaj()) && (
-                                <optgroup label='Varázsló'>
-                                    <option value={Osztaly.Varazslo}>{OsztalyLabel(Osztaly.Varazslo)}</option>
-                                    <option value={Osztaly.Illuzionista}>{OsztalyLabel(Osztaly.Illuzionista)}</option>
-                                </optgroup>
-                            )}
-                        </select>
-                    </div>
-                    <div className='row m-2'>
-                        <label className='col-lg-1 col-sm-2 form-label'></label>
-                        <div className='col' >
-                            <p className='row pt-2'>
-                                {OsztalyDescription(currentOsztaly())}
-                            </p>
-                            <ul>
-                                {OsztalyProperties(currentOsztaly()).map(((special, i) => (<li key={currentOsztaly() + i}>{special}</li>)))}
-                            </ul>
-                            <p className='d-inline-flex gap-1'>
-                            {OsztalySpecialSkills(currentOsztaly()).map(skill => (
-                                <button
-                                    key={'btn' + skill.Id}
-                                    className='btn btn-outline-dark'
-                                    type='button'
-                                    data-bs-toggle="collapse"
-                                    data-bs-target={"#collapse-"+skill.Id}
-                                    aria-expanded="false"
-                                    aria-controls="collapseExample"
-                                >
-                                    {skill.Name}
-                                </button>
-                            ))}
-                            </p>
-                            {OsztalySpecialSkills(currentOsztaly()).map(skill => (
-                                <div key={'collapse'+skill.Id} className="collapse" id={'collapse-' + skill.Id}>
-                                    <div className="card card-body">
-                                        {skill.Description}
-                                    </div>
-                                </div>
-                            ))}
-                        </div>
-                    </div>
+                    <OsztalySelector
+                        currentFaj={currentFaj}
+                        currentOsztaly={currentOsztaly}
+                        register={() => register(OSZTALY_FIELD_NAME, {required: true})}
+                    />
 
                     <div className='d-grid gap-2'>
                         <button className='btn btn-danger btn-lg' type='submit'>Létrehozás</button>