Skip to main content
se añadieron 92 caracteres en el cuerpo
Origen Enlace
Pablo Lozano
  • 49.9k
  • 8
  • 55
  • 92

Te propongo este cambio:

var cards = [
    { name: "manzana", points: 1, img: "http://www.dummy.com/manzana.jpg" },
    { name: "manzana", points: 2, img: "http://www.dummy.com/manzana.jpg" },
    { name: "manzana", points: 3, img: "http://www.dummy.com/manzana.jpg" },
    { name: "pera", points: 1, img: "http://www.dummy.com/pera.jpg" },
    { name: "pera", points: 2, img: "http://www.dummy.com/pera.jpg" },
    { name: "pera", points: 3, img: "http://www.dummy.com/pera.jpg" },
    { name: "melon", points: 1, img: "http://www.dummy.com/melon.jpg" },
    { name: "melon", points: 2, img: "http://www.dummy.com/melon.jpg" },
    { name: "melon", points: 3, img: "http://www.dummy.com/melon.jpg" },
];

function random_card() {
    if (cards.length > 0) {
        const rand = Math.floor(Math.random() * cards.length);
        const card=cards[rand];
        const html = "card: " + cards[rand]card.name + "<br/>points: " +
            cards[rand]card.points + "<br/><br/>";
        const imgSrc = cards[rand]card.img;
        // ... hacer algo con la url, quizá añadir un <img> en el div?
        document.getElementById("myDiv").innerHTML += html;
        cards.splice(rand, 1);
    }
}

Usar el contructor de Array no es necesario, es más breve y claro usar directamente corchetes. Y me parece mucho más claro guardar un objeto con campos con nombres que un array para guardar difentes tipos de información

Te propongo este cambio:

var cards = [
    { name: "manzana", points: 1, img: "http://www.dummy.com/manzana.jpg" },
    { name: "manzana", points: 2, img: "http://www.dummy.com/manzana.jpg" },
    { name: "manzana", points: 3, img: "http://www.dummy.com/manzana.jpg" },
    { name: "pera", points: 1, img: "http://www.dummy.com/pera.jpg" },
    { name: "pera", points: 2, img: "http://www.dummy.com/pera.jpg" },
    { name: "pera", points: 3, img: "http://www.dummy.com/pera.jpg" },
    { name: "melon", points: 1, img: "http://www.dummy.com/melon.jpg" },
    { name: "melon", points: 2, img: "http://www.dummy.com/melon.jpg" },
    { name: "melon", points: 3, img: "http://www.dummy.com/melon.jpg" },
];

function random_card() {
    if (cards.length > 0) {
        const rand = Math.floor(Math.random() * cards.length);
        const html = "card: " + cards[rand].name + "<br/>points: " +
            cards[rand].points + "<br/><br/>";
        const imgSrc = cards[rand].img;
        document.getElementById("myDiv").innerHTML += html;
        cards.splice(rand, 1);
    }
}

Usar el contructor de Array no es necesario, es más breve y claro usar directamente corchetes. Y me parece mucho más claro guardar un objeto con campos con nombres que un array para guardar difentes tipos de información

Te propongo este cambio:

var cards = [
    { name: "manzana", points: 1, img: "http://www.dummy.com/manzana.jpg" },
    { name: "manzana", points: 2, img: "http://www.dummy.com/manzana.jpg" },
    { name: "manzana", points: 3, img: "http://www.dummy.com/manzana.jpg" },
    { name: "pera", points: 1, img: "http://www.dummy.com/pera.jpg" },
    { name: "pera", points: 2, img: "http://www.dummy.com/pera.jpg" },
    { name: "pera", points: 3, img: "http://www.dummy.com/pera.jpg" },
    { name: "melon", points: 1, img: "http://www.dummy.com/melon.jpg" },
    { name: "melon", points: 2, img: "http://www.dummy.com/melon.jpg" },
    { name: "melon", points: 3, img: "http://www.dummy.com/melon.jpg" },
];

function random_card() {
    if (cards.length > 0) {
        const rand = Math.floor(Math.random() * cards.length);
        const card=cards[rand];
        const html = "card: " + card.name + "<br/>points: " +
            card.points + "<br/><br/>";
        const imgSrc = card.img;
        // ... hacer algo con la url, quizá añadir un <img> en el div?
        document.getElementById("myDiv").innerHTML += html;
        cards.splice(rand, 1);
    }
}

Usar el contructor de Array no es necesario, es más breve y claro usar directamente corchetes. Y me parece mucho más claro guardar un objeto con campos con nombres que un array para guardar difentes tipos de información

Origen Enlace
Pablo Lozano
  • 49.9k
  • 8
  • 55
  • 92

Te propongo este cambio:

var cards = [
    { name: "manzana", points: 1, img: "http://www.dummy.com/manzana.jpg" },
    { name: "manzana", points: 2, img: "http://www.dummy.com/manzana.jpg" },
    { name: "manzana", points: 3, img: "http://www.dummy.com/manzana.jpg" },
    { name: "pera", points: 1, img: "http://www.dummy.com/pera.jpg" },
    { name: "pera", points: 2, img: "http://www.dummy.com/pera.jpg" },
    { name: "pera", points: 3, img: "http://www.dummy.com/pera.jpg" },
    { name: "melon", points: 1, img: "http://www.dummy.com/melon.jpg" },
    { name: "melon", points: 2, img: "http://www.dummy.com/melon.jpg" },
    { name: "melon", points: 3, img: "http://www.dummy.com/melon.jpg" },
];

function random_card() {
    if (cards.length > 0) {
        const rand = Math.floor(Math.random() * cards.length);
        const html = "card: " + cards[rand].name + "<br/>points: " +
            cards[rand].points + "<br/><br/>";
        const imgSrc = cards[rand].img;
        document.getElementById("myDiv").innerHTML += html;
        cards.splice(rand, 1);
    }
}

Usar el contructor de Array no es necesario, es más breve y claro usar directamente corchetes. Y me parece mucho más claro guardar un objeto con campos con nombres que un array para guardar difentes tipos de información