Si le nombre est inférieur à zéro et d'un nombre négatif cacher jquery

Ayant le plus dur moment de l'écriture de la fonction pour masquer les numéros qui sont égal ou inférieur à zéro. Voici la fonction que j'ai envie de l'écrire à:

function isItANumber() {
var increased = parseInt($("#increasedRevenueValue"));

if(isNaN(increased)) {
    $("#increasedRevenueValue").hide();
}

}

Toute pensée? Dois-je utiliser OR?

EDIT: Voici le violon de mon code http://jsfiddle.net/YVcj7/

  • Ce type d'élément ID d' increasedRevenueValue?
  • Je pense qu'il devrait être $("#increasedRevenueValue").val() si elle est un élément de saisie ou de $("#increasedRevenueValue").texte() si elle est span/div/html élément de regroupement.
  • La condition isNaN.. doit être étendue, comme cela ne prend soin de valeurs non numériques
  • Cette communauté est aussi sur des recherches approfondies et écrit correctement des questions.
  • Je dirais que j'ai décrit ce que je suis après très bien. Il y a même un charme pour aller avec elle.
  • Pourquoi ne serait-il pas un simple <= 0 travail?
  • Il a fait un travail, mais vous ne devez pas avoir regardé le violon. C'était une partie de la réponse, mais pas l'ensemble de la solution.
  • Le violon, devrait soutenir votre question, ne pas contenir de clé/pièces essentielles à elle. Toutes les critiques du code doit être collé ici.
  • C'est plus utile de répondre à la downvotes que j'obtiens.
  • Haha, c'est ce que je suis ici. Je ne peux pas parler pour les downvoters, mais la raison de mon commentaire, c'est que si jsfiddle est en baisse (godforbid), nous avons toujours besoin d'un regard sur/votre code d'accès pour vous aider.
  • Du sens. J'apprécie le commentaire. Je suis ouvert à la critique, mais de la réception vers le bas de votes sans raison donnée est frustrant.

InformationsquelleAutor Sethen | 2012-03-15
  1. 6

    Essayez ce qui suit, Im prenant une deviner à quel élément de votre aide.

    function isItANumber() {
    var increased = parseInt($("#increasedRevenueValue").text());

    if(isNaN(increased) || increased <= 0) {
        $("#increasedRevenueValue").hide();
    }
}
isItANumber();
​

    Démonstration En Direct

    Et pure js.. parce que ce que nous avons vraiment besoin jQuery pour cela?

    function isItANumber(el) {
    var increased = parseInt(el.innerText);

    if(isNaN(increased) || increased <= 0) {
        el.style.display = 'none';
    }
}
var element = document.getElementById("increasedRevenueValue");
isItANumber(element);

    pure js démo

    • Voici un violon de ce que je fais peut-être vous pouvez jeter un peu de lumière dans ce contexte: jsfiddle.net/YVcj7
    • Le #increasedRevenueValue est un champ de saisie.. Vous devriez utiliser .val() pour obtenir la valeur du champ de saisie.
    • Pour une raison quelconque, chaque fois que j'essaie de trouver la valeur et à écrire, et si l'instruction pour qu'il soit inférieure à 0, ce n'est pas le travail.
    • href="http://jsfiddle.net/loktar/YVcj7/1/" >jsfiddle.net/loktar/YVcj7/1 dans la dernière fonction où vous ne la case à cocher, cliquez sur événement déplacer isItANumber() vers le bas. Vous avez le contrôle de l'ancienne valeur avant de mettre à jour la nouvelle valeur. Aussi j'ai changé text() à val()
    • C'était comme ça, bravo. Je suppose que je n'ai pas encore vraiment comprendre la logique derrière cela. Je que c'était la vérification de l'ancienne valeur, mais pourquoi nous devons mettre de val-là?
    • val se réfère à un apport d'éléments de données. text est utilisé pour les divs/enjambe/etc.
    • Je reçois, je me demandais simplement pourquoi nous devons parseInt() que la valeur?
    • fondamentalement, il suffit de le transformer en un certain nombre, sinon ça va être une chaîne de caractères.

    InformationsquelleAutor Loktar
  2. 0

    Je pense que vous voulez 

    if(isNaN(increased) || increased <= 0){
    $("#increasedRevenueValue").hide();

}
    InformationsquelleAutor devstruck
  3. 0

    Je ne pense pas que vous pouvez parseInt un objet jQuery. Je voudrais essayer parseInt($("#increasedRevenueValue").text()); ou parseInt($("#increasedRevenueValue").val()); selon ce #increasedRevenueValue est.

    InformationsquelleAutor Paul Sham
  4. 0

    Il ressemble à votre fonction est de faire un peu de double duty (obtenir la valeur d'un champ, de vérifier si c'est un nombre, etc.).

    Tout d'abord, je voudrais vous recommandons de vérifier que la valeur est numérique. Cette question a une très bonne isNumeric function que vous pouvez en tirer.

    Dans ce cas, vous avez juste besoin de faire ce qui suit:

    function isItANumber(value) {
    var value = $("#increasedRevenueValue").val() || $("#increasedRevenueValue").text() //get the value if this is an input field, or fallback on the text otherwise.
    var isNumber = isNumeric(value);
    var isNegative = value < 0;

    if (isNumber && isNegative) {
        $("#increasedRevenueValue").hide();
    }
}
    InformationsquelleAutor NT3RP
  5. 0

    Comme les gens l'ont mentionné, si l'élément est un champ de saisie, de votre "augmenté" devrait être:

    var increased = parseInt($("#increasedRevenueValue").val(), 10);

    Si ses un div/span/paragraphe ou quelque chose le long de ces lignes, il devrait l'être.

    var increased = parseInt($("#increasedRevenueValue").text(), 10);

    Puis il suffit de faire une boucle.

    if(increased =< 0) {
    $("#increasedRevenueValue").hide();
}

    En fonction de votre utilisation de cette fonction, vous devrez peut-être envisager si $("#increasedRevenueValue").val() est une valeur vide, paresint fera en 0.

    InformationsquelleAutor ninja
  6. 0

    Je vais supposer que $("#increasedRevenueValue") est un élément d'entrée,

    function isItANumber() {
   //change it to $("#increasedRevenueValue").text() if it is div/span
   var increased = parseInt($("#increasedRevenueValue").val(), 10);

   if(isNaN(increased) || increased <= 0) {
       $("#increasedRevenueValue").hide();
   }

}
    InformationsquelleAutor Selvakumar Arumugam