Compléter un test

Résolu
Configuration: Windows 10 / Edge et Excel 2007


Salut a tous , forum salut,

Je fais des essais de tests afin d'apporter des améliorations dans ma compta.

Ci joint un lien vers un fichier avec quelques explications dans une bulle.
https://www.cjoint.com/c/JBjqIY5ym6Y
1) un test pour replacer le curseur d'une colonne a gauche.

2) un test pour interdire d'entrer une valeur de zéro.

Ci-joint le fichier avec explicatifs a l'intérieur.

Merci a vous pour votre aide, bonne après midi.

Cdlt Raymond

7 réponses

  1. Modérateur
    Vu le nombre de cellules concernées par ton code... l'utilisation du SELECT ne pose aucun problème.

    Lorsque tu as lu ce genre d'indication... c'est sur du traitement de centaines/millier de lignes ou les débutant code comme ci ils "manipulaient" le classeur à la souris au lieu de passer par les propriétés/méthodes des objets.
    Toi tu veux positionner la souris.. donc.. faire un SELECT de cellule.

    1. Re salut jordane45,

      Merci pour ta réponse et la modification du fichier,

      Pour te répondre du NB que tu me mets: dans le fichier joint et dans la bulle j'écris que j'ai bien consulter le lien attentivement que tu m'a proposer et que j'ai fait des essais divers par rapport a ce que j'ai pu comprendre mais hélas je n'ai pas compris le fonctionnement de "Select"

      Pour "Target.Select" proposer je l'ai placer également a plusieurs endroits, mais ca plante.

      '================

      -1) Pour le test des chiffres négatifs, c'est pas bon, ca me mets Ex pour entrée " -5" affichage message "Versement effectuer" en vert, donc devrai rien se passer comme pour le zéro.

      -2) Pour le zéro c'est bon, dans la situation actuelle.

      Mais voila je fait mon possible mais je n'ai pas tes connaissances.

      Merci a toi pour ton aide, bonne après midi.

      Cdlt Raymond
      1. Modérateur
        Option Explicit
        
        Private Sub Worksheet_Change(ByVal Target As Range)
        
        If Target.Address <> "$F$31" And Target.Address <> "$F$32" And _
           Target.Address <> "$F$33" And Target.Address <> "$F$34" Then Exit Sub
         
         
        Application.EnableEvents = False ' désactive les autres évènements que pourraient déclencher ta macro
        Select Case Target.Value
          Case IsNumeric(Target.Value) And Target.Value = 0 And Not IsEmpty(Target.Value) = False       'Si zéro
            Target.Value = ""
            Target.Select
          Case Is < 0           'Si chiffre négatif
            Target.Value = ""
            Target.Select
          Case Is <> ""          'Si cellule non vide
            With Target.Offset(0, 1)
              .Value = "Versement en attente"
              .Font.ColorIndex = 3    'Rouge
            End With
        
          Case Is = ""             'Si cellule vide
             With Target.Offset(0, 1)
              .Value = "Versement effectuer"
              .Font.ColorIndex = 4     'Vert
            End With
        
          Case Else: Range("F31:F34").Value = ""
             Range("G31:G34").Value = ""
        End Select
        Application.EnableEvents = True
        
        End Sub
        
      2. @jordane45Re jordane45,

        De nouveau merci pour le retour du fichier modifier, cette fois c'est bon pour moi.

        plus de plantage ni d'erreur d'entrée.

        Mais dans mes lectures sur "Case is" j'ai lu plusieurs fois qu'il n'était pas conseillé d'utiliser "Select" car cela ralenti le code.

        Mais plutôt "Offset(0,1)" par exemple qui serait beaucoup plus rapide, tu en penses quoi ???

        Sinon merci beaucoup pour ta patience qui a fini par payer, grâce a toi j'ai pu finir mon programme.

        Bonne fin d'après midi et encore merci a toi pour ton savoir partager, ca été cool.

        Bien cordialement Raymond
    2. Modérateur
      Bonjour,

      Poster un fichier c'est bien.
      Expliquer (dans le forum) le souci rencontré exactement ET montrer le code qui pose problème est mieux.
      Pourquoi ? .. par ce que les soucis que tu rencontres peuvent interesser d'autres internautes... et le moteur de recherche n'est pas capable d'aller voir le texte explicatif que tu mets dans tes liens.

      Brefs..
      Ton souci semble être la "non" différenciation entre les cellules vides et les zéro.
      Essaye ça :
      Option Explicit
      
      Private Sub Worksheet_Change(ByVal Target As Range)
      
      If Target.Address <> "$F$31" And Target.Address <> "$F$32" And _
         Target.Address <> "$F$33" And Target.Address <> "$F$34" Then Exit Sub
      
      Select Case Target.Value
        Case IsNumeric(Target.Value) And Target.Value = 0 And Not IsEmpty(Target.Value) = False       'Si zéro
          Application.EnableEvents = False
          Target.Value = ""
      
        Case Is < 0           'Si chiffre négatif
          Target.Value = ""
      
        Case Is <> ""          'Si cellule non vide
          With Target.Offset(0, 1)
            .Value = "Versement en attente"
            .Font.ColorIndex = 3    'Rouge
          End With
      
        Case Is = ""             'Si cellule vide
           With Target.Offset(0, 1)
            .Value = "Versement effectuer"
            .Font.ColorIndex = 4     'Vert
          End With
      
        Case Else: Range("F31:F34").Value = ""
           Range("G31:G34").Value = ""
      End Select
      Application.EnableEvents = True
      End Sub
      


      NB: Vu que tu n'as pas essayé de gérer la position du curseur (via la méthode .Select que je t'ai donné avant...) je n'ai pas pris en compte cette demande.
      Mais en gros... la réponse est simplement un truc du genre :
      Target.Select
      

      1. Salut a tous, forum
        Salut jordane45

        Après maintes essais je n'arrive pas a terminer ce que je souhaitai faire.

        Pas faute de chercher une solution mais ca marche pas.

        je remets le fichier pour essais, j'ai mis ce que je rencontre comme soucis et donc
        ce qui ne fonctionne pas.

        Bonne journée et merci.

        Cdlt Raymond

        Lien pour le fichier

        https://www.cjoint.com/c/JBkkb7mQaWY
        1. Modérateur
          Bonjour,

          Private Sub Worksheet_Change(ByVal Target As Range)
          
          If Target.Address <> "$F$31" And Target.Address <> "$F$32" And _
             Target.Address <> "$F$33" And Target.Address <> "$F$34" Then Exit Sub
          
          Select Case Target.Value
           Case Is = 0                     'Si zéro
              Application.EnableEvents = False
              Target.Value = ""
           Case Is <> ""                          'Si cellule non vide
             With Target.Offset(0, 1)
                 .Value = "Versement en attente"
                 .Font.ColorIndex = 3 'Rouge
             End With
           Case Is = ""                            'Si cellule vide
             With Target.Offset(0, 1)
                 .Value = "Versement effectuer"
                 .Font.ColorIndex = 4 'Vert
             End With
          
           Case Else
             Range("F31:F34").Value = ""
             Range("G31:G34").Value = ""
          End Select
          Application.EnableEvents = True
          End Sub
          

          1. Bonsoir jordane45,

            Merci pour ta réponse rapide, c'est gentil a toi,

            Donc pour les zéros ca fonctionne bien, merci.

            Serait t'il possible d'interdire de rentrer aussi des chiffres négatifs qui me sont pas utiles du tout Ex: interdire -1 ou - 45

            et la possibilité que le curseur reste ou revienne a gauche enfin qu'il reste en colonne F31:F34

            car quand j'entre un chiffre plage F31:F34 le curseur se déplace d'une colonne a droite pour écrire le message et reste dans la cellule G31:G34

            j'ai essayer un Target.Offset(0,-1) pour forcer le curseur a revenir d'une colonne a gauche donc plageF31:F34

            Merci pour ton aide, je te souhaite la bonne soirée

            Cdlt Raymond
          2. Modérateur
            @dudulerayPour définir le focus sur une cellule utilise la propriété Select
            Pour interdire les nombres négatifs il suffit de vérifier si ils sont inférieurs à zéro