This project is much simpler than the build a bike store project prior.
The test requirements do not require as much checking as you were doing, this could be complicating your logic a little bit.
For example, this bit of code can simply be condense to inserting the time and checking to see if there is a result.
Since time is unique, your database will not allow you to insert at overlapping time.
However, it is very simplified since time is being used as a text and it’s easy to put in. “ 11am” as well as “11:00” and both would be allowed.
But this project is just a simplified version of the bike store project. Try reviewing this code, and see if you can simplify other parts of your solution.
echo -e “\nWhat time would you like your $SERVICE_NAME_TO_SELECT, ‘$CUSTOMER_NAME’”
read SERVICE_TIME
TIME_IN_DATABASE=$($PSQL “SELECT time FROM appointments WHERE time = ‘$SERVICE_TIME’”)
if [[ -z $TIME_IN_DATABASE ]]
then
INSERT_APPOINTMENT_TIME=$($PSQL “INSERT INTO appointments(customer_id, service_id, time) VALUES($NEW_CUSTOMER_ID, $SERVICE_ID_SELECTED, ‘$SERVICE_TIME’)”)
echo $INSERT_APPOINTMENT_TYPE
echo -e “\nI have put you down for a$SERVICE_NAME_TO_SELECT at $SERVICE_TIME, $CUSTOMER_NAME.”
fi
your case statements are also unnecessary, simply use the number entered to get the service ID.
Finally, it seems like you were checking the entered, phone number twice, which doesn’t seem necessary to me. There are a lot of lines you could aluminate, which will help you see what’s really going on.
Try writing it out on paper reviewing the test requirements. Using pseudo code, that could help.
I wrote the majority of this code, while the power was out in a notepad on my laptop.
I simply went through each test and wrote the code for that test, and for the most part, it worked pretty well with a few corrections. But it was not nearly as complicated as yours.
