169 похожих чатов

Подскажите по стилю кода, плс Есть 2 функции: /** * @throws

...
*/
function fn1 () {
...
if (...) throw new Error()
}

/**
* @throws ... <здесь тоже самое что и в fn1>
*/
function fn2 () {
fn1()
}

// клиентский код
fn2()

Вопрос, стоит ли в fn2 ловить все исключения явно и прокидывать их дальше?

function fn2 () {
try {
fn1()
} catch (e) {
throw e
}
}

Или же стоит оставлять вот такой неявный проброс (как в первом варианте), ограничиваясь подписью жсдока?

4 ответов

5 просмотров

Есть assert функции

Alexey-Stepanov Автор вопроса
Dmitry Remezov
Есть assert функции

Ну заменить throw на assert, все равно функция выбрасывает исключение же, и его нужно ловить в условной fn2

Ловить чтобы дальше пробросить, ничего не сделав - это бессмысленный бойлерплейт, так делать не надо

Alexey-Stepanov Автор вопроса
Andrei Kurosh
Ловить чтобы дальше пробросить, ничего не сделав -...

Ну вдруг здесь работает правило "явное лучше неявного". Т.е. первый вариант лучше и подписи jsdoc хватит?

Похожие вопросы

Обсуждают сегодня

Всем привет, написал код ниже, но он выдает сегфолт, в чем причина? #include <stdio.h> #include <stdlib.h> #include <string.h> struct product { char *name; float price; };...
buzz базз
70
Всем доброго дня, ребят подскажите пожалуйста, если в курсе по ассемблеру используется MASM32, могу ли я использовать FASM? В чем явная разница и будет ли у меня все работать?
Botsman
17
Хотел бы спросить у знающих, правильную ли я выбрал книгу для начала изучения ассемблера Юрова В.И ? Или есть более лучшие книги для начала обучения?
Botsman
25
Книга Юрова В.И пойдёт для обучения?
Botsman
24
$params = [ 'formid' => 'feedbackForm', 'formTpl' => '@CODE: <form class="form-validate" data-id="ajax_form"> <fieldset class="margin-bottom-md"> ...
Pathologic
1
Люди добрые, помогите с идеями, потому что свои закончились. У клиента падает софтина в момент инициализации модуля OtlEventMonitor на RegisterWindowMessage('Gp/OtlTaskEvents/...
Михаил Усков
7
> Примечательно, что новый владелец удаляет из GitHub любые жалобы, указывающие на подозрительную активность или смену владельца, и, видимо, рассчитывает на то, что пользовате...
Alex Sherbakov
2
GridView fully ignored first parent(SizedBox), and take width from second parent(Container). How can I constrain GridView by first parent? Widget build(BuildContext context) {...
Hamster
1
Коллеги, добрый день. Есть такой вопрос: Есть модуль, который надо запустить через супервизор как дочерний процесс. Пока инстансов было нужно 8, всё было окей, но когда их ст...
Δημήτηρ
4
Hey there Which is the best Linux destro for developers (coding)? To my research on reddit, they said Linux mint is good for mid level spec and Ubuntu for high Lev hardwar...
Wiz 🪄
11
Карта сайта