Skip to content

1004 wordle#96

Open
varhegyimeli wants to merge 3 commits intobmeaut:gh-pagesfrom
varhegyimeli:1004_Wordle
Open

1004 wordle#96
varhegyimeli wants to merge 3 commits intobmeaut:gh-pagesfrom
varhegyimeli:1004_Wordle

Conversation

@varhegyimeli
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown
Contributor

@csorbakristof csorbakristof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nagyon szép és tanulságos snippet lesz ez. Pár kiegészítést, kommentárt írtam bele.

Comment thread snippets/1004_Wordle/index.md Outdated
áthelyezése után), az összes korábbi kontextus elveszik. Érdemes a kontextus promptot
külön fájlban tárolni és újra beadni.
- **Claude és Copilot jól kiegészítik egymást** — Claude erős volt a tervezésben,
architektúrában és hibakeresésben, a Copilot pedig a kód generálásban.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ezt nem teljesen értem... a Claude Haiku volt erős a tervezésben? A "Copilot" alatt most a Copilot alatt használt Sonnet 4.6-ot értjük?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Claude Sonnet 4.6-ot desktop alkalmazásként használom, sima Claude-ként rá referálok, ő volt erős a tervezésben. A Copilot alatt van a Haiku 4.5 (kicsit butább, mint a Sonnet), amit Visual Studioban használok kód generálásra. Sajnos a GitHub Education csomagból kivették a Sonnet modelleket, így Copilotnak a Haiku maradt.
Javítom a snippetben, hogy egyértelműbb legyen.

**Projekt hivatkozások:**
- `WordleMAUI` → `WordleMAUI.ViewModels` → `WordleMAUI.Core`
- `WordleMAUI.Tests` → `WordleMAUI.ViewModels` és `WordleMAUI.Core`
- `WordleMAUI.Tests` nem hivatkozik `WordleMAUI`-ra (WinRT inicializálási hiba miatt)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Itt érdemes megemlíteni, hogy ennek részleteiről később is lesz szó.

Comment thread snippets/1004_Wordle/index.md Outdated
|---|---|
| `LetterState` | enum: Correct, Misplaced, Absent |
| `GuessResult` | record: char Letter, LetterState State |
| `WordValidator` | 5 betűs, csak alfa, szótárban szereplő szavak ellenőrzése |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Itt mit jelent a "csak alfa"? Csak betűk lehetnek a szavakban?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Igen, van arra egy ellenőrzés, ha a string valamelyik karaktere nem betű, akkor return false. Az alkalmazás jelenlegi állása szerint csak a játék alján kirajzolt billentyűzettel lehet szavakat beírni, amin csak betűk szerepelnek, de ha később fizikai billentyűzetet is támogatnánk, akkor már más karakter is bejöhetne.
Viszont jobban belegondolva a szótári ellenőrzés ezt ugyanúgy megfogja és jelenleg nincs külön hibaüzenettel kezelve, úgy mint az 5 betűnél rövidebb szó, szóval ezt a "minden karakter betű-e" ellenőrzést el is lehetne hagyni.
A snippet szövegéből a "csak alfa" részt kiveszem, mert igazából nem lényeges.


## A feladat leírása

A feladat egy Wordle szójáték klón elkészítése volt .NET MAUI keretrendszerben, C# nyelven,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ide érdemes lenne egyetlen mondatban leírni, hogy mi a Wordle, hátha valaki nem tudja pontosan.

Comment thread snippets/1004_Wordle/index.md Outdated
Comment on lines +256 to +258
| `SelectByDate` eltávolítása után | 80.43% | +6.43% |
| `GameStateManager` tesztek után | 89.13% | +8.70% |
| `WordSelector` tesztek után | 93.48% | +4.35% |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ez így egy kicsit lóg a levegőben mert nem tudni, hogy mit is takarnak ezek az "iterációk". Főleg amit törölni kellett, azt az olvasó biztosan nem tudja visszakeresni még akkor sem, ha mellékeled a forráskódot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants