2

私はgwtの問題5794に直面しています：http ：//code.google.com/p/google-web-toolkit/issues/detail？id = 5794

8か月前のパッチがあるのを見ましたが、gwt 2.5RC1http://gwt-code-reviews.appspot.com/1620804/には含まれていません 。

このパッチがgwt2.5rc2または最終リリースに含まれるかどうか誰かが知っていますか？

そうでない場合は、誰かがその問題の最善の回避策を教えてください。

前もって感謝します。

4

3 に答える 3

3

これは、AbstractRequestContextのcopyBeanAndCollectionsメソッドから思いついたものです。新しいコンテキストでプロキシを別のプロキシに複製したいという仕事をしているようです。これには基本的に、最初に新しいプロキシを作成するか、新しいコンテキストを使用してプロキシを編集する必要がありますが、プロパティを新しいプロキシに繰り返しコピーするためにコピーコンストラクタを使用する必要はなくなりました。私はまだテスト中ですが、単純なGWTTestCaseを作成しましたが、正しく機能しているようです。

クイックアップデート

リクエストファクトリエンティティロケーターがタイプを解決できなかったという火災の問題を解決したので、クローンを作成するためにstableidタグとversionタグを追加し直しました。これは悪い考えかもしれません。元のオブジェクトへの実際の参照のクラスタイプに安定したIDが使用されているかどうかを判断したら、更新します。私の推測では、サーバー側でタイプを解決するために使用されます。

public class ProxyUtils {

    public static <T extends BaseProxy> AutoBean<T> cloneBeanProperties(final T original, final T clone, final RequestContext toContext) {

        AutoBean<T> originalBean = AutoBeanUtils.getAutoBean(original);
        AutoBean<T> cloneBean = AutoBeanUtils.getAutoBean(clone);

        return cloneBeanProperties(originalBean, cloneBean, toContext);
    }

    /**
     * Shallow-clones an autobean and makes duplicates of the collection types.
     * A regular {@link AutoBean#clone} won't duplicate reference properties.
     */
    public static <T extends BaseProxy> AutoBean<T> cloneBeanProperties(final AutoBean<T> toClone, final AutoBean<T> clone, final RequestContext toContext) {
        // NOTE: This may be a bad idea, i don't know if the STABLE_ID is the type or if it is the
        // actual server side reference to this object. Don't want it to update my original object.
        // I added this back in because I was getting an InstantionException in the locator I am
        // pretty sure this is because request factory server side could not resolve the class type. 
        // Maybe someone could shed some light on this one, if you know what the stable id is really
        // used for. 
        clone.setTag(STABLE_ID, toClone.getTag(STABLE_ID));
        clone.setTag(Constants.VERSION_PROPERTY_B64, toClone.getTag(Constants.VERSION_PROPERTY_B64));
        clone.accept(new AutoBeanVisitor() {
            final Map<String, Object> values = AutoBeanUtils.getAllProperties(toClone);

            @Override
            public boolean visitCollectionProperty(String propertyName, AutoBean<Collection<?>> value, CollectionPropertyContext ctx) {
                // javac generics bug
                value = AutoBeanUtils.<Collection<?>, Collection<?>> getAutoBean((Collection<?>) values.get(propertyName));
                if (value != null) {
                    Collection<Object> collection;
                    if (List.class == ctx.getType()) {
                        collection = new ArrayList<Object>();
                    } else if (Set.class == ctx.getType()) {
                        collection = new HashSet<Object>();
                    } else {
                        // Should not get here if the validator works correctly
                        throw new IllegalArgumentException(ctx.getType().getName());
                    }

                    if (isValueType(ctx.getElementType()) || isEntityType(ctx.getElementType())) {
                        /*
                         * Proxies must be edited up-front so that the elements
                         * in the collection have stable identity.
                         */
                        for (Object o : value.as()) {
                            if (o == null) {
                                collection.add(null);
                            } else {
                                collection.add(editProxy(toContext, (Class<T>) ctx.getType(), (T) o));
                            }
                        }
                    } else {
                        // For simple values, just copy the values
                        collection.addAll(value.as());
                    }

                    ctx.set(collection);
                }
                return false;
            }

            @Override
            public boolean visitReferenceProperty(String propertyName, AutoBean<?> value, PropertyContext ctx) {
                value = AutoBeanUtils.getAutoBean(values.get(propertyName));
                if (value != null) {
                    if (isValueType(ctx.getType()) || isEntityType(ctx.getType())) {
                        /*
                         * Value proxies must be cloned upfront, since the value
                         * is replaced outright.
                         */
                        @SuppressWarnings("unchecked")
                        AutoBean<BaseProxy> valueBean = (AutoBean<BaseProxy>) value;
                        ctx.set(editProxy(toContext, (Class<T>) ctx.getType(), (T) valueBean.as()));
                    } else {
                        ctx.set(value.as());
                    }
                }
                return false;
            }

            @Override
            public boolean visitValueProperty(String propertyName, Object value, PropertyContext ctx) {
                ctx.set(values.get(propertyName));
                return false;
            }
        });
        return clone;
    }



