Erreur 1004 formulaire vba

feafe Messages postés 7 Date d'inscription   Statut Membre Dernière intervention   -  
feafe Messages postés 7 Date d'inscription   Statut Membre Dernière intervention   -

Bonjour à tous et à toutes,

Je suis entrain de créer un tableau qui répertorie toute les commandes de mes clients avec une entrée par formulaire. Je suis novice en VBA je me suis donc aidée de cette vidéo : 

https://www.youtube.com/watch?v=PzYUyB2HjgM

Et mon script donne ça : 

'***
'procédure pour ajout d'une nouvelle commande
'***
Private Sub btnAjout_Click()
    Sheets("Commande").Activate
    Range("A1").Select
    Selection.End(x1down).Select 'on se positione sur la dernière ligne non vide
    Selection.Offset(1, 0).Select 'on se décale d'une ligne vers le bas
    ActiveCell = cboagence.Value
    ActiveCell.Offset(0, 1).Value = txtclient
    ActiveCell.Offset(0, 2).Value = txtref
    ActiveCell.Offset(0, 3).Value = cbodate
    ActiveCell.Offset(0, 4).Value = txtcontact
    ActiveCell.Offset(0, 5).Value = txtnumcontact
    ActiveCell.Offset(0, 6).Value = txtadresse
    ActiveCell.Offset(0, 7).Value = txtspec
    ActiveCell.Offset(0, 8).Value = txtdemandepart
    ActiveCell.Offset(0, 9).Value = txtdevis
    ActiveCell.Offset(0, 10).Value = txtdetail
    ActiveCell.Offset(0, 11).Value = cbofournisseur
    ActiveCell.Offset(0, 12).Value = txtvaleur
    
    MsgBox "Votre commande a bien été ajoutée", vbOKOnly + vbInformation, "CONFIRMATION"
End Sub

'****
'procédure pour reinitialiser les champs du formulaire
'****
Private Sub btneffacer_Click()
    cboagence = ""
    txtclient = ""
    txtref = ""
    cbodate = ""
    txtcontact = ""
    txtnumcontact = ""
    txtadresse = ""
    txtspec = ""
    txtdemandepart = ""
    txtdevis = ""
    txtdetail = ""
    cbofournisseur = ""
    txtvaleur = ""
End Sub
'****
'procédure pour fermer le formulaire
'****
Private Sub btnfermer_Click()
    Unload Me
End Sub
'****
'afficher la liste des commandes
'****
Private Sub btnsource_Click()
    Sheets("Commande").Activate
    Range("a1").Select
End Sub

'*****
'procédure permettant de d'activer le bouton d'ajout de
'commande en fonction de si la première case est vide ou non
'*****

Private Sub cboagence_Change()
    If cboagence <> "" Then
        btnAjout.Enabled = True 'activer le bouton
    Else
        btnAjout.Enabled = False 'désactiver le bouton
    End If

End Sub
 

Malgré tout j'ai une erreur 1004 au niveau de la ligne 4 (hors commentaire)

Si quelqu'un a une piste je suis preneur. 

Merci d'avance


Windows / Chrome 92.0.4515.107

5 réponses

yg_be Messages postés 23541 Date d'inscription   Statut Contributeur Dernière intervention   Ambassadeur 1 584
 

bonjour,

Merci de suivre ces instructions quand tu partages du code: https://codes-sources.commentcamarche.net/faq/11288-poster-un-extrait-de-code

Quelle est l'instruction sur la ligne 4?

Il est préférable de ne pas utiliser ainsi Select et Active.

    dim cl as range
    set cl=Sheets("Commande").Range("A1").End(x1down).Offset(1, 0)
    cl = cboagence
    cl.Offset(0, 1) = txtclient
0
ozone_ Messages postés 1518 Date d'inscription   Statut Membre Dernière intervention   478
 

Bonjour à vous deux,

N'est ce pas plutôt 

End(xlDown)

Au lieu de 

End(x1Down)
0
yg_be Messages postés 23541 Date d'inscription   Statut Contributeur Dernière intervention   1 584 > ozone_ Messages postés 1518 Date d'inscription   Statut Membre Dernière intervention  
 

Il est important, aussi, d'avoir une ligne avec "option explicit" en début du module.

0
feafe Messages postés 7 Date d'inscription   Statut Membre Dernière intervention  
 

D'abord merci à vous deux pour votre réponse si rapide. Je m'excuse de la présentation de mon message je n'avais pas trouver comment le mettre en forme correctement. J'y remédie dans celui là.

Option Explicit

'***'
'procédure pour ajout d une nouvelle commande'
'***'
Private Sub btnAjout_Click()
    Dim cl As Range
    Set cl = Sheets("Commande").Range("A1").End(xlDown).Offset(1, 0)
    cl = cboagence
    cl.Offset(0, 3) = txtclient
    cl.Offset(0, 4) = txtref
    cl.Offset(0, 5) = cbodate
    cl.Offset(0, 6) = txtcontact
    cl.Offset(0, 7) = txtnumcontact
    cl.Offset(0, 8) = txtadresse
    cl.Offset(0, 9) = txtspec
    cl.Offset(0, 10) = txtdemandepart
    cl.Offset(0, 11) = txtdevis
    cl.Offset(0, 12) = txtdetail
    cl.Offset(0, 13) = txtvaleur
    cl.Offset(0, 14) = cbofournisseur
    
    MsgBox "Votre commande a bien été ajoutée", vbOKOnly + vbInformation, "CONFIRMATION"
