FTP CLASS met Exceptions

Door Andreas Warnaar, 19 jaar geleden, 3.996x 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

  1. ftp-class-met-exceptions

 

Er zijn 17 reacties op 'Ftp class met exceptions'

PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen
Maarten
Maarten
19 jaar geleden
 
0 +1 -0 -1
Ziet er goed uit :)
Jan geen
Jan geen
19 jaar geleden
 
0 +1 -0 -1
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..
Andreas Warnaar
Andreas Warnaar
19 jaar geleden
 
0 +1 -0 -1
@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?
Thijs X
Thijs X
19 jaar geleden
 
0 +1 -0 -1
Leuke klasse, inderdaad beetje jammer dat het niet gedocumenteerd is.
Lode
Lode
19 jaar geleden
 
0 +1 -0 -1
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
PHP erik
PHP erik
19 jaar geleden
 
0 +1 -0 -1
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.
Jan geen
Jan geen
19 jaar geleden
 
0 +1 -0 -1
Idd dat cls mag wel weg, je doet toch ook niet voor elke variabele var?
Jelmer -
Jelmer -
19 jaar geleden
 
0 +1 -0 -1
Heeft hij wel, voor iedere member 'm_' terwijl er ook al $this-> voor staat... Lang leve de Hongaarse notatie... :/


19 jaar geleden
 
0 +1 -0 -1
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?
Jan geen
Jan geen
19 jaar geleden
 
0 +1 -0 -1
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.
Andreas Warnaar
Andreas Warnaar
19 jaar geleden
 
0 +1 -0 -1
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..
PHP erik
PHP erik
19 jaar geleden
 
0 +1 -0 -1
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?
Andreas Warnaar
Andreas Warnaar
19 jaar geleden
 
0 +1 -0 -1
Een vraagje aan de kenners..

Afbeelding

Is dit correct .. ?
Lode
Lode
19 jaar geleden
 
0 +1 -0 -1
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...

Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
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;
    }
}

?>
Andreas Warnaar
Andreas Warnaar
19 jaar geleden
 
0 +1 -0 -1
Maar geef je in de state / activetie diagram de "exception throws" ? weer???
Jan geen
Jan geen
19 jaar geleden
 
0 +1 -0 -1
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.
PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen
Andreas Warnaar
Andreas Warnaar
19 jaar geleden
 
0 +1 -0 -1
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..

Edit:
Lees nu..
UML in 24 uur.. van Academic Service

Om te reageren heb je een account nodig en je moet ingelogd zijn.

Inhoudsopgave

  1. ftp-class-met-exceptions

Labels

  • Geen tags toegevoegd.

Navigatie

 
 

Om de gebruiksvriendelijkheid van onze website en diensten te optimaliseren maken wij gebruik van cookies. Deze cookies gebruiken wij voor functionaliteiten, analytische gegevens en marketing doeleinden. U vindt meer informatie in onze privacy statement.