Solved

Function to sum a range and set BG color based on result

Posted on 2011-02-15
8
538 Views
Last Modified: 2013-11-25
I cannot get this to work
I have searched for a solution but I have not found one.
Close but not there yet. The statement  rCell.Interior.Color = iCol1 fails. Please help.
The content of the cell: "= SumAndHighlight(D26,D19:D25)"
Also, how can I indicate the cell that contains the formula without explicitly passing it in? i.e. Not have to pass in the first parameter "D26" but have the color in D26 change?
***
Public Function SumAndHighlight(rCell As Range, rSumRange As Range)
  Dim vResult As Integer
  Dim iCol1 As Long
  Dim iCol2 As Long
  Dim iCol3 As Long
  iCol1 = Range("a70").Interior.Color 'Red
  iCol2 = Range("a71").Interior.Color 'Green
  iCol3 = Range("a72").Interior.Color 'Yellow
  vResult = WorksheetFunction.Sum(rSumRange)
  If vResult > 1 Then
    rCell.Interior.Color = iCol1   '<------- This statement fails
  ElseIf vResult < 1 Then
    rCell.Interior.Color = iCol2   '<------- This statement fails
  Else
    rCell.Interior.Color = iCol3   '<------- This statement fails
  End If
  SumAndHighlight = vResult
End Function
0
Comment
Question by:lpollis
  • 3
  • 2
  • 2
  • +1
8 Comments
 
LVL 11

Expert Comment

by:Runrigger
ID: 34897755
Can you tell me the error if you have one?

Can you also show me the calling routine, or attach an example.

  If vResult > 1 Then
    rCell.Interior.Color = iCol1   '<------- I can get this wor work
  ElseIf vResult < 1 Then
    rCell.Interior.Color = iCol2   '<------- I can also get this to work
  Else
    rCell.Interior.Color = iCol3   '<------- This condition will currently not be met because either one of the above will trap first!
  End If
0
 
LVL 24

Accepted Solution

by:
StephenJR earned 250 total points
ID: 34897807
I think a function can only affect the cell in which it is placed, and cannot change formatting.
0
 
LVL 11

Expert Comment

by:Runrigger
ID: 34897855
Ah, that could be why mine works, I converted it to a sub!

Author, what is the reason for you having a function?
0
Is Your AD Toolbox Looking More Like a Toybox?

Managing Active Directory can get complicated.  Often, the native tools for managing AD are just not up to the task.  The largest Active Directory installations in the world have relied on one tool to manage their day-to-day administration tasks: Hyena. Start your trial today.

 

Author Comment

by:lpollis
ID: 34898200
Thanks for your responses.
I'll take them in order.

Runrigger: The error is "A value use in the formula is of the wrong data type." Also, the third condition is met if the value is equal to 1.

StephenJR: I have seen some examples (I believe) where the color can be changed just by selecting the cell in a function call. Or colors of other cells can be set when a cell has the focus. However, you may be right but how to know for sure. Is there any documentation that you can point me to? I would love to know for sure.

Runrigger:  I can try to convert it to a Sub but I wanted the Function call as it is placed in the cell where I want that action. I may have other cells that I do not want this color change. I guess I can pursue this a little further but I really wanted to use the function call.  
0
 
LVL 11

Expert Comment

by:Runrigger
ID: 34898222
You would never believe that I was a mathematician, LOL!

Busy day at work.
0
 
LVL 24

Expert Comment

by:StephenJR
ID: 34898228
See here: http://support.microsoft.com/kb/170787

Runrigger's approach is one way to go.
0
 
LVL 41

Expert Comment

by:dlmille
ID: 34898873
I believe you can do this with conditional formatting.  I don't think a function (or subroutine) is necessary.

Since you initially were developing a UDF, I assume you want to "point at the cell" and do some operation.  Certainly a macro would work on the active cell.

However, isn't it just as easy (or easier) to setup conditional formatting?

Perhaps it depends on how many of these you're putting out there.  But you can't do =subroutine(,) in a formula, so you have to specify the parameters and run the macro - so this could be less labor intensive.

Anyway, another option.

Dave
ColorCellBasedonSumRange.xlsm
0
 

Author Closing Comment

by:lpollis
ID: 34899076
My thanks to all the responders. I am awarding the solution to Stephen because he identified the problem (You cannot do this in a formula!) and pointed me to the documentation that supported his point.
And I agree with dimille in that this can be done with conditional formatting (at least for this requirement). I was looking beyond the capabilities of conditional formatting and I will have to use subroutines if that becomes necessary.
I was surprised that the error message didn't help more. It would have made things easier.
Thanks to all for your assistance.
Lp
0

Featured Post

Is Your AD Toolbox Looking More Like a Toybox?

Managing Active Directory can get complicated.  Often, the native tools for managing AD are just not up to the task.  The largest Active Directory installations in the world have relied on one tool to manage their day-to-day administration tasks: Hyena. Start your trial today.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Freeze panes is an option within all variants of Excel to enable parts of a sheet to remain stationary when the cursor is in another part of the sheet. This is a very useful feature which is overlooked or under used.
When you see single cell contains number and text, and you have to get any date out of it seems like cracking our heads.
This lesson covers basic error handling code in Microsoft Excel using VBA. This is the first lesson in a 3-part series that uses code to loop through an Excel spreadsheet in VBA and then fix errors, taking advantage of error handling code. This l…
Finds all prime numbers in a range requested and places them in a public primes() array. I've demostrated a template size of 30 (2 * 3 * 5) but larger templates can be built such 210  (2 * 3 * 5 * 7) or 2310  (2 * 3 * 5 * 7 * 11). The larger templa…

832 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question