Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

getter de current tileset #37

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

getter de current tileset #37

wants to merge 1 commit into from

Conversation

Jato30
Copy link
Collaborator

@Jato30 Jato30 commented Jan 6, 2018

No description provided.

@Anders1232
Copy link
Owner

Anders1232 commented Jan 6, 2018

Hmm, deve dar erro de compilação, o atrbuto é um vetor de ponteiros para TileSet,e não se pode converter TileSet* para TileSet&.

Pergunta, qual o contexto em que é necessário? Pois por mais que se que o tileMap contenha esse vetor, ele não manipula os TileSets deiretamente. Todos os tileSets que ele contém foram criados externamente e passados para ele no construtor.

Meu questionamento, é que provavelmente essa informação consegue ser obtida no momento em que o vetor de tilesets foi isntanciado. Quem precisa dessa também é instanciado lá?

De qualquer forma, um método que percebo ser necessário no TileMap é void SetCurrentTileSet(int index). Creio que é uma boa oportunidade de fazer isso. Concorda, @bestknighter(ou criados outra branch/PR)?

@Jato30
Copy link
Collaborator Author

Jato30 commented Jan 22, 2018

a AI GoTo precisa

@Anders1232
Copy link
Owner

Por que motivos uma IA precisa de acesso ao TileSet?

@bestknighter
Copy link
Collaborator

Hmmm, talvez eu precise de uma breve revisão do que é o TileSet, TileMap e o Index nesse contexto. Pq realmente, não vejo nenhum motivo do pq qualquer um queria acesso ao TileSet se não o TileMap e quem vai renderizar (que acredito ser o próprio TileMap).
O que a AI GoTo tá querendo fazer que precisa de acesso ao TileSet?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants