Как вы определили, преимущество этого подхода состоит в том, что компилятор гарантирует, что идентификатор, используемый на странице, всегда совпадает с идентификатором в вашем коде Java (например, метод портлета processAction, который читает переданное поле). Изменение значения константы String автоматически обновляет представление. Это также может помочь определить использование строковых констант.
Недостатком является то, что это делает код довольно уродливым, и я бы по возможности избегал кода в стиле скриптлета.
То, изменил бы я это, вероятно, будет зависеть от чистоты кода. Если идентификаторы String смешиваются с беспорядком других констант, а представления представляют собой тысячи джунглей скриптов, я бы, вероятно, оставил их в покое.
Так как это портлеты, если бы я чувствовал себя абсолютно неуверенно в отношении чистоты кода, я бы назвал это входное имя с помощью функции EL.