Програма за магазин

[Meniak]

Реших с наученото до сега за Java (до колко правилно и до колко кривилно вие кажете) да напиша някаква програмка.

Племеницата ми има магазинче - железария - и артикулите ги описваше на един екселски файл като сама смяташе отписваше дописваше и т.н.т. когато продава или доставя артилули. Аз я попитах защо не използва някоя складова програма? има ги безплатни в интернет а тя каза че са прекалено заангажираци, на нея и трябвало нещо по просто.
И реших аз да напиша нещо просто което да е като екселския файл ама поне изчисленията да прави само.
Написах тази програмка която споделям с вас и и я дадох да я тества под предлог че не съм сигурен това дали ще сработи но не стига че работелело правилно ми и неини колеги на който тя се похвалила я искаха да си работят с нея.

Та понеже интереса кам тази програмка ме запали да си я развивам искам да попитам за вашето мнение за кода, за програмата, изобщо тръгва ли на вашите компщтри, за абсолючно всичко. И втората причина поради която искам да си развивам програмката е че тя е първото мое проектче което не е мъртво родено а вижда реализация за разлика от доста код дето сам изписал до сега.

Програмата е писана на Java и изискава инсталиран JDK минимум 1.8.0
Програмата не е тествана какработи под Linux и се съмнявам да няма кривинки в дизайна на прозорците.

Линк за сваляне: http://dox.bg/files/dw?a=dc41d88dad

Линк към Github: https://github.com/Meniak/Magazin в папката src се намира кода на програмата - NetBeans качи целия проект

[Wanderer]

Не съм ти пробвал приложението, но кода ти е ужасен! 

[Meniak]

Много ше съм благодарен. Не съм на ясно с много неща и искам да ги ознам.

[HD]

@Wanderer a твоя първи проект как изглеждаше?
@Маniak браво за положените усилия. Прави добро впечатление, че си направил github репо, което е нетипично за прохождащи в занаята.

[Meniak]

Здравейте. Качвам представям подобрена версия на програмата ми.

Новото в програмата е че когато в сметка имаме записи надписа на бутона сметка става червен и не е нужно да проверяваме всеки път дали имаме записи в сметката.

Всъщност целта на бутона сметка е да избегнете работата с калкулатор за да изчислите сумата която трябва да плати клиента, просто продавате артикулите и те се добавят към сметка след като клиента приключи с поръчките отваряте сметка и виждате каква е сумата за плащане на всеки един артикул по единично и общо.

Другото ново нещо е че програмата копира папката Store ( която всъщност е базата данни на програмата ) в папката Backup при всяко стартиране на програмата и ако базата данни на програмата се повреди ( както вече ми се случи ) можем да си заредим работеща база данни от последното включване на програмата.

Програмата вече е и с инсталатор.

Програмата е лицензирана под GNU GPL лиценз и изисква инсталиран JDK 1.8.0 минимум.

Линк за сваляне  :  http://dox.bg/files/dw?a=cdffdd8ba4

Линк към Github  :  https://github.com/Meniak/Magazin

Кода все още ми е ужасно написан защото не знам как да го напиша както трябва.

[HanKrum]

Дай шорткъти скрийншоти за да видим как изглежда и ко има вътре.

[Wanderer]

@Wanderer a твоя първи проект как изглеждаше?

И моя, то между другото все още си ми е ужасен 

Дай шорткъти за да видим как изглежда и ко има вътре

Да нямаш предвид screenshots ?


-----------------------------------------------------------------------------------------------------------------
Защо кода  е лош:

-Не е ООП
--кода е само статични методи
-- дори тези които не са статични - могат да бъдат, защото “класовете“ ти нямат никакво вътрешно състояние например виж Reports.java, Edit.java, Help.java  и други
--някои класове имат ужасни имена - DS, DD, VP, BackupDatabase  Името на класовете трябва да е съществително
--защо гореизброените класове ползват Simple-Whatever-Property вместо string, double , etc ? Така и така получават примирно string и връщат string - от какъв зор тогава ползваш SimpleStringProperty ?
--getter, setter са antipatterns, нищо, че java beans са изградени върху тях. Правилото е “TELL! DON'T ASK!" - или казано с повече думи - класовете, които съдържат информацията трябва да вършат работата.

По-късно може да добавя още неща

[Meniak]

[HanKrum]

Изглежда доста прилично

[Meniak]

Мерси аз по принцип не мога да правя дизайни защото нямам усет да направя нещо красиво, но щом това ти харесва значи имам напредък. Епа ако се барне това и с малко CSS ще светне.

[Meniak]

Защо кода  е лош:

-Не е ООП
--кода е само статични методи
-- дори тези които не са статични - могат да бъдат, защото “класовете“ ти нямат никакво вътрешно състояние например виж Reports.java, Edit.java, Help.java  и други
--някои класове имат ужасни имена - DS, DD, VP, BackupDatabase  Името на класовете трябва да е съществително
--защо гореизброените класове ползват Simple-Whatever-Property вместо string, double , etc ? Така и така получават примирно string и връщат string - от какъв зор тогава ползваш SimpleStringProperty ?
--getter, setter са antipatterns, нищо, че java beans са изградени върху тях. Правилото е “TELL! DON'T ASK!" - или казано с повече думи - класовете, които съдържат информацията трябва да вършат работата.

По-късно може да добавя още неща

Благодаря че ме насочи какво да гледам.

[jazzman]

Няма какво да добавя щото съм далеч от Java, но браво за усилията които полагаш, виждам резултати и още..... благодаря, че пишеш и споделяш в научиМе.

[HD]

Браво за усилията и от мен. Хубаво изглежда на външен вид.  Само остава да изпълниш съветите на @wanderer,  които общо взето важат за всеки един език за програмиране с обектен модел. Давай смело и продължавай напред. Пък с каквото може ще помагаме.

[Meniak]

Мерси много. работа по съветите на @wanderer но ми е трудно да асимилирам това

Защо кода  е лош:

-Не е ООП
--кода е само статични методи
-- дори тези които не са статични - могат да бъдат, защото “класовете“ ти нямат никакво вътрешно състояние например виж
Reports.java, Edit.java, Help.java  и други.

[Wanderer]

Съществуват няколко подхода (ако може да се нарекат така) към програмирането:
-процедурно - данните и логиката , която се изпълнява върху тях са разделени.
-ООП - данните и логиката, която се изпълнява върху тях са едно.

Двата подхода имат различни плюсове и минуси. Например при ООП е много по-лесно да дефинираш нов тип данни (клас) или да промениш вече съществуващ, но пък е много по-трудно да добавиш логика, която се изпълнява върху него. Докато при процедурното програмиране е обратното. Там е много по лесно да добавиш нова функция, която работи върху дадена структура, отколкото да промениш вече съществуваща структура.

Виждам, че си преименувал файловете, та сега ми е трудно да се ориентирам Нека разгледаме:

Код: Java

1. public class ReportsWindow {
2.  
3.     public void referenceSales() throws IOException {
4.         Parent root = FXMLLoader.load(getClass().getResource("/screens/DailySalesScreen.fxml"));
5.         Stage stage = new Stage();
6.         Scene scene = new Scene(root);
7.  
8.         stage.setTitle("Справка дневни продажби");
9.         stage.initModality(Modality.APPLICATION_MODAL);
10.         stage.setResizable(false);
11.         stage.initStyle(StageStyle.UTILITY);
12.         stage.setScene(scene);
13.         stage.show();
14.     }
15.  
16.     public void referenceSupply() throws IOException {
17.         Parent root = FXMLLoader.load(getClass().getResource("/screens/DailyDeliveriesScreen.fxml"));
18.         Stage stage = new Stage();
19.         Scene scene = new Scene(root);
20.  
21.         stage.setTitle("Справка дневни доставки");
22.         stage.initModality(Modality.APPLICATION_MODAL);
23.         stage.setResizable(false);
24.         stage.initStyle(StageStyle.UTILITY);
25.         stage.setScene(scene);
26.         stage.show();
27.     }
28. }
29.  

Защо горните два метода не са статични ? Защо трябва да направя инстанция на класа за да ги използвам ?
Също се забелязва 100% дублиране на код. Така е по добре, въпреки че такива utility методи са code smell.

Код: Java

1. public class ReportsWindow {
2.  
3.     public static void utilityWIndow(String title, URI pathToFxml) throws IOException  {
4.         Parent root = FXMLLoader.load(pathToFxml);
5.         Stage stage = new Stage();
6.         Scene scene = new Scene(root);
7.  
8.         stage.setTitle(title);
9.         stage.initModality(Modality.APPLICATION_MODAL);
10.         stage.setResizable(false);
11.         stage.initStyle(StageStyle.UTILITY);
12.         stage.setScene(scene);
13.         stage.show();
14.     }
15.  
16.     public static void referenceSales() throws IOException {
17.         utilityWIndow("Справка дневни продажби", getClass().getResource("/screens/DailyDeliveriesScreen.fxml"));
18.     }
19.  
20.     public static referenceSupply() throws IOException {
21.         utilityWIndow("Справка дневни доставки", getClass().getResource("/screens/DailySalesScreen.fxml"));
22.     }
23. }

Не съм запознат с JavaFX-a така че не мога да ти кажа как точно да си организираш кода, но само статични методи не е добре. Също избягвай повторението, където е възможно.