TauCetiStation/TauCetiClassic

[Task] Рефакторинг телепортов

Open

#9831 aperta il 15 ago 2022

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)DM (433 fork)batch import
Good First IssueTask

Metriche repository

Star
 (153 star)
Metriche merge PR
 (Merge medio 19g 19h) (63 PR mergiate in 30 g)

Descrizione

В билде есть утилитарный do_teleport и вариации /datum/teleport, но не все телепорты в игре используют эти инструменты. В результате, у нас куча независимых методов, каждый из которых во многом делает то же, что и другие, повторяя код, и иногда лучше или хуже.

Нужно убрать копипасту и свести всё к do_teleport, по необходимости дописать функционал. Стоит еще смотреть на код с /tg/, как пример.

Многие телепорты в билде сейчас можно найти по COMSIG_ATOM_INTERCEPT_TELEPORT.

Guida contributor

Direzione di ricerca
Le funzioni esistenti do teleport e /datum/teleport possono essere trovate nella codebase. Il segnale COMSIG ATOM INTERCEPT TELEPORT viene utilizzato per intercettare i teletrasporti; cercandolo si rivelerà il codice pertinente. La refactoring dovrebbe identificare tutte le funzioni relative al teletrasporto e assicurarsi che utilizzino do teleport, eventualmente estendendolo. Si consiglia di controllare la codebase di /tg/ per implementazioni simili.
Tech stack
Nessuno
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70