Help with validation

Question about the task

CRUD 2

Java Core, Level 7, Lesson 10

Under discussion

Hey guys, just wondering if anyone can spot a bug in my code which isn't allowing validation? Thanks so much

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.ParseException; 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 ParseException { // Start here if (args.length != 0){ switch(args[0]){ case "-c": synchronized (allPeople) { int sex = 2; int db = 3; for (int i = 1; i < args.length; i += 3) { Date date = new SimpleDateFormat("dd mm yyyy", Locale.ENGLISH).parse(args[db]); if (args[sex].equals("m")) { allPeople.add(Person.createMale(args[i], date)); } else if (args[sex].equals("f")) { allPeople.add(Person.createFemale(args[i], date)); } System.out.println(allPeople.size() - 1); sex += 3; db += 3; } break; } case "-u": synchronized (allPeople) { int name = 2; int sex2 = 3; int db2 = 4; for (int i = 1; i < args.length; i += 4) { Person updatePerson = allPeople.get(i); Date date2 = new SimpleDateFormat("dd mm yyyy", Locale.ENGLISH).parse(args[db2]); updatePerson.setName(args[name]); updatePerson.setBirthDate(date2); if (args[sex2].equals("m")) { updatePerson.setSex(Sex.MALE); } else if (args[sex2].equals("f")) { updatePerson.setSex(Sex.FEMALE); } name += 4; sex2 += 4; db2 += 4; } break; } case "-d": synchronized (allPeople) { for (int i = 1; i < args.length; i++) { Person delete = allPeople.get(i); delete.setSex(null); delete.setBirthDate(null); delete.setName(null); } break; } case "-i": synchronized (allPeople) { SimpleDateFormat sdf = new SimpleDateFormat("MMM dd yyyy", Locale.ENGLISH); for (int i = 1; i < args.length; i++) { Person print = allPeople.get(i); String name2 = print.getName(); String gender = ""; if (print.getSex().equals(Sex.MALE)) { gender = "m"; } else { gender = "f"; } System.out.println(name2 + " " + gender + " " + sdf.format(print.getBirthDate())); } break; } } } } }

Comments (1)

Popular
New
Old

You must be signed in to leave a comment

Thomas

Level 41 , Bayreuth, Germany

22 June 2020, 07:34

One problem may be

Person print = allPeople.get(i);

this is always 1 (here, as you start the loop with 1 and the list.size() is 2). You sould do something like

Person print = allPeople.get(Integer.parseInt(args[1]));

Then you get (for a start) the correct selection