What am I missing?

Question about the task

CRUD

Java Core, Level 7, Lesson 10

Resolved

I would appreciate the help here on what I am missing to pass the conditions.

CrUD: Create, Update, Delete.

The program runs with one of the following sets of arguments:
-c name sex bd -u id name sex bd -d id -i id

Argument values:
name (String)
sex ("m" or "f")
bd (birth date in the following format: "04 15 1990")
-c (adds a person with the specified arguments to the end of allPeople; displays the id on the screen)
-u (updates the data of the person with the specified id)
-d (performs the logical deletion of the person with the specified id; replaces all of its data with null)
-i (displays information about the person with the specified id: name sex (m/f) bd (format Apr 15 1990))

The id corresponds to the index in the list.
All the people should be stored in allPeople.
Use Locale.ENGLISH as the second argument for SimpleDateFormat.

Example arguments:
-c Washington m "04 15 1990"

Example output for the -i argument:
Washington m Apr 15 1990

Requirements:

The Solution class must contain a public static List<Person> field called allPeople.
The Solution class must have a static block where two people are added to the allPeople list.
When you start the program with the -c argument, the program should add the person with the specified arguments to the end of the allPeople list and display the id (index).
When you run the program with the -u argument, the program must update the data of the person with the specified id in the allPeople list.
When you run the program with the -d argument, the program must perform the logical deletion the person with the specified id in the allPeople list.
When you run the program with the -i argument, the program should display the data about the person with the specified id in accordance with the format given in the task.

package com.codegym.task.task17.task1710; import java.text.ParseException; import java.text.SimpleDateFormat; import java.util.ArrayList; import java.util.Date; import java.util.List; import java.util.Locale; /* CRUD */ public class Solution { public static List<Person> allPeople = new ArrayList<>(); static { allPeople.add(Person.createMale("Donald Chump", new Date())); // id=0 allPeople.add(Person.createMale("Larry Gates", new Date())); // id=1 } public static void main(String[] args) throws ParseException{ // Start here if (args.length != 0){ switch (args[0]) { case "-c": Date date1 = new SimpleDateFormat("MMM dd yyyy", Locale.ENGLISH).parse(args[3]); if (args[2].equals("m")) { allPeople.add(Person.createMale(args[1], date1)); System.out.println(allPeople.size() - 1); } else if(args[2].equals("f")) { allPeople.add(Person.createFemale(args[1], date1)); System.out.println(allPeople.size() - 1); } break; case "-u" : Person updatePerson = allPeople.get(Integer.parseInt(args[1])); Date date2 = new SimpleDateFormat("MMM dd yyy", Locale.ENGLISH).parse(args[3]); updatePerson.setName(args[2]); updatePerson.setBirthDate(date2); if (args[3].equals("m")){ updatePerson.setSex(Sex.MALE); } else { updatePerson.setSex(Sex.FEMALE); } break; case "-d" : Person deletePerson = allPeople.get(Integer.parseInt(args[1])); deletePerson.setName(null); deletePerson.setSex(null); deletePerson.setBirthDate(null); break; case "-i" : Person infoPerson = allPeople.get(Integer.parseInt(args[1])); System.out.println(infoPerson + " " + infoPerson.getSex() + " " + infoPerson.getBirthDate()); break; default: throw new IllegalStateException("Unexpected value: " + args[0]); } } // if(args[0].equals("-c")) { // SimpleDateFormat sdf = new SimpleDateFormat("MMM dd yyyy", Locale.ENGLISH); // Date d = sdf.parse(args[3]); // if(args[2].equals("m")) // allPeople.add(Person.createMale(args[1], d)); // else // allPeople.add(Person.createFemale(args[1], d)); // System.out.println(allPeople.size()-1); // } // // else if(args[0].equals("-u")) { // SimpleDateFormat sdf = new SimpleDateFormat("MMM dd yyyy", Locale.ENGLISH); // Date d = sdf.parse(args[4]); // Person person = allPeople.get(Integer.parseInt(args[1])); // person.setName(args[2]); // if(args[3].equals("m")) // person.setSex(Sex.MALE); // else // person.setSex(Sex.FEMALE); // person.setBirthDate(d); // } // // else if(args[0].equals("-d")) { // //allPeople.remove(Integer.parseInt(args[1])); // Person person = allPeople.get(Integer.parseInt(args[1])); // person.setName(null); // person.setSex(null); // person.setBirthDate(null); // } // // else if(args[0].equals("-i")) { // Person person = allPeople.get(Integer.parseInt(args[1])); // String gender = ""; // if(person.getSex().equals(Sex.MALE)) // gender = "m"; // else // gender = "f"; // System.out.print(person.getName() + " " + gender + " " + person.getBirthDate()); // } } }

Comments (8)

Popular
New
Old

You must be signed in to leave a comment

Seb

Level 41 , Crefeld, Germany

13 February 2020, 01:19

You need to adjust the SimpleDateFormat on line 31. You are reading in a string in the form of "04 15 1990". The pattern "MMM dd yyyy" is not correct. Try to use only two M's... ;-) The same accounts for line 45. In addition, when updating a user with the -u option args[3] is not the date since a full command line for the update looks like -u 1 Washington m "04 15 1990". There is an additional argument for the id. The date therefore is args[4]. The program's output for running with the -i option is something like - Person@60e53b93 MALE Thu Feb 13 00:05:13 CET 2020. This is not quite the required format either. Therefore on line 66 call infoPerson.getName() - instead of just infoPerson. And last, but not least, the date needs to be printed out in the format ~~"MM dd yyyy"~~ (Correction: "MMM dd yyyy"). Set a SimpleDateFormatter as you did when reading in dates, and use it to output the date. It could be something like SimpleDateFormat simpleDateFormat = new SimpleDateFormat("MMM dd yyyy", Locale.ENGLISH); And then maybe simpleDateFormat.format(infoPerson.getBirthDate()); All the best. ;-)

Ivan

Level 22 , Nope, Bulgaria

13 February 2020, 12:48

Stupid little mistakes :) and the hints weren't very clear. Can't pass the last one, tried your way and also with String to take the date, still can't. Thank you for all your help!

