java - Why is an integer resetting to 100 when it should update to something different -

-

the wall of code below should change value of "lowestguess" , "highestguess" time loop runs through. ideally, it's meant lower highest guess 100 , raise lowest guess 1 program can't guess lower or higher number has guessed (reducing amount of loops takes complete program.

however, each time runs "takestab" function, lowestguess , highest guess reset defaults, 1 , 100 respectively. no idea what's going on here seeing i'm not getting errors.

is possible because methods here private, don't update variables outside of respective scopes?

import java.util.scanner;  public class aiguessing {     public static void main(string[]args)      {         //declarations*         scanner inputreader = new scanner(system.in);         int number;         int guess;         int guesses = 0;         int lowestguess = 1;         int highestguess = 100;          //code         system.out.println("welcome!");         system.out.println("please enter number ai guess");         number = inputreader.nextint();          //computer tries guess number                 {             guesses++;             guess = takestab(lowestguess, highestguess, number);             system.out.println("\"i guess " +guess+ ".\"");         }while(guess != number);          if(guess == number)         {             system.out.println("woo! got it!");         }          if(guesses >= 1 && guesses ==2)         {             system.out.println(guesses + " guesses? i..am...god!!!!");         }         else if(guesses >= 3 && guesses <= 5)         {             system.out.println(guesses + " guesses? hooray! i'm average gravy!!!");         }         else if(guesses >= 6 && guesses <= 8)         {             system.out.println(guesses + " guesses? guess when i'm drunk");         }         else if(guesses >= 9)         {             system.out.println(guesses + " guesses? bugger me... turn me scrap");         }         //end code     }      public static int takestab(integer lowestguess, integer highestguess, integer number)     {            int estimate;         estimate = estimate(lowestguess, highestguess);         system.out.println("estimate is: "+estimate+".");          if(estimate < number && estimate > lowestguess)         {             lowestguess = estimate;             barkchange(lowestguess, highestguess, estimate);         }         else if(estimate > number && estimate < highestguess)         {             highestguess = estimate;             barkchange(lowestguess, highestguess, estimate);         }                 return estimate;     }      //function generate , return random number     public static int estimate(int low, int high)     {         int comguess;         comguess = (low + (int)(math.random() * ((high - low) + low)));         return comguess;     }      //function 'bark' changes of lowest , highest guesses     public static void barkchange(integer lowestguess, integer highestguess, integer guess)     {         system.out.println("current guess is: "+guess+".");         system.out.println("lowest guess is: "+lowestguess+".");         system.out.println("highest guess is: "+highestguess+".\n");     } }

integer immutable object.

lowestguess = estimate;

at line(and 1 highestguess), you're changing local copy of lowestguess , not 1 passed it.

the simple solution in case, declare lowestguess , highestguess "globally" (static).

above main put

static int highestguess = 100; static int lowestguess = 1;

then, can remove parameters method, , don't have pass them in anymore. can reference static ones directly.

an aside nitpickers: used "globally" here because there's no oo here. static variable in context intents , purposes global variable.


Comments

Post a Comment

Popular posts from this blog

c# - How Configure Devart dotConnect for SQLite Code First? -

-
can configure following sample code ? when run following code, error : no entity framework provider found ado.net provider invariant name 'devart.data.sqlite' setting in machine.config <system.data> <dbproviderfactories> <add name="dotconnect sqlite" invariant="devart.data.sqlite" description="devart dotconnect sqlite" type="devart.data.sqlite.sqliteproviderfactory, devart.data.sqlite, version=4.6.287.0, culture=neutral, publickeytoken=09af7300eec23701" /> </dbproviderfactories> </system.data> after add below block code in app.config <?xml version="1.0"?> <configuration>   <configsections> <!-- more information on entity framework configuration, visit http://go.microsoft.com/fwlink/?linkid=237468 --> <section name="entityframework" type="system.data.entity.internal.configfile.entityframeworksection, entit...
Read more

java - Copying object fields -

-
what best way of copying classes when need have 2 independent variables? have simple class: public class myclass { boolean = false; string b= "not empty"; } do need make method : assign(myclass data ) { a= data.a; b= data.b; } is there automatic method copy (duplicate) objects in java? do need make method : pretty close. instead of making method, should make constructor. such constructors called copy constructor , , create them this: myclass(myclass data) { = data.a; b = data.b; } and create copy of instance, use constructor this: myclass obj1 = new myclass(); myclass obj2 = new myclass(obj1); copy constructor can tedious : using copy constructor create deep-copy can tedious, when class has mutable fields. in case, assignment create copy of reference, , not object itself. have create copy of fields (if want deep copy). can go recursive. a better way create deep copy serialize , deserialize object. why not use clone()? ...
Read more

c++ - Clear the memory after returning a vector in a function -

-
i have function processes , stores lot of data in , returns results vector of class. amount of data stored in function tremendous , want clear storage memory of function after finished job. necessary (does function automatically clear memory) or should clear memory function? update: vector<customers> process(char* const *filename, vector<int> id) { vector<customers> list_of_customers; (perform actions) return list_of_customers; } variables defined locally within function automatically released @ end of function scope. destructors objects called, should free memory allocated objects (if objects coded correctly). example of such object std::vector . anything you've allocated new must have corresponding delete release storage. try avoid doing own allocation , use raii instead, i.e. containers or smart pointers.
Read more