Command line seems to work but verify step is failing on create/update

Question about the task

Resolved

I use this is debug input : -c Washington m 04 15 1990 Jefferson f 05 03 1994 BAC m 12 12 2012 and see 3 records get added to allPeople and it prints out 2 3 4 but the verify step doesn't complete without an error. Is there some other condition I am not taking into account? Threading somehow?

Batch CrUD: multiple Creations, Updates, Deletions.

The program runs with one of the following sets of arguments:
-c name1 sex1 bd1 name2 sex2 bd2 ... -u id1 name1 sex1 bd1 id2 name2 sex2 bd2 ... -d id1 id2 id3 id4 ... -i id1 id2 id3 id4 ...

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

The id corresponds to the index in the list.
The birth date format is Apr 15 1990.
All the people should be stored in allPeople.
The order in which data is displayed corresponds to the order in which is input.
Be sure the program is thread safe (works correctly with multiple threads without corrupting the data).
Use Locale.ENGLISH as the second argument for SimpleDateFormat.

Example output for the -i argument with two ids:
Washington m Apr 15 1990 Ross f Apr 25 1997

Requirements:

The Solution class must contain a public volatile List<Person> field called allPeople.
The Solution class must have a static block where two people are added to the allPeople list.
With the -c argument, the program must add all people with the specified arguments to the end of the allPeople list, and display the id of each of them.
With the -u argument, the program must update the data of the people with the specified ids in the allPeople list.
With the -d argument, the program must perform the logical deletion of the people with the specified ids in the allPeople list.
With the -i argument, the program should display data about all the people with the specified ids according to the format specified in the task.
The Solution class's main method must contain a switch statement based on args[0].
Each case label in the switch statement must have a synchronization block for allPeople.

package com.codegym.task.task17.task1711; import java.text.SimpleDateFormat; import java.util.ArrayList; import java.util.Date; import java.util.List; import java.util.Locale; /* CRUD 2 */ public class Solution { public static volatile 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 Exception { SimpleDateFormat sdfOut = new SimpleDateFormat("MMM dd yyyy", Locale.ENGLISH); SimpleDateFormat sdfIn = new SimpleDateFormat("M d yyyy", Locale.ENGLISH); int id; if (args.length < 2) return; switch (args[0]) { case "-i": synchronized (allPeople) { for (int i = 1; i < args.length; i++) { id = Integer.parseInt(args[i]); if (id < allPeople.size()) System.out.println(allPeople.get(id).getName() + " " + (Sex.MALE.equals(allPeople.get(id).getSex()) ? "m" : "f") + " " + sdfOut.format(allPeople.get(id).getBirthDate())); } break; } case "-d": synchronized (allPeople) { for (int i = 1; i < args.length; i++) { id = Integer.parseInt(args[i]); if (id < allPeople.size()) { Person person = allPeople.get(id); person.setBirthDate(null); person.setSex(null); person.setName(null); allPeople.set(id, person); } } break; } case "-u": synchronized (allPeople) { int argCount = 1; while (argCount <= args.length) { id = argCount; String name = args[argCount+1]; String sex = args[argCount+2]; Date bd = sdfIn.parse(args[argCount+2] + " " + args[argCount+3] + " " + args[argCount+4]); if (sex.equals("m")) { allPeople.set(id, Person.createMale(name, bd)); } else { allPeople.set(id, Person.createFemale(name, bd)); } } break; } case "-c": synchronized (allPeople) { int argCount = 1; while (argCount < args.length) { String name = args[argCount]; String sex = args[argCount+1]; Date bd = sdfIn.parse(args[argCount+2] + " " + args[argCount+3] + " " + args[argCount+4]); if (sex.equals("m")) { allPeople.add(Person.createMale(name, bd)); } else { allPeople.add(Person.createFemale(name, bd)); } System.out.println(allPeople.size() - 1); argCount +=5; } break; } } } }

Comments (3)

Popular
New
Old

You must be signed in to leave a comment

Guadalupe Gagnon

Level 37 , Tampa, United States

31 January 2019, 14:50

The problem that I see is that the arg 'birthdate' only takes up one index. For example, with '-c', in the args array the information is set like this: [{0: -c}, {1: name}, {2: sex}, {3: birthday}, {4: name2}, {5, sex2}, {6:birthday2}, etc..] To input these values properly into the IDE for args you need to enter them like this: -c Washington m "04 15 1990" Jefferson f "05 03 1994" BAC m "12 12 2012"

jim smith

Level 22 , Cedar Park, United States

31 January 2019, 15:12

Thanks that did fix it. However in my defense if you read the description of CRUD2 question it doesn't have the date in quotes but in CRUD 1 it did. I thought they must have done that for a reason so I changed my code it explicitly to use them as separate parameters. That is what I get for following the directions. tldr; it verified after making that change and fiddling with the update logic. Thanks a bunch, your answers have helped me immensely to get this far.

Guadalupe Gagnon

Level 37 , Tampa, United States

31 January 2019, 15:15

No need for a defense. I nearly pulled all of my hair out when I solved this myself. My biggest complaint about codegym is the task specifications hardly make any sense, and the examples they give only create more confusion.