sruehl commented on a change in pull request #14: WIP: Api Refactoring URL: https://github.com/apache/incubator-plc4x/pull/14#discussion_r214042283
########## File path: plc4j/api/src/main/java/org/apache/plc4x/java/api/messages/PlcReadResponse.java ########## @@ -18,40 +18,81 @@ Licensed to the Apache Software Foundation (ASF) under one */ package org.apache.plc4x.java.api.messages; -import org.apache.plc4x.java.api.messages.items.ReadRequestItem; -import org.apache.plc4x.java.api.messages.items.ReadResponseItem; - -import java.util.Collections; -import java.util.List; -import java.util.Optional; +import java.time.LocalDate; +import java.time.LocalDateTime; +import java.time.LocalTime; +import java.util.Collection; /** * Response to a {@link PlcReadRequest}. - * Contains the values read from the PLC but untyped. - * <p> - * Values are extracted using the {@link ReadRequestItem}s that were send in the read request. - * <p> - * If only a variables of one type are requested it is better to use - * {@link org.apache.plc4x.java.api.messages.specific.TypeSafePlcReadRequest} which leads to a - * {@link org.apache.plc4x.java.api.messages.specific.TypeSafePlcReadResponse}. */ -public class PlcReadResponse extends PlcResponse<PlcReadRequest, ReadResponseItem<?>, ReadRequestItem<?>> { +public interface PlcReadResponse extends PlcFieldResponse<PlcReadRequest> { + + int getNumValues(String name); + + boolean isRaw(String name); Review comment: I just thought about why not convert these methods into `is(Class clazz, String name)`, `get(Class clazz, String name)` `getAll(Class clazz, String name)`. From a user perspective this opens some more possibilities. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services