Как не возвращать данные из конструктора если их нет?

Question

[kawiyibit]

Надо строить объект по схеме, исходя из вводных данных. Проблема в том, что иногда данные не полностью соответсвуют схеме и в этом случае надо не возращать это поле вообще.

class Article {
  constructor(data) {
    this.author = data.author;
    this.description = data.description
  }
}
const authorOne = new Article({
  author: 'Vasya'
});

В этом случае создастся объект

{
  author: 'Vasya',
  description: undefined
}

А как вернуть только заполненные поля? Класс это самый правильный способ в данном случае иил есть альтернативы?

Comments

[hzzzzl]

if (data.description) { 
  this.description = data.description 
}

[kawiyibit]

hzzzzl, спасибо, такого не надо

Answer 1

[Антон Швец]

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

function objectMaker(schema, src) {
  return Object.keys(schema).reduce((acc, el) => {
      if (src.hasOwnProperty(el)) {
         acc[el] = src[el]
      }
      return acc;
  }, {})
}

const mySchema = {
  author: 'string',
  description: 'string',
}
const source = {
  author: 'Vasya',
  garbage: 'dfsfsdf',
}

const authorOne = objectMaker(mySchema, source);

Домашнее задание
joi
class-transformer

Comments

[kawiyibit]

Огромная благодарность

Answer 2

[tvsjke]

class Article {
  constructor(data) {
  	Object.keys(data).forEach(key => {
    	this[key] = data[key];
    })
  }
}

Comments

[kawiyibit]

Не все возвращают просто значение, на некоторых есть дополнительная обработка. Типо

this.tags = data.tags
      ? data.tags.filter(Boolean).map(tag => 'hello ' + tag)

[Сергей Егоров]

kawiyibit, конструктор не место для таких экзерсисов... он просто для инициализации свойств объекта значениями. И чтобы не вводить в заблуждение пользователей, он должен принимать отдельные аргументы, для каждого из своих свойств:

class Article {
  constructor(author, description) {
    this.author = author;
    this.description = description;
  }
}

[kawiyibit]

Сергей Егоров, скажите это ребятам которые делают Mongoose

[Сергей Егоров]

kawiyibit, а в какой конструктор там передается 100 параметров?

[kawiyibit]

Сергей Егоров, в схему https://mongoosejs.com/docs/guide.html

[Сергей Егоров]

kawiyibit, так это ж полувырожденный вариант шаблона builder, что подтверждается присутствием метода add.

Ну и сто параметров там не подразумевается, ибо мы инстанциируем схему для одной единственной коллекции объектов.

[Сергей Егоров]

kawiyibit, исходники кода метода Schema я конечно не читал, но на вид это выглядит как проверка правил описания метаданных модели, перед тем, как пользователь начнет модели эти в коллекцию складывать. То есть это не есть тупое клонирование того, что пришло в аргумент конструктора.

Если мы хотим просто создавать экземпляры объектов с разными наборами свойств, то в Javascript нам вообще не нужна функция конструктор, мы можем пользоваться чистыми литералами объектов:

const man = {
  firstName: 'Jack',
  lastName: 'Sparrow',
  gender: 'Male'
};

const identity = { 
  number: '92839872498',
  issued: '2019-01-04',
  origin: 'us',
  expires: '2029-01-04'
};

[kawiyibit]

Сергей Егоров, понятно что можем))) Но мне нужна заранее определенная схема - какие поля сохранять в бд а какие нет.

[kawiyibit]

Сергей Егоров,

Ну и сто параметров там не подразумевается, ибо мы инстанциируем схему для одной единственной коллекции объектов.

ты какой-то странный. конечно сто параметров не подразумевается - это ты мне предлагаешь каждое поле пихать отдельынм параметром))) у меян ОДИН аргумент передается, и из него уже строится объект. в монгузе тоже - передается объект, он определяет модель/схему. а ты предлагаешь безумство - сто параметров передавать отдельно.

Answer 3

[Сергей Егоров]

class Article {
  constructor(data) {
    this.author = data.author;
    if(data.description){
        this.description = data.description;
    }
  }
}

Comments

[kawiyibit]

там 100 значений

[Сергей Егоров]

kawiyibit, 100 аргументов передается в конструктор? Это отвратительный дизайн. Не боитесь что кто-нибудь когда-нибудь будет читать ваш код?

[kawiyibit]

Сергей Егоров, я всего лишь примитивно повторяю то что делает ODM, типо монгуза. Они не боятся что кто-то увидит.

[Сергей Егоров]

kawiyibit, не вопрос, повторяйте...

Answer 4

[Игорь]

class Article {
  constructor(data) {
    for (let key in data) {
      this[key] = data[key]
    }
  }
}

const authorOne = new Article({
  author: 'Vasya'
  
});

console.log(authorOne)