0

Всем привет. Есть метод, в котором производятся различные проверки. И, хотелось бы сделать так, чтобы производилась проверка, и если она выдала какую-то ошибку, то метод прерывался и возвращал результат этой проверки. Я реализовал это через if, но у меня есть четкое ощущение, что есть какой-то паттерн для этого, а я его просто не знаю

checkGraphicsAndGetErrorMessage = (boundariesOptions?: BoundariesOptions): ErrorI18KeyAndOptions | undefined => {
  if (mapStore.selectedGraphic) {
    const res = this.checkGraphic(mapStore.selectedGraphic)
    if (res) {
      return res
    }
  } else {
    const res = this.checkGraphicsCollection(mapStore._graphicsLayer.graphics)
    if (res) {
      return res
    }
  }

  if (boundariesOptions) {
    return this.checkBoundaries(boundariesOptions)
  }
}
6
  • 1
    Ну почему вы так любите код картинками прикладывать?.. Его нельзя скопировать, читать неудобно. А переписывать с картинки его никто не будет.
    – rusgeli
    20 окт 2022 в 11:00
  • код нужно прикладывать кодом, а не картинкой
    – Grundy
    20 окт 2022 в 11:01
  • @rusgeli надо наверное на мете сделать вопрос и отсылать всех тех, кто картинки вставляет туда, чтобы оставили ответы. Потому что это какая-то тотальная жесть с ними последний год точно. Интересно понять почему люди так делают 20 окт 2022 в 11:08
  • @АлексейШиманский ru.meta.stackoverflow.com/q/12119
    – insolor
    20 окт 2022 в 13:52
  • @insolor не, этот вопрос говорит о том, "почему так не надо делать" и "как надо", а я говорю о том, чтобы эти самые люди писали ответы почему они так делают. Это немного другое. Да и тему не загрязнять 20 окт 2022 в 13:56

2 ответа 2

0
return (
         mapStore.selectedGraphic
         ? this.checkGraphic(mapStore.selectedGraphic)
         : this.checkGraphicsCollection(mapStore._graphicsLayer.graphics)
       )
       || boundariesOptions && this.checkBoundaries(boundariesOptions)
       || undefined // скорее всего, можно убрать
8
  • тогда уж все надо было логическими делать (mapStore.selectedGraphic && (this.checkGraphic(mapStore.selectedGraphic) || this.checkGraphicsCollection(mapStore._graphicsLayer.graphics))) || boundariesOptions && this.checkBoundaries(boundariesOptions)
    – Grundy
    20 окт 2022 в 13:35
  • @Grundy, во-первых, ты со скобками накосячил, во-вторых, даже если исправить, то твой код будет работать не так, как у автора.
    – Qwertiy
    20 окт 2022 в 13:43
  • скобки вроде поправил, теперь не отличается от твоего :) твой тоже не работает так как у автора :)
    – Grundy
    20 окт 2022 в 13:44
  • @Grundy, у тебя || внутри аргумента функции, так что не поправил. Мой работает как у автора если считать, что единственным ложным возвращаемым значением является undefined. Твой отличается и от авторского, и от моего тем, что вызывает не те проверки.
    – Qwertiy
    20 окт 2022 в 13:46
  • Поправил поправил :) А, все, понял в чем подвох :)
    – Grundy
    20 окт 2022 в 13:46
-1

И, хотелось бы сделать так, чтобы производилась проверка, и если она выдала какую-то ошибку, то метод прерывался и возвращал результат этой проверки.

const res = this.checkGraphic

esma js 2015 полагаю?


пожалуй да, есть для таких случаев кое что давно известное и вроде удобное

if(x==false){trow('x false');}
/*...*/
try{...}
catch(e){...}

https://developer.mozilla.org/ru/docs/Web/JavaScript/Reference/Statements/throw https://developer.mozilla.org/ru/docs/Web/JavaScript/Reference/Statements/try...catch

сленг конечно местами мерзкий в статьях об этом - эдакий рекурсивный канцелярит, но вроде оно того стоит, чтоб разобраться.

Конструктор Error создаёт объект ошибки. Экземпляры объекта Error выбрасываются при возникновении ошибок во время выполнения. Объект Error также может использоваться в качестве базового для пользовательских исключений. Смотрите ниже стандартные встроенные типы ошибок.

чтоб вы понимали, это вступительный абзац к error() ...)

5
  • Хах, спасибо) Думаю, потерплю обилие "объект Error" ради такого)
    – DimaEff
    20 окт 2022 в 14:43
  • Нафига использовать исключения там, где их не должно быть?
    – Qwertiy
    20 окт 2022 в 15:28
  • @Qwertiy серьёзно? не должно быть?) ты вообще хоть частично читал топик? цитирую и если она выдала какую-то ошибку, то метод прерывался и возвращал результат этой проверки вообще-то если верить MDN, это ровно то для чего они и предназначены. В - внимательность))) в конце концов, то на чьем ответе поставили галочку, говорит лучше всяких слов.
    – puffleeck
    20 окт 2022 в 20:28
  • @puffleeck, названия функций прочитай.
    – Qwertiy
    21 окт 2022 в 12:05
  • @Qwertiy, ну да, это самодельный обработчик ошибок судя по всему, только.... Я реализовал это через if, но у меня есть четкое ощущение, что есть какой-то паттерн для этого, а я его просто не знаю автор явно хотел либо заменить его целиком, либо хотя бы просто увидеть аналог, что я и показал. неужели это всё настолько не очевидно, что требуется объяснять? эм... отдохни-ка от строгой типизации, желательно склепав что-то на динамике. никаких "зачем" и "почему", просто сделай и всё.
    – puffleeck
    21 окт 2022 в 12:42

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.