Leçon chapitre-nio2

[pbettens]

Vous pouvez-faire vos remarques ici.

(Je remarque que la mise en forme n'est pas identique à celle de Marco. Je mettrai à jour lorsque j'aurai vos remarques).

(Au sujet des sources, je n'ai pas encore fait de pull request pour cette leçon. Je le ferai après vos premières remarques)

[mcodutti]

Mes remarques de celui qui ne connaissait pas le sujet.

• sl 521 : comme présenté, on dirait que ce sont les 3 points qui, ensemble, idenftifient un fichier. La différence n'est pas claire entre ces 3 points et on ne sait pas exactement ce qu'est un FQN. L'exemple devrait être plsu explicite avec des annotations pour indiquer ce qui est quoi.
• sl 522 : "Path ... permet de le manipuler" et "Paths ... permettant de manipuler un path" La différence ne saite pas aux yeux. Est-ce que Paths n'est pas plutôt une Factory qui permet de créer des instrances de Path.
• sl 522 : s/premattant/permettant/
• sl 525 : toRealPath() : finalement les 2 premiers points (absolu ne change pas, relatif -> absolu) sont les mêmes que pour toAbsolutePath() au point précédant. Je les mettrais plutôt avec toAbsolutePath() et indiquerait ici : idem + gestion du lien symbolique.
• sl 527 : L'exemple est boiteux parce que si exception -> path reste à null -> NullPointerException après. En plus, je ne vois pas ce qui pourrait provoquer l'exception -> je mettrais simplement les 3 lignes utiles. Autre chose : avec ".", l'exemple donné est équivalent à ce qu'on pourrait avoir avec toAbsolutePath() il me semble.
• sl 528 : c'est marrant qu'une interface propose une méthode equals(). C'est juste pour pouvoir expliquer le contrat de la méthode via la javadoc j'imagine.
• sl 528 : Je ne mettrais pas le Comparable dont la javadoc n'est pas des plus limpides. Iterable est plus intéressant mais on ne parle plus de cette interface dans la leçon IO.
• sl 530 : Tu parles de ça parce que les méthodes de Files ont des paramètres variables mais ça ne parait pas clairement quand tu les décrits aux slides suivants.
• sl 532 : "Ces 2 méthodes ne sont pas complémentaires". L'API indique en quoi elles ne sont pas complémentaires mais je n'ai pas compris pourquoi l'existence du fichier pourrait ne pas être déterminée (manque de permissions d'aller voir ?)
• sl 532 : La remarque finale est OK comme méta-informations mais pas dans les slides finaux. On ne sait pas qui est "moi"
• sl 535 : un "\" en trop dans l'exemple
• sl 540 : est-ce vraiment la méthode qui est Closeable ? Ce ne serait pas plutôt l'obet retourné ?

Rq générale : cf. ma remarque sur Fora; j'ai peur que ça fasse trop de matière pour les 18h qui nous restent. En plus cette leçon va prendre du temps car :

• beaucoup de notions nouvelles qui prennent du temps à expliquer
• besoin de souvent aller dans l'API pour éclaircir des phrases des slides.

S'il fallait réduire un peu, j'enlèverais ce qui concerne : les liens symbolqiues, la notion d'URI et de MIME (haut slide 524, slides 525, 526 et 546.

Merci.

[cathleruste]

En première lecture :

sl 519 : Tu peux signaler que "NIO" est le nom du package stp ? sl 521 : Rappel point 2 : s/peut etre/peut être sl 521 : Rappel point 3 : s/certains système de fichiers/certains systèmes de fichiers sl 522 : s/La classe Path/L'interface Path sl 522 : La classe Paths : on n'a jamais parlé de final pour une classe, n'est-ce pas l'occasion ? sl 522 : "La classe Paths, " : retirer la "," sl 522 : s/premettant/permettant sl 523 : ce serait bien d'avoir vu varargs avant, histoire de pouvoir leur montrer l'API de get de Paths sl 527 : Pq la "->" après le "java ResolvePath" qui suit l'encadré sl 528 : Ah ben oui, tu dis que c'est une interface ^^ sl 530 : s/peut-être/peut être sl 532 : un mot sur LinkOption ? sl 532 : s/exists(Path, LinkOption)/exists(Path, LinkOption...) sl 532 : s/notExists(Path, LinkOption)/notExists(Path, LinkOption...) sl 534 : s/Effacer (delete) un fichier ou un répertoire/Effacer (delete) un fichier ou un répertoire vide sl 535 : s/copy(Path, Path, CopyOption)/copy(Path, Path, CopyOption...) sl 535 : encadré : s/.REPLACE_EXISTING/.REPLACE_EXISTING sl 535 : s/move(Path, Path, CopyOption)/move(Path, Path, CopyOption...) sl 543 : s/input stream/InputStream sl 543 : s/output stream/OutputStream sl 549 : tiens, c'est marrant, là tu mets un "\n" alors que tu as utilisé plein de "%n" dans les slides précédents :) sl 551 : il y a des "-" dans tes liens.

J'aurai peut-être encore d'autres remarques quand j'aurai digéré le sujet...

Merci pour le boulot!