È buona pratica abbreviare le funzioni? [duplicare]

Question

È buona pratica abbreviare le funzioni? [duplicare]

#1 da (5 voti)

0

È buona pratica abbreviare le funzioni? Ad esempio questo codice:

      using System;
      namespace Hello
      {
          class Program
          {
                public static void Main(String[] args)
                {
                     Write("Hello, World!");
                     Stagger();
                }

                public static void Write(string s)
                {
                    Console.WriteLine(s);
                }

                public static void Stagger()
                {
                     Console.ReadLine();
                }
           }
      }

Divento pigro e faccio cose del genere Ma è corretto?

c# programming-practices

posta XiKuuKy 13.02.2015 - 03:56

fonte

1 risposta

Leggi altre domande sui tag c# programming-practices

Progettazione della biblioteca: meglio lasciare una funzionalità potenzialmente distruttiva con un avvertimento o tagliarlo completamente? Posso creare una funzione con parametri variabili in C?

score 5 · Accepted Answer

Se lo fai per migliorare la leggibilità del codice, questa è una buona idea.

Se lo fai per digitare meno, probabilmente lo fai male.

Nel tuo esempio, i tuoi metodi non migliorano la leggibilità; Direi anche che aggiungono complessità e riducono la leggibilità:

Che cos'è Write ? Sta scrivendo qualcosa in un file? Un registro? Bene, dovresti leggere come il metodo è implementato (dato che non hai nemmeno commenti XML) per sapere qual è lo scopo del metodo.
Che cosa significa Stagger ? A meno che questo termine non sia usato molto nel dominio dell'applicazione, lo eviterei: Console.ReadLine è molto più esplicito.

Potrebbero aiutarti a digitare meno, se stai scrivendo le tue app in Blocco note. Se si utilizza Visual Studio, Intellisense renderà irrilevanti quei metodi. Senza contare il fatto che se conti usando lo stile C # ufficiale, dovresti scrivere Program.Write invece di solo Write , il che significa che il numero di tasti premuti durante l'utilizzo di Intellisense sarà esattamente lo stesso.