[Bitrepository-devel] Tomme elementer.

[Mikis Seth Sørensen]

Jeg syntes vi skal prøve at lade være med at bruge tomme elementer. Når
det så er sagt skal der stadigvæk kodes som at de andre kan finde på det.

Hvis man vil undgå at skrive:
request.getChecksumRequestForNewFile() != null &&
request.getChecksumRequestForNewFile().getChecksumType().trim().length() >
0
kan man jo lave en metode til at indkapsel forskellige måder at skrive
'Ingen checksum request'
private boolean isChecksumRequestDefined(ChechsumSpecTYPE spec) {
  return
    request.getChecksumRequestForNewFile() != null &&
request.getChecksumRequestForNewFile().getChecksumType().trim().length() >
0 &&
    .........;
}


Mvh
Mikis 


On 07/02/12 08.26, "Kim Teglgaard Christensen" <ktc at statsbiblioteket.dk>
wrote:

>Hej
>
>Jeg er selv ude i at checke på om elementerne er null og om de har
>længden 0. Da jeg ser et tomt element som værende ikke eksisterende.
>
>Angående et tomt salt element vil jeg mene at det skal tolkes på samme
>måde, som at det er null.
>Altså at null eller længde 0 element skal medføre almindelig checksum.
>
>Men det kan da godt være at webclienten skal laves om så der ikke bliver
>sat tomme elementer?
>
>Mvh Kim
>
>On Mon, 2012-02-06 at 15:54 +0000, Michael Rasmussen wrote:
>> Hejsa.
>> 
>>  
>> 
>> Jeg kan se, at webclienten, når extra checksummer
>> (ChecksumRequestFor*File) er ²disabled², stadig sender elementerne
>> med:
>> 
>>  
>> 
>> 
>><ChecksumRequestForNewFile><ChecksumType></ChecksumType><ChecksumSalt></C
>>hecksumSalt></ChecksumRequestForNewFile>
>> 
>>  
>> 
>> Er dette tilladt eller er det bare en forglemmelse?Jeg vil helst undgå
>> at 
>> 
>>  
>> 
>>             if (request.getChecksumRequestForNewFile() != null) {
>> 
>>  
>> 
>> skal blive til..
>> 
>>  
>> 
>>             if (request.getChecksumRequestForNewFile() != null &&
>> 
>>request.getChecksumRequestForNewFile().getChecksumType().trim().length()
>>> 0) {
>> 
>>  
>> 
>> alle steder, hvor noget sådant kan optræde.
>> 
>>  
>> 
>> I samme forbindelse: Hvordan skal udeladt/tomt ChecksumSalt element
>> opfattes? Jeg anlægger p.t. følgende tolkning:
>> 
>>  
>> 
>> Intet ChecksumSalt element => alm. checksum  (f.eks. MD5)
>> 
>> Tomt ChecksumSalt element => HMAC med salt = byte[] { 0 }
>> 
>>  
>> 
>> Hvilket betyder at det ikke er muligt at få en ikke-HMAC checksum ud
>> med webklienten.
>> 
>>  
>> 
>> ~ Michael
>> 
>>  
>> 
>>  
>> 
>> http://support.kb.dk/images/kb_logo.jpg
>> 
>> 
>> Det Kongelige Bibliotek
>> Nationalbibliotek og Københavns
>> Universitetsbibliotek
>> 
>> 
>> Michael Rasmussen
>> IT-Konsulent/Civil værnepligtig |
>> Consultant
>> 
>> Det Kongelige Bibliotek | The Royal
>> Library
>> Digital Infrastruktur og Service |
>> P.O. Box 2149 | DK-1016 København K
>> tel +45 3347 4574 | Fax +45 3393
>> 2218 | mira at kb.dk | www.kb.dk
>> 
>> Besøgsadresse | Visiting address |
>> Søren Kierkegaards Plads 1
>> Leveringsadresse | Delivery address
>> | Christians Brygge 8 | 1219
>> København K
>> 
>> EAN 5798 000 79 52 97 | Bank 0216
>> 4069032583 | CVR 28 98 88 42
>> IBAN DK2002164069032583 | Swiftcode
>> DABADKKK
>> 
>> 
>> 
>>  
>> 
>> 
>
>
>_______________________________________________
>Bitrepository-devel mailing list
>Bitrepository-devel at ml.sbforge.org
>http://ml.sbforge.org/mailman/listinfo/bitrepository-devel