basic conditional with javascript

Question

I'm trying to make an IMC calculator that shows an alert depending your IMC value. But when I try to make a conditional for show an alert depending the body-status it doesn't come

var k =
  { nombre       : 'Antonio'
  , peso         : 85
  , altura       : 1.80
  , masacorporal : function ()
    {
    var r = this.peso / (this.altura*this.altura);
    return Math.round(r*100)/100;
    }
  }
function f()
  {
  k.peso   = document.getElementById("dato1").value;
  k.altura = document.getElementById("dato2").value;
  
  document.getElementById("resultado").innerHTML = k.masacorporal();
  
  var error = 0;
  if (k.peso == '' && error == 0)
    {
    alert("El valor del peso es obligatorio");
    k.peso.focus();
    error=1;
    }
  if (k.altura=='' && error==0)
    {
    alert("El valor de la altura es obligatorio");
    k.altura.focus();
    error=1;
    }
  if (!/^([0-9])*$/.test(k.peso))
    {
    alert("El valor " + k.peso + " no es un número");
    m1.focus();
    error=1;
    }
  if (error==0)
    {
    f()
    }
  }
function m()
  {
  if (k.masacorporal < 18.5) 
    { 
    alert("Bad healt status"); 
    }
  }

PESO (kg): <input type="text" id="dato1" onkeypress=""/>
<br/>
Altura (metros): <input type="text" id="dato2"/>
<br/>
<input   type="button" value="Calcular" onclick="f()"/>
<br/>
IMC:<span id="resultado"> </span>

Thats the full code if u want to try it, for me the alert doesnt show even if i have corrected the last part

PMI but what's IMC?

j08691
– j08691

2021-03-12 22:55:36 +00:00
Commented Mar 12, 2021 at 22:55 — j08691
– j08691, Commented Mar 12, 2021 at 22:55
@j08691 BMI (body mass index) in Spanish.

GirkovArpa
– GirkovArpa

2021-03-12 22:56:59 +00:00
Commented Mar 12, 2021 at 22:56 — GirkovArpa
– GirkovArpa, Commented Mar 12, 2021 at 22:56
masacorporal is a function off of k

Daniel A. White
– Daniel A. White

2021-03-12 22:58:39 +00:00
Commented Mar 12, 2021 at 22:58 — Daniel A. White
– Daniel A. White, Commented Mar 12, 2021 at 22:58
Do you have a script tag in the middle of your code?

trincot
– trincot

2021-03-12 22:59:54 +00:00
Commented Mar 12, 2021 at 22:59 — trincot
– trincot, Commented Mar 12, 2021 at 22:59
yes, im going to post my full code

Joselito13
– Joselito13

2021-03-12 23:00:55 +00:00
Commented Mar 12, 2021 at 23:00 — Joselito13
– Joselito13, Commented Mar 12, 2021 at 23:00

Masood · Accepted Answer · 2021-03-12 23:04:31Z

1

You are trying to access method inside object directly. You should add object name before method.

function m() {
    if (k.masacorporal < 18.5){alert("Bad healt status");}
}

edited Mar 12, 2021 at 23:04

answered Mar 12, 2021 at 22:58

Masood

1,6041 gold badge23 silver badges33 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Daniel A. White Over a year ago

its a function.

trincot Over a year ago

It is perfectly fine to call functions that are defined as object properties methods.

trincot Over a year ago

Joselito, where do you call m?

GirkovArpa · Accepted Answer · 2021-03-12 23:04:37Z

0

You need to write masocorporal as k.masacorporal().

It has to be prefixed with k. because it's a property of k, and it has to have parentheses to execute it because it's function. Otherwise you're checking if the function is smaller than 18.5 which makes no sense; you want to compare the value the function returns.

var k = {
  nombre: "Antonio",
  peso: 0,
  altura: 1.80,
  masacorporal: function () {
    var r = this.peso / (this.altura * this.altura);
    return Math.round(r * 100) / 100;
  }
};

function m() {
  if (k.masacorporal() < 18.5) { alert("Bad health status"); }
}

m();

answered Mar 12, 2021 at 23:04

GirkovArpa

5,1015 gold badges19 silver badges52 bronze badges

Collectives™ on Stack Overflow

basic conditional with javascript

2 Answers 2

3 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related