Написание метода с ArrayList строк в качестве параметров

Эта...

if(n.length() ==4){
    themarklength.add("****");
}

Просто пытается добавить "****" в конец списка. Это терпит неудачу, потому что Iterator используемый циклом for-each, не позволит изменениям в базовом List когда он был повторен.

Вы можете сначала создать копию List...

List<String> values = new ArrayList<String>(themarklength);

Или преобразовать его в массив String

String[] values = themarklength.toArray(new String[themarklength.size()]);

И использует их, как вы итерации точки...

for (String value : values) {

Затем вам нужно будет вставить новый элемент в ArrayList в определенную точку. Для этого вам нужно знать исходный индекс значения, с которым вы работаете...

if (value.length() == 4) {
    int index = themarklength.indexOf(value);

А затем добавьте новое значение в нужное место...

    themarklength.add(index, "****");

Это добавит "****" в index точку, нажав все остальные записи вниз

обновленный

Как было правильно указано на мне, использование themarklength.indexOf(value) не будет учитывать случай использования, когда список themarklength содержит два элемента одного и того же значения, которые возвращают неверный индекс.

Я также не фокусировался на производительности в качестве основного требования для обеспечения возможного решения.

Обновление...

Как указывалось JohnGarnder и AnthonyAccioly, вы можете использовать for-loop вместо for-each который позволит вам обойтись без themarklength.indexOf(value)

Это устранит риск дублирования значений, испортивших местоположение индекса и улучшающих общую производительность, поскольку вам не нужно создавать второй итератор...

// This assumes you're using the ArrayList as the copy...
for (int index = 0; index < themarklength.size(); index++) {
    String value = themarklength.get(index);
    if (value.length() == 4) {
        themarklength.add(index, "****");
        index++;

Но что вы используете, зависит от вас...

Проблема в том, что в вашем методе вы не изменяли каждую строку в arraylist, а добавляли только 4 звезды к списку. Итак, правильный способ сделать это - вам нужно изменить каждый элемент массива и заменить старую строку на новую:

void marklength4(ArrayList<String> themarklength){
    int index = 0;
    for(String n : themarklength){
        if(n.length() ==4){
            n = "****" + n;
        }
        themarklength.set(index++, n);
    }
    System.out.println(themarklength);
}

Если это не то, что вам нужно, но вы хотите добавить новую строку "**" перед каждым элементом в arraylist, вы можете использовать метод listIterator в ArrayList чтобы добавить новый дополнительный элемент до строки EACH, если длина равна 4.

    ListIterator<String> it = themarklength.listIterator();
    while(it.hasNext()) {
        String name = it.next();
        if(name.length() == 4) {
            it.previous();
            it.add("****");
            it.next();
        }
    }

Разница заключается в следующем: ListIterator позволяет вам изменять список при повторении через него, а также позволяет вам вернуться назад в список.

Я бы использовал ListIterator вместо a для каждого, listiterator.add скорее всего сделает именно то, что вы хотите.

public void marklength4(List<String> themarklength){
    final ListIterator<String> lit = 
        themarklength.listIterator(themarklength.size());
    boolean shouldInsert = false;   
    while(lit.hasPrevious()) {
        if (shouldInsert) {
            lit.add("****");
            lit.previous();
            shouldInsert = false;
        }
        final String n = lit.previous();
        shouldInsert = (n.length() == 4);
    }
    if (shouldInsert) {
        lit.add("****");
    }
}

---

Рабочий пример

сделать временный arraylist, изменить этот список и скопировать его содержимое в конце в исходный список

    import java.util.ArrayList;

    public class MarkLength {
    void marklength4(ArrayList <String> themarklength){
        ArrayList<String> temp = new ArrayList<String>();
        for(String n : themarklength){
            if(n.length() ==4){
                temp.add(n);
                temp.add("****");
            }
        }
        themarklength.clear();
        themarklength.addAll(temp);
        System.out.println(themarklength);
    }
}

О, я помню эту прекрасную ошибку из старых добрых дней. Проблема заключается в том, что ваш ArrayList не полностью заполнен к тому моменту, когда элемент массива должен быть доступен. Подумайте об этом, вы создаете объект, а затем сразу начинаете его зацикливать. Следовательно, объект должен заполнять себя значениями, когда цикл будет запущен.

Простой способ решить это - предварительно заполнить свой ArrayList.

public class MarklengthTestDrive {
    public static void main(String[] args){

        ArrayList <String> words = new ArrayList<String>() {{ 

        words.add("Kane");
        words.add("Cane");
        words.add("Fame");
        words.add("Dame");
        words.add("Lame");  
        words.add("Same");
        }};
    }
}

Скажите мне, исправляет ли это это. Вы также можете использовать static инициализатор.