convert the list into array

Question

Hi team, 
 i have an expression rule to get the doctors with count of patients assigned to them with the disease but i want to get the total patience count for that particular doctor with array of disease like eg: 
 swift:3:{cance, Gallstones,Emotional Disorder }, Please help me to achieve this format 
 
 substitute(substitute(substitute(substitute(
 substitute(
 substitute(
 a!queryEntity(
 entity: cons!PO_MEDICALINFO,
 query: a!query(
 logicalexpression: a!queryLogicalExpression(
 operator: "AND",
 filters: {
 a!queryFilter(
 field: "reasonForVisit", 
 operator: "in",
 value:ri!reasonForVisit 
 ),
 a!queryFilter(
 field:"appointmentDate",
 operator: "between",
 value: {
 todatetime(ri!createddatefrom),
 todatetime(ri!createddateto)
 },
 applywhen: and(
 not(rule!APN_isEmpty(ri!createddatefrom)),
 not(rule!APN_isEmpty(ri!createddateto)))
 
 ),

 
 },
 ignorefilterswithemptyvalues: true()
 
 ),
 aggregation: a!queryAggregation(
 aggregationColumns: {
 a!queryAggregationColumn(
 field: "reportingDoctor",
 isGrouping: true,
 ),
 a!queryAggregationColumn(
 field: "patientId",
 alias:"patientCount",
 aggregationFunction:"COUNT"

 ),
 a!queryAggregationColumn(
 field: "reasonForVisit",
 isGrouping: true
 )
 
 
 },

 ),
 pagingInfo: a!pagingInfo(
 startIndex: 1,
 batchSize: -1,
 sort: a!sortInfo(
 field: "patientCount",
 ascending: true()
 )
 )
 ),
 fetchTotalCount: true
 ).data,"patientCount",""),"reportingDoctor:",""),",",""),"[",""),"]",""),"reasonForVisit","")

ajhick · Accepted Answer

Haha, I hadn't heard of property()!! Thanks. 
 I agree with everything you've said. I've used reject() since before a!forEach() existed but never thought of using apply() either so I've just always used it. I might spawn different discussions (but also search for other discussions first) to dive deeper into why one is better than the other.

ajhick · Answer

Hi Mike, looked further into property() versus index() and I can see your crusade over quite some time! Unfortunately you've not gained a new follower I didn't know property() ever existed and I've never had difficulty in discerning what was being indexed and in my mind positional indexing is simply one type of index. 
 I get your point but it just doesn't solve any problem for me. Especially as I abstract away nested indexes into a rule I stole from Stefan (He calls it deep index). Converting to an exact replica of the same function just with another name doesn't hold much value and would break consistency in my environment which would actually lead to more confusion and harder to read code. 
 Good luck spreading the good word!

ajhick · Answer

Hi mamathak, 
 Does this help? I stepped through each transformation very slowly so that there wasn't one big expression that is hard to pick apart. Let me know if this hits the mark. 
 Please note I've assumed that the 'reasonForVisit' is always delimited by a semicolon and then a space "; " and that you want this to be a unique list of all reasons for each doctor. 
 a!localVariables(
 local!data: {
 {
 reportingDoctor: "Meenakshi",
 patientCount: 1,
 reasonForVisit: "Cancer"
 },
 {
 reportingDoctor: "Meenakshi",
 patientCount: 1,
 reasonForVisit: "Gout; Fainting Spells"
 },
 {
 reportingDoctor: "Sriram",
 patientCount: 1,
 reasonForVisit: "Diabetes; High Blood Pressure; Other"
 },
 {
 reportingDoctor: "Sai Kiran",
 patientCount: 1,
 reasonForVisit: "Heart Disease"
 },
 {
 reportingDoctor: "Meenakshi",
 patientCount: 1,
 reasonForVisit: "Gout"
 },
 {
 reportingDoctor: "Sai Kiran",
 patientCount: 1,
 reasonForVisit: "Gallstones"
 },
 {
 reportingDoctor: "Sai Kiran",
 patientCount: 1,
 reasonForVisit: "Fainting Spells"
 }
 },
 /*Get the unique list of doctors*/
 local!uniqueListOfDoctors: reject(
 fn!isnull,
 union(
 touniformstring(
 index(
 local!data,
 "reportingDoctor",
 {}
 )
 ),
 tostring({})
 )
 ),
 /*Iterate through this list of doctors to construct the string*/
 a!forEach(
 items: local!uniqueListOfDoctors,
 expression: a!localVariables(
 /*Get the indexes of were this doctor is in the data*/
 local!indexesOfThisDoctor: wherecontains(
 fv!item,
 touniformstring(
 index(
 local!data,
 "reportingDoctor",
 {}
 )
 )
 ),
 
 /*Get the data for this doctor*/
 local!dataForThisDoctor: index(
 local!data,
 local!indexesOfThisDoctor,
 {}
 ),
 
 /*Get the number of times this docter appears in the data*/
 local!countForThisDoctor: length(local!dataForThisDoctor),
 
 /*Get the reasons for this visit for this doctor (In a "raw" format where some entries may be delimited by "; " */
 local!reasonsForVisitRaw: index(
 local!dataForThisDoctor,
 "reasonForVisit",
 {}
 ),
 
 /*Turn the raw reasons for a visit into an array*/
 local!reasonsForVisitArray: touniformstring(
 a!flatten(
 a!forEach(
 items: local!reasonsForVisitRaw,
 expression: split(
 fv!item,
 "; "
 )
 )
 )
 ),
 
 /*Reasons may be duplicated so get a unique list*/
 local!reasonsForVisitUniqueArray: reject(
 fn!isnull,
 union(
 local!reasonsForVisitArray,
 tostring({})
 )
 ),
 
 /*Convert the unique array into a string*/
 local!reasonsForVisitAsString: joinarray(
 local!reasonsForVisitUniqueArray,
 ","
 ),
 
 /*Construct the final string*/
 concat(
 fv!item,
 ":",
 local!countForThisDoctor,
 ":{",
 local!reasonsForVisitAsString,
 "}"
 )
 
 )
 )
)

Mike Schmitt · Answer

This is along the lines of what I was thinking of suggesting. 
 My only personal code-cleanliness nitpicks would be 1) to avoid calling reject() at all costs, instead preferring to call a!forEach and return "{}" for any empty entries as necessary, and 2) avoid calling " union() " against an empty set to de-duplicate, instead preferring to set up a helper rule in advance such as " rule!APN_Distinct() " which takes an array input and unions it against itself (this by itself reduces some chunks of code from 4 or 5 lines to 1). 
 [And of course, my personal soap-box position STILL maintains that optimally-readable code should use property() when getting a CDT property instead of index() , which should be used to get indexes (i.e. a positional index of an array). But around here you'd think nobody's ever even heard of property() ... ]

Mike Schmitt · Answer

When you say "upon giving the doctor name..." - maybe you could pass the doctor name into the Query Entity rule as a filter, then the resulting list would only be visits to that particular doctor, and you could arrange the "reasonForVisit" properties in whatever way you require? Or are you after some sort of enumeration of ALL doctors followed by an array of ALL visit reasons for that doctor, all at once? This is trickier but should be doable.

convert the list into array

Top Replies