संकलित समय पर लैम्ब्डा रिटर्न प्रकार की जाँच क्यों नहीं की जाती है?


38

प्रयुक्त विधि संदर्भ में वापसी का प्रकार है Integer। लेकिन Stringनिम्नलिखित उदाहरण में एक असंगत की अनुमति है।

withमैन्युअल रूप से कास्टिंग के बिना विधि संदर्भ प्रकार को सुरक्षित करने के लिए विधि घोषणा को कैसे ठीक करें ?

import java.util.function.Function;

public class MinimalExample {
  static public class Builder<T> {
    final Class<T> clazz;

    Builder(Class<T> clazz) {
      this.clazz = clazz;
    }

    static <T> Builder<T> of(Class<T> clazz) {
      return new Builder<T>(clazz);
    }

    <R> Builder<T> with(Function<T, R> getter, R returnValue) {
      return null; //TODO
    }

  }

  static public interface MyInterface {
    Integer getLength();
  }

  public static void main(String[] args) {
// missing compiletimecheck is inaceptable:
    Builder.of(MyInterface.class).with(MyInterface::getLength, "I am NOT an Integer");

// compile time error OK: 
    Builder.of(MyInterface.class).with((Function<MyInterface, Integer> )MyInterface::getLength, "I am NOT an Integer");
// The method with(Function<MinimalExample.MyInterface,R>, R) in the type MinimalExample.Builder<MinimalExample.MyInterface> is not applicable for the arguments (Function<MinimalExample.MyInterface,Integer>, String)
  }

}

उपयोग मामला: एक प्रकार का सुरक्षित लेकिन सामान्य बिल्डर।

मैंने एनोटेशन प्रोसेसिंग (ऑटोवैल्यू) या कंपाइलर प्लगइन (लम्बोक) के बिना एक सामान्य बिल्डर को लागू करने की कोशिश की

import java.lang.reflect.Array;
import java.lang.reflect.InvocationHandler;
import java.lang.reflect.Method;
import java.lang.reflect.Proxy;
import java.util.HashMap;
import java.util.concurrent.atomic.AtomicReference;
import java.util.function.Function;

public class BuilderExample {
  static public class Builder<T> implements InvocationHandler {
    final Class<T> clazz;
    HashMap<Method, Object> methodReturnValues = new HashMap<>();

    Builder(Class<T> clazz) {
      this.clazz = clazz;
    }

    static <T> Builder<T> of(Class<T> clazz) {
      return new Builder<T>(clazz);
    }

    Builder<T> withMethod(Method method, Object returnValue) {
      Class<?> returnType = method.getReturnType();
      if (returnType.isPrimitive()) {
        if (returnValue == null) {
          throw new IllegalArgumentException("Primitive value cannot be null:" + method);
        } else {
          try {
            boolean isConvertable = getDefaultValue(returnType).getClass().isAssignableFrom(returnValue.getClass());
            if (!isConvertable) {
              throw new ClassCastException(returnValue.getClass() + " cannot be cast to " + returnType + " for " + method);
            }
          } catch (IllegalArgumentException | SecurityException e) {
            throw new RuntimeException(e);
          }
        }
      } else if (returnValue != null && !returnType.isAssignableFrom(returnValue.getClass())) {
        throw new ClassCastException(returnValue.getClass() + " cannot be cast to " + returnType + " for " + method);
      }
      Object previuos = methodReturnValues.put(method, returnValue);
      if (previuos != null) {
        throw new IllegalArgumentException("Value alread set for " + method);
      }
      return this;
    }

    static HashMap<Class, Object> defaultValues = new HashMap<>();

    private static <T> T getDefaultValue(Class<T> clazz) {
      if (clazz == null || !clazz.isPrimitive()) {
        return null;
      }
      @SuppressWarnings("unchecked")
      T cachedDefaultValue = (T) defaultValues.get(clazz);
      if (cachedDefaultValue != null) {
        return cachedDefaultValue;
      }
      @SuppressWarnings("unchecked")
      T defaultValue = (T) Array.get(Array.newInstance(clazz, 1), 0);
      defaultValues.put(clazz, defaultValue);
      return defaultValue;
    }

    public synchronized static <T> Method getMethod(Class<T> clazz, java.util.function.Function<T, ?> resolve) {
      AtomicReference<Method> methodReference = new AtomicReference<>();
      @SuppressWarnings("unchecked")
      T proxy = (T) Proxy.newProxyInstance(clazz.getClassLoader(), new Class[] { clazz }, new InvocationHandler() {

        @Override
        public Object invoke(Object p, Method method, Object[] args) {

          Method oldMethod = methodReference.getAndSet(method);
          if (oldMethod != null) {
            throw new IllegalArgumentException("Method was already called " + oldMethod);
          }
          Class<?> returnType = method.getReturnType();
          return getDefaultValue(returnType);
        }
      });

      resolve.apply(proxy);
      Method method = methodReference.get();
      if (method == null) {
        throw new RuntimeException(new NoSuchMethodException());
      }
      return method;
    }

    // R will accep common type Object :-( // see /programming/58337639
    <R, V extends R> Builder<T> with(Function<T, R> getter, V returnValue) {
      Method method = getMethod(clazz, getter);
      return withMethod(method, returnValue);
    }

    //typesafe :-) but i dont want to avoid implementing all types
    Builder<T> withValue(Function<T, Long> getter, long returnValue) {
      return with(getter, returnValue);
    }

    Builder<T> withValue(Function<T, String> getter, String returnValue) {
      return with(getter, returnValue);
    }

    T build() {
      @SuppressWarnings("unchecked")
      T proxy = (T) Proxy.newProxyInstance(clazz.getClassLoader(), new Class[] { clazz }, this);
      return proxy;
    }

    @Override
    public Object invoke(Object proxy, Method method, Object[] args) {
      Object returnValue = methodReturnValues.get(method);
      if (returnValue == null) {
        Class<?> returnType = method.getReturnType();
        return getDefaultValue(returnType);
      }
      return returnValue;
    }
  }

  static public interface MyInterface {
    String getName();

    long getLength();

    Long getNullLength();

    Long getFullLength();

    Number getNumber();
  }

  public static void main(String[] args) {
    MyInterface x = Builder.of(MyInterface.class).with(MyInterface::getName, "1").with(MyInterface::getLength, 1L).with(MyInterface::getNullLength, null).with(MyInterface::getFullLength, new Long(2)).with(MyInterface::getNumber, 3L).build();
    System.out.println("name:" + x.getName());
    System.out.println("length:" + x.getLength());
    System.out.println("nullLength:" + x.getNullLength());
    System.out.println("fullLength:" + x.getFullLength());
    System.out.println("number:" + x.getNumber());

    // java.lang.ClassCastException: class java.lang.String cannot be cast to long:
    // RuntimeException only :-(
    MyInterface y = Builder.of(MyInterface.class).with(MyInterface::getLength, "NOT A NUMBER").build();

    // java.lang.ClassCastException: java.lang.String cannot be cast to java.lang.Long
    // RuntimeException only :-(
    System.out.println("length:" + y.getLength());
  }

}

1
आश्चर्यजनक व्यवहार। ब्याज से बाहर: क्या यह वही है जब आप बिल्डर के classबजाय उपयोग करते हैं interface?
GameDroids

वह अस्वीकार्य क्यों है? पहले मामले में, आप का प्रकार नहीं देते हैं getLength, इसलिए इसे स्ट्रिंग पैरामीटर से मिलान करने के लिए वापस Object(या Serializable) समायोजित किया जा सकता है ।
थिलो

1
मुझसे गलती हो सकती है, लेकिन मुझे लगता है कि आपकी विधि withसमस्या का हिस्सा है क्योंकि यह वापस आती है nullwith()वास्तव में फ़ंक्शन के Rप्रकार का उपयोग करके विधि को लागू करने के दौरान Rआपको पैरामीटर से त्रुटि मिलती है। उदाहरण के लिए<R> R with(Function<T, R> getter, T input, R returnValue) { return getter.apply(input); }
GameDroids

2
jukzi, शायद आपको कोड या एक स्पष्टीकरण प्रदान करना चाहिए कि आपके विधि के साथ वास्तव में क्या करना चाहिए और आपको क्यों Rहोना चाहिए Integer। इसके लिए, आपको हमें यह दिखाना होगा कि आप रिटर्न वैल्यू का उपयोग कैसे करना चाहते हैं। ऐसा लगता है कि आप किसी प्रकार के बिल्डर-पैटर्न को लागू करना चाहते हैं, लेकिन मैं एक सामान्य पैटर्न या आपके इरादे को नहीं पहचान सकता।
sfiss

1
धन्यवाद। मैंने पूर्ण आरंभीकरण के लिए जाँच के बारे में भी सोचा। लेकिन जब से मैं इसे संकलन समय पर करने का कोई रास्ता नहीं देखता हूं मैं डिफ़ॉल्ट मानों के साथ रहना पसंद करता हूं null / 0। मुझे यह भी पता नहीं है कि संकलित समय पर गैर-इंटरफ़ेस विधियों की जांच कैसे करें। रनटाइम पर ".with (m -> 1)" .returning (1) "जैसे गैर इंटरफ़ेस का उपयोग करके पहले ही java.lang.NoSuchMethodException
jukzi

जवाबों:


27

पहले उदाहरण में, MyInterface::getLengthऔर "I am NOT an Integer"सामान्य मापदंडों को हल करने में मदद की Tऔर Rकरने के लिए MyInterfaceऔर Serializable & Comparable<? extends Serializable & Comparable<?>>क्रमशः।

// it compiles since String is a Serializable
Function<MyInterface, Serializable> function = MyInterface::getLength;
Builder.of(MyInterface.class).with(function, "I am NOT an Integer");

MyInterface::getLengthFunction<MyInterface, Integer>जब तक आप स्पष्ट रूप से ऐसा नहीं कहते हैं, तब तक हमेशा ऐसा नहीं होता है , जो दूसरे उदाहरण के रूप में संकलन-समय त्रुटि को जन्म देगा।

// it doesn't compile since String isn't an Integer
Function<MyInterface, Integer> function = MyInterface::getLength;
Builder.of(MyInterface.class).with(function, "I am NOT an Integer");

यह जवाब पूरी तरह से इस सवाल का जवाब देता है कि इसकी व्याख्या दूसरे से क्यों की जाती है। दिलचस्प। R जैसी आवाज़ बेकार है। क्या आप समस्या का कोई समाधान जानते हैं?
जुकज़ी

@jukzi (1) स्पष्ट रूप से विधि प्रकार के मापदंडों (यहां, R) को परिभाषित करें : Builder.of(MyInterface.class).<Integer>with(MyInterface::getLength, "I am NOT an Integer");इसे संकलित नहीं करने के लिए, या (2) इसे
संक्षिप्त

11

इसका प्रकार अनुमान है जो यहां अपनी भूमिका निभा रहा है। Rविधि हस्ताक्षर में सामान्य पर विचार करें :

<R> Builder<T> with(Function<T, R> getter, R returnValue)

सूचीबद्ध मामले में:

Builder.of(MyInterface.class).with(MyInterface::getLength, "I am NOT an Integer");

के प्रकार Rको सफलतापूर्वक अनुमान लगाया गया है

Serializable, Comparable<? extends Serializable & Comparable<?>>

और Stringइस प्रकार से एक का मतलब है, इसलिए संकलन सफल होता है।


Rअसंगतता के प्रकार को स्पष्ट रूप से निर्दिष्ट करने और पता लगाने के लिए, कोई व्यक्ति केवल कोड की पंक्ति को बदल सकता है:

Builder.of(MyInterface.class).<Integer>with(MyInterface::getLength, "not valid");

