Posté le 16/07/2013 à 23:37 Membre depuis le 10/06/2001, 45104 messages
Je devais modifier un truc qui dépendait directement du code effacé.
On doit livrer une version intermédiaire demain soir.
Posté le 17/07/2013 à 08:28 Membre depuis le 11/11/2001, 116494 messages
Kevin Kofler (./29) :
Bah, pourquoi pas 1. faire un revert et 2. écrire un mail au stagiaire que son commit était foireux et a donc été inversé et que par conséquent il doit tout refaire de zéro le lendemain (vu qu'il ne sera probablement pas capable de consulter l'historique, vu sa manière incompétente d'utiliser le SCM)? Ça lui apprendra…

c'est complètement à l'opposé de la philosophie du truc... c'est un merge qu'il faut faire, sinon c'est comme si tu travaillais tout seul dans ton coin avec un backup pour un rollback de ci de là...
avatarWebmaster du site Ti-FRv3 (et aussi de DevLynx)
Si moins de monde enculait le système, alors celui ci aurait plus de mal à nous sortir de si grosses merdes !
"L'erreur humaine est humaine"©Nil (2006) // topics/6238-moved-jamais-jaurais-pense-faire-ca
Posté le 17/07/2013 à 19:08 Membre depuis le 02/06/2004, 3569 messages
c'est svn ou git ? Sur svn on ne peut pas pousser les fichiers qui n'ont pas la même version mais je suppose qu'il a updaté et qu'il a un IDE qui ne recharge pas les fichiers et donc il a tout écrasé tes modifs. Faut le prévenir qu'il doit updater ces fichiers en ayant aucun fichier ouvert sinon tu risques de retrouver le problème.
Posté le 17/07/2013 à 19:11 Membre depuis le 10/06/2001, 45104 messages
C'est svn et Visual Studio, qui prévient qu'il y a des modifs externes.
Je pense qu'il a simplement écrasé avec une sauvegarde externe.
Cela dit, il utilise aussi notepad++, possible que ça vienne de là.
Merci pour la suggestion, en tout cas !
Posté le 17/07/2013 à 19:12 Membre depuis le 16/06/2001, 69776 messages
svn refuse le commit si ta wc n'est pas updatée.
git refuse le pull si ta wc est en conflit avec les modifs pullées.

donc non
Posté le 17/07/2013 à 19:19 Membre depuis le 10/06/2001, 45104 messages
Mmmm ?
Ma théorie, c'est qu'il a en effet mis à jour sa copie locale, et seulement ensuite repris sa sauvegarde manuelle.
Posté le 17/07/2013 à 19:20 Membre depuis le 02/06/2004, 3569 messages
non pas rapport a quoi ? la sauvegarde externe ? ça pour moi ce n'est pas possible il a simplement pas recharger les fichiers dans les éditeurs et resauvegarder le tout. Donc dans le référentiel en local les fichiers sont à jours mais ces fichiers non.
Posté le 17/07/2013 à 19:25 Membre depuis le 11/11/2001, 116494 messages
après, y'a la solution : "tout écraser sans demander" au moment du merge ou sinon : dév offline, ctrl+a, ctrl+c, update, ctrl+a, ctrl+v, commit
avatarWebmaster du site Ti-FRv3 (et aussi de DevLynx)
Si moins de monde enculait le système, alors celui ci aurait plus de mal à nous sortir de si grosses merdes !
"L'erreur humaine est humaine"©Nil (2006) // topics/6238-moved-jamais-jaurais-pense-faire-ca
Posté le 17/07/2013 à 19:43 Membre depuis le 10/06/2001, 45104 messages
grincheux (./36) :
la sauvegarde externe ? ça pour moi ce n'est pas possible
Pourquoi ? confus
Posté le 18/07/2013 à 00:29 Membre depuis le 10/06/2001, 40265 messages
Pen^2 (./30) :
Je devais modifier un truc qui dépendait directement du code effacé.On doit livrer une version intermédiaire demain soir.

Dommage. Ce n'est pas normal que tu passes ton temps à réparer les erreurs de quelqu'un d'autre.
vince (./31) :
Kevin Kofler (./29) :
Bah, pourquoi pas 1. faire un revert et 2. écrire un mail au stagiaire que son commit était foireux et a donc été inversé et que par conséquent il doit tout refaire de zéro le lendemain (vu qu'il ne sera probablement pas capable de consulter l'historique, vu sa manière incompétente d'utiliser le SCM)? Ça lui apprendra…
c'est complètement à l'opposé de la philosophie du truc...

C'est le stagiaire qui n'a pas utilisé correctement l'outil, inverser son commit ne serait que la manière la plus simple de réparer les dégâts qu'il a causés.
avatarMes news pour calculatrices TI: Ti-Gen (fr/en), MobiFiles (de)
Mes projets PC pour calculatrices TI: TIGCC, CalcForge (CalcForgeLP, Emu-TIGCC)
Mes chans IRC: #tigcc et #inspired sur irc.freequest.net (UTF-8)

Liberté, Égalité, Fraternité
Posté le 18/07/2013 à 00:40 Membre depuis le 10/06/2001, 45104 messages
Kevin Kofler (./39) :
Dommage. Ce n'est pas normal que tu passes ton temps à réparer les erreurs de quelqu'un d'autre.
Ouep, je t'avoue que j'étais *un peu* énervé (surtout vu le contexte du projet, mais bref cheeky)
Posté le 19/07/2013 à 07:33 Membre depuis le 03/11/2002, 14538 messages
squalyl (./34) :
svn refuse le commit si ta wc n'est pas updatée.
git refuse le pull si ta wc est en conflit avec les modifs pullées.

donc non

WC: updated?
1281075291_1384919555.jpg

WC en conflit avec d'autres outils ?
modern_toilet_restaurant_010.jpg
avatarHighway Runners, mon jeu de racing à la Outrun qu'il est sorti le 14 décembre 2016 ! N'hésitez pas à me soutenir :)

https://itunes.apple.com/us/app/highway-runners/id964932741
Posté le 19/07/2013 à 07:40 Membre depuis le 16/06/2001, 69776 messages
trilove owi
Posté le 19/07/2013 à 08:18 Membre depuis le 27/04/2006, 60465 messages
#pointvince#
avatarZeroblog

« Tout homme porte sur l'épaule gauche un singe et, sur l'épaule droite, un perroquet. » — Jean Cocteau
« Moi je cherche plus de logique non plus. C'est surement pour cela que j'apprécie les Ataris, ils sont aussi logiques que moi ! » — GT Turbo
Posté le 25/07/2013 à 17:41 Membre depuis le 13/06/2001, 73049 messages
Kevin Kofler (./39) :
C'est le stagiaire qui n'a pas utilisé correctement l'outil, inverser son commit ne serait que la manière la plus simple de réparer les dégâts qu'il a causés.
Tiens, c'est exactement ce que j'aurais fait (s'il n'y a pas eu de commit par un tiers après, bien sûr... là, ça serait plus problématique pour démêler les ficelles), mais en envoyant un petit mail juste assez poli pour lui expliquer la situation et lui indiquer comment récupérer son boulot.
avatar
Posté le 25/07/2013 à 17:49 Membre depuis le 10/06/2001, 45104 messages
(oué, mais là, on avait une deadline cheeky)
Posté le 25/07/2013 à 18:53 Membre depuis le 13/06/2001, 73049 messages
Et la nuit, ça sert à quoi ? embarrassed
avatar
Posté le 25/07/2013 à 19:00 Membre depuis le 16/06/2001, 69776 messages
à exploser la tête du stagiaire.
Posté le 25/07/2013 à 19:02 Membre depuis le 10/06/2001, 45104 messages
Nil (./46) :
Et la nuit, ça sert à quoi ? embarrassed
Pen^2 (./30) :
23:37
%'(
Posté le 25/07/2013 à 19:08 Membre depuis le 16/06/2001, 69776 messages
awww tsss
Posté le 25/07/2013 à 19:10 Membre depuis le 10/06/2001, 45104 messages
En plus je ne suis pas habitué à veiller tsss
Posté le 25/07/2013 à 19:44 Membre depuis le 18/06/2001, -26081 message
grin
avatar<<< Kernel Extremist©®™ >>>
Feel the power of (int16) !
Posté le 25/07/2013 à 21:28 Membre depuis le 13/06/2001, 73049 messages
Chuis sûr que c'est un challenger de plptp qui a soudoyé le stagiaire !

(Au fait, il a un anus comment, maintenant, ledit stagiaire ?)
avatar
Posté le 25/07/2013 à 21:36 Membre depuis le 16/06/2001, 69776 messages
trisotfl

ne mélange pas Pen^2 et bearbecue embarrassed
Posté le 25/07/2013 à 21:40 Membre depuis le 10/06/2001, 45104 messages
#trihum#
(Je lui demanderai de ta part #itm#)
Posté le 25/07/2013 à 22:57Edité par Kevin Kofler le 25/07/2013 à 22:59 Membre depuis le 10/06/2001, 40265 messages
Pen^2 (./45) :
(oué, mais là, on avait une deadline cheeky)

Tant pis, ce serait son travail qui aurait raté la deadline, pas le tien. tongue

(Cela dit, on aurait peut-être eu droit à une histoire sur The Daily WTF au sujet du méchant "vieux" qui sabote le travail d'un pauvre stagiaire. gni)
avatarMes news pour calculatrices TI: Ti-Gen (fr/en), MobiFiles (de)
Mes projets PC pour calculatrices TI: TIGCC, CalcForge (CalcForgeLP, Emu-TIGCC)
Mes chans IRC: #tigcc et #inspired sur irc.freequest.net (UTF-8)

Liberté, Égalité, Fraternité
Posté le 25/07/2013 à 22:59 Membre depuis le 10/06/2001, 45104 messages
Ben, si, puisqu'il avait justement viré mon code, et que je devais encore ajuster des choses cheeky
EDIT : ah oui, si, en revertant, OK... Mais bon, le client, c'est pas vraiment son problème.

(grin)
Posté le 25/07/2013 à 23:42 Membre depuis le 16/06/2001, 69776 messages
cette après midi en ayant une nouvelle fonctionnalité à implémenter, on a vu des trucs étranges dans le code à modifier, donc on a voulu faire un peu de ménage dans le code, démarré par moi il y a 3 ans, puis confié à une collègue (partie depuis) pour qu'elle le reprenne et le continue:

class chose {
  private int fvalue;
    public void setvalue(int value) {
    value = value;
  }
}


une classe remplie, en gros de :
class chose {
  private int fvalue;
    public void setvalue(int value) {
    fvalue = value;
    autreclasse.setvalue(value);
  }
  etc etc
}


des chaines d'appels qui enchainent les conversions entre byte[] et String à coup de bin2hex et hex2bin, au moins 4 à la suite à travers les différents appels. Et bien sûr, sans test de nullité nulle part.

une réimplémentation d'une méthode bin2hex bien optimale à coup d'appels à la classe Formatter

en inlinant du code de classes inutiles, on a réussi à générer un stack overflow à cause un getinstance() de sigleton dont le constructeur privé fait appel à un autre getinstance dans un autre singleton, lui même faisant appel au premier. C'était masqué à cause d'une instruction indispensable mais commentée (on comprend pourquoi).

On a plein de form générées par netbeans et remplies de code correspondant aux évènements, et puis j'ai découvert tout un tas d'appels à des classes machinchouettecontroller qui ne font que forwarder des appels

et le mieux, notre produit a des plugins, qui reposent sur une api à base d'interfaces dont les implémentations se trouvent dans le code principal. Eh bien quand elle a eu besoin de nouvelles fonctionnalités, au lieu d'ajouter des entrées dans l'interface, elle a juste fait une méthode getotherfunctions() qui renvoie directement une implémentation de ces fonctions (implémentation stockée dans le projet API directement, pas dans le code principal) ce qui lui a demandé des tonnes de classes proxies mal écrites, et des modifs cheloues un peu partout.

Et bien entendu, quand elle a fait tout ça, elle ne m'a posé aucune question d'architecture, en me disant "oui oui" chaque fois que je lui demandais si elle s'en sortait avec l'architecture de ce que j'avais écrit avant.

Résultat, j'ai l'impression de voir un tapis volant qui cache un gros tas de merde et que mon projet a été amélioré graphiquement, mais niqué en interne. Je précise que là où on bosse on a chacun un projet, on est censés être indépendants, et que personne n'a de relation de garde chiourme avec les autres développeurs. On lui a fait confiance après qu'elle aie réussi quelques trucs corrects, mais non, ce truc là, elle l'a bien foiré, et elle s'est surtout posé aucune question quant a la complexité et à la torducité de ce qu'elle écrivait.

Résultat, 2h de refactoring cette aprem pour virer un tas de classes inutiles, bon c'est pas perdu, ça a permis au nouveau collègue de bien comprendre le genre d'archi du code qu'on attend, et ce qu'il faut pas faire grin Lui est bien plus compétent, je sens en codant en tandem qu'il comprend ce qu'il voit, c'est marrant, c'est la première fois que je codais en live binome avec quelqu'un sans m'engueuler avec lui parce qu'il comprend pas grin

FUUUUUUUUUUUUUUUU
Posté le 25/07/2013 à 23:55 Membre depuis le 10/06/2001, 45104 messages
Classe.
Posté le 26/07/2013 à 00:10 Membre depuis le 11/07/2003, 54841 messages
.class, même.
avatar<<< Kernel Extremis©®™ >>> et Inventeur de la différence administratif/judiciaire ! (©Yoshi Noir)

<Vertyos> un poil plus mais elle suce bien quand même la mienne ^^
<Sabrina`> tinkiete flan c juste qu'ils sont jaloux que je te trouve aussi appétissant
Posté le 26/07/2013 à 01:13 Membre depuis le 27/04/2006, 60465 messages
^^

Ça me rappelle mon ancien boulot... ils avaient confié une partie du code à un gars dont ils se sont aperçus plus tard qu'il était complètement incompétent... et pareil, ils ne demandait rien, il disait "oui oui" mais faisait n'importe quoi. Des mois après son départ (pas vraiment volontaire ^^), on ramassait encore les morceaux, et chaque découverte d'une nouvelle partie de son code était une occasion de facepalm grin
avatarZeroblog

« Tout homme porte sur l'épaule gauche un singe et, sur l'épaule droite, un perroquet. » — Jean Cocteau
« Moi je cherche plus de logique non plus. C'est surement pour cela que j'apprécie les Ataris, ils sont aussi logiques que moi ! » — GT Turbo