Originariamente inviato da Mauro84
è corretto un tale utilizzo di eval nella funzione select?
No. L'utilizzo di eval() dovrebbe essere limitato il più possibile, oltretutto la tua chiamata ad eval() è totalmente inutile.

Originariamente inviato da Mauro84
il ciclo while nella funzione select potrebbe essere migliorato secondo voi?
Il modo in cui hai implementato la gestione degli errori va migliorato. Es. non verifichi il valore restituito da mysql_query().

Inoltre una classe per la manipolazione del DB non dovrebbe in linea teorica inviare dei dati in output, dovrebbe restituirli come valore di ritorno del metodo, etc.

Originariamente inviato da Mauro84
utilizzare return nelle prime tre funzioni mi sembra brutale ma non sono riuscito a far di meglio conoscete alternative?
Aumenta l'astrazione: non restituire semplicemente il valore di ritorno delle funzioni mysq_*, restituisci true o false, etc. etc.