TACHE PLANIFIÉE

Commentaire de ynizon le 30/03/2004 17:13:57

Il n'y a aucun echange avec le planificateur Windows (pas de gestion de fichier .job). Tu crees juste un timer qui lance un shell pour executer un fichier.

Commentaire de surfzoid le 06/04/2004 12:59:14

je vois que pas mal de gens ont au moins  jeté un coup d oeil à ma source mais que personne ne laisse de notes ou de commentaire ( a part ynizon qui m a mis 1 sans prendre le temps de vraiment regardé ce que g fait,cf : ses sources)
pouriez vous m éclairé sur la nulluté et l inintêret de cette source
thx
@++ et BP

Commentaire de threadom le 14/06/2004 13:25:24

Bon j'avou j'aime pas trop la facon dont tu code :) trop direct :) heureusement que y'a des commentaire mais ton code parle pas assez tout seul :p Malgre tout ca ma permis d'apprendre comment mettre en l'icon en dans le systeme tray :) merci. ca merite pas un 10 mais bon :) ca ce note bien quand meme.

Commentaire de threadom le 06/07/2004 12:32:24

Je vais déjà commencé par l'indentation. Un coup tu met fait une tabulation le coups suivant non ... Apres il ya les espaces, defois tu saute une ligne avant la fin de ta fonction (ou procedure) d'autre fois c'est au début.

Niveau code : très peut de fonctions ce qui donne des instruction dans tout les sens, exemple avec le fait de rajouter 0 si il n'y a qu'un chiffre. C'est sur ta method est peut etre plus rapide mais je rapelle que c'est du vb... Ensuite le fait que tu ne change pas le nom de tes composants (ca aide pas a faire parler le code). Commente ligne a ligne n'est pas non plus une solution, autant faire 10 ligne et faire un gros bloque au dessus pour les explique (surtout pour les declaration de dll). Essaye de donnez une forme a ton code, un code propre permet de pouvoir y revenir (meme au bout de 5000ligne) sinon generalement on lache l'affaire et on recommence. Autre petite chose quand je parle du manque de fonction entend par la de crée des petites fonction basic et réutilisable dans d'autre programme.

Je suis désolé si c'est un peu flou mais j'avou que ca fait deja longtemps pour moi que j'avais posté et je ne me souvenait pas trop du code (j'en suis pas mécontent d'ailleur :p).

Commentaire de surfzoid le 08/07/2004 13:53:59

Désoler g dis une grosse bourde c que en fait j avais mis l alerte sur un nouvel email et étant absent un certaint temp j avais finit par carrément l oublié ; encore désolé pour ce "faux" bug de forum
LoL

Commentaire de cqui789 le 05/02/2005 20:59:39

pour ce qui est de la modif des combo, je te propose la methode suivante, qui remplace le .change et transfert les verif au .lostfocus (qui pourrait etre un .validate, je ne connait pas la difference)
je n'ai pas compris l'interet du combo1.click


' on bloque tout ce qui n'est pas un chiffre ou un backspace (emprunte a Magidev)
Private Sub Combo1_KeyPress(KeyAscii As Integer)
If InStr("1234567890" & Chr(8), Chr$(KeyAscii)) = 0 Then KeyAscii = 0
End Sub


Private Sub Combo1_LostFocus()
If Combo1.Text > "59" Then
   Combo1.Text = "59"
End If

If Quotidien.Value = True Then
    Text2 = Combo3 & ":" & Combo2 & ":" & Combo1
Else
    If Horaire.Value = True Then
        Text2 = Combo2 & ":" & Combo1
    Else
        Text2 = Combo2
    End If
End If
Combo1.ListIndex = Combo1
End Sub

Commentaire de cqui789 le 05/02/2005 23:08:16

j'ai l'air de donner une lecon mais j'apprend beaucoup en lisant ton programme (par exemple les settings ne sont nul part dans mon manuel...)

j'ai, a ce sujet, une autre suggestion, pour la premiere partie du gaveur:

la fonction format avec le format "00" garantie que si la valeur n'a pas autant de chiffres que voulu, un 0 est ajoute au debut.

de plus je trouve plus joli de gaver tous les combos dans la meme boucle, quite a metre un test pour les heures

Function gaveur()
Dim i
For i = 0 To 59
    Combo1.AddItem Format(i, "00"), i
    Combo2.AddItem Format(i, "00"), i
    If i < 24 Then Combo3.AddItem Format(i, "00"), i
Next i

On Err......

Commentaire de surfzoid le 05/02/2005 23:44:30

exacte et merci
je me justifirai un petit peut en t avouant que j ai fais cette source vite fait il y plus de 7 mois pour un besoin spécifique qui vient d evolué aujoudh ui LOL
Bonne prog et @++

Commentaire de cqui789 le 06/02/2005 00:02:31

Le systray, tres bien.

encore une question, je n'ai pas bien compris le fonctionnement des fonctions invisible et firsttime, je me demande si le fait que la fenetre ne s'ouvre pas au demarrage n'est pas du a une petite erreur a ce niveau.

Commentaire de cqui789 le 07/02/2005 12:31:07

bon, cote parametre X, j'ai rien trouve de mieux que ce que l'on trouve dans ton code en lisant les constantes 'click'
(merci de les donner toutes meme si tu n'en utilise que 2)