Muito massa mano! Eu dei uma olhada no source, e se tu me permite eu queria dar umas sugestões:
- Evite trocar os objetos originais do jogo por novos, por exemplo, esse objeto que você usou pro vendedor não é usado no jogo original, mas caso a pessoa use outros mods que usem ele, vai bugar. Eu por exemplo tive esse problema aqui com o mod de praia realista:
O melhor seria você carregar o personagem fazendo uma pose e posicionar ele mesmo, assim não só o personagem seria alterado junto com os mods da pessoa (caso ela use mods de personagens em HD ou um colete customizado), como você também pode fazer umas coisas legais tipo permitir que o jogador mate ele ou fazer ele mexer a boca quando usar as falas do mod
- Teve vários casos, como o de selecionar a posição do dia atual, onde seria melhor ter usado SWITCH ao invés de IF. Isso além de todo o código ser meio confuso, eu recomendo que você releia o tutorial de script e se não chegou a terminar ele, tente terminar também. Principalmente
essa parte, é muito importante isso não só pra você não se perder como também pras pessoas que quiserem te ajudar a corrigir algo não se perderem também
- Você fez o carro sempre ser criado, o que não é muito legal em questão de otimização, o mais ideal seria que você definisse as coords de acordo com o dia e trabalhasse em cima delas, e criasse ele só quando a camera estivesse perto, uns 250m de distância por exemplo (talvez até menos). Detalhe também que você só muda a posição dele conforme o dia da semana, mas e se a semana mudar enquanto o jogador estiver na frente dele? Ou pior, e se ele estiver comprando? São grandes as chances de dar um soft lock. Verificando se o carro tá a essa distância da câmera, você pode fazer com que ele continue lá enquanto o jogador estiver perto e só troque caso se distancie, talvez até fazer com que o carro troque de posição se não estiver dentro da câmera mesmo (mas não fique criando e apagando ele caso estiver fora porque vai pesar muito quando a pessoa for mexer a câmera)
- Você deixou pra criar o carro primeiro e usar a posição dele como referência pra todo o funcionamento do mod. É errado? De certa forma não, mas seria muito mais inteligente você usar variaveis pra definir a posição de funcionamento e trabalhar em cima delas, spawnando o carro e o restante das mecanicas nela, ao inves de ter q ficar pegando a posição do carro sempre
- Você usou IS_VEHICLE_ON_ALL_WHEELS pra checar se o carro tá no chão e deixou num loop só pegando as coordenadas e fazendo nada além se não estivesse, o mais correto seria sim verificar porém depois usar GET_GROUND_Z_FOR_3D_COORD pra redefinir a posição do carro caso não estivesse
- Isso é uma tarefa secundária pro player fazer no mapa, não tem porque usar LOAD_ALL_MODELS_NOW, é melhor só chamar o modelo e esperar que ele seja carregado pra não pesar no jogo da pessoa
- Não vi muita utilidade pra van não poder ser danificada - pô, a gente tá falando de GTA, sempre que der pra fazer algo ser destrutível ou morto, é bom fazer com que seja, mesmo que dê mais trabalho - ainda mais sendo que você já fez um sisteminha que tem até voz personalizada pra reconhecer se você tá atirando nela ou não
- Acho que é bom definir a van como veículo de missão também, pra outros mods não interferirem no funcionamento dela
- Você verificou se o jogador tá atirando na van mas colocou num loop enquanto ele atira e só botou as consequências depois que para de atirar, então em teoria se eu entrasse e saísse de cena atirando não aconteceria nada (mas não testei)
- Seria interessante colocar mais checagens, como por exemplo se o carro não tá em chamas ou se o player não atirou no vendedor, tanto antes quanto durante a compra, e interromper a compra e fechar o menu caso essas coisas aconteçam
- Você esqueceu de fazer os sons seguirem a configuração do menu, se uma pessoa jogar com o som desligado pelo menu ela ainda vai ouvir o personagem falando
- Não achei muito útil ter que apertar Y pra entrar na loja visto que tem uma animação pra abrir a porta, então dá pra passar correndo pela frente do carro bem de boa
- Sempre tente usar o máximo possível dos textos originais do jogo, assim qualquer pessoa pode jogar com a tradução que ela tenha instalada. Vc pode pegar os textos originais do jogo
aqui, só dar um ctrl + f. também é interessante usar o
weapon names do junior_djjr, pra pessoa sempre ter os mesmos nomes de armas pra todos os mods (mas vai ter que incluir esse no download, lembra de colocar um recado pra pessoa não substituir caso já tenha)
- Também podia ter colocado duas colunas no menu e ter formatado pra ficar igual nos menus da Ammu-Nation, e não formatar o menu com cores pra seguir o padrão do jogo
- Tinha umas leituras de um .ini que aqui pelo menos não chegou no download
- Teve uma parte também que você adicionou comandos de menu manualmente sendo que você já tinha adicionado menus, mas vocÊ deixou lá no código fazendo nada, ficou bem poluído
- Ao invés de ficar definindo os ids das armas e dos modelos delas toda vez, prefira simplesmente definir/guardar o WeaponType numa variavel e chamar os modelos dela com GET_WEAPONINFO e GET_WEAPONINFO_MODELS, deixando esse processo num gosub e só chamando quando trocar o id da arma. Isso vai organizar muito melhor o seu código e vai deixar muito mais fácil de adaptar pra armas adicionadas sem substituir, além de melhorar a compatibilidade com armas adicionadas pelo AutoID
- Uma sugestão besta, mas seria legal também fazer com que a porta do lado do vendedor abrisse/fechasse antes da outra, pra ficar mais natural, e também utilizar os easings da CLEO+ pra animação não ficar tão robótica.
Curti muito seu mod e apesar de que algumas coisas q mencionei ali quebram o funcionamento dele, a maioria são só críticas construtivas. Eu cheguei a recriar parte do código mas não terminei ainda, se quiser ver como tá o progresso ou quiser ajuda só me chamar no discord @jovemblvkfriday
