Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > Java: Общие вопросы > Получание значения переменой


Автор: Asafj 11.1.2011, 22:59
Есть метод с помощью которого можно получить значение переменой-true false. Нужно написать функцию, которая вызывает 
другой метод после того как значение переменой стало true. Как это можно сделать?

Автор: priam220 11.1.2011, 23:53
Код

public class MyClass{    
    private Boolean checker;

    public static void main(String[] args){
     MyClass mc = new MyClass();
     mc.setChecker(false);
     mc.setChecker(true);
    }
    
    public void setChecker(boolean checker){
     this.checker = checker;
     if (checker)
         modifier();        
    }
    
    private void modifier(){
     System.out.println("if checker is " + checker + " this method is called");
    }
}


Вывод:
if checker is true this method is called


P.S. В java нет функций.

Автор: Asafj 12.1.2011, 00:08
Это не совсем, что нужно
Я получаю из другого класса только значение переменной
Это значение сначало false потом становится true
И после того как оно поменялось надо взвать другой метод
т.е
Код

boolean upd;
void b(){
if (upd) {
e();
}
} 

но если b = false метод не вызовится
я пробовал что-то вроде этого но в многопоточной программе я думаю это не вариант:
Код

boolean upd;
void b(){
if (upd) {
e();
}
else {Thread t = new thread();
t.start();
t.sleep(1000);
b();}
} 
 

Автор: priam220 12.1.2011, 00:19
Цитата(Asafj @  11.1.2011,  22:59 Найти цитируемый пост)
которая вызывает другой метод после того как значение переменой стало true


Цитата(Asafj @  12.1.2011,  00:08 Найти цитируемый пост)
но если b = false метод не вызовится




не могу связать эти предложения, - так тебе и не надо вызывать modifier() когда false?...

Автор: lazycat 12.1.2011, 00:22
Думаю, что Вам надо нечто в таком роде:

Код

boolean upd;
boolean lastupd  = null;

  void b(){
    if ((lastupd  == null) || (lastupd  != upd)) {
      e();
      lastupd = upd;
    } 
  
  } 

Автор: Asafj 12.1.2011, 00:23
значение меняется в другой части приложения

Добавлено через 3 минуты и 49 секунд
lazycat,  в этом случае если upd=false метод не вызовется 

Автор: priam220 12.1.2011, 01:04
Цитата(Asafj @  12.1.2011,  00:23 Найти цитируемый пост)
значение меняется в другой части приложения

так ты будешь долго писать свой код, особенно в многопоточной программе. Свойство класса может меняться только из одного места, - из самого класса, с помощью метода-сеттера,  и в твоем случае, он должен быть синхронизирован (либо свойство должно быть объявлено как volatile).
Надеюсь, что, наконец, понял условие. 

Автор: Asafj 12.1.2011, 01:15
А если доступен тоько get - метод из этого класса?

Автор: priam220 12.1.2011, 01:25
Цитата(Asafj @  12.1.2011,  01:15 Найти цитируемый пост)
А если доступен тоько get - метод из этого класса?

если доступен только get, значит:

1. комуто было лень писать  set
2. значение свойства не должно меняться.

Если этот класс скомпилин  (просто добавить нужный метод не можешь) и он не финальный, то ты его можешь расширить, дописав set в потомке. 
плохое решение. 

Автор: Temdegon 12.1.2011, 01:26
Объясни, ты это делаешь в своем коде? Или у тебя есть готовый класс, который ты изменить не можешь, и можешь только вызывать геттер?

Автор: Asafj 12.1.2011, 01:29
Temdegon, да

Автор: priam220 12.1.2011, 01:49
Надеюсь, я один не понял, что означало слово "да".  smile 

Автор: Temdegon 12.1.2011, 01:52
Краткость конечно с.т., но я лично не понял, что значит "да" в этом контексте. 

Автор: Asafj 12.1.2011, 02:03
Ну собственно ответ на твой вопрос. Второй.

Автор: lazycat 12.1.2011, 08:33
Цитата(Asafj @  12.1.2011,  00:23 Найти цитируемый пост)
lazycat,  в этом случае если upd=false метод не вызовется 


Ошибаетесь, метод будет вызываться при каждом ИЗМЕНЕНИИ значении переменной, а при первом обращении к ней вызовется при любом значении. 
Правда с кодом я протупил (писал его практически засыпая) и написал чушь. Реально он должен выглядеть так:

