07-Proxy, Федоров Андрей, 393 группа

[AndreosGhost]

И да, теперь JSON реализован с нормальными символами экранирования \.

[seregakol007]

DBTableProvider:
1)

final class DBTableProvider implements AutoCloseableProvider {
    public static final char ESCAPE_CHARACTER = '\\';
    private static final char QUOTE_CHARACTER = '\"';
    //    public static final String QUOTED_STRING_REGEX =
    //            Utility.getQuotedStringRegex(QUOTE_CHARACTER + "", ESCAPE_CHARACTER + "" +
    // ESCAPE_CHARACTER);

а также

    public Storeable deserialize(Table table, String value) throws ParseException {
        try (UseLock useLock = validityController.use()) {
            Utility.checkNotNull(table, "Table");
            Utility.checkNotNull(value, "Value");

            //            String partRegex = "null|true|false|-?[0-9]+(\\.[0-9]+)?";
            //            partRegex += "|" + QUOTED_STRING_REGEX;
            //            partRegex = "\\s*(" + partRegex + ")\\s*";
            //            String regex = "^\\s*\\[" + partRegex + "(," + partRegex + ")*" + "\\]\\s*$";
            //
            //            if (!value.matches(regex)) {
            //                throw new ParseException(
            //                        "wrong type (Does not match JSON simple list regular expression)", -1);
            //            }
            //

и

строки 284-356

вроде закомментированный код не принято пулить, либо надо в комментариях объяснить, почему так вышло.
2)

    private static final Collection<Class<?>> SUPPORTED_TYPES = new ConvenientCollection<>(
            new HashSet<Class<?>>()).addNext(Integer.class).addNext(Long.class).addNext(Byte.class)
                                    .addNext(Double.class).addNext(Float.class).addNext(Boolean.class)
                                    .addNext(String.class);

HashSet< Class < ? > > лучше diamond types HashSet<>
3)

    private static final Map<Class<?>, Function<String, Object>> PARSERS =
            new ConvenientMap<>(new HashMap<Class<?>, Function<String, Object>>())

аналогично

[seregakol007]

StoreableTableImpl:

this.columnTypes = Collections.unmodifiableList(new ArrayList<Class<?>>(columnTypes));

желателен diamond types

[seregakol007]

класс ArrayMatcher можно наверно удалить

[seregakol007]

Может я не туда смотрю, но после работы с БД у меня в System.getProperty("user.home") появляется файл с логом, который достаточно далек от заданного формата логов.

[AndreosGhost]

Это мой давнишний лог, отключу его. Лог, который нужен, появляется в той же директории, откуда исполняется программа.

[AndreosGhost]

Закомментированный код просто забыл удалить)

[AndreosGhost]

Diamond types не везде можно применить - не скомилится. Где возможно, применю.

[AndreosGhost]

Пиши, пожалуйста, комментарии к коду в таком стиле, т.к. в таком случае можно писать ответ в том же месте.

[AndreosGhost]

Some fixes & improvements:
#562

[vpavlenko]

Не лучше ли бросать исключение в этом случае? Ведь пользователь библиотеки будет думать, что всё корректно сериализовалось, а на самом деле - нет.

[AndreosGhost]

Есть куча классов из стандартной Java. Я же не могу их всех аннотировать. Но большая их часть выводит нечто осмысленное в toString().