我想知道,内联小型私有函数是否是一个好主意?
就我而言,这些功能仅出于可读性目的而存在,而且我知道,它们仅被调用过几次,因此较大的字节码大小无关紧要。
我知道,性能提升也可能是微不足道的,因为我没有传递函数类型(并且编译器实际上警告过我),但是让我们假设它是我们应用程序中的热点。
实际示例:
我有一个理论上的无限符号带(像图灵机一样),它由两个阵列(位置<0和位置> = 0分别为左和右)建模。现在,我已经进行了很多读写操作。
在Java中,我有:
/**
* @param cell the cell
* @return the symbol at the specified cell
*/
public char read(int cell) {
char[] tape;
if (cell < 0) {
tape = left;
cell = -cell - 1;
} else {
tape = right;
}
return cell < tape.length ? tape[cell] : blank;
}
/**
* Writes a symbol to the specified cell.
* @param c the symbol
* @param cell the cell
*/
public void write(char c, int cell) {
char[] tape;
if (cell < 0) {
cell = -cell - 1;
if (cell >= left.length) left = expandArray(left, cell, blank);
tape = left;
} else {
if (cell >= right.length) right = expandArray(right, cell, blank);
tape = right;
}
tape[cell] = c;
}
现在我想将代码片段翻译为kotlin,阅读内联函数并提出以下内容:
fun read(cell: Int = headPosition) = when {
cell < 0 -> read(left, -cell - 1)
else -> read(right, cell)
}
private inline fun read(tape: CharArray, cell: Int): Char {
return if (cell < tape.size) tape[cell] else blank
}
fun write(c: Char, cell: Int = headPosition) = when {
cell < 0 -> left = write(c, left, -cell - 1)
else -> right = write(c, right, cell)
}
private inline fun write(c: Char, tape: CharArray, cell: Int): CharArray = when {
cell >= tape.size -> expandArray(tape, cell, blank)
else -> tape
}.also { it[cell] = c }
我个人认为,尤其是读取功能很容易读取。
所以这是一个好主意,我可以忽略IDE的警告吗?还是我错过了什么?也许有一种最佳实践或其他方式来编写这些函数,而不必两次(几乎)重复相同的行(对于位置<0和位置> = 0)。
答案 0 :(得分:5)
不需要内联非常小的重复使用的函数,因为JVM JIT在认为合适的情况下很可能会为您执行此操作。如果内联函数,则确实会在生成的字节码中造成一点膨胀,但是对于未从代码中许多不同点调用的函数,也没有太大的危害。对于较大的内联函数,代码中许多地方使用的函数以及调用其他内联函数的内联函数,字节码膨胀更糟。您的案件根本不会造成任何影响。
短方法从内联中受益匪浅,正如您在When Short Methods Pay Off: JIT Inlining中所读到的:
在以下情况下,一种方法可以进行内联
:
- 它很小-字节码大小小于35个字节(可以被-XX:MaxInlineSize = X标志覆盖)。
- 它经常被调用(很热),并且小于325个字节(可以被-XX:MaxFreqInlineSize = X标志覆盖)。
如果您超出了这些参数,或者想预热内联,则可以使用inline
关键字使其在编译时发生。
在以下情况下,您还希望在小型函数上使用inline
: