I don't understand why this did not function..

Question about the task

Under discussion

Please help.

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 = fix(list); for (String s : list) { System.out.println(s); } } public static ArrayList<String> fix(ArrayList<String> list) { ArrayList<String> copy = new ArrayList<>(); copy.addAll(list); for (int i = 0; i <copy.size() ; i++) { String[] word = list.get(i).split(""); for (int j = 0; j < word.length; j++) { if ( (!word[j].equalsIgnoreCase("r")) || (!word[j].equalsIgnoreCase("l")) || (!word[j].equalsIgnoreCase("r")) || (!word[j].equalsIgnoreCase("l")) ) { if ((word[j].equalsIgnoreCase("r")) && (!word[j].equalsIgnoreCase("l"))) { copy.remove(i); } else if ((word[j].equalsIgnoreCase("l")) && (!word[j].equalsIgnoreCase("r"))) { System.out.println("entra 3"); System.out.println(copy.get(i)); String temp = list.get(i); copy.add(temp); } } } } // write your code here return copy; } }

Comments (1)

Popular
New
Old

You must be signed in to leave a comment

Guadalupe Gagnon

Level 37 , Tampa, United States

29 November 2019, 01:20

A different way to solve, and should be easier to code. The code above already makes a new array at line 24, instead of adding all the data from list to it at line 25 do this instead: Make one loop that loops through list and for each word a) does nothing for words that have r but no L (use continue) else b) adds words that contain L but don't contain r and outside the if statement adds all other words. The code:

foreach (word in list){
    if (word contains r but not l)
      continue;
    else if (word contains l but not r)
      copy.add(word);

   copy.add(word);
}

That code will solve this entire task and is simple.