
Новые сообщения [новые:0]
Дайджест
Горячие темы
Избранное [новые:0]
Форумы
Пользователи
Статистика
Статистика нагрузки
Мод. лог
Поиск
|
|
18.04.2012, 13:11:47
|
|||
|---|---|---|---|
Нужно мнение по следующему коду |
|||
|
#18+
Начали спорить качественный ли код пишет наш коллега, выскажите плиз тут любое свое мнение. спасибо Код: php 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. 288. 289. 290. 291. 292. 293. 294. 295. 296. 297. 298. 299. 300. 301. 302. 303. 304. 305. 306. 307. 308. 309. 310. 311. 312. 313. 314. 315. 316. 317. 318. 319. 320. 321. 322. 323. 324. 325. 326. 327. 328. 329. 330. 331. 332. 333. 334. 335. 336. 337. 338. 339. 340. 341. 342. 343. 344. 345. 346. 347. 348. 349. 350. 351. 352. 353. 354. 355. 356. 357. 358. 359. 360. 361. 362. 363. 364. 365. 366. 367. 368. 369. 370. 371. 372. 373. 374. 375. 376. 377. 378. 379. 380. 381. 382. 383. 384. 385. 386. 387. 388. 389. 390. 391. 392. 393. 394. 395. 396. 397. 398. 399. 400. 401. 402. 403. 404. 405. 406. 407. 408. 409. 410. 411. 412. 413. 414. 415. 416. 417. 418. 419. 420. 421. 422. 423. 424. 425. 426. 427. 428. 429. 430. 431. 432. 433. 434. 435. 436. 437. 438. 439. 440. 441. 442. 443. 444. 445. 446. 447. 448. 449. 450. 451. 452. 453. 454. 455. 456. 457. 458. 459. 460. 461. 462. 463. 464. 465. 466. 467. 468. 469. 470. 471. 472. 473. 474. 475. 476. 477. 478. 479. 480. 481. 482. 483. 484. 485. 486. 487. 488. 489. 490. 491. 492. 493. 494. 495. 496. 497. 498. 499. 500. 501. 502. 503. 504. 505. 506. 507. 508. 509. 510. 511. 512. 513. 514. 515. 516. 517. 518. 519. 520. 521. 522. 523. 524. 525. 526. 527. 528. 529. 530. 531. 532. 533. 534. 535. 536. 537. 538. 539. 540. 541. 542. 543. 544. 545. 546. 547. 548. 549. 550. 551. 552. 553. 554. 555. 556. 557. 558. 559. 560. 561. 562. 563. 564. 565. 566. 567. 568. 569. 570. 571. 572. 573. 574. 575. 576. 577. 578. 579. 580. 581. 582. 583. 584. 585. 586. 587. 588. 589. 590. 591. 592. 593. 594. 595. 596. 597. 598. 599. 600. 601. 602. 603. 604. 605. 606. 607. 608. 609. 610. 611. 612. 613. 614. 615. 616. 617. 618. 619. 620. 621. 622. 623. 624. 625. 626. 627. 628. 629. 630. 631. 632. 633. 634. 635. 636. 637. 638. 639. 640. 641. 642. 643. 644. 645. 646. 647. 648. 649. 650. 651. 652. 653. 654. 655. 656. 657. 658. 659. 660. 661. 662. 663. 664. 665. 666. 667. 668. 669. 670. 671. 672. 673. 674. 675. 676. 677. 678. 679. 680. 681. 682. 683. 684. 685. 686. 687. 688. 689. 690. 691. 692. 693. 694. 695. 696. 697. 698. 699. 700. 701. 702. 703. 704. 705. 706. 707. 708. 709. 710. 711. 712. 713. 714. 715. 716. 717. 718. 719. 720. 721. 722. 723. 724. 725. 726. 727. 728. 729. 730. 731. 732. 733. 734. 735. 736. 737. 738. 739. 740. 741. 742. 743. 744. 745. 746. 747. 748. 749. 750. 751. 752. 753. 754. 755. 756. 757. 758. 759. 760. 761. 762. 763. 764. 765. 766. 767. 768. 769. 770. 771. 772. 773. 774. 775. 776. 777. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
18.04.2012, 13:27:53
|
|||
|---|---|---|---|
Нужно мнение по следующему коду |
|||
|
#18+
1. Режет глаза что все переменые имею разный формат: $LayoutName, $kcaptcha, $auto_blocks, $maxElements, $E_URI 2. Используеться global, переместите их чтоли в аргумент конструктора хотябы 3. метод action_path просто убил, обычный цикл так убить 4. debugOutput, getTable html в php коде? 5. buildLinks html в параметру, ужс 6. 7572559ca86e781ba8fe8073a0b725c6 в константу бы Первый два пункта - это минус вашего ведущего програмиста. Нужно иметь регамент как писать код чтобы не было бардака. Остальное конечно мдамс... ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
18.04.2012, 13:30:44
|
|||
|---|---|---|---|
Нужно мнение по следующему коду |
|||
|
#18+
Умп, мне непонравилось. ничего супер плохого, но: -комментарии в стиле КО -использование global переменных, при этом внутри есть статика типа getInstance и т.д. зачем? -инициализация всего и сразу. почему бы не ленивая? -класс вроде View но зачемто в нем есть и используется объект Db -прописанные намертво теги, стили какието хеши в куках, пути до каталогов вообще код какойто непонятный, бессвязный. очень большие методы - сигнал о том что пора их рефакторить. покажите еще пример кода как и где этот класс используется. возможно станет понятно где есть еще косяки или неоднозначности. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
18.04.2012, 16:00:48
|
|||
|---|---|---|---|
Нужно мнение по следующему коду |
|||
|
#18+
Код: php 1. дальше можно не смотреть. за global $E_URI, $E_DB, $E_USER можно убивать. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
18.04.2012, 16:45:02
|
|||
|---|---|---|---|
Нужно мнение по следующему коду |
|||
|
#18+
автор3. метод action_path просто убил, обычный цикл так убить array_shift не модно походу ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
19.04.2012, 16:35:01
|
|||
|---|---|---|---|
Нужно мнение по следующему коду |
|||
|
#18+
ИМХО, ничего криминального в данном коде нету. Небольшие некрасивости, я уверен, есть у всех ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
20.04.2012, 05:42:15
|
|||
|---|---|---|---|
Нужно мнение по следующему коду |
|||
|
#18+
artasИМХО, ничего криминального в данном коде нету. Небольшие некрасивости, я уверен, есть у всех эти некрасивости при поддержке вызовут много головной боли. например вообще непонятно что в классе Представления делают методы buildLimit() и saveQueries(), которым самое место в классе БД.. ну и другие подобные некрасивости ) ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
20.04.2012, 15:42:09
|
|||
|---|---|---|---|
|
|||
Нужно мнение по следующему коду |
|||
|
#18+
Код: php 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. скуля инжекшен ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
20.04.2012, 15:44:27
|
|||
|---|---|---|---|
Нужно мнение по следующему коду |
|||
|
#18+
... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
20.04.2012, 18:59:04
|
|||
|---|---|---|---|
Нужно мнение по следующему коду |
|||
|
#18+
phpz, расскажите, как Вы в СЕССИЮ сделаете инжект ? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
20.04.2012, 19:08:51
|
|||
|---|---|---|---|
|
|||
Нужно мнение по следующему коду |
|||
|
#18+
artasphpz, расскажите, как Вы в СЕССИЮ сделаете инжект ? В куки, в куки.. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
20.04.2012, 21:02:05
|
|||
|---|---|---|---|
Нужно мнение по следующему коду |
|||
|
#18+
artasphpz, расскажите, как Вы в СЕССИЮ сделаете инжект ? Легко ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
21.04.2012, 07:59:38
|
|||
|---|---|---|---|
Нужно мнение по следующему коду |
|||
|
#18+
Мои глаза... Я ненавижу PHP считаю, что код ужасен ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
03.05.2012, 15:50:28
|
|||
|---|---|---|---|
|
|||
Нужно мнение по следующему коду |
|||
|
#18+
Класс отвечает за загрузку скриптов и вывод инфромации пользователю. - что простите? Это так так? Я думаю не стоит применять ООП не понимая зачем оно нужно), кроме неправильной архитектуры класса тут еще очень много чего: 1) Публичные данные? Чем это обоснованно? 2) Не закоментирован 3) Название класса View, а отвечает за загрузку скриптов? Очень интересно) 4) Методы не обладают связностью 5) Интерфейса нет 6) По ОПП тут вообще много, небуду останавливаться) 7) Принято инициализировать все данные класса в конструкторе 8) $values - значения? Хм... А чего? И так ко многим переменным можно придраться 9) Просто жопа 10) Из - за таких как ваш коллега php - програмистов за программистов не считают. 11) Что читать: Мак Коннелл Совершенный код. На каждой странице он учит как писать/проектировать приложения чтоб небыло того что вы нам привели( ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
03.05.2012, 15:53:14
|
|||
|---|---|---|---|
|
|||
Нужно мнение по следующему коду |
|||
|
#18+
NekZ, Я думаю такое не только на php можно увидеть ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
03.05.2012, 16:02:00
|
|||
|---|---|---|---|
|
|||
Нужно мнение по следующему коду |
|||
|
#18+
Это как? вы угараете над нами или что? Код: php 1. 2. 3. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
03.05.2012, 16:03:29
|
|||
|---|---|---|---|
|
|||
Нужно мнение по следующему коду |
|||
|
#18+
if ($b+11>$pages) { $b = $pages-11+1; } 11, 1 - магические числа? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|

start [/forum/topic.php?fid=23&mobile=1&tid=1465207]: |
0ms |
get settings: |
8ms |
get forum list: |
18ms |
check forum access: |
2ms |
check topic access: |
2ms |
track hit: |
160ms |
get topic data: |
7ms |
get forum data: |
2ms |
get page messages: |
44ms |
get tp. blocked users: |
1ms |
| others: | 220ms |
| total: | 464ms |

| 0 / 0 |
