Separate nested if-blocks into helper methods (refactor step 1)
Change-Id: I87ef8b174295fb12a91fec35acd3b0cc152ad8f0
This commit is contained in:
Tadashi G. Takaoka
parent
33aa6e3cf0
commit
8b449c6dda
1 changed files with 127 additions and 112 deletions
|
|
@ -893,15 +893,134 @@ public final class PointerTracker implements PointerTrackerQueue.Element {
|
||||||
onMoveEventInternal(x, y, eventTime);
|
onMoveEventInternal(x, y, eventTime);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private void slideInToNewKey(final Key newKey, final int x, final int y, final long eventTime) {
|
||||||
|
// The pointer has been slid in to the new key, but the finger was not on any keys.
|
||||||
|
// In this case, we must call onPress() to notify that the new key is being pressed.
|
||||||
|
// This onPress call may have changed keyboard layout. Those cases are detected at
|
||||||
|
// {@link #setKeyboard}. In those cases, we should update key according to the
|
||||||
|
// new keyboard layout.
|
||||||
|
Key key = newKey;
|
||||||
|
if (callListenerOnPressAndCheckKeyboardLayoutChange(key)) {
|
||||||
|
key = onMoveKey(x, y);
|
||||||
|
}
|
||||||
|
onMoveToNewKey(key, x, y);
|
||||||
|
startLongPressTimer(key);
|
||||||
|
setPressedKeyGraphics(key, eventTime);
|
||||||
|
}
|
||||||
|
|
||||||
|
private void slideFromOldKeyToNewKey(final Key newKey, final int x, final int y,
|
||||||
|
final long eventTime, final Key oldKey, final int lastX, final int lastY) {
|
||||||
|
// The pointer has been slid in to the new key from the previous key, we must call
|
||||||
|
// onRelease() first to notify that the previous key has been released, then call
|
||||||
|
// onPress() to notify that the new key is being pressed.
|
||||||
|
Key key = newKey;
|
||||||
|
setReleasedKeyGraphics(oldKey);
|
||||||
|
callListenerOnRelease(oldKey, oldKey.mCode, true);
|
||||||
|
startSlidingKeyInput(oldKey);
|
||||||
|
mTimerProxy.cancelKeyTimers();
|
||||||
|
startRepeatKey(key);
|
||||||
|
if (mIsAllowedSlidingKeyInput) {
|
||||||
|
// This onPress call may have changed keyboard layout. Those cases are detected
|
||||||
|
// at {@link #setKeyboard}. In those cases, we should update key according
|
||||||
|
// to the new keyboard layout.
|
||||||
|
if (callListenerOnPressAndCheckKeyboardLayoutChange(key)) {
|
||||||
|
key = onMoveKey(x, y);
|
||||||
|
}
|
||||||
|
onMoveToNewKey(key, x, y);
|
||||||
|
startLongPressTimer(key);
|
||||||
|
setPressedKeyGraphics(key, eventTime);
|
||||||
|
} else {
|
||||||
|
// HACK: On some devices, quick successive touches may be reported as a sudden
|
||||||
|
// move by touch panel firmware. This hack detects such cases and translates the
|
||||||
|
// move event to successive up and down events.
|
||||||
|
// TODO: Should find a way to balance gesture detection and this hack.
|
||||||
|
if (sNeedsPhantomSuddenMoveEventHack
|
||||||
|
&& getDistance(x, y, lastX, lastY) >= mPhantonSuddenMoveThreshold) {
|
||||||
|
if (DEBUG_MODE) {
|
||||||
|
Log.w(TAG, String.format("[%d] onMoveEvent:"
|
||||||
|
+ " phantom sudden move event (distance=%d) is translated to "
|
||||||
|
+ "up[%d,%d,%s]/down[%d,%d,%s] events", mPointerId,
|
||||||
|
getDistance(x, y, lastX, lastY),
|
||||||
|
lastX, lastY, Constants.printableCode(oldKey.mCode),
|
||||||
|
x, y, Constants.printableCode(key.mCode)));
|
||||||
|
}
|
||||||
|
// TODO: This should be moved to outside of this nested if-clause?
|
||||||
|
if (ProductionFlag.IS_EXPERIMENTAL) {
|
||||||
|
ResearchLogger.pointerTracker_onMoveEvent(x, y, lastX, lastY);
|
||||||
|
}
|
||||||
|
onUpEventInternal(eventTime);
|
||||||
|
onDownEventInternal(x, y, eventTime);
|
||||||
|
}
|
||||||
|
// HACK: On some devices, quick successive proximate touches may be reported as
|
||||||
|
// a bogus down-move-up event by touch panel firmware. This hack detects such
|
||||||
|
// cases and breaks these events into separate up and down events.
|
||||||
|
else if (sNeedsProximateBogusDownMoveUpEventHack
|
||||||
|
&& sTimeRecorder.isInFastTyping(eventTime)
|
||||||
|
&& mBogusMoveEventDetector.isCloseToActualDownEvent(x, y)) {
|
||||||
|
if (DEBUG_MODE) {
|
||||||
|
final float keyDiagonal = (float)Math.hypot(
|
||||||
|
mKeyboard.mMostCommonKeyWidth, mKeyboard.mMostCommonKeyHeight);
|
||||||
|
final float radiusRatio =
|
||||||
|
mBogusMoveEventDetector.getDistanceFromDownEvent(x, y)
|
||||||
|
/ keyDiagonal;
|
||||||
|
Log.w(TAG, String.format("[%d] onMoveEvent:"
|
||||||
|
+ " bogus down-move-up event (raidus=%.2f key diagonal) is "
|
||||||
|
+ " translated to up[%d,%d,%s]/down[%d,%d,%s] events",
|
||||||
|
mPointerId, radiusRatio,
|
||||||
|
lastX, lastY, Constants.printableCode(oldKey.mCode),
|
||||||
|
x, y, Constants.printableCode(key.mCode)));
|
||||||
|
}
|
||||||
|
onUpEventInternal(eventTime);
|
||||||
|
onDownEventInternal(x, y, eventTime);
|
||||||
|
} else {
|
||||||
|
// HACK: If there are currently multiple touches, register the key even if
|
||||||
|
// the finger slides off the key. This defends against noise from some
|
||||||
|
// touch panels when there are close multiple touches.
|
||||||
|
// Caveat: When in chording input mode with a modifier key, we don't use
|
||||||
|
// this hack.
|
||||||
|
if (getActivePointerTrackerCount() > 1 && sPointerTrackerQueue != null
|
||||||
|
&& !sPointerTrackerQueue.hasModifierKeyOlderThan(this)) {
|
||||||
|
if (DEBUG_MODE) {
|
||||||
|
Log.w(TAG, String.format("[%d] onMoveEvent:"
|
||||||
|
+ " detected sliding finger while multi touching",
|
||||||
|
mPointerId));
|
||||||
|
}
|
||||||
|
onUpEvent(x, y, eventTime);
|
||||||
|
mKeyAlreadyProcessed = true;
|
||||||
|
}
|
||||||
|
if (!mIsDetectingGesture) {
|
||||||
|
mKeyAlreadyProcessed = true;
|
||||||
|
}
|
||||||
|
setReleasedKeyGraphics(oldKey);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
private void slideOutFromOldKey(final Key oldKey, final int x, final int y) {
|
||||||
|
// The pointer has been slid out from the previous key, we must call onRelease() to
|
||||||
|
// notify that the previous key has been released.
|
||||||
|
setReleasedKeyGraphics(oldKey);
|
||||||
|
callListenerOnRelease(oldKey, oldKey.mCode, true);
|
||||||
|
startSlidingKeyInput(oldKey);
|
||||||
|
mTimerProxy.cancelLongPressTimer();
|
||||||
|
if (mIsAllowedSlidingKeyInput) {
|
||||||
|
onMoveToNewKey(null, x, y);
|
||||||
|
} else {
|
||||||
|
if (!mIsDetectingGesture) {
|
||||||
|
mKeyAlreadyProcessed = true;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
private void onMoveEventInternal(final int x, final int y, final long eventTime) {
|
private void onMoveEventInternal(final int x, final int y, final long eventTime) {
|
||||||
final int lastX = mLastX;
|
final int lastX = mLastX;
|
||||||
final int lastY = mLastY;
|
final int lastY = mLastY;
|
||||||
final Key oldKey = mCurrentKey;
|
final Key oldKey = mCurrentKey;
|
||||||
Key key = onMoveKey(x, y);
|
final Key newKey = onMoveKey(x, y);
|
||||||
|
|
||||||
if (sShouldHandleGesture) {
|
if (sShouldHandleGesture) {
|
||||||
// Register move event on gesture tracker.
|
// Register move event on gesture tracker.
|
||||||
onGestureMoveEvent(x, y, eventTime, true /* isMajorEvent */, key);
|
onGestureMoveEvent(x, y, eventTime, true /* isMajorEvent */, newKey);
|
||||||
if (sInGesture) {
|
if (sInGesture) {
|
||||||
mCurrentKey = null;
|
mCurrentKey = null;
|
||||||
setReleasedKeyGraphics(oldKey);
|
setReleasedKeyGraphics(oldKey);
|
||||||
|
|
@ -909,119 +1028,15 @@ public final class PointerTracker implements PointerTrackerQueue.Element {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if (key != null) {
|
if (newKey != null) {
|
||||||
if (oldKey == null) {
|
if (oldKey == null) {
|
||||||
// The pointer has been slid in to the new key, but the finger was not on any keys.
|
slideInToNewKey(newKey, x, y, eventTime);
|
||||||
// In this case, we must call onPress() to notify that the new key is being pressed.
|
} else if (isMajorEnoughMoveToBeOnNewKey(x, y, eventTime, newKey)) {
|
||||||
// This onPress call may have changed keyboard layout. Those cases are detected at
|
slideFromOldKeyToNewKey(newKey, x, y, eventTime, oldKey, lastX, lastY);
|
||||||
// {@link #setKeyboard}. In those cases, we should update key according to the
|
|
||||||
// new keyboard layout.
|
|
||||||
if (callListenerOnPressAndCheckKeyboardLayoutChange(key)) {
|
|
||||||
key = onMoveKey(x, y);
|
|
||||||
}
|
|
||||||
onMoveToNewKey(key, x, y);
|
|
||||||
startLongPressTimer(key);
|
|
||||||
setPressedKeyGraphics(key, eventTime);
|
|
||||||
} else if (isMajorEnoughMoveToBeOnNewKey(x, y, eventTime, key)) {
|
|
||||||
// The pointer has been slid in to the new key from the previous key, we must call
|
|
||||||
// onRelease() first to notify that the previous key has been released, then call
|
|
||||||
// onPress() to notify that the new key is being pressed.
|
|
||||||
setReleasedKeyGraphics(oldKey);
|
|
||||||
callListenerOnRelease(oldKey, oldKey.mCode, true);
|
|
||||||
startSlidingKeyInput(oldKey);
|
|
||||||
mTimerProxy.cancelKeyTimers();
|
|
||||||
startRepeatKey(key);
|
|
||||||
if (mIsAllowedSlidingKeyInput) {
|
|
||||||
// This onPress call may have changed keyboard layout. Those cases are detected
|
|
||||||
// at {@link #setKeyboard}. In those cases, we should update key according
|
|
||||||
// to the new keyboard layout.
|
|
||||||
if (callListenerOnPressAndCheckKeyboardLayoutChange(key)) {
|
|
||||||
key = onMoveKey(x, y);
|
|
||||||
}
|
|
||||||
onMoveToNewKey(key, x, y);
|
|
||||||
startLongPressTimer(key);
|
|
||||||
setPressedKeyGraphics(key, eventTime);
|
|
||||||
} else {
|
|
||||||
// HACK: On some devices, quick successive touches may be reported as a sudden
|
|
||||||
// move by touch panel firmware. This hack detects such cases and translates the
|
|
||||||
// move event to successive up and down events.
|
|
||||||
// TODO: Should find a way to balance gesture detection and this hack.
|
|
||||||
if (sNeedsPhantomSuddenMoveEventHack
|
|
||||||
&& getDistance(x, y, lastX, lastY) >= mPhantonSuddenMoveThreshold) {
|
|
||||||
if (DEBUG_MODE) {
|
|
||||||
Log.w(TAG, String.format("[%d] onMoveEvent:"
|
|
||||||
+ " phantom sudden move event (distance=%d) is translated to "
|
|
||||||
+ "up[%d,%d,%s]/down[%d,%d,%s] events", mPointerId,
|
|
||||||
getDistance(x, y, lastX, lastY),
|
|
||||||
lastX, lastY, Constants.printableCode(oldKey.mCode),
|
|
||||||
x, y, Constants.printableCode(key.mCode)));
|
|
||||||
}
|
|
||||||
// TODO: This should be moved to outside of this nested if-clause?
|
|
||||||
if (ProductionFlag.IS_EXPERIMENTAL) {
|
|
||||||
ResearchLogger.pointerTracker_onMoveEvent(x, y, lastX, lastY);
|
|
||||||
}
|
|
||||||
onUpEventInternal(eventTime);
|
|
||||||
onDownEventInternal(x, y, eventTime);
|
|
||||||
}
|
|
||||||
// HACK: On some devices, quick successive proximate touches may be reported as
|
|
||||||
// a bogus down-move-up event by touch panel firmware. This hack detects such
|
|
||||||
// cases and breaks these events into separate up and down events.
|
|
||||||
else if (sNeedsProximateBogusDownMoveUpEventHack
|
|
||||||
&& sTimeRecorder.isInFastTyping(eventTime)
|
|
||||||
&& mBogusMoveEventDetector.isCloseToActualDownEvent(x, y)) {
|
|
||||||
if (DEBUG_MODE) {
|
|
||||||
final float keyDiagonal = (float)Math.hypot(
|
|
||||||
mKeyboard.mMostCommonKeyWidth, mKeyboard.mMostCommonKeyHeight);
|
|
||||||
final float radiusRatio =
|
|
||||||
mBogusMoveEventDetector.getDistanceFromDownEvent(x, y)
|
|
||||||
/ keyDiagonal;
|
|
||||||
Log.w(TAG, String.format("[%d] onMoveEvent:"
|
|
||||||
+ " bogus down-move-up event (raidus=%.2f key diagonal) is "
|
|
||||||
+ " translated to up[%d,%d,%s]/down[%d,%d,%s] events",
|
|
||||||
mPointerId, radiusRatio,
|
|
||||||
lastX, lastY, Constants.printableCode(oldKey.mCode),
|
|
||||||
x, y, Constants.printableCode(key.mCode)));
|
|
||||||
}
|
|
||||||
onUpEventInternal(eventTime);
|
|
||||||
onDownEventInternal(x, y, eventTime);
|
|
||||||
} else {
|
|
||||||
// HACK: If there are currently multiple touches, register the key even if
|
|
||||||
// the finger slides off the key. This defends against noise from some
|
|
||||||
// touch panels when there are close multiple touches.
|
|
||||||
// Caveat: When in chording input mode with a modifier key, we don't use
|
|
||||||
// this hack.
|
|
||||||
if (getActivePointerTrackerCount() > 1 && sPointerTrackerQueue != null
|
|
||||||
&& !sPointerTrackerQueue.hasModifierKeyOlderThan(this)) {
|
|
||||||
if (DEBUG_MODE) {
|
|
||||||
Log.w(TAG, String.format("[%d] onMoveEvent:"
|
|
||||||
+ " detected sliding finger while multi touching",
|
|
||||||
mPointerId));
|
|
||||||
}
|
|
||||||
onUpEvent(x, y, eventTime);
|
|
||||||
mKeyAlreadyProcessed = true;
|
|
||||||
}
|
|
||||||
if (!mIsDetectingGesture) {
|
|
||||||
mKeyAlreadyProcessed = true;
|
|
||||||
}
|
|
||||||
setReleasedKeyGraphics(oldKey);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
if (oldKey != null && isMajorEnoughMoveToBeOnNewKey(x, y, eventTime, key)) {
|
if (oldKey != null && isMajorEnoughMoveToBeOnNewKey(x, y, eventTime, newKey)) {
|
||||||
// The pointer has been slid out from the previous key, we must call onRelease() to
|
slideOutFromOldKey(oldKey, x, y);
|
||||||
// notify that the previous key has been released.
|
|
||||||
setReleasedKeyGraphics(oldKey);
|
|
||||||
callListenerOnRelease(oldKey, oldKey.mCode, true);
|
|
||||||
startSlidingKeyInput(oldKey);
|
|
||||||
mTimerProxy.cancelLongPressTimer();
|
|
||||||
if (mIsAllowedSlidingKeyInput) {
|
|
||||||
onMoveToNewKey(key, x, y);
|
|
||||||
} else {
|
|
||||||
if (!mIsDetectingGesture) {
|
|
||||||
mKeyAlreadyProcessed = true;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue