Не могу перевести число в десятичную систему без чисел после запятой используя Math.round()?

Question

[OleksandrPrante]

class menuCard{
		constructor(price, parentSelector){
			this.price = price;
			this.parent = document.querySelector(parentSelector);
			this.transfer = 42;
			this.changeToUAH();
		}
		changeToUAH() {
			 Math.round(this.price = +this.price * this.transfer)
		}
	}

	new menuCard(5, '.menu .container');

Comments

[Алексей Ярков]

Как насчёт того, чтобы нормально оформить код? И зачем дублировать вопросы?

[VoidVolker]

OleksandrPrante https://qna.habr.com/help/rules#3.8

[Adamos]

Оформление не поможет, ТС не хватает самых базовых знаний информатики и языка.
"Десятичная система" не имеет к вопросу никакого отношения в принципе.
В формуле

Math.round(this.price = +this.price * this.transfer)

ни плюс, ни round не играют никакой роли и просто не имеют смысла. Причем второй - даже в том случае, если тут просто что-то "съел" парсер и перед плюсом на самом деле что-то есть.

[OleksandrPrante]

Спасибо за помощь получается джентльмены

[Adamos]

OleksandrPrante, в программировании методом тыка единственная возможная помощь - это таки послать читать учебник. У тебя в коде происходит магия, которой ты совершенно не понимаешь.
Для понимания конкретно этого косяка нужно просто прочесть https://learn.javascript.ru/function-basics
Чтобы вообще программировать - все-таки учебник целиком.

[OleksandrPrante]

Adamos, Adamos, Спасибо я прочитаю, но я функции достаточно изучил, понимаю что метод это функция которую я вызываю в свойстве, не думаю что мне поможет данный раздел книги, прочитаю конечно, но я думал попадется человек для которого не составит труда потратить 1 минуту что бы помочь начинающему, уверен здесь максимально простая проблема, я просто не вижу почему метод не работает, сейчас буду пробовать вне класса методом тыка поделать функции с Math.round.

Answer 1

[Алексей Ярков]

А чего ради где-то на странице должно округляться? Метод changeToUAH ничего не возвращает даже, но зачем-то вызывается. Вы понимаете вообще что вы там накодили и что происходит?

Comments

[OleksandrPrante]

я создал класс в котором есть свойства price родитель и число после создал метод который умножает мое число на число атрибута объекта и на странице метод отрабатывает то есть выводится уже число обработанные методом, но это число с знаком после запятой, я хотел это пофиксить и знаю что этим занимается метод round библиотеки Math, попробовал использовать его но нечего не происходит
Я понимаю что возможно что-то упустил либо не так понял, иначе бы я тут не был, поэтому и создал вопрос,

Еще если вам не сложно просто ради интереса можете подсказать почему если я пишу
changeToUAH() {
console.log(Math.round(this.price = +this.price * this.transfer))
}
то в консоле у меня Math.round работает
а на странице нет
changeToUAH() {
Math.round(this.price = +this.price * this.transfer)
}

[Алексей Ярков]

OleksandrPrante,

а на странице нет

ДА ЧЕГО РАДИ ДОЛЖНО ЧТО-ТО НА СТРАНИЦЕ МЕНЯТЬСЯ? Ты нигде на странице результат не выводишь ))

[OleksandrPrante]

Алексей Ярков, я не дописал код с выводом на страницу, ведь к чему он если в коде вывода я просто рендрю, не к чему ваш caps lock,

class menuCard{
constructor(price, parentSlector){
this.price = price;
this.parent = document.querySelector(parentSlector);
this.transfer = 42;
this.changeToUAH();
}
changeToUAH() {
Math.round(this.price = +this.price * this.transfer)
}
render() {
const element = document.createElement('div');
element.innerHTML = `


Цена:
${this.price} грн/день


`;
this.parent.append(element);
}
}
new menuCard(
4,
'.menu .container'
).render();

[Алексей Ярков]

OleksandrPrante,

не к чему ваш caps lock

Мой CAPS LOCK очень даже "к чему", ибо вы напрочь не понимаете что написано у вас в коде. По вашему получается что вызов

Math.round(this.price = +this.price * this.transfer)

должен мутировать свойство this.price, которое выводится на странице. Вот я и спрашиваю: ЧЕГО РАДИ? Вам там в комментариях дали ссылки на материалы для изучения. Рекомендую ознакомиться.
А чтобы ваш вариант заработал, надо из метода возвращать результат и выводить на странице так:

class menuCard {
  constructor(price, parentSlector) {
    this.price = price;
    this.parent = document.querySelector(parentSlector);
    this.transfer = 42;
    this.changeToUAH();
  }
  changeToUAH() {
    // возвращаем результат вызова Math.round
    return Math.round((+this.price * this.transfer));
  }
  render() {
    const element = document.createElement('div');
    element.innerHTML = `


  Цена:
  ${this.changeToUAH()} грн/день
  

  `;
    this.parent.append(element);
  }
}
new menuCard(4, '.menu .container').render();

Или хотя бы так

changeToUAH() {
    this.price = Math.round(+this.price * this.transfer)
}

[OleksandrPrante]

Алексей Ярков, Большое спасибо, я сегодня начну читать книгу, я я просто недавно перешил с верстки в js пока что не понимаю некоторые истины, поэтому и обратился за помощью, интресно одно вы написали что вызов Math.round(this.price = +this.price * this.transfer) с чего ради должен мутировать this.price, я этого и не мог понять, в моем понимании я в объект передаю вызов метода this.changeToUAH(); после пишу сам метод
changeToUAH() {
Math.round(this.price = +this.price * this.transfer)
} то в объекте он должен вызываться без Math.round() он рассчитывает стоимость, а вот с ним нет, видимо это мое слабое знание функций, сегодня буду с этим разбираться
Спасибо!

[Алексей Ярков]

OleksandrPrante, рад если удалось хоть немного помочь. И таки да, начните с базы. Не бегите впереди паровоза ))

[Михаил Ливач]

Алексей Ярков, у Вас проскочила смесь из двух вариантов. Либо вариант с return, но тогда this.changeToUAH(); в конструкторе лишний. Либо вызов в конструкторе и присвоение в методе (последний блок кода), но тогда ${this.changeToUAH()} заменяется на ${this.price}. Первый вариант выглядит более перспективным; проще добавить конвертацию в другие валюты.
Это не в упрёк Вам, а чтобы показать OleksandrPrante: это был не вылизанный, законченный код, а только правильно работающий [в ключевой части] пример.