Seb

Level 41 , Crefeld, Germany

13 February 2020, 12:58

Uuups, sorry - I just realize that I've given you the wrong date format for the output. For the output (displaying the person's infos) it should actually be "MMM dd yyyy". Try that. If it still does not pass, I'll have another look.

Seb

Level 41 , Crefeld, Germany

13 February 2020, 13:05

The output for -i should look like: Washington m Apr 15 1990 Also make sure that you are outputting m or f for sex and not MALE or FEMALE.

Ivan

Level 22 , Nope, Bulgaria

13 February 2020, 14:53

You are right, thanks! Stepping back from programming for 2months is not helpful :( Solution: case "-i" : Person infoPerson = allPeople.get(Integer.parseInt(args[1])); SimpleDateFormat sdf = new SimpleDateFormat("MMM dd yyyy", Locale.ENGLISH); String gender = ""; //sdf.format(infoPerson.getBirthDate()); if(infoPerson.getSex().equals(Sex.MALE)){ gender = "m"; } else { gender = "f"; } System.out.println(infoPerson.getName() + " " + gender + " " + sdf.format(infoPerson.getBirthDate())); break;

Seb

Level 41 , Crefeld, Germany

13 February 2020, 15:59

Ah, no worries - sometimes a little break is necessary. :-) Good to hear you've now found a proper solution. Please set the status of this question to resolved then. That helps others as well. Cheers

Ivan

Level 22 , Nope, Bulgaria

13 February 2020, 16:30

I did my friend. Thank you again, hope you'll stay for a while here to support :) Will be going in hard these days

Seb

Level 41 , Crefeld, Germany

13 February 2020, 16:44

haha - sounds good. :-) And sure, in case there's a bigger problem, you're also welcome to directly get in touch. Bye for now.