1

Hola estoy haciendo un programa que realiza votes en la cadena de bloques de steemjs.

Mi codigo

var u = 0;
const friendsz = [
      'juan',
      'luis',
      'carlos'
    ];  
var myInt = setInterval(function () {

    if (u < friendsz.length) {
        //console.log(friendsz[u++]);
        wkd.api.getDiscussionsByBlog({tag: friendsz[u++], limit: 1}, (err, res) => { 
            var author = res[0].author;
            var permlink = res[0].permlink;
            console.log(author);
            console.log(permlink);

        wkd.api.getActiveVotes(author, permlink, (err, res) => {

        for (var i = 0; i < res.length; i++) {
            var post = res[i].voter;

        if (post == voter) {
            console.log("[Weku Buster Bot] says:".yellow, "The Post:", permlink, "of the user", author, "has already been voted.");
        }else{
            wkd.broadcast.vote(wif, voter, author, permlink, weight, (err, res) => { 
                console.log("[Weku Buster Bot] says: ".yellow + "Your Friend " + author + " Got " + weight/100 + "% Upvote For his last post.");
            });
            wkd.broadcast.comment(wifPosting, author, permlink, voter, comment_permlink, title, body, jsonMetadata, (err, res) => {
                console.log("[Weku Buster Bot] says: ".yellow + "We have announced to your friend " + author + " that you have voted his last post a weight: " + weight/100 + " %.");
        });
        }
        }
    });
    });
}

}, 15000);

EL problema que surge es que al realizar la comparación de post y user hace que que todo el if se ejecute y me causa errores.

post == voter verifica que el usuario ya ha votado si no ha votado, tendría que saltar a else y ejecutar las funciones. pero en este caso ejecuta las 2 cosas a la ves y me provoca errores. ¿Entonces como puedo hacer bien esa condicional?

14
  • que lenguaje es.. y no se entiende que queres?!?!????
    – gbianchi
    Commented el 18 ene. 2019 a las 21:58
  • olvide poner que era javascript. pues solo quiero realizar bien esa condicional. Commented el 18 ene. 2019 a las 22:00
  • Pero sigo sin entender cual es el problema. si esta haciendo lo que queres...
    – gbianchi
    Commented el 18 ene. 2019 a las 22:03
  • es que me trae a la ves el else. se supone que "post == usuario" son correctos, no tendría que traerme el else. con traerme me refiero a mostrar. Commented el 18 ene. 2019 a las 22:05
  • me parece que estas entendiendo muy mal tu codigo. Si es correcto, el if muestra "son similares", si es incorrecto, muestra el else.. en tu array tenes 3 incorrectos y un correcto.
    – gbianchi
    Commented el 18 ene. 2019 a las 22:07

1 respuesta 1

1

Aca solo hay un error de concepto en el codigo.

Si tu for solo tiene que verificar si hay uno correcto, no deberias tener un if con un else adentro...

En realidad, lo que tendrias que hacer es comprobar si existe o no ese usuario en la cadena, y despues dependiendo de eso, hacer lo que queres hacer

Algo asi:

var encontrado = false;
for (var i = 0; i < res.length; i++) {
    var post = res[i].voter;
    if (post == voter) {
        encontrado = true;
    }
}
if (encontrado) {
    console.log("[Weku Buster Bot] says:".yellow, "The Post:", permlink, "of the user", author, "has already been voted.");
}else{
    wkd.broadcast.vote(wif, voter, author, permlink, weight, (err, res) => { 
    console.log("[Weku Buster Bot] says: ".yellow + "Your Friend " + author + " Got " + weight/100 + "% Upvote For his last post.");
 });
 ... todo el resto de tu codigo

De esta forma, solo comprobas si voto o no...

Es mas, como simplificacion, podrias no recorrer todo el array usando un while de la siguiente forma...

var encontrado = false;
var i = 0;
var tamaño = res.length
while (!encontrado && i < tamaño) {
    var post = res[i].voter;
    if (post == voter) {
        encontrado = true;
    }
    i++;
}

Como aclaración, este codigo es puro, porque las librerias que buscan dentro de arrays, hacen exactamente esto que escribi.

6
  • Excelente. Funciona correctamente ahora si. Gracias. Ahora tendré que repasar mas sobre esto. Commented el 18 ene. 2019 a las 23:27
  • paso algo bro. cuando ejecuto el programa si el primer post ya ha sido votado se queda con que los demás post han sido votados. es decir ya no ejecuta la función para verificar si ya ha sido votado o no cada post. Commented el 19 ene. 2019 a las 0:18
  • @HadrienRivere habria que ver como metiste este codigo dentro de tu funcion...
    – gbianchi
    Commented el 19 ene. 2019 a las 1:03
  • lo acabo de actulizar :p Commented el 19 ene. 2019 a las 1:13
  • Actualizar tu codigo en este caso es mala idea, porque la respuesta queda desactualizada.... sobre todo porque mi solucion ya no es la solucion porque es lo que escribiste.. volve atras el post.. el error es que var verifyVote = false; tiene que estar adentro de la funcion.. justo encima del for.. si no, una vez que es true, nadie lo vuelve a poner en false. @HadrienRivere
    – gbianchi
    Commented el 19 ene. 2019 a las 1:15

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.