I used IntelliJ for "Inspect Code", and one of its results is:
Problem synopsis Can be package local (at line 18(public class HeartBeat)
)
What does it mean, how can I fix it?
it whole class is like this:
package com.xxxxxxxxxxx.app.xxxx;
public class HeartBeat
{
private static final Logger LOG = LoggerFactory.getLogger( HeartBeat.class );
private final File heartBeatFile;
public HeartBeat( File heartBeatFile )
{
this.heartBeatFile = heartBeatFile;
}
public void beat()
{
try
{
FileUtils.writeStringToFile( heartBeatFile, String.valueOf( System.currentTimeMillis() ) );
}
catch( IOException e )
{
LOG.error( "Error while writing heart beat log", e );
}
}
}
Your HeartBeat class isn't used anywhere outside of package com.xxxxxxxxxxx.app.xxxx. In this case you can declare the class 'protected' or 'private' to be more precise in your access.
If you do not intend to use this class outside of this package, you should change the class declaration. If you intend to use this class outside this package, leave it and the warning will go away.
E.g.:
Coming from the Android app background, you also need to consider that sometimes those warnings are redundant. So for
debug build
classes, it can bepackage local
but forrelease build
it might be used outside of package.I have disabled it on my end and @ashario answer was quite helpful in finding how to do it.
Each of these lint warnings can be suppressed on a case-by-case basis with the following code.
IDEA is referring to package-private visibility.
For more information, see Controlling Access to Members of a Class.
You can solve the problem by removing
public
keyword from the class (if the class is not intended to be used outside the package), or by using the class from a different package.If you want to suppress this warning:
EDIT: in the latest IDEA release step 4 this looks to have changed to "Suggest package-private visibility level for ..." and includes several options for various conditions