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.

Sign up for GitHub

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

Jump to bottom

Apenas um esclarecimento #34

Open
alexsmonte opened this issue Feb 1, 2020 · 5 comments
Open

Apenas um esclarecimento #34

alexsmonte opened this issue Feb 1, 2020 · 5 comments

Comments

@alexsmonte
Copy link

Gostaria de entender o pq usar ele fora do modulo?

eu percebi que ele nao é usado em lugar nenhuma alem desse momento, é só uma duvida :D

meu-primeiro-jogo-multiplayer/playground/1st-release/public/index.html

Line 36 in 5b036fc

const screen = document.getElementById('screen')

@JarnDev
Copy link

JarnDev commented Feb 2, 2020 •
edited

Acredito que da forma que foi criado, o código fica mais modular, o arquivo render-screen.js, renderiza em qualquer screen enviada, caso essa linha fosse colocada dentro do render-screen.js seria um hardcoded e o codigo só renderizaria em um elemento que tivesse um ID 'screen'.

<body>
        <canvas id="gameScreen" width="10" height="10"></canvas>

Caso no seu código seu index.html tenha um canvas com outro id como acima, seria necessario apenas enviar esse elemento para o 'renderScreen'


socket.on('connect', () => {
                const playerId = socket.id
                console.log(`Player connected on Client with id: ${playerId}`)

                const screen = document.getElementById('gameScreen')  << apenas seria necessario alterar aqui
                renderScreen(screen, game, requestAnimationFrame, playerId)
            })

isolando completamente o módulo responsável pela renderização.

@alexsmonte
Copy link
Author

O meu pensamento em passar aquele trecho para dentro era simplesmente eu poder fazer isso

renderScreen('gameScreen', game, requestAnimationFrame, playerId)

creio que o resultado seria o mesmo e de fato isolando a renderização

@JarnDev
Copy link

JarnDev commented Feb 2, 2020

Da sua forma ficaria até um pouco mais conciso.

renderScreen('id_do_canvas', game, requestAnimationFrame, playerId)

e antes dessa linha

meu-primeiro-jogo-multiplayer/playground/1st-release/public/render-screen.js

Line 2 in 5b036fc

const context = screen.getContext('2d')

colocaria : const screen = document.getElementById(screen)

confere?

@alexsmonte
Copy link
Author

Eh q tow criando um jogo de xadrez baseado na ideia do companheiro, entao eu so queria entender mesmo se tem alguma diferença criar dentro ou fora :D

@JarnDev
Copy link

JarnDev commented Feb 2, 2020

Acredito que tenha sido só para isolamento mesmo, só ele pode confirmar kkk. mas pelo que vimos pode ser isolado colocando essa linha dentro do modulo daquela forma.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@alexsmonte @JarnDev