javascript - Looping through an array and look for repeating numbers -


ok, i've spent 2 days on problem , give up.

what i'm trying create array of random numbers between 0-17, each number repeat once - 36 numbers in total.

i wrote function check how many times number occurs, if it's twice, returns false.

it works until undefined number, breaks...

here's have:

var numbers = [];      function checknumberrepeat(n,o,c){         var count = 0;         for(var in o){             if(o[i] == n){                 count ++;             }         }         if(count == c){              return false;         }else{             return n;            }     }      function makerand(){         var rand = checknumberrepeat(math.floor(math.random()*18),numbers,2);          if(rand){             return rand;         }else{             makerand();         }        }       for(var i=0;i<36;i++){           numbers.push(makerand());         console.log(numbers);     }

i'm pretty sure problem happening in myrand() function

any idea?

thanks

you missing return

return makerand()

it has no return returns undefined

update:

you had mistake in code. 0 falsy if (rand) when rand 0 returns false, hence 0 never inserted => endless loop

makerand shoul this:

function makerand() {     var rand = checknumberrepeat(math.floor(math.random()*18),numbers,2);      if (rand !== false){         return rand;     } else {         return makerand();     }    }

updated jsfiddle: http://jsfiddle.net/nayish/c9sk9suc/4/


Comments

Post a Comment

Popular posts from this blog

java - Spring Data JPA: Why findOne(id) executing delete query internally? -

i have 1 application using @manytomany annotation link 2 different entities 1 parent entity. not sure if have implemented properly. when execute .findone() on parent entity's repository find it, @ first fetch data along executing delete queries internally delete record mapping table (a table jpa creates after adding @manytomany annotation.). following generated sql queries , code snippet: java class @entity @table(name = "inventory") public class inventory { @id long id; // many other properties @manytomany(fetch=fetchtype.lazy) private list<stream> streams; @manytomany(fetch=fetchtype.lazy) private list<subject> subjects; } there no other annotation i've put in stream , subject entity class link inventory class. when run app, create following tables: inventory stream subject inventory_stream inventory_subject (inventory_stream , inventory_subject, 2 tables created automatically, don't k...
Read more

python - Mongodb How to add addtional information when aggregating? -

i beginner , wrote line pipeline works, wanna add other information output, screen name , or number of tweets.i tried add under $group gave me syntax error everytime here pipeline: def make_pipeline(): # complete aggregation pipeline pipeline = [ { '$match': { "user.statuses_count": {"$gt":99 }, "user.time_zone": "brasilia" } }, { "$group": { "_id": "$user.id", "followers": { "$max": "$user.followers_count" } } }, { "$sort": { "followers": -1 } }, { "$limit" : 1 } ]; i using on example : { "_id" : objectid("5304e2e3cc9e684aa98bef97"), "text" : "first week of school on :p", "in_reply_to_status_id" : null, "retweet_count" : null, ...
Read more

java - Incorrect order of records in M-M relationship in hibernate -

Image
i have 4 records in bridge table in mysql database. using @manytomany in hibernate, accurate order of employees incorrect incorrect of projects. following required material figure out doing mistake. this source. @entity @table(name="project") public class projectbean { @id @generatedvalue @column(name="project_id") private int projectid; @column(name="title") private string projecttitle; @manytomany(mappedby="projects") private collection<employeebean> employees; public projectbean() { employees = new arraylist<employeebean>(); } //getters & setters } @entity @table(name="employee") public class employeebean { @id @generatedvalue @column(name="employee_id") private int employee_id; @column(name="first_name") private string ...
Read more