Skip to content

Improved shooter design#30

Open
mariorodriguespt wants to merge 4 commits into
masterfrom
shooter-design-enhancement
Open

Improved shooter design#30
mariorodriguespt wants to merge 4 commits into
masterfrom
shooter-design-enhancement

Conversation

@mariorodriguespt

Copy link
Copy Markdown

changes

  1. Minor improvement on the shooter design
  2. Minor code cleanup

Screenshot 2020-03-01 at 07 11 34

@mariorodriguespt mariorodriguespt requested a review from RuiOrey March 1, 2020 07:14

@RuiOrey RuiOrey left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Fora o comentario que fiz sobre o uso do THREE.Shape, ja que estas na mao na massa, eu acho que isto o PlayerShooter devia-se decompor em mais GameObject ou ter mais GameComponent.
Isto porque ha energia que desaparece. Os pares de geometrias esquerdas e direitas (a parte junto a bola central e o mini cubo) deviam ser dois prefabs LeftShooterEnergy e RightShooterEnergy que deveriam ser referenciados num GameComponent novo do shooter chamado Energy ou algo do genero, que pode ser parameterizado com nivel de energia e reutilizado para inimigos tb.
Ou seja, é necessario e ainda não existe um GameComponent para a energia, que podia ter como parametros opcionais os nomes dos prefabs de cada barra de energia. Mas depois se quiseres explico melhor isto no chat.


//leftBackEnergy

path = new THREE.Shape();

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

tanto neste como no direito podes usar o THREE.BoxGeometry https://threejs.org/docs/#api/en/geometries/BoxGeometry, esta cena do Shape é mais para cenas que nao sao primitivas geometricas

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