End Sub

'****'
'procédure pour reinitialiser les champs du formulaire'
'****'
Private Sub btneffacer_Click()
    cboagence = ""
    txtclient = ""
    txtref = ""
    cbodate = ""
    txtcontact = ""
    txtnumcontact = ""
    txtadresse = ""
    txtspec = ""
    txtdemandepart = ""
    txtdevis = ""
    txtdetail = ""
    cbofournisseur = ""
    txtvaleur = ""
End Sub

'****'
'procédure pour fermer le formulaire'
'****'
Private Sub btnfermer_Click()
    Unload Me
End Sub

'****'
'afficher la liste des commandes'
'****'

Private Sub btnsource_Click()
    Sheets("Commande").Activate
    Range("a1").Select
End Sub

'*****'
'procédure permettant de d'activer le bouton d ajout de'
'commande en fonction de si la première case est vide ou non'
'*****'

Private Sub cboagence_Change()
    If cboagence <> "" Then
        btnAjout.Enabled = True 'activer le bouton'
    Else
        btnAjout.Enabled = False 'désactiver le bouton'
    End If

End Sub

Alors j'ai ajouté mon option explicit au début, j'ai modifié mon code selon vos conseils et en effet c'est bien xldown et non x1down. Ca fonctionne beaucoup mieux comme ça néanmoins j'arrive à rentrer 2 commandes et ensuite ça stagne sur la même ligne. Je me demande si il ne faudrait pas que je fasse une boucle qui incrémente le nombre de ligne à chaque fois ?

En tout cas merci de vos réponses elles m'ont bien aidées

0
yg_be Messages postés 23541 Date d'inscription   Statut Contributeur Dernière intervention   1 584
 

Rien de spécial dans la colonne A de la ligne où cela stagne?

0
feafe Messages postés 7 Date d'inscription   Statut Membre Dernière intervention   > yg_be Messages postés 23541 Date d'inscription   Statut Contributeur Dernière intervention  
 

Rien de spécial c'est à dire ? le tableau ne s'étend pas et les valeurs sont remplacés par celles entrées dans le formulaire au lieu d'ajouter une ligne.

0
yg_be Messages postés 23541 Date d'inscription   Statut Contributeur Dernière intervention   1 584 > feafe Messages postés 7 Date d'inscription   Statut Membre Dernière intervention  
 

A toi de nous donner des informations, ou de partager ton fichier.

La colonne A de la ligne où cela stagne est bien remplie, rien de spécial comparé aux lignes précédentes?

0
feafe Messages postés 7 Date d'inscription   Statut Membre Dernière intervention  
 

Biensur je vous partage le fichier. Non la colonne est bien remplie et la ligne aussi. Y a t il une procédure pour vous l'envoyer ou un lien wetransfer suffit ?

0
yg_be Messages postés 23541 Date d'inscription   Statut Contributeur Dernière intervention   1 584
 

Je pense que la méthode que tu utilises n'ajoute pas des lignes dans le tableau.

Je pense préférable de faire ainsi: https://docs.microsoft.com/fr-fr/office/vba/api/excel.listrows.add

0
feafe Messages postés 7 Date d'inscription   Statut Membre Dernière intervention   > yg_be Messages postés 23541 Date d'inscription   Statut Contributeur Dernière intervention  
 

Bonjour yg_be,

Merci de votre réponse, néanmoins le lien me mène sur le site de Microsoft comme ci dessous

0
yg_be Messages postés 23541 Date d'inscription   Statut Contributeur Dernière intervention   1 584 > feafe Messages postés 7 Date d'inscription   Statut Membre Dernière intervention  
 

fais alors une recherche "excel listrows add".

0
feafe Messages postés 7 Date d'inscription   Statut Membre Dernière intervention  
 

https://wetransfer.com/downloads/948017aab1d56b42e2cabc7e4c29d3f420220912065532/15d347

0
ozone_ Messages postés 1518 Date d'inscription   Statut Membre Dernière intervention   478
 

Bonjour,

Voici une proposition

'***
'procédure pour ajout d'une nouvelle commande
'***
Private Sub btnAjout_Click()
    Dim cl As Range
    Dim cel As Range

    Application.ScreenUpdating = False
    Sheets("Commande").Select
    
    For Each cel In Range("A:A")
        If cel.Value = "" Then cel.Select: Exit For
    Next cel
    
    'Set cl = Sheets("Commande").Range("A1").End(xlDown).Offset(1, 0)
    cel = cboagence
    cel.Offset(0, 3) = txtclient
    cel.Offset(0, 4) = txtref
    cel.Offset(0, 5) = cbodate
    cel.Offset(0, 6) = txtcontact
    cel.Offset(0, 7) = txtnumcontact
    cel.Offset(0, 8) = txtadresse
    cel.Offset(0, 9) = txtspec
    cel.Offset(0, 10) = txtdemandepart
    cel.Offset(0, 11) = txtdevis
    cel.Offset(0, 12) = txtdetail
    cel.Offset(0, 13) = txtvaleur
    cel.Offset(0, 14) = cbofournisseur
    
    MsgBox "Votre commande a bien été ajoutée", vbOKOnly + vbInformation, "CONFIRMATION"
    Application.ScreenUpdating = True
End Sub
0

Vous n’avez pas trouvé la réponse que vous recherchez ?

Posez votre question
feafe Messages postés 7 Date d'inscription   Statut Membre Dernière intervention  
 

Merci ozone_,

Je teste ça et je vous redis. 

0