    /**
     * Take ownership of a proxy instance and make it editable.
     */
    private static <T extends BaseProxy> T editProxy(RequestContext ctx, Class<T> clazz, T object) {
        AutoBean<T> toClone = AutoBeanUtils.getAutoBean(object);

        // Create editable copies
        AutoBean<T> parent = toClone;

        AutoBean<T> clone = (AutoBean<T>) ctx.create(clazz);
        AutoBean<T> cloned = cloneBeanProperties(toClone, clone, ctx);

        cloned.setTag(Constants.PARENT_OBJECT, parent);
        return cloned.as();
    }


    private static boolean isEntityType(Class<?> clazz) {
        return isAssignableTo(clazz, EntityProxy.class);
    }

    private static boolean isValueType(Class<?> clazz) {
        return isAssignableTo(clazz, ValueProxy.class);
    }

    public static boolean isAssignableTo(Class<?> thisClass, Class<?> assignableTo ) {
        if(thisClass == null || assignableTo == null) {
          return false;
        }

        if(thisClass.equals(assignableTo)) {
            return true;
        }

        Class<?> currentSuperClass = thisClass.getSuperclass();
        while(currentSuperClass != null) {
            if(currentSuperClass.equals(assignableTo)) {
                return true;
            }
            currentSuperClass = thisClass.getSuperclass();
        }
        return false;
    }
}

これが正常に完了した私の簡単な単体テストです。isAssignableFromメソッドで調査する必要があるエラーがありました。

   public void testCloneProxy() {
        DaoRequestFactory requestFactory = GWT.create(DaoRequestFactory.class);
        RequestContext fromContext = requestFactory.analyticsTaskRequest();

        AnalyticsOperationInputProxy from = fromContext.create(AnalyticsOperationInputProxy.class);

        from.setDisplayName("DISPLAY 1");
        from.setInputName("INPUT 1");


        RequestContext toContext = requestFactory.analyticsTaskRequest();

        AnalyticsOperationInputProxy to = toContext.create(AnalyticsOperationInputProxy.class);


        ProxyUtils.cloneBeanProperties(from, to, toContext);

        System.out.println("Cloned output " + to.getDisplayName());
        System.out.println("Cloned output " + to.getInputName());

        Assert.assertTrue("Display name not equal" , from.getDisplayName().equals(to.getDisplayName()));
        Assert.assertTrue("Input name not equal" , from.getInputName().equals(to.getInputName()));

    }
于 2013-04-13T02:37:54.127 に答える
2

Chris Hinshawはプロキシのクローンを作成するための優れたコードを作成したと思いますが、129行目の無限ループを避けて、1行を変更して改善します。