Код


Boolean lastupd  = null;

  void b(){
    boolean upd = someObject.getUpd();

    if (lastupd  != upd) {
      e();
      lastupd = upd;
    } 
  
  }

Автор: techmax 12.1.2011, 08:58
посмотри может есть какой-нибудь листнер на изменение переменной. 
или Напиши какой-нибудь класс для изменения этого свойства
Код

  class A{

static setSomeValue(SomeClass sc,boolean flag){
sc.setFlag(flag);
if(flag){
//сделать чтонибудь

}

Автор: Asafj 12.1.2011, 11:13
Я не понимаю - пусть Upd примет значение. Метод е() выполнится или нет от ситуации. 
Каким образом выполнение сюда вернется после изменения значения. Он должен ждать изменения с false на true.

Автор: Asafj 12.1.2011, 13:14
листнер на изменение переменной я думаю подошел бы
но где ж его взять?

Автор: Kircul 12.1.2011, 13:37
Цитата(Asafj @  12.1.2011,  12:14 Найти цитируемый пост)
листнер на изменение переменной я думаю подошел бы
но где ж его взять? 

Ну что значит где? Возможно некоторые ЯП поддерживают данные возможности на уровне языка, но не Java (философия не та). Если вам нужен слушатель то напишите этот код сами.
Код

class TestClass {
    private Object value;
    private Listener listener;

...

    public void setValue(Object value) {
         this.value = value;
         if (listener != null) {
             listener.onValueChanged(value);   
         }
    }

    public void setListener(Listener listener) [
        this.listener = listener;
    }
}

interface Listener {
    void onValueChanged(Object newValue);
}

Собственно ничего сверхъестественного. Читаем http://goo.gl/lkbpM.

Автор: jk1 12.1.2011, 14:10
Цитата

Если вам нужен слушатель то напишите этот код сами.


Или используйте http://download.oracle.com/javase/tutorial/javabeans/properties/bound.html

Автор: priam220 12.1.2011, 14:18
lazycat,  у тебя NPE выскочит.


Код

public void setValue(Object value) {


исходя из условия нет этого метода в классе. 
Мне кажется ни листенеры ни наблюдатели тут не нужны. Человеку надо почитать что такое инкапсуляция и переписать свой код нормально.

Автор: math64 12.1.2011, 16:47
Если это свой код или чужой с исходниками - переписать, чтобы были сеттер и листенер.
Если исходников нет - только опрос в Thread'е, что будет с запаздыванием.

Автор: Asafj 12.1.2011, 19:26
опрос в треде  - эт что значит?

Автор: priam220 12.1.2011, 20:09
Цитата(Asafj @  12.1.2011,  19:26 Найти цитируемый пост)
опрос в треде  - эт что значит?

это то, что ты пытался с потоком навоять.  

Автор: math64 13.1.2011, 08:53
Код

class A {
int x;
public int getX() { return x; }
protected void setX(int x) { this.x = x; } 
}

a) Сеттера нет, но есть исходник - рефакторинг "добавить сеттер" с установленным флажком "заменить обращения к переменной на вызов сеттера"
b) Сеттер есть, но он не public, а protected, его можно переопределить в производном классе:
Код

class MyListener {
  public void e(A a) {
    ...
  }
}
class MyA {
  MyListener l;
  public MyA(MyListener l) {
    super();
    this.l = l;
  }
  protected void setX(int x) {
    if (x != getX()) {
      super.setX(x);
      l.e(this);
    } 
  }
}
MyListener l = new MyListener();
A a = new MyA(l);

c) Сеттера нет или он private -  опрос в Thread'e:
Код

boolean terminate;
void f() {
  final A a = new A();
  Runnable r = new Runnable() {
    public void run() {
      int oldX = a.getX();
      try {
        while(!terminate) {
          int newX = a.getX();
          if (newX != oldX) {
            oldX = newX;
            e(a);
          } else {
            Thread.sleep(1000);
          }
        } 
      } catch(InterruptedException e) {
      }
    }
  };
  terminate = false;
  Thread t = new Thread(r);
  t.start();
  ...
  terminate = true;
}
void e(A a) {
  ...
}

