Code snippets Razmena snippets, funkcija, dodataka, algoritama,... pod uslovom je da ste vi autor ili imate dozvolu autora ili da navedete autora uz odgovarajucu licencu. |
|
Alati teme | Način prikaza |
|
12. 03. 2011. | #1 |
@SasaKostic
Na probnom radu
Datum učlanjenja: 15.12.2009
Poruke: 34
Hvala: 17
1 "Hvala" u 1 poruci
|
Nema problema Sinisa,
Dobro vidis HTML kod u kontroleru. To nije definisana praksa ali je moj nacin iskljucivo za ovaj (backup) kontroler koji nije deo finalne app, nije obavezno koriscenje. 1. Backup web aplikacije koju eventualno radite, jos u naslovu se vidi 2. Ne znam tvoju definiciju lepog koda, ulepsaj ga po ukusu 3. Ako ti se folder ne zove proj, preimenuj ga. Sva sreca da je kod kratak inace bi se namucio da me jos vise iskritikujes. Ako nisi probao kod, ni ne moras. Prihavatam kritike cim sam objavio nesto. Poruka je namenjena ljudima koji rade web app u CI, ne svima, ali naravno procitaju je usput i oni kojima nije namenjena. |
12. 03. 2011. | #2 |
@SasaKostic
Na probnom radu
Datum učlanjenja: 15.12.2009
Poruke: 34
Hvala: 17
1 "Hvala" u 1 poruci
|
Uzevsi u obzir neke od Sinisinih kritika, kod sad izgleda ovako:
Kôd:
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en"> <head> <meta http-equiv="Refresh" content="2; URL=http://localhost/proj" /> <meta name="author" content="" /> <title>Backup</title> </head> <body> <?php class Backup extends Controller { function __construct() { parent::Controller(); } function index() { $path = './';//directory in archive $archive_path = 'd:/htdocs/backup/proj/';//destination directory for backup files $date = strftime("%d-%b-%y at %Hh%M ");//date format in backup file names $db_backup_file = $date . 'db.gz';//filename for database backup $proj_backup_file = $date . 'proj.zip';//filename for code backup //DB backup: $this->load->dbutil(); $backup = &$this->dbutil->backup(); $this->load->helper('file'); write_file($archive_path . $db_backup_file, $backup); //<-DB backup //proj backup: $this->load->lib('zip'); $this->zip->read_dir($path); $date = strftime("%d-%b-%y at %Hh%M "); $this->zip->archive($archive_path . $proj_backup_file); // Creates a file named myarchive.zip $data = 'Done. Redirecting in 2 sec<br> <strong>' . $db_backup_file . '</strong> - '.round((filesize($archive_path . $db_backup_file)/1024),1).' KB <br> <strong>' . $proj_backup_file . '</strong> - '.round((filesize($archive_path . $proj_backup_file)/1024),1).' KB <br>'; $this->out->set_out($data);//display status info //<-proj backup } } ?> </body> </html> |
12. 03. 2011. | #3 |
član
Certified
|
Za pocetak nisam uopste zeleo da te uvredim ili bilo sta nalik na to, samo skrecem paznju na neke meni nedopustive stvari.
Npr ovo: da ali na jednom mestu, u nekoj varijabli, konstanti ili cemu god drugom, mada si to u CI-ju mogao da napravis automatski da vidi koji je folder i da sam to resi bez potrebe za insertovanjem bilo cega, no u tvom kodu je to proj na nekoliko mesta dakle svaki put kad koristis skriptu menjaj na x mesta... Sorry ako sam te uvredio, ali to nije bilo u planu Edit: vidim da si sredio neke od stvari |
|
|