[C-Builder] Proiect Sah - Incarcare imagine din fisier (OOP)

Enumul e un intreg, n-ai nevoie de if/if/if.

 andreim77, on 23 decembrie 2016 - 17:46, said:

1. Nu, pentru renderingul imaginii ai avea un helper in view layer care accepta ChessPieceType ca parametru, si returneaza un string ce poate fi folosit in crearea caii catre imagine.

2. Sau view helper ia intreaga Piece, care are si pointer catre Player, si returneaza calea.

Oricare ar fi, 1 sau 2, e oarecum un detaliu pana la urma urmei.

Nu ai if in if cu oricare dintre abordari in acest view helper, ci un switch.

dar tot il testezi, nu?
if(piesa==turn) {
if(output==jpg) desenazaTurn()
else if (output==wav) playTurn()
...
} else if(piesa=pion) {
....
}

switch, if, tot niste ramuri care trebuie actualizate pt orice tip nou de output.

Edited by andreim77, 23 December 2016 - 17:55.

 andreim77, on 23 decembrie 2016 - 17:54, said:

Ok, imi voi rafina raspunsul anterior:

prin faptul ca ai diferite interfete nu ma refer la ingramadirea diferitelor interfete intr-o singura componenta, ci la posibilitatea (flexibilitatea) de a avea diferite componente (numeste-le plug-ins, namespaces, packages, whatever fits you), fiecare cu specificul sau, fara ca, in libChessGame, care e generic, sa fortezi o anumita structura.

Daca ai returna un string din libChessGame, ai avea o functie care "atarna in aer" dpv logistic in acele interfete in care nu e vorba despre niciun fisier - ci de asset-uri in RAM sau cine mai stie ce.

Tocmai asta e ideea: ca nu stii, si nu ar trebui sa-ti pese, ce fel de interfete vor folosi biblioteca ta generica libChessGame.

Quote

Deci nu, nu il testezi, in interfata pe care o va construi in C++ Builder, el stie cu certitudine ca va desena imagini.

Dar poate peste un an va face o interfata in OpenGL. Un alt proiect, complet nou, diferit de interfata scrisa in C++ Builder.

Edited by OriginalCopy, 23 December 2016 - 18:02.

Cum e mai simplu de facut? Sa fac o variabila id si sa am cate un id = ? la fiecare piesa? Sau sa pun cu #define ? (#define Pawn 1, #define Rook 2, #define white/black)

Sa pui un ... enum?

 enterspeed, on 23 decembrie 2016 - 18:02, said:

Defineste problema pe care incerci sa o rezolvi.

Diagrama lui dani.user iti spune practic ce ai de facut, nu vad loc de niciun id deocamdata.

 OriginalCopy, on 23 decembrie 2016 - 18:05, said:

In ChessPieceType alea le scriu cu enum?

 enterspeed, on 23 decembrie 2016 - 18:09, said:

ChessPieceType este un enum, da, asta inseamna E-ul in cercul rosu din diagrama.

as vrea sa vad interfata care asigura "afisarea" pieselor catre jucator (jpg, wav, etc) cu acele enum la treaba. care cumva duplica ierarhia pieselor.

In proiectul C++ Builder, ar avea ceva de genul:

imPiesa->Picture->LoadFromFile(build_image_path_for_piece(piece));

In alt proiect pentru dizabilitati ar avea:

play_sound(build_sound_path_for_piece(piece));

Sunt proiecte complet diferite, disjuncte, nu ai if-uri in functie de tipul interfetei, pentru ca interfata insasi stie deja ce fel de interfata este.

build_image_path_for_piece sau build_sound_path_for_piece sunt helperele despre care vorbeam. In ele ai switch in functie de acel ChessPieceType. Nu if in if.

 andreim77, on 23 decembrie 2016 - 18:13, said:

Nu duplica nimic, din contra, separa concerns: logica jocului (libChessGame) de vizualizare.

 andreim77, on 23 decembrie 2016 - 17:54, said:

switch-ul ar trebui actualizat pentru noi tipuri de piese de sah, nu pentru tipurile de interfata.

Interesting fact: in Rust, programul nici nu ar compila intr-o astfel de situatie, daca scrii codul curat, pentru ca match necesita matching exhaustiv: https://doc.rust-lan...ook/match.html.

Edited by OriginalCopy, 23 December 2016 - 18:26.

si in toate build alea o sa ai un switch sa testezi piesa. ok. noroc ca tipul de piese in jocul asta e fix. altfel...

bine, mai filozofam zilele urmatoare.

 andreim77, on 23 decembrie 2016 - 18:27, said:

Normal ca ai switch in fiecare, pentru ca fiecare interfata are specificul ei.

Sunt multe lucruri pe marginea carora am putea filozofa, dar acesta nu este unul dintre ele.

Vorbim despre separation of concerns, despre "model vs view", sau cum vrei tu sa le botezi.

Sau sa punem altfel problema:

daca ai un bug in afisare, unde cauti mai degraba cauza, in view layer (proiectul C++ Builder) sau in biblioteca generica libChessGame?

De ce sa testezi piesa?

std::string imagePath = std::string("chess_piece-") + std::to_string(static_cast<int>(piece->getType())) + std::to_string(static_cast<int>(piece->getPlayer->getColor())) + std::string(".jpg");

Edited by dani.user, 23 December 2016 - 18:32.

enum ChessPieceType { Pawn, Knight, Bishop, Rook, Queen, King }; -> se pun in .h sau .cpp?

Edit: nvm, se pun in .h

Edited by enterspeed, 23 December 2016 - 18:57.

In .h, e ceva de interes larg, nu doar local.

 enterspeed, on 23 decembrie 2016 - 18:55, said:

Tare am senzatia ca tu nu faci TDD.

 OriginalCopy, on 23 decembrie 2016 - 18:59, said:

Nu fac ca nu-mi ajunge timpul.

Cum scriu getName() : String ?

Edited by enterspeed, 23 December 2016 - 19:31.