FTP CLASS met Exceptions
Door Andreas Warnaar, 20 jaar geleden, 4.356x bekeken
FTP CLASS die ik geschreven heb voor mijn examen project. Ik ben er nog mee bezig maar ik verwacht niet veel wijzigingen aan deze klassen.
Deze klasssen maakt onderdeel uit van een reeks andere klassen die ik nog aan het schrijven ben. maar deze komen later..
Het engige wat deze klassen anders maakt dan de eerder geplaatste klassen van Sean( http://phphulp.nl/php/scripts/4/263/ ) is de array die je kunt op vragen, een download functie en de exceptions handler..
er ontbreekt nog een method en dat is die voor het verwijderen van een directory, maar deze zal zeker nog komen..
De ftpException class is anders dan het orginele, maar ik heb een header aan toe moeten voegen voor de foutmelding en css. Normaal zal deze bepaald worden door de boven liggende klassen..
Voorbeeld: http://www.andreaswarnaar.nl/phphulp/cls_ftp.php
Gesponsorde koppelingen
PHP script bestanden
Er zijn 17 reacties op 'Ftp class met exceptions'
0 
-0 
-0
ff snel gekeken, ziet er niet uit. Wel heel weinig commentaar en wat ik ook niet zo mooi vind is dat je bij getFileSize() een String teruggeeft ipv het aantal bytes. Laat de gebruiker maar beslissen hoe ie dat wil afbeelden..
0 -0
-0
@Rik
Het is idd geen of weinig commentaar..
wat betreft de getFileSize($p_File='',$int=false)
De gebruiker kan wel kiezen hoe hij zijn result krijgt d.m.v. de $int > 1.
Hoe zou het anders mooier zijn? gewoon niet?
Het is idd geen of weinig commentaar..
wat betreft de getFileSize($p_File='',$int=false)
De gebruiker kan wel kiezen hoe hij zijn result krijgt d.m.v. de $int > 1.
Hoe zou het anders mooier zijn? gewoon niet?
0 -0
-0
ik snap echt geen bal van je exception idee....
- de namen vind ik slecht en je hebt standaard Exceptions in SPL voor dit soort dingen...
- opmaak in de exception? NEE! je handelt exceptions af in de catch en/of exception_handler
- de namen vind ik slecht en je hebt standaard Exceptions in SPL voor dit soort dingen...
- opmaak in de exception? NEE! je handelt exceptions af in de catch en/of exception_handler
0 -0
-0
Paar aanmerkingen:
- In getFile() moet je exceptions gooien als fopen niet lukt
- echo $fileStream; <--- dit kan natuurlijk niet, een echo
- Een aantal van je methods zouden protected of private moeten zijn in mijn ogen
- De naam van de class is erg raar (cls...)
- Ik vind de namen van je methods een beetje onhandig. Ik zou gewoon delete(), chmod(), etc, gebruiken. Dit is makkelijker te onthouden en logischer voor mensen die het voor het eerst gebruiken.
- Documentatie (zoals gezegd)
Op zich allemaal relatief kleine aanmerkingen, ik vind het een goede toevoeging.
- In getFile() moet je exceptions gooien als fopen niet lukt
- echo $fileStream; <--- dit kan natuurlijk niet, een echo
- Een aantal van je methods zouden protected of private moeten zijn in mijn ogen
- De naam van de class is erg raar (cls...)
- Ik vind de namen van je methods een beetje onhandig. Ik zou gewoon delete(), chmod(), etc, gebruiken. Dit is makkelijker te onthouden en logischer voor mensen die het voor het eerst gebruiken.
- Documentatie (zoals gezegd)
Op zich allemaal relatief kleine aanmerkingen, ik vind het een goede toevoeging.
0 -0
-0
Idd dat cls mag wel weg, je doet toch ook niet voor elke variabele var?
0 -0
-0
Heeft hij wel, voor iedere member 'm_' terwijl er ook al $this-> voor staat... Lang leve de Hongaarse notatie... :/
0 -0
-0
dat cls is gewoon een kenmerk zodat je kunt zien dat je een klasse zelf hebt geschreven. Vind je dat onnodig haal je het lekker weg ga niet zeuren dat het er staat. Ik weet dat de meeste hier geen boeken lezen (moet je lekker zelf weten). Maar een manier van schrijven afkraken vind ik een beetje zwak. ''ik ken de code niet lezen of ik begrijp deze niet dus ik ga ze schrijf wijze maar afkraken'' Ik kan ook nog wel genoeg commentaar geven als ik iedereen ga aanspreken die code anders schrijft hier dan mij. Waarom is jou of mijn schrijf wijze beter dan die van Andreas Warnaar?
0 -0
-0
Quote:
''ik ken de code niet lezen of ik begrijp deze niet dus ik ga ze schrijf wijze maar afkraken"
Grappig mannetje ben je, het is gewoon een mening. Als je goeie benamingen gebruikt heb je al die domme, ja domme, voorvoegsels niet nodig.
0 -0
-0
De benamingen die gebruikt worden in het boek: PHP 5 SuperBoek http://www.phphulp.nl/php/boeken/42/
heb ik zoveel mogelijk geprobeerd aan te houden. De recenties over dit boek waren goed. Maar ik had niet begrepen dat deze manier van benoemen vreemd is.
Wat betreft de namen van de methods zal ik idd nog wat verbeteringen doen, maar ik zal de namen van de variabels aan houden..
Badankt voor jullie feedback en ik zal de nodige opmerkingen wijzigingen aan de code toepassen..
heb ik zoveel mogelijk geprobeerd aan te houden. De recenties over dit boek waren goed. Maar ik had niet begrepen dat deze manier van benoemen vreemd is.
Wat betreft de namen van de methods zal ik idd nog wat verbeteringen doen, maar ik zal de namen van de variabels aan houden..
Badankt voor jullie feedback en ik zal de nodige opmerkingen wijzigingen aan de code toepassen..
0 -0
-0
Het is het meest gebruikelijk om je klasse de naam te geven van het object dat hij representeert. Dus als je een klasse van een boom hebt dan noem je 'm Boom en als je een klasse van een FTP-client hebt dan noem je 'm FTPClient of FTP ofzo.
@Pascal
Volgens mij lees jij de verkeerde boeken. Ga eens universitaire boeken, Java-boeken, Zend-boeken of PEAR-pagina's lezen. Verder werd er ook niet over gezeurd, er werd een aanmerking gemaakt. Jij bent zeker ook tegen de ISO en standaarden?
@Pascal
Volgens mij lees jij de verkeerde boeken. Ga eens universitaire boeken, Java-boeken, Zend-boeken of PEAR-pagina's lezen. Verder werd er ook niet over gezeurd, er werd een aanmerking gemaakt. Jij bent zeker ook tegen de ISO en standaarden?
0 -0
-0
lijkt mij niet kloppen omdat je een request en response pool hebt...
dit is puur een request pool...
En nogmaals exceptions moet je gewoon afhandelen in het catch block of je exception_handler...
dit is puur een request pool...
En nogmaals exceptions moet je gewoon afhandelen in het catch block of je exception_handler...
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
<?php
class blaat_exception extends Exception{}
class schaap_exception extends Exception{}
try{
//throw new schaap_exception(NULL);
throw new blaat_exception(NULL);
}
catch(Exception $e){
//situatie a
if($e instanceof blaat_exception){
//doe iets anders...
}
if($e instanceof schaap_exception){
//doe weer iets anders...
}
//situatie b
switch(get_class($e)){
case 'blaat_exception' : /* doe iets anders */ break;
case 'schaap_exception' : /* doe weer iets anders */ break;
}
}
?>
class blaat_exception extends Exception{}
class schaap_exception extends Exception{}
try{
//throw new schaap_exception(NULL);
throw new blaat_exception(NULL);
}
catch(Exception $e){
//situatie a
if($e instanceof blaat_exception){
//doe iets anders...
}
if($e instanceof schaap_exception){
//doe weer iets anders...
}
//situatie b
switch(get_class($e)){
case 'blaat_exception' : /* doe iets anders */ break;
case 'schaap_exception' : /* doe weer iets anders */ break;
}
}
?>
0 -0
-0
Maar geef je in de state / activetie diagram de "exception throws" ? weer???
0 -0
-0
Wat je hier hebt getekend is een Activity diagram en die gebruik je voor het weergeven van activiteiten ;). Volgens mij kan je beter een sequence diagram gebruiken.
0 -0
-0
Is dat je persoonlijke voorkeur? of is een sequence digram nodig voordat je aan een activity diagram begint?
Ik heb meerder documentatie gelezen maar niet echt begrepen warneer wel of geen gebruik maken van de verschillenden diagrammen..
Ik heb meerder documentatie gelezen maar niet echt begrepen warneer wel of geen gebruik maken van de verschillenden diagrammen..
Edit:
Lees nu..
UML in 24 uur.. van Academic Service
UML in 24 uur.. van Academic Service
Om te reageren heb je een account nodig en je moet ingelogd zijn.
PHP hulp
0 seconden vanaf nu