Uwagi do treści projektu

kamirus · 1 Czerwiec 2018 10:01

operacja new nie ma zaktualizowanego parametru emp tak jak w operacji root
operacja new wymaga sprawdzania czy admin jest dzieckiem emp1 albo jego przodkiem, do czego przydaje się zbudowany indeks w pamięci, który chcemy budować nie w fazie init. Czy to zamierzone? Może w fazie init możemy założyć, że ten warunek nie musi być sprawdzany, albo odłożyć jego sprawdzenie na koniec fazy init?

przemek · 1 Czerwiec 2018 14:50

specyfikacja:

[…] admin musi być pracownikiem emp1

Ja rozumiem, że “być pracownikiem emp1” oznacza “być pracownikiem o id emp1”. Gdyby, jak mówisz, znaczyło to “być podwładnym (dzieckiem) pracownika o id emp1”, to pracownik nie mógłby dodawać swoich padawanów; w szczególności szef (korzeń) nie mógłby nikogo zatrudnić.

Ja za to mam inne pytanie: jakiego typu jest identyfikator zadawany w root()?

przemek · 1 Czerwiec 2018 16:00

W specyfikacji nie ma doprecyzowane:

Czy użytkownik init będzie istniał przed pierwszym uruchomieniem programu z argumentem -init?
Czy baza danych, na której będziemy operować też już będzie istniała? Czy może jednak pierwsze-ever wywołanie open(database, …, …) z argumentem -init ma stworzyć bazę o nazwie database?

@mzr

mzr · 1 Czerwiec 2018 19:33

Czy dostępny będze moduł pgcrypto ?

pwi · 2 Czerwiec 2018 06:34

Naniosłem kilka poprawek do treści specyfikacji. Dziękuję za uwagi.

WojtekFica · 2 Czerwiec 2018 09:35

Czy mógłby Pan stworzyć jakieś, niekoniecznie duże, przykładowe testy?
Jest takie przykładowe wejście na stronie:

{ "open": { "baza": "student", "login": "init", "password": "qwerty"}}
{ "new": { "arg1": "value1", "arg2": "value2" } }

Czy funkcja new ma argumenty nazwane arg1, arg2?

przemek · 2 Czerwiec 2018 11:29

I czy argument open to baza jak w przykładzie czy database jak w dalszym opisie?

Co należy zwracać jeżeli zapytamy sie o dane użytkownika, który nie istnieje?
{"status": "OK", "data": "Null"} czy {"status": "ERROR"}?

WojtekFica · 2 Czerwiec 2018 14:52

I czy passwd czy password? Teraz nie wiadomo bo są używane zamiennie.

przemek · 3 Czerwiec 2018 12:01

Małe testy:

gist.github.com

https://gist.github.com/kainoj/26c3184724e48a8da20fe0df82cad34d

test.md

Init data (binary tree)
```json
{ "open": { "database": "student", "login": "init", "password": "qwerty"}}
{ "root": { "secret": "qwerty", "newpassword": "asd", "data": "IAMBOSS", "emp":0} }
{ "new": { "admin": 0, "passwd": "asd", "data":"lvl1", "newpasswd":"asd", "emp1":0, "emp":1 } }
{ "new": { "admin": 0, "passwd": "asd", "data":"lvl1", "newpasswd":"asd", "emp1":0, "emp":2 } }
{ "new": { "admin": 0, "passwd": "asd", "data":"lvl2", "newpasswd":"asd", "emp1":1, "emp":3 } }
{ "new": { "admin": 0, "passwd": "asd", "data":"lvl2", "newpasswd":"asd", "emp1":1, "emp":4 } }
{ "new": { "admin": 0, "passwd": "asd", "data":"lvl2", "newpasswd":"asd", "emp1":2, "emp":5 } }
{ "new": { "admin": 0, "passwd": "asd", "data":"lvl2", "newpasswd":"asd", "emp1":2, "emp":6 } }

This file has been truncated. show original

mzr · 4 Czerwiec 2018 20:58

Relacje descentant i ancestor nie są zwrotne, tak?

pwi · 4 Czerwiec 2018 21:11

Relacje descentant i ancestor nie są zwrotne.

radekmie · 6 Czerwiec 2018 11:16

new <admin> <passwd> <data> <newpasswd> <emp1> <emp> dodawanie nowego pracownika o identyfikatorze <emp> z danymi <data> i hasłem dostępu <newpasswd>, pracownik <emp> staje się podwładnym pracownika <emp1>, <admin> musi być pracownikiem <emp1> lub jego bezpośrednim lub pośrednim przełożonym, <passwd> to hasło pracownika <admin>
// nie zwraca danych

Z opisu wychodzi, że <admin> musi być różny od <emp1> (bo nie jestem swoim pracownikiem ani przełożonym). W takim razie nigdy nie można wykonać new, bo nawet sam root nie może dodać kogoś sobie.

Stąd, przykładowy test jest niezgodny specyfikacją (choć zgodny ze zdrowym rozsądkiem):

{ "root": { "secret": "qwerty", "newpassword": "qwerty", "data": "dane prezesa", "emp":0} }
{ "new": { "admin": 0, "passwd": "123456", "data":"dane wiceprezesa", "newpasswd":"qwerty", "emp1":0, "emp":1 } }

Rozumiem, że jednak powinno być to możliwe i mogę sam sobie dodać podwładnego, tak?

kamirus · 6 Czerwiec 2018 12:56

<admin> musi być pracownikiem o identyfikatorze <emp1>

mbuszka · 9 Czerwiec 2018 15:01

@pwi Czy program musi weryfikować poprawność wejścia (brakujące argumenty, niepoprawny json itp)?

pwi · 10 Czerwiec 2018 07:39

Wejście będzie zawsze poprawne od strony składniowej (json będzie poprawny, będą podane wszystkie argumenty i będą odpowiednich typów).
Może się zdarzyć, że operację będzie próbował wykonać ktoś nieuprawniony, ktoś poda nieprawidłowe hasło itp.