Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > it.comp.www.php > #20434

Re: Manipolare il DB al volo senza PHPMyAdmin

Show all headers | View raw

On 28/02/2016 18:40, Francesco Bianco wrote:

> https://github.com/Javanile/SchemaDB
>
> mi piacerebbe un vostro parere e dei feedback di utilizzo

L'hai chiesto tu! :P

Prima di tutto complimenti per l'impegno. Non ho provato a usarlo, ma ho 
un paio di consigli a livello codice.

- adotta uno stile e usa un tool per forzarlo. Se proprio va male, usa 
PSR-2, che ormai ha formatting tools ovunque. Il tuo codice è davvero 
difficile da leggere, con l'indentazione "a caso".

- Togli tutti i commenti vuoti. Occupano spazio e non portano nessun 
vantaggio

- Il modo in cui usi i trait è sbagliato. Un trait non dovrebbe mai 
dipendere da altri trait o da funzionalità di una classe base. Dentro la 
cartella Model, per esempio, nessuno dei trait può essere usato 
indipendentemente dagli altri. Tanto vale fare una classe unica, o usare 
composizione di classi, e non "finta ereditarietà multipla".

- Non mettere echo o print, nemmeno per debug. E soprattutto non mettere 
HTML. Usa una classe per il logging (vedi PSR-3), e nel dumpSchema crea 
un array bidimensionale coi dati e ritornalo, non stampare tabelle in 
HTML. In questo modo puoi riciclare quella funzione per altri output 
(json, csv, sql, ecc.)

- Usa un framework standard (come PHPUnit) per i test. Anche qui dai per 
scontato che uno veda i test con un browser, il che ti impedisce di 
renderli automatici.

- Non ho analizzato a fondo, ma mi pare di aver visto più di qualche 
vulnerabilità a SQL injection. In una libreria che astrae un DB, 
dovrebbe essere la priorità assoluta. Tutto il resto è secondario.

Bye.

Back to it.comp.www.php | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Manipolare il DB al volo senza PHPMyAdmin Francesco Bianco <info.francescobianco@gmail.com> - 2016-02-28 10:40 -0800
  Re: Manipolare il DB al volo senza PHPMyAdmin Alex <tommaso5ita@yahoo.it> - 2016-02-29 12:08 +0100
  Re: Manipolare il DB al volo senza PHPMyAdmin fmassei@gmail.com - 2016-02-29 10:16 -0800
  Re: Manipolare il DB al volo senza PHPMyAdmin Alessandro Pellizzari <shuriken@amiran.it> - 2016-03-01 10:47 +0000

csiph-web