La legge di Demorgan nel codice open source è un ostacolo al contributo?

1

Recentemente ho aggiunto questo codice a a un progetto Github su cui sto lavorando :

    if not initial_push=='y' and not initial_push=='yes':
        print('Aborting.')
        return False

Dato che questo è un repository pubblico che spero che le persone possano contribuire, sto considerando il refactoring quel bit come il codice leggermente più leggibile:

    if initial_push=='y' or initial_push=='yes':
        pass
    else:
        print('Aborting.')
        return False

Se il mio obiettivo è di mantenere il progetto open source accessibile al contributo esterno , dovrei utilizzare il secondo metodo più leggibile ma maldestro ? In realtà sono interessato alla risposta per il caso generale , non solo a questo specifico esempio.

    
posta dotancohen 23.02.2015 - 11:16
fonte

2 risposte

3

Invertire qui la logica, per mantenere lo spirito della legge di Demorgan:

Vorresti che i "programmatori" contribuissero al tuo progetto anche se hanno problemi con la logica elementare? È un vantaggio netto per te? O saranno tempo sprecato? Il tuo progetto ha abbastanza volontari per rivedere il loro codice?

    
risposta data 23.02.2015 - 11:51
fonte
2

Non penso che nessuno dei due sia davvero più complesso o più difficile dell'altro. Sceglierei quello senza il ramo pass non necessario, perché quello è un ramo non necessario.

Se consideri tutte le cose che impediscono alle persone di contribuire all'Open Source (ha bisogno di tempo, capacità di programmazione, motivazione, ottenere un account Github, dare un senso all'organizzazione di un intero programma, superare la paura di non essere abbastanza buono da contribuire, in realtà creando una buona richiesta di pull, eccetera), non posso immaginare che la scelta di come si scrive una dichiarazione if come questa farà mai la differenza tra qualcuno che contribuisce o meno.

Ma in generale, leggibile e maldestro di solito non vanno insieme . Quasi sempre, il modo in cui è più leggibile è anche il modo meno maldestro, e il modo goffo richiede più linee, più inutili cruft ed è quindi meno leggibile.

Quindi: scegli sempre la massima leggibilità

    
risposta data 23.02.2015 - 12:12
fonte

Leggi altre domande sui tag