Вы никогда не вызываете метод факультета (...):
public static void main(String[] args)
{
faculty( 5 );
}
Вы бы также избавились от:
n=5;
, который побеждает цель передачи параметра вметод, если вы всегда жестко кодируете значение.
Кроме того, почему вы бы назвали метод "факультет"? Может быть, что-то более описательное, например calculateFactorial
для имени метода, поскольку именно это вы и пытаетесь вычислить. Сделайте имена методов описательными, чтобы у нас было представление о том, что делает метод.