Dziwny błąd. (choć parsuje , zwraca typ String)

Question

Jak wiadomo, pola można modyfikować dowolny sposób (zmieniać typ, setować itd). Cóż, tak samo mi się wydawało w JS)

Mam taki sobie obiekt:

 

var Validator = function(salaryZl,salaryGr,dayHour,nightHour,isEmploy){
    this.salaryZl = salaryZl;
    this.salaryGr = salaryGr;
    this.dayHour = dayHour;
    this.nightHour = nightHour;
    this.isEmploy = isEmploy;
    this.state = false;
    this.salary;
    }

I pewnym miejscu wywołuje taką metodę:

    this.dayHour = parseInt(this.dayHour,10);
    console.log ("Its String? " + typeof (this.dayHour)); // String

Jak widać próbuje zmienić typ danych (Z string to int, a właściwie number). Myślałem, że mogę dowolny sposób modyfikować te pola, jednakże się myliłem, gdyż kolejne wywołanie odpowiada, że jest to String. Dlaczego?

 

Answer 1

Pokaż cały kod, w którym tak się dzieje, bo prawdę mówiąc nie ma prawa się tak dziać.

Comments

var Validator = function(salaryZl,salaryGr,dayHour,nightHour,isEmploy){
    this.salaryZl = salaryZl;
    this.salaryGr = salaryGr;
    this.dayHour = dayHour;
    this.nightHour = nightHour;
    this.isEmploy = isEmploy;
    this.state = false;
    this.salary;
    }

module.exports = Validator;

Validator.prototype.init = function (){
        if (!this.checkerValue() || !this.parserValue()  ){
            this.setState(false);
            return;
        }
        if (this.checkerValue() && this.parserValue()){
            if(this.checkCriteria()){
                this.setState(true);
             return;  
            }
   
}

}


Validator.prototype.checkerValue = function (){
     if (!this.iSeeSalary()){
              var Error = new (require('./Error.js'))();
              Error.emptyValueSalary();
              return false;
    }
    
    if (!this.iSeeHour()){
                var Error = new (require('./Error.js'))();
                Error.emptyValueHour();
                return false;
    }
   return true;
}
Validator.prototype.parserValue = function (){
    if (!this.isNum(this.salaryGr) || !this.isNum(this.salaryZl) ){
        return false;
    }
    
    if (!this.isNum(this.dayHour) || !this.isNum(this.nightHour)){
        return false;
    }
    if (!this.isGoodLength(this.salaryGr) || !this.isGoodLength(this.salaryZl)){
         var Error = new (require('./Error.js'))();
         Error.tooLength();
         return false;
    }
    this.salaryGr = parseInt(this.salaryGr);
    this.salaryZl = parseInt(this.salaryZl);
    this.dayHour = parseInt(this.dayHour,10);
    console.log ("Its String? " + typeof (this.dayHour));
    console.log (this);
    this.nightHour = parseInt(this.nightHour);
    this.convertToFloat();
    return true;
}
Validator.prototype.checkCriteria = function (){
    var JSON = new (require('./JSON.js'))('JS/lib/modules/JSON/options.json');
    var getOptions = JSON.getOptions();
     getOptions.then(function( JSON ) {
             alert ("Its OK. I see" + this);
             return console.log(this.isCorrect(JSON));
             
             if (this.isCorrect(JSON)) return true;    return false;
            }.bind(this));
            
}

Validator.prototype.iSeeSalary  = function (){
      if (this.isEmpty(this.salaryZl) && this.isEmpty(this.salaryGr)){
     return false;
   } 
     if (this.isEmpty(this.salaryZl) && !this.isEmpty(this.salaryGr)){
     return false;
   } 
   return true;
}
Validator.prototype.iSeeHour = function (){
     if (this.isEmpty(this.dayHour) && this.isEmpty(this.nightHour)){
       return false;
   }
   return true; 
}
Validator.prototype.isNum = function (value){
    if (value < 0 ){
          var Error = new (require('./Error.js'))();
           Error.lessAsZero();
           return false;
    }
    var parserNumber = parseInt(value,10);
    console.log (parserNumber);
    if (isNaN(parserNumber)){
          var Error = new (require('./Error.js'))();
          Error.isNotNumber();
          return false;
    }
    
    return true;
}
Validator.prototype.isCorrect = function (JSON){
    var inputHours = this.dayHour + this.nightHour;
    console.log (typeof (inputHours));
    if (inputHours > parseInt(JSON.options[0].params)){
         var Error = new (require('./Error.js'))();
          Error.tooHours();
          return false;
    }
    
    if (this.salary < parseInt(JSON.options[1].params)){
        var Error = new (require('./Error.js'))();
          Error.lessSalary();
          return false;
    }
    if (this.salary > parseInt(JSON.options[2].params)){
            var Error = new (require('./Error.js'))();
          Error.highSalary();
          return false;
    }
    return true;
}
Validator.prototype.getState = function (){
    return this.state;
}
Validator.prototype.setState = function (value){
    this.state = value;
}
Validator.prototype.isEmpty = function (value){
    if (value == ""){
      
        return true;
    }
     return false;
}
Validator.prototype.isGoodLength = function (value){
   if (value.length > 2)return false; return true;
}
Validator.prototype.convertToFloat = function (value){
    var floatValue = this.salaryZl + "." + this.salaryGr;
    var StringValue = floatValue.toString();
    this.setSalary(parseFloat(StringValue));
}
Validator.prototype.setSalary = function (value){
    this.salary = value;
}
Validator.prototype.getSalary = function (){
    return this.salary;
}

I tu jeszcze większa zagwozdka:

  var parserValue =  parseInt(this.dayHour,10);
    console.log ("Its number?" + typeof (parserValue)); // number;
    this.dayHour = parserValue;
    console.log ("Its String? " + typeof (this.dayHour)); // Dalej String....
    console.log (this); // Wskazuje na właściwy obiekt.

Ok, znalazłem rozwiązanie problemu. Niezadawalające ale niestety. Wskazuje mu typ przed wysłaniem do kolejnego obiektu i teraz w końcu pokazuje Number. Ale... dlaczego nie pozwolił modyfikować typ obiektu zamkniętego w polu?

var form = document.forms[0];
var salaryZl =parseInt (form.salaryHourZl.value);
var salaryGr = parseInt (form.salaryHourGr.value);
    if (salaryGr == "" ) salaryGr = 0;
    var dayHour = parseInt (form.dayHour.value);
    var nightHour = parseInt (form.nightHour.value);
    if (nightHour == "" )nightHour = 0;
    var target = new (require('./modules/Validator.js'))(salaryZl,salaryGr,dayHour,nightHour,this.isEmploy);
    var handler = {
  set: function (target, key, value, receiver) {
   console.log (target.getState());
  }
};

 

Answer 2

Można zrobić to na początku.

var Validator = function(dayHour){
    this.dayHour = +dayHour; // lub this.dayHour = parseInt(dayHour, 10);
    alert(typeof this.dayHour);
}
new Validator("25");

 

Comments

Problem z tym, że dane lecą zewnątrz (nie wiem czy to jest int, float  lub  nic (po konwersji oczywiście), twoja metoda by owszem działała. Ale w razie nie powodzenia, rzucałaby NaN no i musiałbym zmodyfikować kod (ponownie) (. Oczywiście wykorzystam to, jeżeli zajdzie taka potrzeba, ale musi być inny sposób.