Skip to content

Mastermind TDD esettanulmány feltöltése#94

Open
Szakszi04 wants to merge 9 commits intobmeaut:masterfrom
Szakszi04:master
Open

Mastermind TDD esettanulmány feltöltése#94
Szakszi04 wants to merge 9 commits intobmeaut:masterfrom
Szakszi04:master

Conversation

@Szakszi04
Copy link
Copy Markdown

No description provided.

@csorbakristof csorbakristof self-requested a review April 2, 2026 13:38
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.

Kicsit félreértetted a TDD működését (lásd kommentek), ez inkább test first megoldás lett. Ha még van esetleg valami tanulság, amit hozzátennél, az jól jönne, mert kicsit "megkértem rá, megoldotta, kész" jellege van a leírásnak, ami fejlesztési szempontból teljesen jó, de tanulni belőle kicsit nehezebb. De ha nem volt ilyen, akkor marad ez, az is OK.

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.

A screenshot alján ha lehet, ne legyen félig ott maradt sor. Vagy látszódjon teljesen, vagy vágd le teljesen.

Comment on lines +24 to +26



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.

Suggested change

Comment on lines +26 to +27


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.

Suggested change

* **Hibaágak:** Érvénytelen színek és rossz tipp-hossz kezelése.
* **Logikai ágak:** Fekete és fehér tüskék számolása, különös tekintettel a duplikált színekre.


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.

Suggested change

---
layout: post
title: Mastermind fejlesztés TDD módszerrel és AI segítségével
tags: ai python tdd mastermind logic
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.

Suggested change
tags: ai python tdd mastermind logic
tags: python testfirst mieset

A tagekről valóban nem beszéltünk, ezek számát igyekezzük alacsonyan tartani (mert különben túl sok olyan tag lesz, ami csak 1-2 snippetben szerepel. Pl. a mastermind valószínűleg ilyen lenne.

@@ -0,0 +1,43 @@
---
layout: post
title: Mastermind fejlesztés TDD módszerrel és AI segítségével
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.

Suggested change
title: Mastermind fejlesztés TDD módszerrel és AI segítségével
title: Mastermind fejlesztés Test First módszerrel és AI segítségével

Ami a leírásban van, az sajnos nem a teljes TDD, mert ott nem írhatnád meg előre az összes teszt esetet a production kód elkezdése nélkül. Pont azzal garantálja a jó lefedettséget, hogy csak kis lépésekben enged előre haladni. Így maradjunk a "Test First" megnevezésnél.

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.

Az elejére még egy tanulság összefoglaló fejezet jól jönne, meg kérlek nézd át, hogy az olvasónak miket kell tudnia, hogy megértse, amit írsz. Sok helyen feltételezed szerintem, hogy pl. a feladat leírást (a PDF-ben) és a CRD-ket elolvasta. Ha ezt feltételezed, akkor az elején írd oda, hogy olvassa el, anélkül ne menjen tovább.

# Pénztárgép Esettanulmány

## Áttekintés
Ez a projekt egy fokozatosan bővített, tesztvezérelt (TDD) pénztárgép-motor megvalósítása C# nyelven. A cél nem csak egy helyes árkalkulátor volt, hanem egy olyan, folyamatosan fejleszthető domain motor létrehozása, amelyben az új üzleti követelmények (CRD-k) kis lépésekben, regresszió nélkül bevezethetők.
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 ha bár a PDF-et feltöltötted, érdemes lenne hivatkozni rá, mivel ha csak ezt a leírást olvassa valaki, szerintem keveset ért belőle. Csomó helyen feltételezed, hogy az olvasó fejében is benne van az, ami a tiédben benne van a feladatról.

@csorbakristof
Copy link
Copy Markdown
Contributor

Azt jól látom, hogy egy ideig ebben a pull requestben volt a 1001_MastermindTdd esettanulmány is, de most már nincsen benne? Szerencsés külön pull requestet nyitni hozzájuk, mert most a MastermindTdd mintha nem lenne sehol... vagy rosszul látom?

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