Добрый вечер!
В регулярные выражения сознательно не лезу. Они мне тяжело идут, по-этому решала по-старинке. Мне кажется, что программа работает верно, но валидатор не согласен. Код вроде простой, будет обидно, если там в какой-то глупости ошибка. Но еще обиднее будет, если логика в корне не верная(
Заранее спасибо за советы😊
package com.javarush.task.task19.task1918;
/*
Знакомство с тегами
*/
import java.io.BufferedReader;
import java.io.FileReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.List;
public class Solution {
public static void main(String[] args) throws IOException {
String tag = args[0];//"a";//
BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
////"C:/ЛАДА/102.txt"
BufferedReader file = new BufferedReader(new FileReader(reader.readLine()));
reader.close();
String sum = "";
while (file.ready()) {
String line = file.readLine();
sum = sum + line;
}
String finish = "";
String[] list = sum.split("</" + tag + ">");
List<String> list2 = new ArrayList<>();
for (int i=0; i<list.length; i++) {
list[i] = list[i] + "</" + tag + ">";
if (list[i].contains("<" + tag) && list[i].contains("</" + tag + ">")) {
int index = list[i].indexOf("<" + tag);
int indexEnd = list[i].indexOf("</" + tag);
finish = list[i].substring(index, indexEnd) + "</" + tag + ">";
list2.add(finish);
}
else list2.add(list[i]);
}
for (int i=0; i<list2.size(); i++) {
if (!list2.get(i).contains("<" + tag)) {
String s = list2.get(i-1) + list2.get(i);
list2.remove(i);
list2.remove(i-1);
list2.add(i-1,s);
}
}
for (String s : list2){
System.out.println(s); }
file.close();
}
}
Дальше вы обрезаете по индексам и получаете (синий цвет), а дальше еще раз добавляете искусственно "окончание тега". ( И тут я уже устал считать костыли)
Проблема вашего кода в том, что он не учитывает ситуацию, когда тег span находится внутри тега span, а задача требует, чтоб вы это учитывали.
строчка 44-50) ( очередной костыль) это тоже не безопасный кусок кода ( подумайте почему подсказка: связано с индексами массива)
По итогу: ваш код не учитывает, что внутри
< span xml:lang="en" lang="en">< span>Turanga Leela< /span >< /span >
есть еще один span:
< span>Turanga Leela< /span >
который тоже нужно вывести.
Как исправить? Есть множество гибких и хороших решений ( думаю, вы сами догадаетесь ).
Не гибким но простым в понимании и реализации является решение:
1) Линейно пробегать вашу строчку
2) когда встречается < span > "сохранять данные в буфер"
3) если встретился 1 < span >, но прошлый еще не закрылся использовать флаг для пропуска
< /span > ( в теории, span внутри span может быть много. Как правильно использовать флаг?)
4) после "нужного " < /span> "возвращаться внутрь" текущего span если внутри него попался span
Старайтесь избегать костылей и "небезопасных" кусков кода
Что выводится у вас:
Ну... видно, что мы теряем
< span >Turanga Leela< /span >
Теперь давайте по коду:
Я не буду придираться к реализации, просто дам пару советов:
строчка 17: вы открываете поток "считывания данных" на 17 строчке, а закрываете на 20. Лучше его закрыть на 18 строчке ( это условности написания более красивого кода)
строчка 19: вы открываете поток "считывания данных из файла" на 19 строчке, а закрываете вообще на 56. Ну это уже совсем никуда не годится и закрыть стоит на 20;
22 строчка - 27 строчка: обещал не придираться к реализации, но стоит упомянуть, что такая реализация может привести к проблем. В данной задаче размер входных данных небольшой и это не приведет в проблеме, но что будет если у нас не пару строчек, а огромный файл? String не резиновый и, при большом файле, конкатенация может привести к тому, что размера string не хватит ( а что тогда случится? погуглите)
30 строчка: вы "обрезаете" строчку по тегу, в итоге list[] имеет следующие элементы:
