0

ユーザーを自動化できる Automator クラスがあります。Windowsでシステムクリップボードを設定する際に特に問題があります。Automator クラスは、システム クリップボードを設定するスレッドである ClipSetThread クラスを利用します。ClipSetThread のインスタンスは入力としてスレッドを取り、null の場合は結合します (完了するのを待ちます)。

私は ClipSetThread を正しく呼び出していないと感じています。これは、信頼性に関して以前に発生したエラーがまだ残っているためです。ClipSetThread の前。このコードは実行時にエラーをスローしませんが、約 2/3 の時間で動作します。それ以外の場合は、1134、_234 などを出力します。スレッドが互いに結合 (待機) していないか、スキップされているようです。

コード：

import java.awt.AWTException;
import java.awt.Robot;
import java.awt.event.KeyEvent;

import org.jnativehook.GlobalScreen;
import org.jnativehook.NativeHookException;
import org.jnativehook.mouse.NativeMouseEvent;
import org.jnativehook.mouse.NativeMouseInputListener;

public class Automator extends Thread implements NativeMouseInputListener 
{
Robot rob = null;
TheAppClass theApp = null;
ClipSetThread lastClipSet = null; 
boolean doit = false;
boolean settingClip = false;

public void run()
{
    try // to make the Global hook
    {
        GlobalScreen.registerNativeHook();
    }
    catch (NativeHookException ex){theApp.updateOutput("No Global Keyboard or Mouse Hook");return;}
    try // to create a robot (can simulate user input such as mouse and keyboard input)
    {
        rob = new Robot();
    } 
    catch (AWTException e1) {theApp.updateOutput("The Robot could not be created");return;}

    while(true) {}
}

public void setApp(TheAppClass app)
{
    theApp = app;
    theApp.updateOutput("Succesfully started automator");
}

public void setClip(String arg)
{
    ClipSetThread set = new ClipSetThread(theApp, lastClipSet);
    lastClipSet = set;
    set.setClip(arg);
}

public void DOit()
{
    theApp.updateOutput("Starting");
    pasteAtCursorLocation("1");
    tab(1);
    pasteAtCursorLocation("2");
    tab(1);
    pasteAtCursorLocation("3");
    tab(1);
    pasteAtCursorLocation("4");
    tab(1);
    theApp.updateOutput("Complete");
}

public void nativeMouseReleased(NativeMouseEvent e) 
{
    //System.out.println("Mouse Released: " + e.getButton());
    if(doit)
    {
        DOit();
        doit = false;
    }
}

public void pasteAtCursorLocation(String text)
{
    setClip(text);
    rob.keyPress(KeyEvent.VK_CONTROL);
    rob.keyPress(KeyEvent.VK_V);
    rob.keyRelease(KeyEvent.VK_V);
    rob.keyRelease(KeyEvent.VK_CONTROL);
    theApp.updateOutput("Simulated Paste");
}
public void tab(int numTimes)
{
    while(numTimes > 0)
    {
        rob.keyPress(KeyEvent.VK_TAB);
        rob.keyRelease(KeyEvent.VK_TAB);
        numTimes--;
        theApp.updateOutput("Simulated Tab");
    }
}
// Unimplemented
public void nativeMouseClicked(NativeMouseEvent arg0) {}
public void nativeMousePressed(NativeMouseEvent arg0) {}
public void nativeMouseDragged(NativeMouseEvent arg0) {}
public void nativeMouseMoved(NativeMouseEvent arg0) {}
}

クリップセット スレッド:

import java.awt.Toolkit;
import java.awt.datatransfer.Clipboard;
import java.awt.datatransfer.DataFlavor;
import java.awt.datatransfer.StringSelection;
import java.awt.datatransfer.Transferable;