Автор: Asafj 13.1.2011, 11:20
спасибо 
А с вот это объясни: 

Код

terminate = false;
  Thread t = new Thread(r);
  t.start();
  ..
  terminate = true;

Автор: Asafj 13.1.2011, 12:58
А здесь что хуже использовать 
Код

 Runnable r = new Runnable() {
    public void run() {
      int oldX = a.getX();
      try {
        while(!terminate) {
          int newX = a.getX();
          if (newX != oldX) {
            oldX = newX;
            e(a);
          } else {
            Thread.sleep(1000);
          }
        } 
      } catch(InterruptedException e) {
      }
    }
  };

или wihle(!a.getx());

Автор: math64 13.1.2011, 14:28
Переменная terminate управляет завершением Thread. Как только terminated устанавливается в true, поток на очередном обороте завершается.
Если нужно ждать только один раз - можно while(!a.getX() && !terminate) { ... }.
terminate лучше оставить, чтобы можно было завершить Thread и тогда, когда getX() никогда не станет true.
PS: для булевых свойств геттер обычно называется isX().

Автор: Asafj 13.1.2011, 20:52
А все таки когда terminate должен вызываться?

Автор: math64 14.1.2011, 09:24
Установи terminate = true; когда нить больше не нужна. Если она должна работать всегда, это делается перед при завершением программы.

Автор: Kircul 14.1.2011, 12:21
math64, 
Я конечно не гуру, но мне кажется что прерывать поток/нить правильнее будет как в данном http://goo.gl/EoITK. Не стоит изобретать велосипед.

Автор: jk1 14.1.2011, 13:37
Цитата

Не стоит изобретать велосипед.


Ага. И раз уже заговорили о велосипедах, то прерывать такой поток перед выходом из приложения не обязательно.
Можно просто запустить его как daemon:
Код

thread.setDaemon(true);

Автор: Asafj 14.1.2011, 21:30
А если здесь isX() =false несколько секунд насколько плохо использовать вместо потоков 
while(!isX());

Автор: Asafj 15.1.2011, 12:13
А если здесь isX() =false несколько секунд насколько плохо использовать вместо потоков 
while(!isX());

Автор: Asafj 16.1.2011, 15:11
 А здесь вот конкретно для моего случая 
  Нужно сделать break только тогда, когда isX() вернет true
  А сейчас после thread.start() - выход.
switch (key) {
case value:
    final Runnable r = new Runnable() {
        public void run() {
         while(!isX()) {
            try {
                   Thread.sleep(10000);
               } catch (InterruptedException ex) {
            }
         }
         }
    };
    Thread thread = new Thread®;
    thread.start();

    break;

Автор: math64 17.1.2011, 16:03
Thread дожен что-то еще делать кроме опроса isX() - (например, после выхода из цикла) иначе он бесполезен. Ну и вызови setDaemon(), как посоветовали, чтобы программы нормально завершалась, еслы isX() всегда возвращает true.

Автор: Asafj 17.1.2011, 23:10
здесь thread должен ждать пока переменная не станет true и только затем должен быть break.

Автор: Asafj 18.1.2011, 11:23
Код


try {
switch (key) {
case value:
    final Runnable r = new Runnable() {
        public void run() {
         while(!isX()) {
            try {
                   Thread.sleep(10000);
               } catch (InterruptedException ex) {
            }
         }
         }
    };
    Thread thread = new Thread®;
    thread.start();
      try {
     thread .join();
          } 
           catch(InterruptedException e) {
          }
    break; 


Вот здесь после thread.join() поток разве жив?

Автор: Kircul 18.1.2011, 12:00
[offtopic]Asafj, прочитайте вы наконец хоть какую-нибудь книгу по Java![/offtopic]

Автор: Asafj 18.1.2011, 12:42
А все таки - кто-нибудь в состоянии обьяснить- почему поток не завершается после выхода из цикла?

Автор: math64 18.1.2011, 14:15
Если основной поток ждёт завершения второго потока, ждущего isX() == false, зачем тебе второй поток? Опрашивай isX() в основном. Второй поток нужен только для того, чтобы освободить основной поток для какого-либо полезного дела.

Автор: Asafj 18.1.2011, 14:52
А как в основном опрашивать?:
while(!isX()); я думаю здесь не вариант
А если while(!isX()) {Thread.sleep();} зависает

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)