public class ProxyUtils {

public static <T extends BaseProxy> AutoBean<T> cloneBeanProperties(final T original, final T clone, final RequestContext toContext) {

    AutoBean<T> originalBean = AutoBeanUtils.getAutoBean(original);
    AutoBean<T> cloneBean = AutoBeanUtils.getAutoBean(clone);

    return cloneBeanProperties(originalBean, cloneBean, toContext);
}

/**
 * Shallow-clones an autobean and makes duplicates of the collection types.
 * A regular {@link AutoBean#clone} won't duplicate reference properties.
 */
public static <T extends BaseProxy> AutoBean<T> cloneBeanProperties(final AutoBean<T> toClone, final AutoBean<T> clone, final RequestContext toContext) {
    // NOTE: This may be a bad idea, i don't know if the STABLE_ID is the type or if it is the
    // actual server side reference to this object. Don't want it to update my original object.
    // I added this back in because I was getting an InstantionException in the locator I am
    // pretty sure this is because request factory server side could not resolve the class type. 
    // Maybe someone could shed some light on this one, if you know what the stable id is really
    // used for. 
    clone.setTag(STABLE_ID, clone.getTag(STABLE_ID));
    clone.setTag(Constants.VERSION_PROPERTY_B64, toClone.getTag(Constants.VERSION_PROPERTY_B64));
    clone.accept(new AutoBeanVisitor() {
        final Map<String, Object> values = AutoBeanUtils.getAllProperties(toClone);

        @Override
        public boolean visitCollectionProperty(String propertyName, AutoBean<Collection<?>> value, CollectionPropertyContext ctx) {
            // javac generics bug
            value = AutoBeanUtils.<Collection<?>, Collection<?>> getAutoBean((Collection<?>) values.get(propertyName));
            if (value != null) {
                Collection<Object> collection;
                if (List.class == ctx.getType()) {
                    collection = new ArrayList<Object>();
                } else if (Set.class == ctx.getType()) {
                    collection = new HashSet<Object>();
                } else {
                    // Should not get here if the validator works correctly
                    throw new IllegalArgumentException(ctx.getType().getName());
                }

                if (isValueType(ctx.getElementType()) || isEntityType(ctx.getElementType())) {
                    /*
                     * Proxies must be edited up-front so that the elements
                     * in the collection have stable identity.
                     */
                    for (Object o : value.as()) {
                        if (o == null) {
                            collection.add(null);
                        } else {
                            collection.add(editProxy(toContext, (Class<T>) ctx.getType(), (T) o));
                        }
                    }
                } else {
                    // For simple values, just copy the values
                    collection.addAll(value.as());
                }

                ctx.set(collection);
            }
            return false;
        }

        @Override
        public boolean visitReferenceProperty(String propertyName, AutoBean<?> value, PropertyContext ctx) {
            value = AutoBeanUtils.getAutoBean(values.get(propertyName));
            if (value != null) {
                if (isValueType(ctx.getType()) || isEntityType(ctx.getType())) {
                    /*
                     * Value proxies must be cloned upfront, since the value
                     * is replaced outright.
                     */
                    @SuppressWarnings("unchecked")
                    AutoBean<BaseProxy> valueBean = (AutoBean<BaseProxy>) value;
                    ctx.set(editProxy(toContext, (Class<T>) ctx.getType(), (T) valueBean.as()));
                } else {
                    ctx.set(value.as());
                }
            }
            return false;
        }

        @Override
        public boolean visitValueProperty(String propertyName, Object value, PropertyContext ctx) {
            ctx.set(values.get(propertyName));
            return false;
        }
    });
    return clone;
}



/**
 * Take ownership of a proxy instance and make it editable.
 */
private static <T extends BaseProxy> T editProxy(RequestContext ctx, Class<T> clazz, T object) {
    AutoBean<T> toClone = AutoBeanUtils.getAutoBean(object);

    // Create editable copies
    AutoBean<T> parent = toClone;

    AutoBean<T> clone = (AutoBean<T>) ctx.create(clazz);
    AutoBean<T> cloned = cloneBeanProperties(toClone, clone, ctx);

    cloned.setTag(Constants.PARENT_OBJECT, parent);
    return cloned.as();
}


private static boolean isEntityType(Class<?> clazz) {
    return isAssignableTo(clazz, EntityProxy.class);
}

private static boolean isValueType(Class<?> clazz) {
    return isAssignableTo(clazz, ValueProxy.class);
}

public static boolean isAssignableTo(Class<?> thisClass, Class<?> assignableTo ) {
    if(thisClass == null || assignableTo == null) {
      return false;
    }

    if(thisClass.equals(assignableTo)) {
        return true;
    }

    Class<?> currentSuperClass = thisClass.getSuperclass();
    if(currentSuperClass != null) {
        if(currentSuperClass.equals(assignableTo)) {
            return true;
        }
    }
    return false;
}

}

于 2013-11-24T20:31:30.610 に答える
1

そうではありません。

最初のレビューから（たまたま私自身によるものです）：

簡単に言うと、このパッチは十分ではなく、多くのルール（「クロスストリーム」）に違反し、テストに違反します。

問題で提案されているシリアル化/逆シリアル化の回避策を使用できない場合は、を使用してクローンを作成する方法があると思いますAutoBeanVisitor。

現在の「特定のプロキシを特定の時間に1つのRequestContextのみが編集できます-そのstableIdによって識別されます-」は本当に迷惑であり、実際には正当化されません（少なくとも、Request Factoryの初期の反復でした）。これは私が将来取り除こうとしていることですが、最初にやるべきことが他にもいくつかあります。

于 2012-09-17T15:59:53.423 に答える