Creation of custom comparator for map in groovy - groovy
I have class in groovy
class WhsDBFile {
String name
String path
String svnUrl
String lastRevision
String lastMessage
String lastAuthor
}
and map object
def installFiles = [:]
that filled in loop by
WhsDBFile dbFile = new WhsDBFile()
installFiles[svnDiffStatus.getPath()] = dbFile
now i try to sort this with custom Comparator
Comparator<WhsDBFile> whsDBFileComparator = new Comparator<WhsDBFile>() {
#Override
int compare(WhsDBFile o1, WhsDBFile o2) {
if (FilenameUtils.getBaseName(o1.name) > FilenameUtils.getBaseName(o2.name)) {
return 1
} else if (FilenameUtils.getBaseName(o1.name) > FilenameUtils.getBaseName(o2.name)) {
return -1
}
return 0
}
}
installFiles.sort(whsDBFileComparator);
but get this error java.lang.String cannot be cast to WhsDBFile
Any idea how to fix this? I need to use custom comparator, cause it will be much more complex in the future.
p.s. full source of sample gradle task (description of WhsDBFile class is above):
project.task('sample') << {
def installFiles = [:]
WhsDBFile dbFile = new WhsDBFile()
installFiles['sample_path'] = dbFile
Comparator<WhsDBFile> whsDBFileComparator = new Comparator<WhsDBFile>() {
#Override
int compare(WhsDBFile o1, WhsDBFile o2) {
if (o1.name > o2.name) {
return 1
} else if (o1.name > o2.name) {
return -1
}
return 0
}
}
installFiles.sort(whsDBFileComparator);
}
You can try to sort the entrySet() :
def sortedEntries = installFiles.entrySet().sort { entry1, entry2 ->
entry1.value <=> entry2.value
}
you will have a collection of Map.Entry with this invocation. In order to have a map, you can then collectEntries() the result :
def sortedMap = installFiles.entrySet().sort { entry1, entry2 ->
...
}.collectEntries()
sort can also take a closure as parameter which coerces to a Comparator's compare() method as below. Usage of toUpper() method just mimics the implementation of FilenameUtils.getBaseName().
installFiles.sort { a, b ->
toUpper(a.value.name) <=> toUpper(b.value.name)
}
// Replicating implementation of FilenameUtils.getBaseName()
// This can be customized according to requirement
String toUpper(String a) {
a.toUpperCase()
}
Related
Nested JSON with duplicate keys
I will have to process 10 billion Nested JSON records per day using NiFi (version 1.9). As part of the job, am trying to convert the nested JSON to csv using Groovy script. I referred the below Stack Overflow questions related to the same topic and came up with the below code. Groovy collect from map and submap how to convert json into key value pair completely using groovy But am not sure how to retrieve the value of duplicate keys. Sample json is defined in the variable "json" in the below code. key "Flag1" will be coming in multiple sections (i.e., "OF" & "SF"). I want to get the output as csv. Below is the output if I execute the below groovy code 2019-10-08 22:33:29.244000,v12,-,36178,0,0/0,10.65.5.56,sf,sf (flag1 key value is replaced by that key column's last occurrence value) I am not an expert in Groovy. Also please suggest if there is any other better approach, so that I will give a try. import groovy.json.* def json = '{"transaction":{"TS":"2019-10-08 22:33:29.244000","CIPG":{"CIP":"10.65.5.56","CP":"0"},"OF":{"Flag1":"of","Flag2":"-"},"SF":{"Flag1":"sf","Flag2":"-"}}' def jsonReplace = json.replace('{"transaction":{','{"transaction":[{').replace('}}}','}}]}') def jsonRecord = new JsonSlurper().parseText(jsonReplace) def columns = ["TS","V","PID","RS","SR","CnID","CIP","Flag1","Flag1"] def flatten flatten = { row -> def flattened = [:] row.each { k, v -> if (v instanceof Map) { flattened << flatten(v) } else if (v instanceof Collection && v.every {it instanceof Map}) { v.each { flattened << flatten(it) } } else { flattened[k] = v } } flattened } print "output: " + jsonRecord.transaction.collect {row -> columns.collect {colName -> flatten(row)[colName]}.join(',')}.join('\n') Edit: Based on the reply from #cfrick and #stck, I have tried the option and have follow up question below. #cfrick and #stck- Thanks for your response. Original source JSON record will have more than 100 columns and I am using "InvokeScriptedProcessor" in NiFi to trigger the Groovy script. Below is the original Groovy script am using in "InvokeScriptedProcessor" in which I have used Streams(inputstream, outputstream). Is this what you are referring. Am I doing anything wrong? import groovy.json.JsonSlurper class customJSONtoCSV implements Processor { def REL_SUCCESS = new Relationship.Builder().name("success").description("FlowFiles that were successfully processed").build(); def log static def flatten(row, prefix="") { def flattened = new HashMap<String, String>() row.each { String k, Object v -> def key = prefix ? prefix + "_" + k : k; if (v instanceof Map) { flattened.putAll(flatten(v, k)) } else { flattened.put(key, v.toString()) } } return flattened } static def toCSVRow(HashMap row) { def columns = ["CIPG_CIP","CIPG_CP","CIPG_SLP","CIPG_SLEP","CIPG_CVID","SIPG_SIP","SIPG_SP","SIPG_InP","SIPG_SVID","TG_T","TG_R","TG_C","TG_SDL","DL","I_R","UAP","EDBL","Ca","A","RQM","RSM","FIT","CSR","OF_Flag1","OF_Flag2","OF_Flag3","OF_Flag4","OF_Flag5","OF_Flag6","OF_Flag7","OF_Flag8","OF_Flag9","OF_Flag10","OF_Flag11","OF_Flag12","OF_Flag13","OF_Flag14","OF_Flag15","OF_Flag16","OF_Flag17","OF_Flag18","OF_Flag19","OF_Flag20","OF_Flag21","OF_Flag22","OF_Flag23","SF_Flag1","SF_Flag2","SF_Flag3","SF_Flag4","SF_Flag5","SF_Flag6","SF_Flag7","SF_Flag8","SF_Flag9","SF_Flag10","SF_Flag11","SF_Flag12","SF_Flag13","SF_Flag14","SF_Flag15","SF_Flag16","SF_Flag17","SF_Flag18","SF_Flag19","SF_Flag20","SF_Flag21","SF_Flag22","SF_Flag23","SF_Flag24","GF_Flag1","GF_Flag2","GF_Flag3","GF_Flag4","GF_Flag5","GF_Flag6","GF_Flag7","GF_Flag8","GF_Flag9","GF_Flag10","GF_Flag11","GF_Flag12","GF_Flag13","GF_Flag14","GF_Flag15","GF_Flag16","GF_Flag17","GF_Flag18","GF_Flag19","GF_Flag20","GF_Flag21","GF_Flag22","GF_Flag23","GF_Flag24","GF_Flag25","GF_Flag26","GF_Flag27","GF_Flag28","GF_Flag29","GF_Flag30","GF_Flag31","GF_Flag32","GF_Flag33","GF_Flag34","GF_Flag35","VSL_VSID","VSL_TC","VSL_MTC","VSL_NRTC","VSL_ET","VSL_HRES","VSL_VRES","VSL_FS","VSL_FR","VSL_VSD","VSL_ACB","VSL_ASB","VSL_VPR","VSL_VSST","HRU_HM","HRU_HD","HRU_HP","HRU_HQ","URLF_CID","URLF_CGID","URLF_CR","URLF_RA","URLF_USM","URLF_USP","URLF_MUS","TCPSt_WS","TCPSt_SE","TCPSt_WSFNS","TCPSt_WSF","TCPSt_EM","TCPSt_RSTE","TCPSt_MSS","NS_OPID","NS_ODID","NS_EPID","NS_TrID","NS_VSN","NS_LSUT","NS_STTS","NS_TCPPR","CQA_NL","CQA_CL","CQA_CLC","CQA_SQ","CQA_SQC","TS","V","PID","RS","SR","CnID","A_S","OS","CPr","CVB","CS","HS","SUNR","SUNS","ML","MT","TCPSL","CT","MS","MSH","SID","SuID","UA","DID","UAG","CID","HR","CRG","CP1","CP2","AIDF","UCB","CLID","CLCL","OPTS","PUAG","SSLIL"] return columns.collect { column -> return row.containsKey(column) ? row.get(column) : "" }.join(',') } #Override void initialize(ProcessorInitializationContext context) { log = context.getLogger() } #Override Set<Relationship> getRelationships() { return [REL_SUCCESS] as Set } #Override void onTrigger(ProcessContext context, ProcessSessionFactory sessionFactory) throws ProcessException { try { def session = sessionFactory.createSession() def flowFile = session.get() if (!flowFile) return flowFile = session.write(flowFile, { inputStream, outputStream -> def bufferedReader = new BufferedReader(new InputStreamReader(inputStream, 'UTF-8')) def jsonSlurper = new JsonSlurper() def line def header = "CIPG_CIP,CIPG_CP,CIPG_SLP,CIPG_SLEP,CIPG_CVID,SIPG_SIP,SIPG_SP,SIPG_InP,SIPG_SVID,TG_T,TG_R,TG_C,TG_SDL,DL,I_R,UAP,EDBL,Ca,A,RQM,RSM,FIT,CSR,OF_Flag1,OF_Flag2,OF_Flag3,OF_Flag4,OF_Flag5,OF_Flag6,OF_Flag7,OF_Flag8,OF_Flag9,OF_Flag10,OF_Flag11,OF_Flag12,OF_Flag13,OF_Flag14,OF_Flag15,OF_Flag16,OF_Flag17,OF_Flag18,OF_Flag19,OF_Flag20,OF_Flag21,OF_Flag22,OF_Flag23,SF_Flag1,SF_Flag2,SF_Flag3,SF_Flag4,SF_Flag5,SF_Flag6,SF_Flag7,SF_Flag8,SF_Flag9,SF_Flag10,SF_Flag11,SF_Flag12,SF_Flag13,SF_Flag14,SF_Flag15,SF_Flag16,SF_Flag17,SF_Flag18,SF_Flag19,SF_Flag20,SF_Flag21,SF_Flag22,SF_Flag23,SF_Flag24,GF_Flag1,GF_Flag2,GF_Flag3,GF_Flag4,GF_Flag5,GF_Flag6,GF_Flag7,GF_Flag8,GF_Flag9,GF_Flag10,GF_Flag11,GF_Flag12,GF_Flag13,GF_Flag14,GF_Flag15,GF_Flag16,GF_Flag17,GF_Flag18,GF_Flag19,GF_Flag20,GF_Flag21,GF_Flag22,GF_Flag23,GF_Flag24,GF_Flag25,GF_Flag26,GF_Flag27,GF_Flag28,GF_Flag29,GF_Flag30,GF_Flag31,GF_Flag32,GF_Flag33,GF_Flag34,GF_Flag35,VSL_VSID,VSL_TC,VSL_MTC,VSL_NRTC,VSL_ET,VSL_HRES,VSL_VRES,VSL_FS,VSL_FR,VSL_VSD,VSL_ACB,VSL_ASB,VSL_VPR,VSL_VSST,HRU_HM,HRU_HD,HRU_HP,HRU_HQ,URLF_CID,URLF_CGID,URLF_CR,URLF_RA,URLF_USM,URLF_USP,URLF_MUS,TCPSt_WS,TCPSt_SE,TCPSt_WSFNS,TCPSt_WSF,TCPSt_EM,TCPSt_RSTE,TCPSt_MSS,NS_OPID,NS_ODID,NS_EPID,NS_TrID,NS_VSN,NS_LSUT,NS_STTS,NS_TCPPR,CQA_NL,CQA_CL,CQA_CLC,CQA_SQ,CQA_SQC,TS,V,PID,RS,SR,CnID,A_S,OS,CPr,CVB,CS,HS,SUNR,SUNS,ML,MT,TCPSL,CT,MS,MSH,SID,SuID,UA,DID,UAG,CID,HR,CRG,CP1,CP2,AIDF,UCB,CLID,CLCL,OPTS,PUAG,SSLIL" outputStream.write("${header}\n".getBytes('UTF-8')) while (line = bufferedReader.readLine()) { def jsonReplace = line.replace('{"transaction":{','{"transaction":[{').replace('}}}','}}]}') def jsonRecord = new JsonSlurper().parseText(jsonReplace) def a = jsonRecord.transaction.collect { row -> return flatten(row) }.collect { row -> return toCSVRow(row) } outputStream.write("${a}\n".getBytes('UTF-8')) } } as StreamCallback) session.transfer(flowFile, REL_SUCCESS) session.commit() } catch (e) { throw new ProcessException(e) } } #Override Collection<ValidationResult> validate(ValidationContext context) { return null } #Override PropertyDescriptor getPropertyDescriptor(String name) { return null } #Override void onPropertyModified(PropertyDescriptor descriptor, String oldValue, String newValue) { } #Override List<PropertyDescriptor> getPropertyDescriptors() { return [] as List } #Override String getIdentifier() { return null } } processor = new customJSONtoCSV() If I should not use "collect" then what else I need to use to create the rows. In the output flow file, the record output is coming inside []. I tried the below but it is not working. Not sure whether am doing the right thing. I want csv output without [] return toCSVRow(row).toString()
If you know what you want to extract exactly (and given you want to generate a CSV from it) IMHO you are way better off to just shape the data in the way you later want to consume it. E.g. def data = new groovy.json.JsonSlurper().parseText('[{"TS":"2019-10-08 22:33:29.244000","CIPG":{"CIP":"10.65.5.56","CP":"0"},"OF":{"Flag1":"of","Flag2":"-"},"SF":{"Flag1":"sf","Flag2":"-"}}]') extractors = [ { it.TS }, { it.V }, { it.PID }, { it.RS }, { it.SR }, { it.CIPG.CIP }, { it.CIPG.CP }, { it.OF.Flag1 }, { it.SF.Flag1 },] def extract(row) { extractors.collect{ it(row) } } println(data.collect{extract it}) // ⇒ [[2019-10-08 22:33:29.244000, null, null, null, null, 10.65.5.56, 0, of, sf]] As stated in the other answer, due to the sheer amount of data you are trying to convert:: Make sure to use a library to generate the CSV file from that, or else you will hit problems with the content, you try to write (e.g. line breaks or the data containing the separator char). Don't use collect (it is eager) to create the rows.
The idea is to modify "flatten" method - it should differentiate between same nested keys by providing parent key as a prefix. I've simplified code a bit: import groovy.json.* def json = '{"transaction":{"TS":"2019-10-08 22:33:29.244000","CIPG":{"CIP":"10.65.5.56","CP":"0"},"OF":{"Flag1":"of","Flag2":"-"},"SF":{"Flag1":"sf","Flag2":"-"}}' def jsonReplace = json.replace('{"transaction":{','{"transaction":[{').replace('}}','}}]') def jsonRecord = new JsonSlurper().parseText(jsonReplace) static def flatten(row, prefix="") { def flattened = new HashMap<String, String>() row.each { String k, Object v -> def key = prefix ? prefix + "." + k : k; if (v instanceof Map) { flattened.putAll(flatten(v, k)) } else { flattened.put(key, v.toString()) } } return flattened } static def toCSVRow(HashMap row) { def columns = ["TS","V","PID","RS","SR","CnID","CIP","OF.Flag1","SF.Flag1"] // Last 2 keys have changed! return columns.collect { column -> return row.containsKey(column) ? row.get(column) : "" }.join(', ') } def a = jsonRecord.transaction.collect { row -> return flatten(row) }.collect { row -> return toCSVRow(row) }.join('\n') println a Output would be: 2019-10-08 22:33:29.244000, , , , , , , of, sf
Clone of list still correct the original list
In groovy the original value get overwritten when I change values in a clone list. Does anyone know if I am doing it wrong or it is a bug older groovy? I am doing something like this: List<Foo> myFooList = fooList.newFoos.findAll { it.type == "Types} List<Foo> newFoo = fooList.oldFoos.findAll { it.type == "Types}.clone() newFoo.each { it.value = "neeeew value" } Foo fooOne = newFoo.each { foooo -> fooTwo = fooList.oldFoos.find { it.id == foooo.id} if(fooTwo.value != foooo.value) { //Here it should go... but it turns out that fooTwo.value == foooo.value } }
the clone method called on list produces a new list but with the same objects in it. you want to build new list with new objects. here is an example: #groovy.transform.ToString class Foo{ String type String value } def fooList = [ new Foo(type:"Types", value:'old value1'), new Foo(type:"Not", value:'old value2'), new Foo(type:"Types", value:'old value3'), new Foo(type:"Not", value:'old value4'), ] def newFooList = fooList. findAll{it.type=='Types'}. collect{ new Foo(type:it.type, value:"new value") } //build new array with transformed elements //check the original list fooList.each{assert it.value!='new value'} //check new list newFooList.each{assert it.value=='new value'} assert newFooList.size()==2 println fooList println newFooList
I solved the issue by adding clone of the element as well, any way it became to much of cowboy fix: List<Foo> myFooList = fooList.newFoos.findAll { it.type == "Types} List<Foo> newFoo = fooList.oldFoos.findAll { it.type == "Types}.collect {it.clone()} newFoo.each { it.value = "neeeew value" } Foo fooOne = newFoo.each { foooo -> fooTwo = fooList.oldFoos.find { it.id == foooo.id} if(fooTwo.value != foooo.value) { //Here it should go... but it turns out that fooTwo.value == foooo.value } }
Is it possible to mock accessors by Mockito in Kotlin?
Is it possible to mock getter and setter of the property by Mockito? Something like this: #Test fun three() { val m = mock<Ddd>() { // on { getQq() }.doReturn("mocked!") } assertEquals("mocked!", m.qq) } open class Ddd { var qq : String = "start" set(value) { field = value + " by setter" } get() { return field + " by getter" } }
To mock getter just write: val m = mock<Ddd>() `when`(m.qq).thenReturn("42") also i suggest to use mockito-kotlin, to use useful extensions and functions like whenever: val m = mock<Ddd>() whenever(m.qq).thenReturn("42")
Complementing IRus' answer, you could also use the following syntax: val mockedObj = mock<SomeClass> { on { funA() } doReturn "valA" on { funB() } doReturn "valB" } or val mockedObj = mock<SomeClass> { on(it.funA()).thenReturn("valA") on(it.funB()).thenReturn("valB") }
How to do Groovy method signatures with Python-style kwargs AND default values?
I might be asking too much, but Groovy seems super flexible, so here goes... I would like a method in a class to be defined like so: class Foo { Boolean y = SomeOtherClass.DEFAULT_Y Boolean z = SomeOtherClass.DEFAULT_Z void bar(String x = SomeOtherClass.DEFAULT_X, Integer y = this.y, Boolean z = this.z) { // ... } } And to be able to provide only certain arguments like so: def f = new Foo(y: 16) f.bar(z: true) // <-- This line throws groovy.lang.MissingMethodException! I am trying to provide an API that is both flexible and type safe, which is the problem. The given code is not flexible in that I would have to pass in (and know as the user of the API) the default value for x in order to call the method. Here are some challenges for the solution I want: Type safety is a must--no void bar(Map) signatures unless the keys can somehow be made type safe. I realize with this I could do the type checking in the method body, but I'm trying to avoid that level of redundancy as I have many of this "kind" of method to write. I could use a class for each method signature--something like: class BarArgs { String x = SomeOtherClass.DEFAULT_X String y String z } And define it like: void bar(BarArgs barArgs) { // ... } And call it using my desired way using the map constructor: f.bar(z: true), but my problem lies in the object's default on y. There's no way to handle that (that I know of) without having to specify it when calling the method as in: f.bar(y: f.y, z: true). This is fine for my little sample, but I'm looking at 20-30 optional parameters on some methods. Any suggestions (or questions if needed) are welcome! Thank you for taking a look.
Interesting question. I've interpreted your requirements like this The class should have a set of default properties. Each method should have a set of default arguments. The method defaults override the class defaults. Each method can have additional arguments, not existing on the class. The method arguments should not modify the class instance. Provided arguments needs to be checked for type. I was not sure about number 5 since it is not explicitly specified, but it looked like that was what you wanted. As far as I know, there is nothing built-in in groovy to support all this, but there are several ways to make it work in a "simple-to-use" manner. One way that comes to mind is to create specialized argument classes, but only use maps as the arguments in the methods. With a simple super-class or trait to verify and set the properties, it is a one-liner to get the actual arguments for each method. Here is a trait and some examples that can be used as a starting point: trait DefaultArgs { void setArgs(Map args, DefaultArgs defaultArgs) { if (defaultArgs) { setArgs(defaultArgs.toArgsMap()) } setArgs(args) } void setArgs(Map args) { MetaClass thisMetaClass = getMetaClass() args.each { name, value -> assert name instanceof String MetaProperty metaProperty = thisMetaClass.getMetaProperty(name) assert name && metaProperty != null if (value != null) { assert metaProperty.type.isAssignableFrom(value.class) } thisMetaClass.setProperty(this, name, value) } } Map toArgsMap() { def properties = getProperties() properties.remove('class') return properties } } With this trait is it easy to create specialized argument classes. #ToString(includePackage = false, includeNames = true) class FooArgs implements DefaultArgs { String a = 'a' Boolean b = true Integer i = 42 FooArgs(Map args = [:], DefaultArgs defaultArgs = null) { setArgs(args, defaultArgs) } } #ToString(includePackage = false, includeNames = true, includeSuper = true) class BarArgs extends FooArgs { Long l = 10 BarArgs(Map args = [:], FooArgs defaultArgs = null) { setArgs(args, defaultArgs) } } And a class that uses these arguments: class Foo { FooArgs defaultArgs Foo(Map args = [:]) { defaultArgs = new FooArgs(args) } void foo(Map args = [:]) { FooArgs fooArgs = new FooArgs(args, defaultArgs) println fooArgs } void bar(Map args = [:]) { BarArgs barArgs = new BarArgs(args, defaultArgs) println barArgs } } Finally, a simple test script; output of method invocations in comments def foo = new Foo() foo.foo() // FooArgs(a:a, b:true, i:42) foo.foo(a:'A') // FooArgs(a:A, b:true, i:42) foo.bar() // BarArgs(l:10, super:FooArgs(a:a, b:true, i:42)) foo.bar(i:1000, a:'H') // BarArgs(l:10, super:FooArgs(a:H, b:true, i:1000)) foo.bar(l:50L) // BarArgs(l:50, super:FooArgs(a:a, b:true, i:42)) def foo2 = new Foo(i:16) foo2.foo() // FooArgs(a:a, b:true, i:16) foo2.foo(a:'A') // FooArgs(a:A, b:true, i:16) foo2.bar() // BarArgs(l:10, super:FooArgs(a:a, b:true, i:16)) foo2.bar(i:1000, a:'H') // BarArgs(l:10, super:FooArgs(a:H, b:true, i:1000)) foo2.bar(l:50L) // BarArgs(l:50, super:FooArgs(a:a, b:true, i:16)) def verifyError(Class thrownClass, Closure closure) { try { closure() assert "Expected thrown: $thrownClass" && false } catch (Throwable e) { assert e.class == thrownClass } } // Test exceptions on wrong type verifyError(PowerAssertionError) { foo.foo(a:5) } verifyError(PowerAssertionError) { foo.foo(b:'true') } verifyError(PowerAssertionError) { foo.bar(i:10L) } // long instead of integer verifyError(PowerAssertionError) { foo.bar(l:10) } // integer instead of long // Test exceptions on missing properties verifyError(PowerAssertionError) { foo.foo(nonExisting: 'hello') } verifyError(PowerAssertionError) { foo.bar(nonExisting: 'hello') } verifyError(PowerAssertionError) { foo.foo(l: 50L) } // 'l' does not exist on foo
How do I create an OrderBy statement using a reflected value?
I would like to create a method that orders an IEnumerable List by a given property where the property is passed into the method by a string i.e. (Mind you the first code example does not work, but the second does and is what I am trying to emulate dynamically). string sortName = "SerialNumber"; IEnumerable<PartSummary> partList = FunctionToCreateList(); partOrderedList = partList.OrderBy(what do I stick in here); that would be equivalent to IEnumerable<PartSummary> partList = FunctionToCreateList(); partOrderedList = partList.OrderBy(p => p.SerialNumber); How can I accomplish this?
Are you saying you want to pass the order by in to your method? If so, you can use this: Expression<Func<PartSummary, bool>> orderByClause Then you can do this: partOrderedList = partList.OrderBy(orderByClause); Then you can handle your order by in your business layer or wherever you wish. Okay, update: If you want to pass in the column name as a string you can do something like as follows: Create a static class for an extension method (reference: http://social.msdn.microsoft.com/Forums/en-US/linqprojectgeneral/thread/39028ad2-452e-409f-bc9e-d1b263e921f6/): static class LinqExtensions { public static IQueryable<T> OrderBy<T>(this IQueryable<T> source, string sortingColumn, bool isAscending) { if (String.IsNullOrEmpty(sortingColumn)) { return source; } ParameterExpression parameter = Expression.Parameter(source.ElementType, String.Empty); MemberExpression property = Expression.Property(parameter, sortingColumn); LambdaExpression lambda = Expression.Lambda(property, parameter); string methodName = isAscending ? "OrderBy" : "OrderByDescending"; Expression methodCallExpression = Expression.Call(typeof(Queryable), methodName, new Type[] { source.ElementType, property.Type }, source.Expression, Expression.Quote(lambda)); return source.Provider.CreateQuery<T>(methodCallExpression); } } Then you can create your method: static IQueryable<PartSummary> FunctionToCreateList() { IList<PartSummary> list = new List<PartSummary>(); list.Add(new PartSummary { Id = 1, SerialNumber = "A", }); list.Add(new PartSummary { Id = 2, SerialNumber = "B", }); return list.AsQueryable(); } And then call your method: static void Main(string[] args) { IQueryable<PartSummary> partOrderedList = FunctionToCreateList(); PartSummary partSummary = new PartSummary(); string sortBy = "Id"; partOrderedList = partOrderedList.OrderBy(sortBy, false); foreach (PartSummary summary in partOrderedList) { Console.WriteLine(summary.Id + ", " + summary.SerialNumber); } Console.ReadLine(); } Now you can pass in the column name as a string and sort. Hope this helps!
You can also avoid extending and just use a compiled expression tree to accomplish this: public Func<T, object> ResolveToProperty<T>(String propertyName) { Type t = typeof(T); var paramExpression = Expression.Parameter(t, "element"); var propertyExpression = Expression.Property(paramExpression, propertyName); return Expression.Lambda<Func<T, object>>(propertyExpression, paramExpression).Compile(); } string sortName = "SerialNumber"; IEnumerable<PartSummary> partList = FunctionToCreateList(); var partOrderedList = partList.OrderBy(ResolveToProperty<PartSummary>(sortName));