स्पष्ट रूप से R को <Integer> घोषित करना दिलचस्प है और इस सवाल का पूरी तरह से जवाब देता है कि यह गलत क्यों है। हालाँकि मैं अभी भी स्पष्ट प्रकार की घोषणा किए बिना एक समाधान की तलाश कर रहा हूं। कोई उपाय?
jukzi

@jukzi आप किस तरह के समाधान की तलाश कर रहे हैं? यदि आप इसका उपयोग करना चाहते हैं तो कोड पहले से ही संकलित है। आप जिस चीज़ की तलाश कर रहे हैं उसका एक उदाहरण आगे चीजों को स्पष्ट करने के लिए अच्छा होगा।
नमन

11

इसका कारण यह है कि आपके सामान्य प्रकार के पैरामीटर Rको ऑब्जेक्ट होने का अनुमान लगाया जा सकता है, अर्थात निम्नलिखित संकलन:

Builder.of(MyInterface.class).with((Function<MyInterface, Object>) MyInterface::getLength, "I am NOT an Integer");

1
ठीक है, अगर ओपी को विधि का परिणाम चर के चर के लिए सौंपा गया है, तो Integerवह होगा जहां संकलन त्रुटि होती है।
sepp2k

@ sepp2k सिवाय इसके कि Builderकेवल सामान्य में है T, लेकिन में नहीं R। यह Integerबस के रूप में जहाँ तक प्रकार की जाँच बिल्डर का संबंध है पर ध्यान नहीं दिया जा रहा है।
थिलो

2
Rमाना जाता हैObject ... वास्तव में नहीं
नमन

@ थिलो तुम सही हो, बिल्कुल। मैंने मान लिया कि वापसी का प्रकार withउपयोग करेगा R। बेशक इसका मतलब यह है कि वास्तव में उस तरीके को लागू करने का कोई सार्थक तरीका नहीं है जो वास्तव में तर्कों का उपयोग करता है।
sepp2k

1
नमन, आप सही हैं, आपने और एंड्रयू ने इसका उत्तर सही अनुमान के साथ अधिक विस्तार से दिया है। मैं बस एक सरल व्याख्या देना चाहता था (हालाँकि इस प्रश्न को देखने वाला कोई भी जानता है कि टाइप अनुमान और अन्य प्रकार केवल Object)।
sfiss

0

यह उत्तर अन्य उत्तरों पर आधारित है जो बताते हैं कि यह अपेक्षा के अनुरूप काम क्यों नहीं करता है।

समाधान

निम्नलिखित कोड द्विभाजन को "दो धाराप्रवाह कार्यों में" के साथ "और" लौटकर "विभाजित करके समस्या हल करता है:

class Builder<T> {
...
class BuilderMethod<R> {
  final Function<T, R> getter;

  BuilderMethod(Function<T, R> getter) {
    this.getter = getter;
  }

  Builder<T> returning(R returnValue) {
    return Builder.this.with(getter, returnValue);
  }
}

<R> BuilderMethod<R> with(Function<T, R> getter) {
  return new BuilderMethod<>(getter);
}
...
}

MyInterface z = Builder.of(MyInterface.class).with(MyInterface::getLength).returning(1L).with(MyInterface::getNullLength).returning(null).build();
System.out.println("length:" + z.getLength());

// YIPPIE COMPILATION ERRROR:
// The method returning(Long) in the type BuilderExample.Builder<BuilderExample.MyInterface>.BuilderMethod<Long> is not applicable for the arguments (String)
MyInterface zz = Builder.of(MyInterface.class).with(MyInterface::getLength).returning("NOT A NUMBER").build();
System.out.println("length:" + zz.getLength());

(कुछ अपरिचित है)


सीधा समाधान के लिए भी stackoverflow.com/questions/58376589 देखें
jukzi
हमारी साइट का प्रयोग करके, आप स्वीकार करते हैं कि आपने हमारी Cookie Policy और निजता नीति को पढ़ और समझा लिया है।
Licensed under cc by-sa 3.0 with attribution required.