-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upravit adresu #7
Comments
No tak můžeme udělat že uživatel může být a nemusí. Nebo jiné řešení? Adresa by měla být asi jen jedna ne? |
Povolit ve sloupci user_id null je asi nejsnadnější řešení. Ale moc se mi nelíbí. Nelíbí se mi tam ani sloupce jméno a příjmení - protože když tam chci vložit adresu pobočky tak ta žádné jméno ani příjmení nemá. Jméno a příjmení by se podle mě mělo brát odjinud. Nicméně asi nemá smysl to překopávat dokud nemáme splněné zbylé body ze zadání.. |
Aha, ale jak by si to řešil jinak, jednodušeji? Takhle nějak?
Users
Names
Organizations
Asi spíš jináč :) |
Za mě se adresa skládá z města, ulice a psč. Ještě by se tam dalo spekulovat o nějakém oslovení (což by mohlo být jméno osoby nebo firmy) Žádné IČ ani DIČ bych tam ale nedával - z logiky věci IČ a DIČ jsou údaje patřící nějaké firmě/osobě. V našem systému by to mohl být třeba uživatel. Asi bych nevytvářel novou entitu Organizations - pro zjednodušení bych ty údaje prdnul do usera. A ani mi nedává smysl mít firstName a lastName v samostatné tabulce. Za mě tedy spíš takto: Address
Users
UserAddress
Jméno a příjmení bych klidně bral z usera. Osobně bych se na IČ a DIČ vykašlal pokud to teda není v zadání.. Teď když už je to napsané a už nám nezbývá moc času do odevzdání, tak to asi už ale nemá cenu přepisovat. |
Ok, a umoňovalo by to pak jednomu uživateli dát více adres včetně jiného (odebírá příbuzný, firma je evidovaná na někoho jiného atp.)? |
Nyní je adresa navázána na uživatele což není správně. Když pak chci vytvořit pobočku a přiřadit jí adresu tak musím vytvořit i nového uživatele.
The text was updated successfully, but these errors were encountered: