|
|
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
cptngrbа покажите тот же код высокого качества, пожалуйста (кусочек)В идеале такой код подразумевает, что его сможет использовать другой программист для своей задачи с минимальной адаптацией. А сейчас это аццкая смесь бизнес-логики с UI, с отсутствием должной обработки ошибок, завязки на текстовые константы в коде, выбор неоптимальных языковых конструкций, отсутствие вменяемой архитектуры и спагетти в коде. В общем рефакторить и рефакторить. Плюс только один - это работает. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 05.03.2019, 23:22 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
white_niggercptngrbа покажите тот же код высокого качества, пожалуйста (кусочек)В идеале такой код подразумевает, что его сможет использовать другой программист для своей задачи с минимальной адаптацией. А сейчас это аццкая смесь бизнес-логики с UI, с отсутствием должной обработки ошибок, завязки на текстовые константы в коде, выбор неоптимальных языковых конструкций, отсутствие вменяемой архитектуры и спагетти в коде. В общем рефакторить и рефакторить. Плюс только один - это работает. А человек и не претендует на мастер-класс по идеальному коду. Есть рабочий код - он им поделился. А добиваться идеальности - ему не требуется, вам - тоже не требуется. И остаются варианты - выложить что есть или не выложить ничего. Если выложить - то может быть он кому-то поможет, начать делать аналогичное. Или не делать так же плохо :) Большинство коммерческих прикладух вот так и написано, и они работают. Да, не идеально, но и мир не идеален. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 04:50 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
A: Покажи плиз код B: на. A: Спасибо! C: Это не код, это полный ужас, это нельзя показывать, не говоря уже про использовать. Но свой, великолепный код, выполненный по всем правилам, писанным и неписанным, работающий идеально - я вам не покажу. Потому что он не идеальный. Спасибо, последний искатель бревен в чужом глазу - умилил просто до слез. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 06:00 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
Этот код был написан 11 лет назад на заре изучения Delphi, для он-лайн касс немого доработал и все. Выложил в качестве примера работы с ККМ Штрих-М. Программа работает в нескольких магазинах на одном предприятии, поддерживается только мной, у которого еще куча обязанностей. Рефакторить не собираюсь, нет ни времени, ни желания, все и так работает. По хорошему там все программу нужно переписывать и на другую БД, но трудозатратно. (Как там в пословице: работает - не трогай!). А на хороший код бы посмотрел, как правильно нужно писать, я ведь самоучка. А то хаять могут а показать как надо - нет, или объяснить на примерах так не делай, а делай так и будет все по фен-шую. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 07:56 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
Vizit0r, ))) ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 08:39 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
Близнец1980, забей. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 11:53 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
Всего лишь вопрос профессионального развития. Развиваться или нет каждый решает сам. А граммотно оформленный код можно (и нужно) смотреть в коммерческих библиотеках коих полно в том числе на варезниках. Если человеку не указать на ошибки то возможно он их не осознает. Здесь типичный случай отсутствия команды - человек пишет код для себя, не задумываясь, что когда-нибудь кому-то придется разбираться и сопровождать это. Многие через это проходят. То что помог кому-то своим кодом - плюс. Код плохой - минус ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 13:33 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
white_niggerВсего лишь вопрос профессионального развития. Развиваться или нет каждый решает сам. А граммотно оформленный код можно (и нужно) смотреть в коммерческих библиотеках коих полно в том числе на варезниках. Если человеку не указать на ошибки то возможно он их не осознает. Здесь типичный случай отсутствия команды - человек пишет код для себя, не задумываясь, что когда-нибудь кому-то придется разбираться и сопровождать это. Многие через это проходят. То что помог кому-то своим кодом - плюс. Код плохой - минус Вот не показатель... ковырялся в FR4, местами то еще болото. Да и зачем лезть в варезник, когда в среднем хороший код в RTL? Разумеется, исключая некоторые индусские выверты ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 14:17 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
Василий 2Вот не показатель... ковырялся в FR4, местами то еще болото.Всякое встречается, не отрицаю. Но в среднем по больнице, качество кода коммерческих библиотек гораздо выше "кода в интернете", про который у Gunsmoker-a написано ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 15:10 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
white_nigger, человек ругает чужой код не говоря как надо. Это не конструктивно. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 16:27 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
Близнец1980, А с драйвеом 10 версии этот код работает? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 16:58 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
antox, работает с последней версией драйвера штриха 1. открываешь чек 2... операция 3. закрываешь чек Код: pascal 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25. 26. 27. 28. 29. 30. 31. 32. 33. 34. 35. 36. 37. 38. 39. 40. 41. 42. 43. 44. 45. 46. 47. 48. 49. 50. 51. 52. 53. 54. 55. 56. 57. 58. 59. 60. 61. 62. 63. 64. 65. 66. 67. 68. 69. 70. 71. 72. 73. 74. 75. 76. 77. 78. 79. 80. 81. 82. 83. 84. 85. 86. 87. 88. 89. 90. 91. 92. 93. 94. 95. 96. 97. 98. 99. 100. 101. 102. 103. 104. 105. 106. 107. 108. 109. 110. 111. 112. 113. 114. 115. 116. 117. 118. 119. 120. 121. 122. 123. 124. 125. 126. 127. 128. 129. 130. 131. 132. 133. 134. 135. 136. 137. 138. 139. 140. 141. 142. 143. 144. 145. 146. 147. 148. 149. 150. 151. 152. 153. 154. 155. 156. 157. 158. 159. 160. 161. 162. 163. 164. 165. 166. 167. 168. 169. 170. 171. 172. 173. 174. 175. 176. 177. 178. 179. 180. 181. 182. 183. 184. 185. 186. 187. 188. 189. 190. 191. 192. 193. 194. 195. 196. 197. 198. 199. 200. 201. 202. 203. 204. 205. 206. 207. 208. 209. 210. 211. 212. 213. 214. 215. 216. 217. 218. 219. 220. 221. 222. 223. 224. 225. 226. 227. 228. 229. 230. 231. 232. 233. 234. 235. 236. 237. 238. 239. 240. 241. 242. 243. 244. 245. 246. 247. 248. 249. 250. 251. 252. 253. 254. 255. 256. 257. 258. 259. 260. 261. 262. 263. 264. 265. 266. 267. 268. 269. 270. 271. 272. 273. 274. 275. 276. 277. 278. 279. 280. 281. 282. 283. 284. 285. 286. 287. end; ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 17:11 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
cptngrb, Ну, у них последних же три 8,9,10. С последней имеется в виду с 10.х ? А то хотят перейти с 8ки на 10ку ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 17:13 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
antoxБлизнец1980, А с драйвеом 10 версии этот код работает? Здесь про Штрих-М, а не про АТОЛ. И, к слову, про АТОЛ: я тыркнулся в 10 версию, хотел прокатить на халяву, узрев, что там тоже есть OLE-сервер, но напоролся на то, что переписывать практически каждую строчку придется, если до этого пользовались 6-8 версиями :) Так что писать всё заново под АТОЛ 10 . ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 17:16 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
Придется писать... ( ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 17:33 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
V.BorzovИ, к слову, про АТОЛ: я тыркнулся в 10 версию, хотел прокатить на халяву, узрев, что там тоже есть OLE-сервер, но напоролся на то, что переписывать практически каждую строчку придется, если до этого пользовались 6-8 версиями :) Так что писать всё заново под АТОЛ 10Не, ну чё, сходства есть. И примеры хорошие для дельфей в доке. Концептуальную разницу я одну заметил: ИНН оператора передается не при печати чека, а при регистрации кассира. Я у себя сделал абстрактный класс и наследуюсь от него в зависимости от версии. Типа TAtol и наследники TAtol8 или TAtol10. Так вот функции классов TAtol8 и TAtol10 очень похожи, не только названием. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 17:54 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
YuRock, А я все драйверы касс выкинул на скрипты в фастрепорт, OLE-сервера там поддерживаются. В самой программе есть стандартный набор процедур, которые она вызывает в этом скрипте, и ей без разницы, что там стоит, а вся работа непосредственно с кассой зависит уже от установленного скрипта. Правда, из десятка когда-то освоенных драйверов реально работают сейчас только АТОЛ да ШТРИХ, а всякие там Меркурии что-то не попадались с тех пор, как ввели онлайн-кассы. Не говоря обо всяких там АМС :) ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 21:00 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
cptngrbwhite_nigger, человек ругает чужой код не говоря как надо. Это не конструктивно. Со мной в команде 3 года работал программист, он каким-то чудом выполнил тестовое задание, за это его взяли на работу. Так он примерно в таком же стиле писал. Я все три года старался его переучить, но тщетно (ему уж 40 лет, опыт работа на Delphi более 20 лет). Недавно его уволили за профнепригодность. Иногда такой его код "выстреливает" так, что плакать хочется. Недавно пришлось модуль на 2 тыс. строк кода переписать с нуля, поскольку он работал с ошибкой, а разобраться и исправить ошибку было нереально (я даже не думал, что так бывает). После этого случая у меня автоматически возникают нехорошие ассоциации. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 21:32 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
DmSer, В 2 тысячах строк не смогли найти ошибку, оказалось проще переписать? Да ладно... Отмазка это, по-моему :) Были другие причины. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 22:20 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
V.BorzovDmSer, В 2 тысячах строк не смогли найти ошибку, оказалось проще переписать? Да ладно... Отмазка это, по-моему :) Были другие причины. По факту там ошибка была в каждой второй строке. Каким-то чудом до поры до времени этот код работал, потом перестал. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.03.2019, 22:23 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
DmSer, поэтому и хочется, чтобы носом тыкнули и написали, что так и так писать не надо, а то будет бо-бо ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 07.03.2019, 08:22 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
cptngrb, Да они хотят просто стырить твой код как есть, чтобы не заморачиваться и не переделывать, и чтобы всё разжевано было, и чтобы работало, и без ошибок. Ан-нет, придется потрудиться. Вот и расстроились.... ))) ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 07.03.2019, 15:11 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
cptngrbDmSer, поэтому и хочется, чтобы носом тыкнули и написали, что так и так писать не надо, а то будет бо-бо О проблемах в критикуемом мною примере кода уже написал white_nigger. Я бы еще добавил, что работа с ФР ведется в основном потоке, поэтому если ФР подвиснет с ответом, то и у пользователя программа будет висеть в статусе "не отвечает". А вообще обычно почти любые примеры работы со штрихом можно подсмотреть в исходниках теста драйвера. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 07.03.2019, 22:21 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
DmSerЯ бы еще добавил, что работа с ФР ведется в основном потоке, поэтому если ФР подвиснет с ответом, то и у пользователя программа будет висеть в статусе "не отвечает". И что? Все равно пока ФР не заработает кассиру делать нечего. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 08.03.2019, 09:33 |
|
||
|
Delphi + онлайн-кассы
|
|||
|---|---|---|---|
|
#18+
Это смотря какой функционал в программу заложен. В идеале да, программа кассира не должна делать ничего лишнего, управление всем остальным должно быть организовано в отдельных сервисах. Но все ли так делают? Проблема не отвечающей программы в том, что кассир пытается нажимать, получается не то, что он планирует, хуже того, иногда модальные окна на задний план улетают, иногда исчезает модальность и пользователь может закрыть нижележащее окно. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 08.03.2019, 11:14 |
|
||
|
|

start [/forum/topic.php?fid=58&msg=39782762&tid=2037689]: |
0ms |
get settings: |
7ms |
get forum list: |
10ms |
check forum access: |
3ms |
check topic access: |
3ms |
track hit: |
66ms |
get topic data: |
6ms |
get forum data: |
8ms |
get page messages: |
40ms |
get tp. blocked users: |
1ms |
| others: | 229ms |
| total: | 373ms |

| 0 / 0 |
