Solved

Databinding a Checkbox List

Posted on 2010-08-28
9
645 Views
Last Modified: 2012-05-10
Hi all.

I have an edit form that contains a checkbox list, then has an "other" box below it so users can type in an answer if none of the checkboxes apply. I render the list, and then I need to check the boxes that were in the database. Then I need to fill the "other" box if needed. I can't seem to figure out 2 things. First, I can't change the textbox value in the checkboxlist databound sub. That's the hard part. The other part that baffles me is that my code to check the boxes feels very sloppy. If I try to use something like:

cboCategories.Items.FindByValue(dbReader("category")).Selected = True

I get a nasty object error. Any ideas?

Here is my code for the databound sub:

 
Protected Sub cboCategories_DataBound(ByVal sender As Object, ByVal e As System.EventArgs) Handles cboCategories.DataBound

        If IsNumeric(hfCommunicationID.Value) Then
            Dim cboCategoryListItem As ListItem
            Dim foundAMatch As Boolean = False

            '--- load categories
            dbConnection.SPName("sp_Communications")
            dbConnection.SPParameter("Action", "SELECTCATEGORIES")
            dbConnection.SPParameter("communicationID", hfCommunicationID.Value)

            Dim dbReader As SqlDataReader
            dbReader = dbConnection.SPExecuteDataReader
            While dbReader.Read
                For Each cboCategoryListItem In cboCategories.Items
                    If cboCategoryListItem.Value = dbReader("category") Then
                        cboCategoryListItem.Selected = True
                        foundAMatch = True
                    End If
                Next
                If Not foundAMatch Then
                    txtOtherCategory.Text = dbReader("category").ToString
                End If
                foundAMatch = False
            End While
            dbConnection.SPCloseDataReader()
        End If

    End Sub

Open in new window

0
Comment
Question by:DEWebWorks
  • 5
  • 3
9 Comments
 
LVL 30

Expert Comment

by:MlandaT
ID: 33552442
1. The problemn with the check is what happens if the value is not found? You may need to implement this as two lines:

cboCategories.Items.FindByValue(dbReader("category")).Selected = True

should perhaps be:

If not isdbnumm(dbReader("category")) then

      dim chk as checkbox = cboCategories.Items.FindByValue(dbReader("category"))
.Selected = True
0
 
LVL 30

Expert Comment

by:MlandaT
ID: 33552463
1. The problemn with the check is what happens if the value is not found? You may need to implement this as two lines:

cboCategories.Items.FindByValue(dbReader("category")).Selected = True

should perhaps be:

If not isdbnumm(dbReader("category")) then

      dim chk as ListItem = cboCategories.Items.FindByValue(dbReader("category"))
      if chk isnot nothing then chk.Selected = True

End if

2. To implement the code for the "Other" and changing the textbox value in the databound... The problem is probably that you are looping through the datareader adn the value that get's set in the textbox is largely dependant on the dbReader("category") for the very last row in your datareader. I'm not sure about your use of loops there. Doesnt look right.
0
 
LVL 44

Expert Comment

by:Arthur_Wood
ID: 33552600
shouldn't this:

If not isdbnumm(dbReader("category")) then

be

If not isdbnull(dbReader("category")) then

?

AW
0
Efficient way to get backups off site to Azure

This user guide provides instructions on how to deploy and configure both a StoneFly Scale Out NAS Enterprise Cloud Drive virtual machine and Veeam Cloud Connect in the Microsoft Azure Cloud.

 
LVL 30

Expert Comment

by:MlandaT
ID: 33552621
Arthur_Wood:

Typo...

But yes indeed you are right. Thanks. It is just meant to be a check to make sure that we dont try to access the value if it is null.
0
 

Author Comment

by:DEWebWorks
ID: 33552938
Sorry, I didn't outline the structure very well. The categories come from a lookup table - no chance of a null.

MlandaT, your solution to problem 1 worked out perfectly! Now I'm just down to why the "other" box won't fill. The basic structure is this. When someone saves the parent record (tblCommunications), first all records are deleted in the tblCommunications_Categories for that ID (it's a wipe and replace scenario). Then I loop through all checkboxes to save the checked ones. Then if something is in the "Other Category" textbox, I save that. So, upon editing, the dbReader can only have the checked boxes and the value of the extra text box.

This "should" not have anything to do with the order that things are rendered, right?

My code now looks like this:

While dbReader.Read
                cboCategoryListItem = cboCategories.Items.FindByValue(dbReader("category"))
                If cboCategoryListItem IsNot Nothing Then
                    cboCategoryListItem.Selected = True
                    cboCategoryListItem = Nothing
                Else
                    txtOtherCategory.Text = dbReader("category").ToString
                End If
            End While

Open in new window

0
 
LVL 30

Expert Comment

by:MlandaT
ID: 33556238
Let's see the code for saving
0
 

Author Comment

by:DEWebWorks
ID: 33558135
I've confirmed the data exists in the database.


'--- save categories
        Dim cboCategoryListItem As ListItem
        For Each cboCategoryListItem In cboCategories.Items
            If cboCategoryListItem.Selected Then
                dbConnection.SPName("sp_Communications")
                dbConnection.SPParameter("Action", "INSERTCATEGORIES")
                dbConnection.SPParameter("communicationID", newCommunicationID)
                dbConnection.SPParameter("category", cboCategoryListItem.Text)
                dbConnection.SPExecuteNonQuery()

                dbConnection.SPParameterClear()
            End If

        Next
        '--- pick up "other" category
        If txtOtherCategory.Text <> "" Then
            dbConnection.SPName("sp_Communications")
            dbConnection.SPParameter("Action", "INSERTCATEGORIES")
            dbConnection.SPParameter("communicationID", newCommunicationID)
            dbConnection.SPParameter("category", txtOtherCategory.Text)
            dbConnection.SPExecuteNonQuery()

            dbConnection.SPParameterClear()

        End If

Open in new window

0
 
LVL 30

Accepted Solution

by:
MlandaT earned 500 total points
ID: 33558314
Looks to me like all your code should work. However, since we are inserting the 'Other' value into the same table as the rest of the categories... when the combobox binds again the next time... there is nothin there to distinguish the 'Other' value from the rest of them.
0
 

Author Closing Comment

by:DEWebWorks
ID: 33558565
Ok, so my partner dug through my 600 lines of code on the page and found where I was resetting the textbox value to "".

Thank you SOOOO much for your help :)
0

Featured Post

Free Tool: SSL Checker

Scans your site and returns information about your SSL implementation and certificate. Helpful for debugging and validating your SSL configuration.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

Question has a verified solution.

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

Suggested Solutions

Title # Comments Views Activity
exchange, OWA, scripts 5 40
Using an iCal Calendar Subscription in Outlook 4 28
MS Outlook auto update conflicts 5 27
exchange, squid, proxy, linux 6 32
What does UTC stand for?  “Coordinated Universal Time” – Think of this as the true time on Planet Earth that never changes with the exception of minor leap seconds here and there to account for the changes in the planet's rotation.   What does th…
Read this checklist to learn more about the 15 things you should never include in an email signature.
This Experts Exchange video Micro Tutorial shows how to tell Microsoft Office that a word is NOT spelled correctly. Microsoft Office has a built-in, main dictionary that is shared by Office apps, including Excel, Outlook, PowerPoint, and Word. When …
Many of my clients call in with monstrous Gmail overloading issues with Outlook. A quick tip is to turn off the All Mail and Important folders from synching. Here is a quick video I made to show you how to turn off these and other folders in Gmail s…

713 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