public class ClipSetThread extends Thread 
{
Clipboard sysClip = null;
TheAppClass  theApp = null;

public ClipSetThread(TheAppClass  app, Thread waitFor)
{
    theApp = app;
    sysClip = Toolkit.getDefaultToolkit().getSystemClipboard();
    if(waitFor != null)
        {try {waitFor.join();}catch (InterruptedException e) {}}
}
public void setClip(String arg)
{
    // Two strings that will hopefully never be on the clipboard
    String checkStr1 = "9999999999999";
    String checkStr2 = "99999999999999";
    // When we read in the clipboard we want to see if we change these strings from the ones they
    // will never be, if they do change we read the clipboard successfully
    String clipBoardTextBefore = checkStr1;
    String clipBoardTextAfter = checkStr2;

    // First get a copy of the current system clipboard text
    while(true)
    {
        try 
        {
            Transferable contents = sysClip.getContents(null);
            clipBoardTextBefore = (String)contents.getTransferData(DataFlavor.stringFlavor);
        }
        catch(Exception e)
        {
            try {Thread.sleep(20);} catch (InterruptedException e1) {}
            continue;
        }
        break;
    }
    // If we failed to change the string it means we failed to read the text
    if(clipBoardTextBefore.equals(checkStr1)) 
        theApp.updateOutput("Could NOT get sysClip text");
    else
    {
        // If we didn't failed to get the current text try to change it
        while(true) 
        {
            try{sysClip.setContents(new StringSelection(arg), null);}
            catch(Exception e)
            {
                try {Thread.sleep(20);} catch (InterruptedException e1) {}
                continue;
            }
            break;
        }

        // Now again check to see the clipboard text
        while(true)
        {
            try 
            {
                Transferable contents = sysClip.getContents(null);
                clipBoardTextAfter = (String)contents.getTransferData(DataFlavor.stringFlavor);
            }
            catch(Exception e)
            {
                try {Thread.sleep(20);} catch (InterruptedException e1) {}
                continue;
            }
            break;
        }
        // If we failed to read the clipboard text
        if(clipBoardTextAfter.equals(checkStr2)) 
            theApp.updateOutput("Could NOT check if sysClip update was successful");
        else
        { // We re-read the clipboard text, see if it changed from the original clipboard text 
            if(clipBoardTextAfter.equals(checkStr1)) 
                theApp.updateOutput("Could NOT successfully set clipboard text");
            else
                theApp.updateOutput("Set Clipboard Text:" + arg + "\n");
        }
    }
}
}
4

2 に答える 2

2

したがって、まず、 を呼び出すことはありませstartんClipSetThread。また、スレッドに参加する前に、スレッドがまだ生きているかどうかを確認する必要があります。

public class ClipSetThread extends Thread {

    Clipboard sysClip = null;
    TheAppClass theApp = null;

    private String toClipboard;

    public ClipSetThread(TheAppClass app, Thread waitFor, String toClipBoard) {
        theApp = app;
        sysClip = Toolkit.getDefaultToolkit().getSystemClipboard();
        this.toClipboard = toClipBoard;
        // !! Check to see if the thread is also alive before trying to join with it...
        if (waitFor != null && waitFor.isAlive()) {
            try {
                waitFor.join();
            } catch (InterruptedException e) {
            }
        }
    }

