Verwijderd schreef op woensdag 14 februari 2018 @ 22:49:
Het onderstaande werkt:
- Als je voor elke field type die je hebt een if implementeert
- Als je vooraf weet/berekent hoeveel bytes je moet alloceren
- Als je fields public zijn
Java:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
| import java.io.IOException;
import java.lang.reflect.Field;
import java.nio.ByteBuffer;
import java.nio.ByteOrder;
public class UDPPacket {
public float m_time;
public float m_lapTime;
public float m_lapDistance;
public float m_totalDistance;
public ByteBuffer serialize() throws IllegalArgumentException, IllegalAccessException, IOException{
// 4 floats * 4 byte per stuk
ByteBuffer b = ByteBuffer.allocate(4 * 4).order(ByteOrder.LITTLE_ENDIAN);
Field[] fields = this.getClass().getFields();
for(Field field : fields){
if(field.getGenericType().getTypeName().equals("byte")) {
b.put(field.getByte(this));
} else if(field.getGenericType().getTypeName().equals("int")) {
b.putInt(field.getInt(this));
} else if(field.getGenericType().getTypeName().equals("float")) {
b.putFloat(field.getFloat(this));
}
// andere types
}
return b;
}
public void deserialize(ByteBuffer buffer) throws IllegalArgumentException, IllegalAccessException, IOException{
Field[] fields = this.getClass().getFields();
for(Field field : fields){
if(field.getGenericType().getTypeName().equals("byte")) {
field.setByte(this, buffer.get());
} else if(field.getGenericType().getTypeName().equals("int")) {
field.setInt(this, buffer.getInt());
} else if(field.getGenericType().getTypeName().equals("float")) {
field.setFloat(this, buffer.getFloat());
}
// andere types
}
}
public static void main(String[] args) throws IOException, IllegalArgumentException, IllegalAccessException {
byte[] a = new byte[] {-61, 105, 99, 65, 105, 82, -17, 64, -110, -47, 42, 67, -110, -47, 42, 67};
ByteBuffer buffer = ByteBuffer.wrap(a).order(ByteOrder.LITTLE_ENDIAN);
UDPPacket udpPacket = new UDPPacket();
udpPacket.deserialize(buffer);
ByteBuffer buffer2 = udpPacket.serialize();
System.out.println("identical buffers: " + buffer.equals(buffer2));
}
} |
Nee. Dit werkt niet.
getFields() geeft de fields niet gegarandeerd in een bepaalde volgorde terug, dat de huidige implementatie dat wel doet is toevallig. Je raakt hierbij dus de volgorde van de fields kwijt en daarmee zit je dus fields te vullen met "onzin". Dus naast dat reflectie bijna nooit de goede keus is, werkt het in dit geval ook niet.
bille schreef op zaterdag 17 februari 2018 @ 15:21:
Zal de volgorde van variabelen in de buffer altijd gelijk blijven? Stel dat in een volgende versie van de verzendende software er een variabele bij komt, hoe ga je om met die verandering? Maw; in hoeverre wil je jouw code iedere keer moeten aanpassen als de verzender iets toevoegt / verwijdert?
Ik zou een ParamTypes map maken die configureerbaar is dmv een settings file. In de map leg je de link tussen de parameter en de positie in de buffer. Vervolgens iets als (pseudocode)
code:
1
| buffer.map((float val, int index)=>ParamBuilder.withValue(val).forParam(ParamTypes[index]).build()) |
Het resultaat is een array van Param (key/value) objecten. Je zou er evt in plaats van een array ook direct een hashmap van kunnen maken met de ParamType als index.
Deze oplossing werkt alleen voor float waarden, hoe gaat deze datastructuur om met bijvoorbeeld integers en byte values? Met deze oplossing lijkt het alsof je data verliest op het moment dat je gaat proberen om een byte uit te lezen in plaats van een float waarde. Nou kun je daar natuurlijk omheen bouwen, maar dat komt zoals @
EddoH al beschreef aardig in de buurt van overengineering.
Sandor_Clegane schreef op zondag 18 februari 2018 @ 10:43:
Je kan het ook omdraaien, je stopt het array in een class en gebruikt Getters om de juiste waarden eruit te toveren.
Mijn Java is waardeloos maar dit is in C#, zal vast ook wel in Java werken:
code:
1
2
3
4
5
6
7
8
9
10
11
12
| public class UDPPacket
{
private byte[] _packet;
public float LapTime
{ get
{ return _packet[0]; }
}
public UDPPacket(byte[] packet)
{
_packet = packet;
}
} |
Edit: De conversie van byte naar de juiste waarde moet je er maar even bij verzinnen
Ook dit werkt niet fatsoenlijk, het veld Laptime geeft een byte terug, geen float. Dat zal niet eens compileren (ook niet in C# denk ik). Het probleem met deze oplossing is weer dat je niet fatsoenlijk onderscheid kunt maken tussen floats, bytes en integers, tenzij je elke keer dat je een waarde opvraagt opnieuw de benodigde bytes uit de byte array trekt en hier het goede datatype van maakt. Behalve dat dit systeem dus niet robuust genoeg is, is een betere versie al eerder gebruikt (Zelfs door TS zelf) in de vorm van een
ByteBuffer.
Naar mijn mening is de mooiste oplossing gewoon om een builder class met een berg setters te maken, op deze manier blijft het simpel genoeg om het overzichtelijk en onderhoudbaar te houden.
/u/12668/hydra.png?f=community)
/u/296460/beaker_s.png?f=community)
/u/1830/acm.png?f=community)
/u/249034/arrow.png?f=community){kind=icon}
:strip_icc():strip_exif()/u/439769/crop5a510243ea2e3_cropped.jpeg?f=community)
/u/341393/zandloper2.png?f=community)
De TS vraagt om een eenvoudigere manier om zooi uit die byte array te halen... hoeveel precies geeft de TS niet aan, vandaar mijn aanname dat het wel een lijst kan zijn die potentieel veranderbaar is. De oplossing hoeft natuurlijk helemaal niet perse daadwerkelijk een config file te gebruiken. Kan net zo goed aan hardcoded map zijn. 