need a nudge

Question about the task

R or L

Java Syntax, Level 7, Lesson 9

Resolved

got the right output and should meet the second criteria, but it says I don't.

1. Create a list of words and populate it yourself.
2. The fix method should:
2.1. remove all words containing the letter "r" from the list
2.2. duplicate all words containing the letter "l".
2.3. if a word contains both "r" and "l", then leave it unchanged.
2.4. don't do anything to other words.

For example:
rose lyre love

Output:
lyre love love

Requirements:

The program must not read data from the keyboard.
The fix method should remove all words containing the letter "r" from the list. There is an exception: words containing both "r" and "l" should be left alone.
The fix method must duplicate words containing the letter "l" (add another instance of this word to the list). There is an exception: words containing both "l" and "r" should not be duplicated.
The fix method should not do anything with words that contain both "l" and "r".
The fix method should not do anything with words that don't contain either "l" or "r".

package com.codegym.task.task07.task0716; import java.util.ArrayList; /* R or L */ public class Solution { public static void main(String[] args) throws Exception { ArrayList<String> list = new ArrayList<String>(); list.add("rose"); // 0 list.add("love"); // 1 list.add("lyre"); // 2 list.add("meat"); list.add("potatos"); list = fix(list); for (String s : list) { System.out.println(s); } } public static ArrayList<String> fix(ArrayList<String> list) { // write your code here ArrayList<String> x = new ArrayList<String>(); x=list; for (int i=0; i<x.size(); i++){ String g = x.get(i); if (!(g.contains("l")) && (g.contains("r"))) x.remove(g); else ; } for (int i=0; i<x.size(); i++){ String b = x.get(i); if (b.contains("l") && (!(b.contains("r")))){ x.add(i, b); i+=2; } } return x; } }

Comments (3)

Popular
New
Old

You must be signed in to leave a comment

Rich

Level 23 , San Diego, United States

4 June 2020, 21:16

solution

that was it, it wanted me to include the check for both letters instead of implying that it be left alone. great help! thank you

Ashish RajAnand

Level 13 , Bhilai , India

4 June 2020, 02:32

I see you are not checked for if string contain both character l and r.

Lucas Hoage

Level 14 , Savannah, United States

4 June 2020, 01:02

I'm sure there's a better way to deal with this. I'll tell you how I did it. I created a new ArrayList in the beginning, Just like you did. I did not assign my new list as the parameter (list) that was passed into the method. I created 4 conditions: if else if else if else The conditions were based on the values that were in the (list) that was passed into the parameter. If the condition was one that I wanted to keep, I added it to my new ArrayList If I had to remove something, I did so, but then subtracted the value of i like so : i-- This was to account for the array values shifting left. If I did not do this, the next value could very well be null or something entirely different. Once the shift happens, Whatever is in (example) index 2 will go to index 1. At this point I don't believe you've been taught that foundational principle. I think that comes at the end of chapter 8 in an article. Or rather, it's much easier to understand with the pictures. https://codegym.cc/groups/posts/21-arraylist-in-pictures-- My if/EIF/EIF/E condition was set up to first: Check double values Check Opposite Check single value Check Opposite By doing the bulk first and the single conditions after, I was guaranteed to have the remaining values left that I needed. When a value needed to be doubled, I literally added it twice. At the end, I assign the value of list with new arraylist. By this point, the new arraylist has been sanitized to contain exactly what I want to pass back to the call. I hope it helps break down some of the ideas behind solving it and why it works that way. It might also not like you adding more words than what you started off with.