    // You should really put your logic into the `run` method in order to allow
    // the code to actually run in a separate thread...otherwise there is no
    // point in using a thread....
    @Override
    public void run() {
        // Two strings that will hopefully never be on the clipboard
        String checkStr1 = "9999999999999";
        String checkStr2 = "99999999999999";
        // When we read in the clipboard we want to see if we change these strings from the ones they
        // will never be, if they do change we read the clipboard successfully
        String clipBoardTextBefore = checkStr1;
        String clipBoardTextAfter = checkStr2;

        // First get a copy of the current system clipboard text
        while (true) {
            try {
                Transferable contents = sysClip.getContents(null);
                clipBoardTextBefore = (String) contents.getTransferData(DataFlavor.stringFlavor);
            } catch (Exception e) {
                try {
                    Thread.sleep(20);
                } catch (InterruptedException e1) {
                }
                continue;
            }
            break;
        }
        // If we failed to change the string it means we failed to read the text
        if (clipBoardTextBefore.equals(checkStr1)) {
            theApp.updateOutput("Could NOT get sysClip text");
        } else {
            // If we didn't failed to get the current text try to change it
            while (true) {
                try {
                    sysClip.setContents(new StringSelection(toClipboard), null);
                } catch (Exception e) {
                    try {
                        Thread.sleep(20);
                    } catch (InterruptedException e1) {
                    }
                    continue;
                }
                break;
            }

            // Now again check to see the clipboard text
            while (true) {
                try {
                    Transferable contents = sysClip.getContents(null);
                    clipBoardTextAfter = (String) contents.getTransferData(DataFlavor.stringFlavor);
                } catch (Exception e) {
                    try {
                        Thread.sleep(20);
                    } catch (InterruptedException e1) {
                    }
                    continue;
                }
                break;
            }
            // If we failed to read the clipboard text
            if (clipBoardTextAfter.equals(checkStr2)) {
                theApp.updateOutput("Could NOT check if sysClip update was successful");
            } else { // We re-read the clipboard text, see if it changed from the original clipboard text 
                if (clipBoardTextAfter.equals(checkStr1)) {
                    theApp.updateOutput("Could NOT successfully set clipboard text");
                } else {
                    theApp.updateOutput("Set Clipboard Text:" + toClipboard + "\n");
                }
            }
        }
    }

}

以前の会話によると、 を使用するのは危険でwhile (true) {}あり、不必要に CPU サイクルを消費するため、無駄でもあります...

public class Automator extends Thread implements NativeMouseInputListener {

    // A "locking" object...
    private static final Object WAIT_LOCK = new Object();

    Robot rob = null;
    TheAppClass theApp = null;
    ClipSetThread lastClipSet = null;
    boolean doit = false;
    boolean settingClip = false;

    public void run() {
        try // to make the Global hook
        {
            GlobalScreen.registerNativeHook();
        } catch (NativeHookException ex) {
            theApp.updateOutput("No Global Keyboard or Mouse Hook");
            return;
        }
        try // to create a robot (can simulate user input such as mouse and keyboard input)
        {
            rob = new Robot();
        } catch (AWTException e1) {
            theApp.updateOutput("The Robot could not be created");
            return;
        }

        // This is wasteful...
//        while (true) {
//        }
        // Locks do not consume CPU cycles while in the wait state...
        synchronized (WAIT_LOCK) {
            try {
                WAIT_LOCK.wait();
            } catch (Exception exp) {
            }
        }
    }

    public void dispose() {

        // Tell the thread it can terminate...
        synchronized (WAIT_LOCK) {
            WAIT_LOCK.notify();
        }
        // This will STOP the current thread (which called this method)
        // while the lastClipSet finishes...
        if (lastClipSet != null && lastClipSet.isAlive()) {
            lastClipSet.join();
        }

    }

    public void setClip(String arg) {
        ClipSetThread set = new ClipSetThread(theApp, lastClipSet, arg);
        lastClipSet = set;
        // You MUST START the thread...
        set.start();
    }

    /*...*/
}

更新しました

このコードは、無限ループを生成する可能性があります。クリップボードに値が含まれていない場合はどうなりStringますか??

while(true)
{
    try 
    {
        Transferable contents = sysClip.getContents(null);
        clipBoardTextBefore = (String)contents.getTransferData(DataFlavor.stringFlavor);
    }
    catch(Exception e)
    {
        try {Thread.sleep(20);} catch (InterruptedException e1) {}
        continue;
    }
    break;
}

あなたはこれを頻繁に行う傾向があります。何度か再試行した後に失敗できるように、ある種の「エスケープ」メカニズムを提供することをお勧めします...

boolean successful = false;
int retries = 0;
while (!successful && retries < 20) {
{
    try 
    {
        Transferable contents = sysClip.getContents(null);
        clipBoardTextBefore = (String)contents.getTransferData(DataFlavor.stringFlavor);
        successful = true;
    }
    catch(Exception e)
    {
        retries++;
        try {Thread.sleep(20);} catch (InterruptedException e1) {}
    }
}

実際の例で更新

わかりました、それは楽しかったです。（単純な）実際の例をまとめました。何らかのテキスト エディタを開きたいと思うでしょう。プログラムを実行すると、5 秒以内にアクティブになります ;)

