jan-christiansen
commented
1 year ago Hier sind noch ein paar Beispiele aus den Laboraufgaben, um den Ansatz später zu evaluieren.
Im besten Fall würde die Regel für die Methoden containsHelp und isNull bessere Namen vorschlagen.
static <T> boolean containsHelp(final T current, final T value) {
return value == null && current == null || current != null && current.equals(value);
}static <T> boolean isNull(T temp, T value) {
return temp != null && temp.equals(value) || temp == null && value == null;
}Auch für die Methoden helpcountNodes und add sollte die Regel im besten Fall bessere Namen vorschlagen.
public int countNodes() {
return helpcountNodes(root);
}
public int helpcountNodes(final BinTreeNode root) {
if (root == null) {
return 0;
} else {
return 1 + helpcountNodes(root.right) + helpcountNodes(root.left);
}
}private int count(final BinTreeNode check) {
if (check == null) {
return 0;
} else {
return 1 + count(check.left) + count(check.right);
}
}
private void add(final List<Integer> list, final BinTreeNode node) {
if (node != null) {
if (node.right != null) {
add(list, node.right);
}
list.add(0, node.value);
if (node.left != null) {
add(list, node.left);
}
}
}
jokogr25
Averodas
In diesem Issue soll einmal beispielhaft eine Regel implementiert werden, die mithilfe eines wahrscheinlichkeitsbasierten Ansatzes Vorschläge für die Verbesserung von Bezeichnern macht. Im konkreten Fall sollen Vorschläge für Methodennamen gemacht werden. Der Ansatz soll die Technik unter https://code2vec.org einsetzen. Dazu muss erst einmal erarbeitet werden, wie das trainierte neuronale Netz unter https://github.com/tech-srl/code2vec verwendet werden kann. Die Struktur der Regel ist ähnlich zur Regel, welche die Sprache eines Bezeichners überprüft. Mit einem Shell-Kommando wird für eine Methode das neuronale Netz ausgewertet und liefert Vorschläge inklusive Wahrscheinlichkeit des Vorschlages. Als Eingabe verlangt das neuronale Netz allerdings die Definition der Methode. Es muss also erarbeitet werden, welche Form dieses Argument genau haben muss.