Skip to content
This repository was archived by the owner on Sep 27, 2022. It is now read-only.

Дмитрий Морозов, 396, JUnit#541

Open
annoing-morda wants to merge 10 commits intodkomanov:masterfrom
annoing-morda:master
Open

Дмитрий Морозов, 396, JUnit#541
annoing-morda wants to merge 10 commits intodkomanov:masterfrom
annoing-morda:master

Conversation

@annoing-morda
Copy link

Исправил согласно комментариям к предыдущему заданию. Только интерпретатор реализован не как набор классов, а как один класс, в котором есть метод handle, осуществляющий проверку корректности команды и выполнящий запуск соответвующего хэндлера.

@vpavlenko vpavlenko self-assigned this Dec 15, 2014
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

emergency - там нет "e" после "r" )

@TafTafa
Copy link

TafTafa commented Dec 20, 2014

Usage: java -Ddb.file=. Your path could have had incorrect value
Exception in thread "main" java.lang.IllegalArgumentException
at ru.fizteh.fivt.students.dmitry_morozov.junit.interpreter.DBInterpreter.(DBInterpreter.java:31)
at ru.fizteh.fivt.students.dmitry_morozov.junit.Main.main(Main.java:76)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:483)
at com.intellij.rt.execution.application.AppMain.main(AppMain.java:120)

Process finished with exit code 1

Не надо показывать Stacktrace

@TafTafa
Copy link

TafTafa commented Dec 20, 2014

Вроде как задание назад мы стали вот так определять директорию: System.getProperty("fizteh.db.dir").

@TafTafa
Copy link

TafTafa commented Dec 20, 2014

$ afa
No such command: $

Наверное мелочь, но надо на новую строку переносить $, более того "No such command: " как то странно смотрится.

@TafTafa
Copy link

TafTafa commented Dec 20, 2014

Еще мелочь:
$ put a a
Table not chosen
$

В задании прописано, что корректно no table отвечать в таких случаях

@TafTafa
Copy link

TafTafa commented Dec 20, 2014

$ use a
anot exists

Добавь пробел

@TafTafa
Copy link

TafTafa commented Dec 20, 2014

$ create a
created
$ use a
using a
$ put aaa
Not enough parametres for command put
$ use v
vnot exists
$ put a a
Table not chosen
$

Неправда же, таблица выбрана. Это таблица a

@TafTafa
Copy link

TafTafa commented Dec 20, 2014

$ remove a
removed
$ commit
2
$ list
a

Мы же только что удалили а.

@ElinRin
Copy link
Contributor

ElinRin commented Dec 22, 2014

$ put e d
no table
$ create t
created
$ use t
using t
$ put 4 f
new
$ put 4 h
overwrite
f
$ put 3 n
new
$ rollback
3
$ list
$ show tables
t 0
$ put 4 j
new
$ put 4 j
overwrite
j
$ put 2 m
new
$ remove 4
removed
$ commit
4

Фактически тут только одно изменение

@ElinRin
Copy link
Contributor

ElinRin commented Dec 22, 2014

Создалось 16 директорий и ни в одной не было данных.

@ElinRin
Copy link
Contributor

ElinRin commented Dec 22, 2014

$ create T
Error occured wrong tablename

не уверена, что t и Т одинаковые имена

@ElinRin
Copy link
Contributor

ElinRin commented Dec 22, 2014

Пусты заготовки, 16 директорий, создаются ещё на моменте создания таблицы

@ElinRin
Copy link
Contributor

ElinRin commented Dec 22, 2014

$ create R
created
$ put 1 f
no table
$ use R
using R
$ put 5 g
new
$ put 7
Not enough parametres for command put
$ put 8 h
new
$ list
5, 8
$ commit
2
$ list
5, 8
$ exit
good bye

Process finished with exit code 0

$ use R
using R
$ list
$ exit
good bye

Process finished with exit code 0

Ничего не сохранилось =(

@annoing-morda
Copy link
Author

@ElinRin Про директории: они все равно удаляются при закрытии. Про t и T: в Windows имена файлов не чувствительны к регистру. Поэтому t и T там одинаковые имена. Вот у меня рабоатет, напрмер.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants