Od czasu do czasu dostaję błędy TSLint "blok jest pusty". Tak się dzieje np. kiedy mijam zwrotnego no-op do funkcji:Dlaczego TSLint i JSLint zgłaszają puste bloki?

doSomething(() => {});

Z tego co czytałem, JSLint najwyraźniej robi to samo, ale nie sprawdzać.

Uważam, że te zwyczaje są całkowicie poprawne, więc próbowałem znaleźć powód, dla którego puste bloki są uważane za złe. Ale jedyną rzeczą, którą mogę znaleźć (np. W this answer) są instrukcje, aby dodać return;, aby uniknąć błędu. To jest nie co chcę zrobić w każdym pustym wywołaniu zwrotnym.

Dlaczego raport TSLint powyżej pustego bloku jest problemem? Czy jest jakikolwiek powód, dla którego nie powinienem wyłączyć kontroli?

Źródło

2015-07-24 theDmi

Nigdy nie używał; po prostu myśląc głośno: czy to możliwe, że czasy, na które skarży się TSLint, kiedy myśli, że funkcja _nie może_zwrócić wartości, a twoja funkcja "no-op" tego nie robi? Możesz zdefiniować jawną funkcję "no-op" i po prostu przekazać jej nazwę w takim wywołaniu. – TripeHound

@ TripeHound Nie, TSLint narzeka nawet wtedy, gdy określam jawny typ '(() => void) dla wywołania zwrotnego. Odnośnie noopa: właśnie dowiedziałem się, że lody już definiują jedną: '_.noop'.To jest jak dotąd najczystsze rozwiązanie ... – theDmi

Możesz również zawinąć '{} w nawiasach, np. '() => ({})' nie powoduje ostrzeżenia "pustego bloku" z TSLint – danwellman

Dlaczego raport TSLint nad pustym bloku jako problemu

Aby zapobiec błędy. Być może funkcja została zapomniana. Poleć () => undefined jako odbiór.

Więcej

Jeśli chcesz wyłączyć to po prostu dodać "no-empty": false, aby Twój tslint.json (wyłączyć globalnie) lub wyłączyć go inline za pomocą /* tslint:disable:no-empty */ komentarz.

Źródło

2015-07-24 09:26:48 basarat

Chyba nie chcę, aby TSLint powiedział mi, że przegapiłem wypełnianie funkcji, to jest to, do czego mam testy. Więc jeśli naprawdę nie ma nic więcej, dla czego ten test jest przydatny, wolę go wyłączyć niż używając '() => undefined' w każdym pojedynczym wywołaniu zwrotnym. Ale przypuszczam, że to kwestia gustu. – theDmi

Aby wyjaśnić, czy można dokładnie określić, co rozumie się przez "błędy"? Czy to po prostu puste ciało funkcyjne? – theDmi

Podobnie jak powiedział basarat, błędem może być, że zapomniałeś zaimplementować coś, choć zwykle każda funkcja, którą chcesz wdrożyć, powinna kiedyś użyć polecenia "NotImplemented". – MikeSW

Podobnie jak w przypadku wszystkich kontroli, masz ostateczną ocenę, czy ci pomagają, czy nie. Możesz wyłączyć ten test TSLint, korzystając z jednej z poniższych opcji.

Wyłącz regułę w tslint.json

//... 
"no-empty": false, 
//...

Wyłącz regułę w pliku:

/* tslint:disable:no-empty */

Zawsze można przełączyć go ponownie, jeśli kiedyś w przyszłości znaleźć pusty blok to spowodowało problem.

Źródło

2015-07-25 20:07:37 Fenton

To powinna być zaakceptowana odpowiedź. – sss4r

Jeśli czujesz, że nie chcesz korzystać zwrotnego podczas niektórych scenariuszach można zmodyfikować kod

doSomething(() => {});

doSomething(() => void);

stosując() => {}, aby to oznacza, że nie zależy ci na tym oddzwanianiu. i wyraźne typowanie będzie unikać implikacji.

Powodzenia.

Źródło

2018-02-26 11:29:49 spacedev

Dlaczego TSLint i JSLint zgłaszają puste bloki?

Odpowiedz

Więcej

Powiązane problemy