[Java] Fehler im BrickletDualRelay getMonoflop()

[Quantasy]

Ich glaube, Ihr habt in der getMonoflop Methode vergessen, dem Rückgabewert (Monoflop) das Relay anzugeben:

 

Hier Euer code:

public Monoflop getMonoflop(short relay) throws TimeoutException, NotConnectedException {
	ByteBuffer bb = ipcon.createRequestPacket((byte)9, FUNCTION_GET_MONOFLOP, this);
	bb.put((byte)relay);

	byte[] response = sendRequest(bb.array());

	bb = ByteBuffer.wrap(response, 8, response.length - ;
	bb.order(ByteOrder.LITTLE_ENDIAN);

	Monoflop obj = new Monoflop();
	obj.state = (bb.get()) != 0;
	obj.time = IPConnection.unsignedInt(bb.getInt());
	obj.timeRemaining = IPConnection.unsignedInt(bb.getInt());

	return obj;
}

Heissen sollte er aber:

public Monoflop getMonoflop(short relay) throws TimeoutException, NotConnectedException {
	ByteBuffer bb = ipcon.createRequestPacket((byte)9, FUNCTION_GET_MONOFLOP, this);
	bb.put((byte)relay);

	byte[] response = sendRequest(bb.array());

	bb = ByteBuffer.wrap(response, 8, response.length - ;
	bb.order(ByteOrder.LITTLE_ENDIAN);

	Monoflop obj = new Monoflop();
	obj.relay = relay;
	obj.state = (bb.get()) != 0;
	obj.time = IPConnection.unsignedInt(bb.getInt());
	obj.timeRemaining = IPConnection.unsignedInt(bb.getInt());

	return obj;
}

Wäre es möglich, dass Ihr das Relay auch noch dort einfüllt... sonst ist das immer 0 und im weiteren Verlauf eines Programs verliert sich die Information... welches Monoflop da gemeint war.

[photron]

Du hast recht, da ist ein Bug in den Java Bindings dies bezüglich. Aber der Bug ist, dass die Monoflop Klasse überhaupt das relay Member hat.

 

Dieser Bug ist in den Heute veröffentlichen Java Bindings behoben.

[Quantasy]

Misst... nun muss ich mir merken, zu welchem Relay denn diese Monoflop-Parameter gehören...

 

Naja, ist eure Design-Entscheidung... muss ich akzeptieren.