Profit Calculator in Java -

-

i trying build program calculate profit made 4 items.this code have far. im not doing calculation part yet, im trying allow user choose items want purchase , how store values. 

public static void main(string[]args) {     int item=0;     double price=0;     string itemname="";     string yes ="";     string no="";     string answer="";     string response;          scanner list=new scanner(system.in);     system.out.println( "these current items availabe:" );      system.out.println( "item number\t item name" );      system.out.println( "1)\t\t flour\n2)\t\t juice\n3)\t\t crix\n4)\t\t cereal" );     system.out.println("enter item number wish purchase");     item=list.nextint();      if( item == 1 )      {         price = 25;          itemname = "flour";          system.out.println( "you selected flour" );      }     else if( item == 2 )      {         price = 15;          itemname = "juice";          system.out.println( "you selected juice" );      }     else if( item == 3 )      {         price = 10;          itemname = "crix";          system.out.println( "you selected crix" );      }     else if( item == 4 )      {         price = 30;          itemname = "cereal";          system.out.println( "you selected cereal" );      }     else      {         system.out.println( "invalid item number entered!" );      }      return;     system.out.println("would purchase item?");     scanner answer1=new scanner(system.in);     response=answer1.next();      if(answer==yes)     {         system.out.println("enter item number wish purchase");         item=list.nextint();     }     else if(answer==no)     {         system.out.println("thank shopping us");     }

the question is, how go doing or method far accurate?

also if else statements when answer yes or no, still asks enter item number wish purchase if input no. how correct this?

this not right, on many levels:

string yes=""; //this empty string... name not mean anything...  .... if(answer==yes){ //comparing empty string bad way...

should probably

private static final string yes="yes"; //now has content

and later

if(answer.equals(yes)) { //proper string equalitz checking ...

remember: strings objects. use .equals() compare equality.

apply same no part of course.

also:

scanner answer1=new scanner(system.in); response=answer1.next(); //you store result response  if(answer==yes){ //you check answer???

should be:

scanner answer1=new scanner(system.in); response=answer1.next(); //you store result response  if(response.equals(yes)){ //correct check

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