markiewb / nb-additional-hints Goto Github PK

<!description="Convert to ternary">
    if ($cond$){
            $var = $a;
        }else{
            $var = $b;
        }
=>     

$var = ($cond$)?$a:$b;
;;

        if ($cond$){
            return $a;
        }else{
            return $b;
        }

return ($cond$)?$a:$b;


 <!description="Convert ternary to if/else">
return ($cond$)?$a:$b;
=>      if ($cond$){
            return $a;
        }else{
            return $b;
        }
;;

<!description="Convert ternary to if/else">
$var = ($cond$)?$a:$b;
=>      if ($cond$){
            $var = $a;
        }else{
            $var = $b;
        }
;;

         String a = null,b = null;
        String var1 = (a != null) ? a: b;//can be converted
        String var2 = a != null ? a: b;//ternary hints do not match here

//hint applied to
"a"+3+"B";
//will result in 
;

//but
String b="a"+3+"B";// works
//--> String b=MessageFormat.format("a{0}B", 3);

        @TriggerPattern(value = "($var1 != $c) ? $a : $b"),
        @TriggerPattern(value = "($var2 == $c) ? $a : $b"),
        @TriggerPattern(value = "($varc) ? $a : $b"),
        @TriggerPattern(value = "(!$varc) ? $a : $b"),

<!description="Convert to assertTrue/assertFalse">
org.junit.Assert.assertEquals(true, $any)
=> org.junit.Assert.assertTrue($any)
;;
org.junit.Assert.assertEquals($msg, true, $any)
=> org.junit.Assert.assertTrue($msg, $any)
;;
org.junit.Assert.assertEquals(false, $any)
=> org.junit.Assert.assertFalse($any)
;;
org.junit.Assert.assertEquals($msg, false, $any)
=> org.junit.Assert.assertFalse($msg, $any)
;;

        String a = null,b = null;
        //hints apply
        String var1;
        var1 = (suffix == null) ? suffix : suffix;
        //hints do not apply
        String var2 = (suffix == null) ? suffix : suffix;

"foo[inner]bar"  // [ ] shows the selection

"foo"+"inner"+"bar"

public class NewClass {

    public void SomeMethod(ArrayList foo) {
        if (foo instanceof Map) {
            System.out.println("No compile-time error");
        }
    }
}

(a != null) ? a.toString() : ""  
->
(a == null) ? "" : a.toString()

        @TriggerPattern(value = "($var1 != null) ? $a : $b"),
        @TriggerPattern(value = "($var2 == null) ? $a : $b"),
        @TriggerPattern(value = "($var3 > $c) ? $a : $b"),
        @TriggerPattern(value = "($var4 < $c) ? $a : $b"),
        @TriggerPattern(value = "($var5 >= $c) ? $a : $b"),
        @TriggerPattern(value = "($var6 <= $c) ? $a : $b"),

'?'+variable+'?' 

$1 $2;
$2 = $3;
=>
$1 $2 = $3;
;;

int a = 5;
String string = "foo";
double d = .5d;

String out = a + " is an integer and \"" + string + "\" is a string and " + d + " is an double";

String out = String.format("%s is an integer and " % s);

                            String word_orig = "abc"+"bin\\sh.exe";

                            String word_actu = "abcbinsh.exe";
                            String word_expe = "abcbin\\sh.exe";

java.util.Set a = null; if (a instanceof java.util.HashSet) {} // ERROR
java.util.Set e = null; if (e instanceof java.util.ArrayList) {} //ERROR

java.util.Set a = null; if (a instanceof java.util.HashSet) {} // *NO* ERROR
java.util.Set e = null; if (e instanceof java.util.ArrayList) {} //ERROR

String variable = name + " (" + position.getX() + ", " + position.getY() + ')';
return "Found " + variable + " entries";

@Override
    public void foo()
    {
        super.foo();
    }

@Override
    public int foo()
    {
        return super.foo();
    }

@Override
    public void foo()
    {
        return;
    }

@Override
    public void foo()
    {

    }

throw new RuntimeException("type "+ type+" ");

 I want to identify and refactor fields and member functions so that they all have the optional "this." present instead of possibly absent.

new BigDecimal("0.0")
new BigDecimal("0")
new BigDecimal("0.00")
new BigDecimal(0)
new BigDecimal(0L)
new BigDecimal(0.0f)
new BigDecimal(0.0d)
BigDecimal.valueOf(0)
BigDecimal.valueOf(0L)
BigDecimal.valueOf(0.0f)

BigDecimal.ZERO

"package test;\n"
                        + "public class Test {\n"
                        + "    public static void main(String[] args) {\n"
                        + "        String a=\"foo|bar\";\n"
                        + "    }\n"
                        + "}\n"

package test;
public class Test {
    public static void main(String[] args) {
        String a=\"foo|bar\";
    }
}

"foo|bar" 

"foo"+"bar"

        logger.debug("ABC " + foo != null ? "notnull "+ foo : "null");

        logger.debug(String.format("ABC %s", foo) != null ? "notnull "+ foo : "null");

        logger.debug(String.format("ABC %s", foo != null ? "notnull "+ foo : "null"));

    for (final Object t : timerService.getTimers())
    {
        // If the timers aren't Timers, skip
        if (!(t instanceof Timer))
        {
            continue;
        }
        final Timer candidate = (Timer)t;

<!description="Convert to StringUtils.isBlank()">
$v == null || $v.trim().length() == 0
=> org.apache.commons.lang.StringUtils.isBlank($v)
;;
$v == null || $v.trim().isEmpty()
=> org.apache.commons.lang.StringUtils.isBlank($v)
;;
$v != null && $v.trim().length() == 0
=> org.apache.commons.lang.StringUtils.isBlank($v)
;;
$v != null && $v.trim().isEmpty()
=> org.apache.commons.lang.StringUtils.isBlank($v)
;;

<!description="Convert to StringUtils.isNotBlank()">
$v != null && $v.trim().length() > 0
=> org.apache.commons.lang.StringUtils.isNotBlank($v)
;;
$v != null && !$v.trim().isEmpty()
=> org.apache.commons.lang.StringUtils.isNotBlank($v)
;;

<!description="Convert to StringUtils.isEmpty()">
$v == null || $v.length() == 0
=> org.apache.commons.lang.StringUtils.isEmpty($v)
;;
$v == null || $v.isEmpty()
=> org.apache.commons.lang.StringUtils.isEmpty($v)
;;
$v != null && $v.isEmpty()
=> org.apache.commons.lang.StringUtils.isEmpty($v)
;;
$v != null && $v.length() == 0
=> org.apache.commons.lang.StringUtils.isEmpty($v)
;;