Thread: Help me please with my codes

Hi everyone
I've just registered here, and really need your help with my assignment
I have these 2 codes that won't work with me

Q1: Write a program that reads a number of patients and then let the employee enter their names and ages and count them. The program should check if the age of the patient is below 25 years old , otherwise, the patient will not be counted . your program should display total number of patient under the age 25.

Q2: Write a program that reads unlimited number of items and their prices until the user enters “Exit “, after that the program should print out the total bill.

I uploaded my codes and print screen for sample run ,
the problem with Q1 is that the print should be like this :
Enter a Number of Patients > 20
Enter patients Name: sara
Enter patients Age: 23
so I can count the ages under 25 .. ???

the problem with Q2 is when I enter the sentinel "Exit" it won't end the loop !!!

that's all , waiting for your help

Try putting the code between code tags.

[*code]

[*/code]
Without the *

Then you will get more replies then with screens.

Sara92 Hi and welcome to the forum!
Please see the announcements page for the use of code tags and other useful facts.


Q1: The user is to give a name and an age one at a time. If the current age is less than 25 add 1 to the number of people under 25. If not do not add 1 and continue. How would you do it without a computer? Write the steps down on paper. (No, seriously, write them down) When you have the steps in front of you it is much easier to write the code.

Q2: The problem I see with this program is logical. You didn't write the code to do what you wanted. Again write the steps down on paper and use that to code from. Doing so will help keep the steps in order. For one thing you set the boolean variable before the loop, and the value never changes. You need some way to change the value of that variable inside the loop. Another thing I see is duplicated lines of code. Any time you see yourself writing the same thing over again, question the design. Use your favorite search engine and 'java do while loop' and 'java loop and a half'

The following code can be seen here:

You use loop-and-a-half to avoid repeating code from outside the loop to the inside. Example:

read a;
while a != b do
  stuff;
  read a;
end

becomes

while true do
  read a
  if a == b then break
  stuff;
end

Now I only have the read in one place.

Edit
I wouldn't worry about the hash map just yet. That is likely some time down the road from now.

thanks a lot,, I got my mistakes now

I appreciate your advice & i will do it in future..