私が行った唯一の基本的な変更は、イベント間に 250 ミリ秒の自動遅延を追加したことです ( rob.setAutoDelay(250).

を使用して、各キーイベント間に遅延を配置することもできますが、それはRobot#delayあなた次第です

public class Engine extends Thread {

    private Robot rob = null;
    private PasteThread lastClipSet = null;

    public void setClip(String arg) {
        if (lastClipSet != null && lastClipSet.isAlive()) {
            try {
                lastClipSet.join();
            } catch (InterruptedException ex) {
                ex.printStackTrace();
            }
        }
        PasteThread set = new PasteThread(arg);
        lastClipSet = set;
        lastClipSet.start();
    }

    public void pasteAtCursorLocation(String text) {
        System.out.println("Paste " + text);
        setClip(text);
        rob.keyPress(KeyEvent.VK_CONTROL);
        rob.keyPress(KeyEvent.VK_V);
        rob.keyRelease(KeyEvent.VK_V);
        rob.keyRelease(KeyEvent.VK_CONTROL);
    }

    public Engine() throws AWTException {
        rob = new Robot();
        rob.setAutoDelay(250);
        try {
            Thread.sleep(5000);
        } catch (InterruptedException ex) {
        }
        pasteAtCursorLocation("This is a simple test, thanks for watching!");
    }

    public static void main(String[] args) {
        try {
            new Engine();
        } catch (AWTException ex) {
            Logger.getLogger(Engine.class.getName()).log(Level.SEVERE, null, ex);
        }

    }

    public class PasteThread extends Thread {

        private String toPaste;

        public PasteThread(String value) {

            toPaste = value;

        }

        @Override
        public void run() {
            Clipboard sysClip = Toolkit.getDefaultToolkit().getSystemClipboard();
            System.out.println("Current clipboard contents = " + getClipboardContents(sysClip));
            sysClip.setContents(new StringSelection(toPaste), null);
            System.out.println("New clipboard contents = " + getClipboardContents(sysClip));
        }

        public String getClipboardContents(Clipboard clipboard) {
            String value = null;
            boolean successful = false;
            int retries = 0;
            while (!successful && retries < 20) {
                Transferable contents = clipboard.getContents(null);
                if (contents.isDataFlavorSupported(DataFlavor.stringFlavor)) {
                    try {
                        value = (String) contents.getTransferData(DataFlavor.stringFlavor);
                        successful = true;
                    } catch (Exception exp) {
                        retries++;
                        exp.printStackTrace();
                    }
                } else {
                    retries++;
                }
            }
            System.out.println(successful + "/" + retries);
            return value;
        }
    }
}
于 2013-01-29T00:00:14.087 に答える
1

間に 1 秒のスリープを入れて貼り付け操作を繰り返してみてください。

public void pasteAtCursorLocation(String text)
{
setClip(text);
rob.keyPress(KeyEvent.VK_CONTROL);
rob.keyPress(KeyEvent.VK_V);
rob.keyRelease(KeyEvent.VK_V);
rob.keyRelease(KeyEvent.VK_CONTROL);
theApp.updateOutput("Simulated Paste");

// put in a sleep 1 second here

rob.keyPress(KeyEvent.VK_CONTROL);
rob.keyPress(KeyEvent.VK_V);
rob.keyRelease(KeyEvent.VK_V);
rob.keyRelease(KeyEvent.VK_CONTROL);
theApp.updateOutput("Simulated Paste");
}

2 回貼り付けると、異なる結果が得られる可能性があります。この奇妙な動作の理由は、Windows がクリップボードを管理する方法にある可能性があります。クリップボードを 2 回貼り付けると異なる結果が得られる場合、この奇妙な動作の根本的な原因はコード内にあるのではなく、Java と Windows がどのように連携しているかにあることがわかります。

于 2013-01-28T23:54:24.433 に答える