java - restart a thread after exception is caught -

-

I am reading a UDP feed and I want to try to restart the thread if its failure. I think I have written my class correctly, but someone has to see and see if something is wrong or I have written something that will not work. In my catch-up section, I try to restart every 6 second thread for 10 attempts. Is this a good solution, will it work? 

  class UDPReader thread {Private thread t; Private finals string ip, socket, queue, threads; Private string error message; Private Final JTextArea Screen; UDPReader (string IP, string socket, string queue, string thread name, JTextArea screen) {this.ip = ip; this. Socket = socket; This.queue = queue; This.threadName = threadName; This.screen = Screen; } Public Zero Run () {try {byte [i] = null; IpaddrConnection ipaddr = new ipaddrConnection (IP, Socket); ParseUDP P = new parseUDP (); Screen.append ("Thread" + Thread name + "Running \ n"); While (true) {i = ipaddr.getPacket (); P.parseUDP (i); //Thread.sleep(0); }} Hold (IOException pre) {Logger.getLogger (MarketDataReader.class.getName ()). Log (Level.SEVERE, null, ex); Error message = "thread" + thread name + "has failed, attempt to restart"; Screen .append ("thread" + name of thread + "has failed, trying to restart \ n"); Email Email = New Email (Error Message, "Warning Market Data Reader Failure"); For (int i = 0; i & lt; 10 & amp; amp; t.isAlive () == wrong; i ++) {try {start (); Thread.Sleep (6000); } Grip (Interrupted Precision EX1) {Logger.getLogger (UDPReader.class.getName ()). Logs (level SESEE, blank, EX1); Error message = "thread" + thread name + "has failed, contact system system"; Screen.append ("Thread" + name of thread + "has failed, contact system system \ n"); Email = new email (error message, "warning market data reader failure"); }}}} Public Zero Start () {If (T == empty) {t = New thread (this, thread name); T. Start (); }}}}

I do not know the reasoning behind my thread, but I make some suggestions on the design of my code.

  1. It is not clear that inside the class you get from thread, you have another thread.

  2. There is no need to get from the thread (and I believe this is usually a bad practice). It is common to implement the runable interface and to create a new thread using it. 

      Class UDPReader implements Runnable {...}

    and after that one Instant Thread: 

      Thread T = New Thread (New UDPReader ());
    1. As a rule, if a thread fails, it ends ... It does not look like a failed thread "Re" better solution is a thread. UncaughtExceptionHandler will provide and analyze your thread's failing status and you can restart the thread if necessary.

    In general, in concurrent programming, you must explicitly separate the logic of a thread and the external argument, which is the management of this thread (its beginning / interruption / termination). ). I think, this is what you have not done in your code.


Comments

Post a Comment

Popular posts from this blog

mysql - How to enter php data into a html multiple select box -

-
I am working on a page that will create a team. I just want to create a selection box which will list all the users of all the users. After that, the user can select more than one friend (i.e. 5) and can send the request to these 5 people to join the team but I am having problems I did this: Select $ sql = "Select i. *, Join * Connect with M * m.id = i.clique_id where adder_id =: id"; $ Stmt = $ db- & gt; Query ($ sql, array ('id' = & gt; $ _SESSION ['id'])); While ($ record = $ stmt-> fetch ()) {$ surname = $ record ['surname']; } and this: & lt; Select multiple = "true" name = "category" well id = "category" name = "category" category = "" & gt; & Lt; Option value = "& lt ;? php echo $ surname ;? & gt;" & Gt; & Lt ;? Php ek $ nickname; ? & Gt; & Lt; / Options & gt; & Lt; / Select & gt; But only a friend's nickname...
Read more

java - Can't add JTree to JPanel of a JInternalFrame -

-
Image
There are two panels in my JInternalframe. I want to add TopPanel to Jtree to nominate another. But I can not add Jaitari to the top of the panel. Please help me in this piece of code: DefaultMutableTreeNode root = new DefaultMutableTreeNode ("deck"); DefaultMutableTreeNode Item Clubs = New DefaultMutableTreeNode ("Club"); AddAllCard (itemClubs); Root.add (itemClubs); DefaultMutableTreeNode item = New DefaultMutableTreeNode ("Diamonds"); AddAllCard (itemDiamonds); Root.add (itemDiamonds); DefaultMutableTreeNode Itemspads = New DefaultMutableTreeNode ("Spades"); AddAllCard (itemSpades); Root.add (itemSpades); DefaultMutableTreeNode item increases = new DefaultMutableTreeNode ("heart"); AddAllCard (itemHearts); Root.add (itemHearts); Default Trimodel Tree Model = New DefaultTreeModel (Route); Tree = new jetty (tree modal); ScrollPane = new JScrollPane (tree); // scrollPane.setViewportView (tree); . ScrollPane.getViewport () add (tr...
Read more

c++ - Cassandra datastax cpp driver - avoiding unnecessary copies -

-
I use the following method to create my insert statement for Cassandra (using datastax cpp driver) : ssStmt1 & lt; & Lt; "Insert some table (...) value (....)" string sStatement1 = ssStmt1.str (); Castestring cass1 = cass_string_init (sStatement1.c_str ()); CassStatement * pStmt1 = cass_statement_new (cass1, 0); I have also tried the following: CassString cass1 = cass_string_init ("Enter some table (id, date, c) value (? ,?,?); "); CassStatement * pStmt1 = cass_statement_new (cass1, 3); Cass_statement_bind_uuid (pStmt1, 0, uuidKey); Cass_statement_bind_int64 (PSTMT1, 1, timestamp); Cass_statement_bind_string (pStmt1, 2, sSomething); The first pattern works with mixed keys, the second is not. The key has a timestamp and a UUID. Timestamp is one of two errors in the field (based on the used bond statement). If bound in this way: cass_statement_bind_int64_by_name (pStmt1, "date", timestamp); I get the following error: Inva...
Read more