VBA, VB6, Ms access

Dear experts, i have a from on ms access where i have a button save and save some records in different tables. for this i set different openrecordset. the problem is its saving in all tables except for 1

here it is
Set db = CurrentDb
Set rst = db.OpenRecordset("Player_Details")


With rst
    .AddNew
    !Firstname = Me.Text3
    !Lastname = Me.Text5
    !DoB = Me.Text7
    !Height = Me.Text9
    !Weight = Me.Text11
    !Level = Me.Combo33
    pk_value = ![idPlayer_Details]
    .Update

End With
rst.Close
    

Set db8 = CurrentDb
Set rst8 = db8.OpenRecordset("Player_Occupation")
With rst8
    .AddNew
    !idOccupations = Me.Combo37
    !idPlayer_Details = pk_value
    !NameOfInstitute = Me.Text26
    !Hobby = Me.Text26
    .Update
End With
rst8.Close



Set db3 = CurrentDb
Set rst3 = db3.OpenRecordset("Team_History")
With rst3
    .AddNew
    !idTeam_Details = Me.Combo40
    !idPlayer_Details = pk_value
    .Update
End With
rst3.Close

Open in new window


Player_Details and Team_History are saving successfully. but i dont know why Player_Occupation is not updating.. Please help
Alv45525Asked:
Who is Participating?
 
Rey Obrero (Capricorn1)Connect With a Mentor Commented:
i see that you have two "id"  fields    
    !idOccupations = Me.Combo37
    !idPlayer_Details = pk_value

in the table Player_Occupation that look like Foreign keys.(FK)

check the relationship of table Player_Occupation to the tables where the above FK id's are
Primary Keys (PK)
If Referential Integrity (RI) is set in the relationship , those FK must be added first as PK to their respective tables.
0
 
2toriaCommented:
Do you get any error messages at all?  

One hint that might be useful, especially for future readability of your code, is to give your fields descriptive names.  For example, if your team details combobox was called cboTeamDetails it would read much better, especially if you leave this code for a few weeks/months and have to come back at some point to debug.  Other than that I can't see anything particularly wrong.  It is early in the morning here though.

Matt
0
 
Alv45525Author Commented:
^^ thats for response Matt,

Yes i know i need to work on the name of the variables later. it was just a set up.

Actually theres no error message.

all saves in respective tables except for Player_Occupation
0
The new generation of project management tools

With monday.com’s project management tool, you can see what everyone on your team is working in a single glance. Its intuitive dashboards are customizable, so you can create systems that work for you.

 
2toriaCommented:
Can you upload the file for me to have a look at?  It might be easier to check out the problem.

Matt
0
 
BitsqueezerCommented:
Hi,

instead of using a recordset you should better use a SQL command for that purpose like this:

DBEngine(0)(0).Execute "INSERT INTO Player_Occupation " & _
                       "           (idOccupations,idPlayer_Details," & _
                       "            NameOfInstitute,Hobby) " & _
                       "       VALUES " & _
                       "           (" & Nz(Me.Combo37,0) & "," & _
                                    Nz(pk_value,0) & ",'" & _
                                    Nz(Me.Text26) & "','" & Nz(Me.Text26) & "')"

Open in new window


(DBEngine(0)(0) is a kind of equivalent to CurrentDB, but it's faster as it is an always existing instance of the currnet database object.)
I assumed in this example that the id fields are numerical and the Text26 is alphanumerical (you need to enclose the value in single quotes in this case). Is it correct that two fields in this case have the value of the same textbox (Me.Text26)?

It's difficult to say anything about the recordset solution without knowing what type of recordset you declared, how your error handling takes place and how your tables look like so I agree with Matt that in such cases it is easier to post the database (an extract without important data of course).

Cheers,

Christian
0
 
Helen FeddemaCommented:
In addition to giving your controls meaningful names, bear in mind that there is a difference between the control object itself, and the value from the control, which is what you are assigning to fields in the recordsets.  Sometimes it is helpful to assign the values from the controls to variables, so you can test whether they have the correct data, and examine them in the Watch window or write them to the Immediate window.  Here is a sample (for a Date value):

   If IsDate(Me![txtDate].Value) = True Then
      dteSingle = CDate(Me![txtDate].Value)
   End If
0
 
rockiroadsCommented:
Looking at Player_Occupation

    .AddNew
    !idOccupations = Me.Combo37
    !idPlayer_Details = pk_value
    !NameOfInstitute = Me.Text26
    !Hobby = Me.Text26
    .Update

I do not see anything wrong with this so as long as the add was fine, pk_value should have something. Looks like in Player_Details the column idPlayer_Details has been setup as a AutoNumber. Is this right?

If that is the case and you are going on something the same for Player_Occupation, then is idOccupations a autonumber or are you referencing a combo which is a lookup of occupations? If it is the latter then I assume a value has been selected and you do have an autonumber setup? If there is an autonumber set can you not add something like this to check if one has been generated or not i.e.

debug.print  !myOccupationsAutonumber

Do you use

ON ERROR RESUME NEXT

if so then comment out that line so you can tell if an error occurred or add this after the .Update in Player_Occupation

if err.number <> 0 then
    msgbox "Error " & err.number & " when updating occupations. " & err.description
end if

0
 
Alv45525Author Commented:
i am referencing a combo which is a lookup of occupations
0
 
Rey Obrero (Capricorn1)Commented:
Alv45525,


before anything else,  i suggest that you check what was posted in http:#a34127359





0
 
DatabaseMX (Joe Anderson - Microsoft MVP, Access and Data Platform)Commented:
@Bitsqueezer:

As far as DbEngine(0)(0) ... I might suggest you check this out:
http://groups.google.com/group/comp.databases.ms-access/msg/9fe98bb5d7cba5ea?hl=en&

mx
0
All Courses

From novice to tech pro — start learning today.