Thoughts? - java help on CodeGym

I'm just throwing solutions at the wall at this point to see what sticks. While loops do not satisfy any of my missing requirements. My real problem is with the 1000 repetitions, all forms of my solution fail to pass Thread.sleep(1) 1000x requirement, though I know they are repeating 1000 times (0-999). My original form for line 38, produced the right format, no real index relation, but will not validate.

addNote(this.getName() + "-Note" + i)

ex: "Thread-0-Note4" Changing line 38 to

addNote(getName() + "-Note" + notes.indexOf(getName())+1);

relates to the notes index proper, and never gets over 0 or 1. And does not validate. ex: "Thread-0-Note1" Changing the addNote method to

public static void addNote(String note, int index) {
    notes.add(index, note);
}

and line 38 to

addNote(getName() + "-Note" + index, index);

Throws errors, but is in the correct format and is related to the index that is actually referenced. Separating the run method with multiple for loops smooths out the errors, but also defeats what I am guessing is the purpose of the lesson. And does not validate either.

Comments (4)

Popular
New
Old

You must be signed in to leave a comment

Michael Martin

Level 19 , Arlington, United States

3 December 2018, 01:42

Don't have the entire method in a try{} block. Only the line Thread.sleep(1) needs to be in the try{} block. Also, you don't need 3 for loops, just do all 3 actions in one loop:

for (int i = 0; i < 1000; i++){
   addNote(getName() + "-Note" + i);
   try {
      Thread.sleep(1);
   } catch (Exception e) {}
   removeNote(getName())
}

Robert P

Level 20 , Nashville, United States

10 December 2018, 20:12

I feel like i tried something like this, but reworked it, and still no progress. If, I could get even one of these conditions to change, I know I would be on the right track. Thanks though

11 December 2018, 13:46

What's your latest code?

jorge ives murillo paredes

Level 40 , New York City, United States

8 October 2019, 23:45

package com.codegym.task.task17.task1701; import java.util.ArrayList; import java.util.List; /* Notes */ public class Solution { public static void main(String[] args) { new NoteThread().start(); new NoteThread().start(); } public static class Note { public static final List<String> notes = new ArrayList<>(); public static void addNote(String note) { notes.add(0, note); } public static void removeNote(String threadName) { String note = notes.remove(0); if (note == null) { System.out.println("Another thread deleted our note"); } else if (!note.startsWith(threadName)) { System.out.println("Thread [" + threadName + "] deleted someone else's note [" + note + "]"); } else { System.out.println("Thread [" + threadName + "] deleted its own note [" + note + "]"); } } } public static class NoteThread extends Thread{ @Override public void run() { for (int i = 0; i < 1000; i++) { Note.addNote ( getName () + "-Note" + i ); try{ Thread.sleep ( 1 ); }catch (InterruptedException o){ } } for(int i = 0 ; i < 1000;i++) { Note.removeNote ( this.getName () ); } } } }