Domanda

Prima di tutto, questa non è davvero una domanda di programmazione. È più una questione di "come posso soddisfare le convenzioni di denominazione di McConnels?" & Quot;

Ho questa applicazione Delphi che manipola documenti di parole. Una delle cose che devo fare è scorrere tutti i segnalibri nel documento ed eliminarne alcuni in base a una semplice regola: se attualmente lavoro su un preventivo eliminerò tutti i segnalibri in cui il nome inizia con "cw_orderspecific". Se lavoro su una conferma d'ordine, rimuoverò tutti i segnalibri con nomi che iniziano con " cw_quotespecific " ;.

Il metodo è attivo e tutto funziona perfettamente, ma ho un piccolo problema. Come dovrei chiamare il mio metodo? Il nome corrente (" DeleteBookmarksNotAllowedForCurrentDocumentType ") è troppo lungo.

Qualche suggerimento?

È stato utile?

Soluzione

DeleteNonRelevantBookmarks?

Altri suggerimenti

Cambia la funzione per prendere il prefisso da eliminare e chiamalo DeleteBookmarksWithPrefix :

DeleteBookmarksWithPrefix("cw_quotespecific")

o

DeleteBookmarksWithPrefix(otherWorkTypePrefix)

Ciò rende il codice molto leggibile e rende la funzione riutilizzabile in altre circostanze, anziché essere sempre utile solo per un'attività molto specifica.

Nota che se avrai mai un terzo tipo di lavoro, potresti voler invertire il senso di esso:

DeleteBookmarksWithoutPrefix(currentWorkTypePrefix)

Quanto corto vuoi che sia? Il mio primo istinto è di abbandonare alcune parole che non sembrano così importanti. Come DeleteBookmarksNotAllowedForDocumentType, DeleteBookmarksNotAllowedForDocument o semplicemente DeleteBookmarksNotAllowed. Anche se probabilmente preferirei qualcosa come DeleteInvalidBookmarks. Naturalmente questo funziona solo se non ci sono altri metodi che eliminano i segnalibri secondo regole diverse.
Oppure, se sei disposto a cambiare il modo in cui il tuo metodo funziona, lo renderei più generico fornendo il prefisso come parametro, quindi la denominazione sarebbe più semplice.

Usa gli acronimi? Definisci nei commenti che CDT è " Tipo di documento corrente " ;, e otterrai DeleteBookmarks_NotAllowedForCDT . Abbastanza corto.

Che ne dici di DelBookmarksStartingWith ? È abbastanza chiaro ed esprime l'intento.

Autorizzato sotto: CC-BY-SA insieme a attribuzione
Non affiliato a StackOverflow
scroll top