Read employee number and salary (exercise to use static keyword)
No.

You use a setter to modify an existing employee. But it makes no sense to change the id and salary together. In fact, you should never change the id of an employee.

This should be the constructor of the class instead. In your main I would expect to see the folowing flow:

Scanner kb=new Scanner(System.in);
int id,sal;
System.out.println("Enter Employee Id: ");
id=kb.nextInt();
System.out.println("Enter Salary: ");
sal=kb.nextInt();
System.out.println("The Values entered are :");
Employee e=new Employee(id, sal);


A second thing to note is that a method with prefix "get" is expected to return some value of the object. Either rename that getValue() method to something like printInfo() or provide actual getters:

public String getId() {
    return id;
}

public static int getNbOfEmployees(){
    return count;
}


These can then be used somewhere else to print out the info, for example in your main:

System.out.println("EmpId = " + e.getId());
System.out.println("Salary =" + e.alary());
System.out.println("Number of employees: = " + Employee.getNbOfEmployees());