Анализ кода парсера

@saphira777 · Регистрация: 13.02.2015

Студворк — интернет-сервис помощи студентам

Ребята помогите провести анализ парсера

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
 import java.io.File;
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.IOException;
/**
 * This class is thread safe.
 */
public class Parser {
  private File file;
  public synchronized void setFile(File f) {
    file = f;
  }
  public synchronized File getFile() {
    return file;
  }
  public String getContent() throws IOException {
    FileInputStream i = new FileInputStream(file);
    String output = "";
    int data;
    while ((data = i.read()) > 0) {
      output += (char) data;
    }
    return output;
  }
  public String getContentWithoutUnicode() throws IOException {
    FileInputStream i = new FileInputStream(file);
    String output = "";
    int data;
    while ((data = i.read()) > 0) {
      if (data < 0x80) {
        output += (char) data;
      }
    }
    return output;
  }
  public void saveContent(String content) throws IOException {
    FileOutputStream o = new FileOutputStream(file);
    for (int i = 0; i < content.length(); i += 1) {
      o.write(content.charAt(i));
    }
  }
}

@turbanoff · 03.03.2017, 11:59

Что значит "провести анализ"? Сказать работает ли он или нет?

@xoraxax · 03.03.2017, 12:14

если я все правильно понимаю, то этот код ужасен и его надо удалить и больше никому не показывать.

@korvin_ · 03.03.2017, 15:40

Сообщение от saphira777

Ребята помогите провести анализ парсера

И что этот "парсер" парсит?

@ntlinuxnt · 03.03.2017, 17:02

xoraxax, Чем он по вашему ужасен? Методы не большие и выполняют нужный функционал, чем он плох объясните?

@xoraxax · 03.03.2017, 18:23

Не по теме:

Сообщение от ntlinuxnt

Чем он по вашему ужасен?

вот зачем там, например synchronized написан? Да и, скорее всего, код можно сократить до пары строк.

@ntlinuxnt · 03.03.2017, 18:26

Не по теме:

Сообщение от xoraxax

вот зачем там, например synchronized написан? Да и, скорее всего, код можно сократить до пары строк.

Но мы же не знаем где оно используется? Возможно он грузит несколько файлов многопоточно извне.

@xoraxax · 03.03.2017, 18:32

Не по теме:

Сообщение от ntlinuxnt

Возможно он грузит несколько файлов многопоточно извне.

возможно тогда синхронизировать надо не только геттеры и сеттеры?

@ntlinuxnt · 03.03.2017, 18:35

Не по теме:

Сообщение от xoraxax

возможно тогда синхронизировать надо не только геттеры и сеттеры?

Возможно, но без пояснений автора, где используется этот код мы этого не узнаем:(

@KEKCoGEN · 03.03.2017, 18:51

ntlinuxnt, xoraxax прав. В синхронизации нет смысла. Состояние объекта файл может быть изменено после того как сделали set и до того как начали читать.

@korvin_ · 03.03.2017, 19:32

Сообщение от xoraxax

вот зачем там, например synchronized написан?

А ещё ТС не знает, что такое Unicode и что возвращает FileInputStream.read

Либо, если так сделано намеренно, это ну очень странный код.

Новые блоги и статьи Все статьи Все блоги /
Автоматическое создание документа при проведении другого документа Maks 29.03.2026 Реализация из решения ниже выполнена на нетиповых документах, разработанных в конфигурации КА2. Есть нетиповой документ "ЗаявкаНаРемонтСпецтехники" и нетиповой документ "ПланированиеСпецтехники". В. . .	Настройка движения справочника по регистру сведений Maks 29.03.2026 Решение ниже реализовано на примере нетипового справочника "ТарифыМобильнойСвязи" разработанного в конфигурации КА2, с целью учета корпоративной мобильной связи в коммерческом предприятии. . . .	Автозаполнение реквизита при выборе элемента справочника Maks 27.03.2026 Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. При выборе "Спецтехники" (Тип Справочник. Спецтехника), заполняется. . .	Сумматор с применением элементов трёх состояний. Hrethgir 26.03.2026 Тут. https:/ / fips. ru/ EGD/ ab3c85c8-836d-4866-871b-c2f0c5d77fbc Первый документ красиво выглядит, но без схемы. Это конечно не даёт никаких плюсов автору, но тем не менее. . . всё может быть. . .
Автозаполнение реквизитов при создании документа Maks 26.03.2026 Программный код из решения ниже размещается в модуле объекта документа, в процедуре "ПриСозданииНаСервере". Алгоритм проверки заполнения реализован для исключения перезаписи значения реквизита,. . .	Команды формы и диалоговое окно Maks 26.03.2026 1. Команда формы "ЗаполнитьЗапчасти". Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. В качестве источника данных. . .	Кому нужен AOT? DevAlt 26.03.2026 Решил сделать простой ланчер Написал заготовку: dotnet new console --aot -o UrlHandler var items = args. Split(":"); var tag = items; var id = items; var executable = args;. . .	Отправка уведомления на почту при создании или изменении элементов справочника Maks 24.03.2026 Программная отправка письма электронной почты на примере типового справочника "Склады" в конфигурации БП3. Перед реализацией необходимо выполнить настройку системной учетной записи электронной. . .

@turbanoff 4094 / 3828 / 745 Регистрация: 18.05.2010 Сообщений: 9,331 Записей в блоге: 12
	03.03.2017, 11:59
	Что значит "провести анализ"? Сказать работает ли он или нет? 0

@xoraxax 3639 / 2971 / 918 Регистрация: 05.07.2013 Сообщений: 14,220
	03.03.2017, 12:14
	если я все правильно понимаю, то этот код ужасен и его надо удалить и больше никому не показывать. 1

@ntlinuxnt $ su 1605 / 520 / 97 Регистрация: 18.11.2010 Сообщений: 2,807 Записей в блоге: 2
	03.03.2017, 17:02
	xoraxax, Чем он по вашему ужасен? Методы не большие и выполняют нужный функционал, чем он плох объясните? 0

@KEKCoGEN 2399 / 2224 / 565 Регистрация: 28.12.2010 Сообщений: 8,672
	03.03.2017, 18:51
	ntlinuxnt, xoraxax прав. В синхронизации нет смысла. Состояние объекта файл может быть изменено после того как сделали set и до того как начали читать. 0