[picocontainer-scm] [jira] Commented: (PICO-382) CollectionComponentParameter having trouble with GenericArrayType

    [ https://jira.codehaus.org/browse/PICO-382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=282121#comment-282121 ] 

Chris Simmons commented on PICO-382:
------------------------------------

It seems that you need a wildcard to cause this problem and the wildcard need not occur on the array itself.  Here's a similar test showing the problem.  The test fails for a constructor like this:-

(final Paramaterized<?> paramaterized, final String...strings)

> CollectionComponentParameter having trouble with GenericArrayType
> -----------------------------------------------------------------
>
>                 Key: PICO-382
>                 URL: https://jira.codehaus.org/browse/PICO-382
>             Project: PicoContainer
>          Issue Type: Bug
>    Affects Versions: 3.0
>            Reporter: Michael Rimov
>         Attachments: TestWildcardTypesInPico.java
>
>
> The following test case will throw IllegalArgumentException from CollectionComponentParameter.getCollectionType(Type)
> I tried changing the code to:
> {code}
>         if (expectedType instanceof Class) {
>             return getCollectionType((Class) expectedType);
>         } else if (expectedType instanceof ParameterizedType) {
>             ParameterizedType type = (ParameterizedType) expectedType;
>             return getCollectionType(type.getRawType());
>         } else if (expectedType instanceof GenericArrayType) {
>         	return getCollectionType( ((GenericArrayType)expectedType).getGenericComponentType());
>         }
> {code}
> but that wouldn't resolve the collection type and my understanding of the JDK 1.5 Type system is so low that I'm shooting in the dark.  Paul, do you have any ideas how to fix this?  (Or is my test case wrong usage?)
> {code}
> public class GenericArrayTestCase {
> 	public static interface GenericInterface<T> {
> 		
> 	}
> 	
> 	public static class GenericOne implements GenericInterface<String>{
> 		
> 	}
> 	
> 	public static class GenericReceiver {
> 		public final GenericInterface<?>[] items;
> 		public GenericReceiver(GenericInterface<?>[] items) {
> 			this.items = items;
> 			
> 		}
> 	}
> 	
> 	@Test
> 	public void testArraysWithGenericArguments() {
> 		MutablePicoContainer pico = new PicoBuilder().withCaching().build();
> 		
> 		pico.addComponent("one", GenericOne.class)
> 			.addComponent("two", GenericOne.class)
> 			.addComponent(GenericReceiver.class, GenericReceiver.class, new CollectionComponentParameter(false));
> 		
> 		GenericReceiver receiver = pico.getComponent(GenericReceiver.class);
> 		assertEquals(2, receiver.items.length);
> 	}
> }
> {code}

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

<hr/>
<p>
To unsubscribe from this list please visit:
</p>
<p>
    <a href="http://xircles.codehaus.org/manage_email">http://xircles.codehaus.org/manage_email</a>