elhacker.net cabecera Bienvenido(a), Visitante. Por favor Ingresar o Registrarse
¿Perdiste tu email de activación?.

 

 


Tema destacado: Arreglado, de nuevo, el registro del warzone (wargame) de EHN


+  Foro de elhacker.net
|-+  Programación
| |-+  Programación General
| | |-+  Java
| | | |-+  Acceder y modificar una variable global desde distintos threads
0 Usuarios y 1 Visitante están viendo este tema.
Páginas: [1] Ir Abajo Respuesta Imprimir
Autor Tema: Acceder y modificar una variable global desde distintos threads  (Leído 5,249 veces)
fileteruso

Desconectado Desconectado

Mensajes: 18



Ver Perfil
Acceder y modificar una variable global desde distintos threads
« en: 6 Febrero 2019, 20:20 pm »

Buenas,

estoy intentando que una variable global de tipo int sea modificada por dos tipos de procesos: uno que le suma 1 y otro que le resta 1. Para ello he creado dos clases que implementan Runnable: una llamada Incrementador y otra llamada Decrementador. El problema que tengo es que no se cómo hacer para que desde la clase principal que crea e inicia los procesos se comparta la variable para ir sumándole y restándole uno simultáneamente un número de veces concretas. He intentado pasarla cuando instancio ambas clases en la principal antes de iniciar los procesos pero en el momento que paso por parámetro la variable que se modifica es la propia de las clases Incrementador y Decrementador.

(En el código de la clase principal de a continuación solo ejecuto el proceso del incrementador para comprobar si le suma 1)

Código clase principal:
Código:
public class Principal {
public static volatile int x = 0;

public static void main(String[] args) {
Incrementador inc = new Incrementador(x);
Thread hilo = new Thread(inc);
hilo.start();
System.out.println(x);
}
}

Código del incrementador:
Código:
public class Incrementador implements Runnable {
private int x;

public Incrementador(int x) {
this.x = x;
}

public void run() {
x=x+1;
}
}

Código del Decrementador:
Código:
public class Decrementador implements Runnable {
private int x;

public Decrementador(int x) {
this.x = x;
}

public void run() {
x=x-1;
}
}


Muchas gracias de antemano!


« Última modificación: 6 Febrero 2019, 20:25 pm por fileteruso » En línea

ivancea96


Desconectado Desconectado

Mensajes: 3.412


ASMático


Ver Perfil WWW
Re: Acceder y modificar una variable global desde distintos threads
« Respuesta #1 en: 6 Febrero 2019, 20:54 pm »

La razón de que actualmente no te funcione es que los tipos primitivos como int se copian cuando los pasas como parámetro. Al copiarse, es otro objeto diferente, y aunque lo modifiques, el original se queda igual. Para pasar objetos, tendrías que utilizar clases en vez de tipos primitivos.

Tendrías que meter la variable dentro de una clase y compartir un objeto de esa clase con las 2 que lo van a modificar. Otra opción es tenerlo como variable static y que accedan directamente a ella.

Dado que el acceso concurrente te acabaría dando problemas, puedes usar la clase AtomicInteger que sirve precisamente para eso: Modificar un int desde varios hilos. Podrías crearla en el main, y pasárselo al constructor de los otros 2 como haces ahora.


En línea

fileteruso

Desconectado Desconectado

Mensajes: 18



Ver Perfil
Re: Acceder y modificar una variable global desde distintos threads
« Respuesta #2 en: 6 Febrero 2019, 20:57 pm »

Tomo nota, muchísimas gracias.
En línea

rub'n


Conectado Conectado

Mensajes: 1.218


(e -> λ("live now")); tatuar -> λ("α");


Ver Perfil WWW
Re: Acceder y modificar una variable global desde distintos threads
« Respuesta #3 en: 6 Febrero 2019, 22:57 pm »

fileteruso que tal usa geshi, para que en un entorno de concurrencia un recurso compartido se actualize de manera correcta necesitas un lock con synchronized usandolo tanto en el método(no recomendado) y/O forma de bloque.

@ivancea96 con clases Atómicas no basta para que un recurso compartido sea actualizado correctamente por múltiples hilos a la misma ves, tampoco basta con volatile se necesita un lock, para asegurar la correcta sincronización.

Lo podemos ver con un ejemplo simple con 20 Hilos, linea 18, y usando un ExecutorService que es mejor que usar directamente la clase Thread o Runnable.



Código
  1. package foro
  2.  
  3. import java.util.Objects;
  4. import java.util.concurrent.*;
  5. import java.util.concurrent.atomic.AtomicInteger;
  6. import java.util.logging.Logger;
  7. import java.util.stream.IntStream;
  8.  
  9. public class UsoSynchronized {
  10.  
  11.    private static final Logger LOGGER = Logger.getLogger(Principal.class.getSimpleName());
  12.    private ExecutorService executors = null;
  13.    private volatile int count = 0;
  14.    private AtomicInteger atomicInteger = new AtomicInteger(0);
  15.  
  16.    public UsoSynchronized() {
  17.        try {
  18.            executors = Executors.newFixedThreadPool(20);
  19.            IntStream.rangeClosed(1,10)
  20.                    .forEach(e -> {
  21.                        executors.submit(()-> withAtomicInteger());
  22.                    });
  23.        }finally {
  24.            if(Objects.nonNull(executors)) {
  25.                executors.shutdown(); // ayuda a evitar memory leaks, fugas de memoria
  26.            }
  27.        }
  28.    }
  29.  
  30.    public void hola() {
  31.        synchronized (this) {
  32.           LOGGER.info("Hola " + (++count));
  33.        }
  34.    }
  35.  
  36.    public void withAtomicInteger() {
  37.        //synchronized (this) {
  38.            LOGGER.info(""+ atomicInteger.incrementAndGet());
  39.        //}
  40.    }
  41.  
  42.    public static void main(String... haga) {
  43.        new UsoSynchronized();
  44.    }
  45. }
  46.  
  47. }


Código
  1. Feb 06, 2019 10:48:29 PM foro withAtomicInteger
  2. INFORMACIÓN: 5
  3. Feb 06, 2019 10:48:29 PM foro withAtomicInteger
  4. INFORMACIÓN: 10
  5. Feb 06, 2019 10:48:29 PM foro withAtomicInteger
  6. INFORMACIÓN: 9
  7. Feb 06, 2019 10:48:29 PM foro withAtomicInteger
  8. INFORMACIÓN: 8
  9. Feb 06, 2019 10:48:29 PM foro withAtomicInteger
  10. INFORMACIÓN: 7
  11. Feb 06, 2019 10:48:29 PM foro withAtomicInteger
  12. INFORMACIÓN: 6
  13. Feb 06, 2019 10:48:29 PM foro withAtomicInteger
  14. INFORMACIÓN: 2
  15. Feb 06, 2019 10:48:29 PM foro withAtomicInteger
  16. INFORMACIÓN: 3
  17. Feb 06, 2019 10:48:29 PM foro withAtomicInteger
  18. INFORMACIÓN: 4
  19. Feb 06, 2019 10:48:29 PM foro withAtomicInteger
  20. INFORMACIÓN: 1
  21.  

Ese resultado de arriba es única y llanamente debido por que no se tiene un lock lo comente apropósito al invocar al método withAtomicInteger() en la linea 21, y cada Hilo entra a modificar el recurso count, e incluso lo ideal siempre es un bloque lock, es decir no usar synchronized en el método debido a que hay mas costo en el performance de ejecución, es mejor localizar justo el recurso que se modificara, tarea no tan fácil de hacer la mayoría de las veces



Otro ejemplo con 3 hilos, lo hize por la brevedad, nasty, usando atomic Integer y con lock comentado

Código
  1. public class UsoSynchronized2 {
  2.  
  3.    private int count= 0;
  4.    private AtomicInteger atomicInteger = new AtomicInteger(0);
  5.  
  6.    public UsoSynchronized2() {
  7.  
  8.    }
  9.  
  10.    private void init(final String msg) {
  11.       // synchronized (this) {
  12.            System.out.println(msg + atomicInteger.incrementAndGet());
  13.       // }
  14.    }
  15.  
  16.  
  17.  
  18.    public static void main(String ...blabl) {
  19.        UsoSynchronized2 usoSynchronized2 = new UsoSynchronized2();
  20.  
  21.        IntStream.rangeClosed(1,100)
  22.                .forEach( e -> {
  23.                    new Thread(() -> usoSynchronized2.init("Hilo 1 ")).start();
  24.                    new Thread(() -> usoSynchronized2.init("Hilo 2 ")).start();
  25.                    new Thread(() -> usoSynchronized2.init("Hilo 3 ")).start();
  26.                });
  27.  
  28.  
  29.    }
  30.  

Código
  1. Hilo 1 1
  2. Hilo 2 2
  3. Hilo 3 3
  4. Hilo 3 4
  5. Hilo 1 5
  6. Hilo 2 6
  7. Hilo 1 7
  8. Hilo 2 8
  9. Hilo 3 9
  10. Hilo 3 10
  11. Hilo 2 11
  12. Hilo 3 12
  13. Hilo 2 13
  14. Hilo 1 14
  15. Hilo 2 15
  16. Hilo 1 16
  17. Hilo 1 17
  18. Hilo 2 18
  19. Hilo 2 19
  20. Hilo 1 20
  21. Hilo 1 21
  22. Hilo 2 22
  23. Hilo 1 23
  24. Hilo 2 24
  25. Hilo 3 25
  26. Hilo 1 26
  27. Hilo 1 27
  28. Hilo 3 28
  29. Hilo 1 29
  30. Hilo 1 30
  31. Hilo 3 31
  32. Hilo 1 32
  33. Hilo 2 33
  34. Hilo 2 34
  35. Hilo 1 35
  36. Hilo 3 36
  37. Hilo 2 43
  38. Hilo 3 42
  39. Hilo 3 41
  40. Hilo 2 40
  41. Hilo 2 39
  42. Hilo 1 38
  43. Hilo 1 37
  44. Hilo 2 44
  45. Hilo 3 45
  46. Hilo 1 46
  47. Hilo 1 47
  48. Hilo 1 48
  49. Hilo 2 49
  50. Hilo 3 50
  51. Hilo 2 51
  52. Hilo 2 52
  53. Hilo 3 53
  54. Hilo 1 54
  55. Hilo 2 55
  56. Hilo 2 56
  57. Hilo 1 57
  58. Hilo 1 60
  59. Hilo 3 61
  60. Hilo 2 59
  61. Hilo 2 62
  62. Hilo 3 58
  63. Hilo 1 65
  64. Hilo 2 66
  65. Hilo 3 64
  66. Hilo 1 63
  67. Hilo 3 67
  68. Hilo 1 68
  69. Hilo 2 69
  70. Hilo 3 70
  71. Hilo 1 71
  72. Hilo 1 72
  73. Hilo 3 73
  74. Hilo 3 75
  75. Hilo 1 76
  76. Hilo 3 77
  77. Hilo 2 78
  78. Hilo 2 74
  79. Hilo 1 80
  80. Hilo 1 81
  81. Hilo 2 82
  82. Hilo 3 79
  83. Hilo 2 83
  84. Hilo 1 84
  85. Hilo 3 85
  86. Hilo 1 88
  87. Hilo 3 87
  88. Hilo 2 86
  89. Hilo 3 98
  90. Hilo 1 104
  91. Hilo 3 97
  92. Hilo 2 96
  93. Hilo 1 95
  94. Hilo 2 109
  95. Hilo 2 94
  96. Hilo 3 93
  97. Hilo 3 92
  98. Hilo 1 91
  99. Hilo 1 113
  100. Hilo 1 114
  101. Hilo 1 90
  102. Hilo 1 89
  103. Hilo 1 123
  104. Hilo 1 132
  105. Hilo 2 133
  106. Hilo 3 121
  107. Hilo 2 120
  108. Hilo 1 119
  109. Hilo 2 118
  110. Hilo 2 117
  111. Hilo 3 115
  112. Hilo 2 116
  113. Hilo 3 112
  114. Hilo 2 111
  115. Hilo 3 110
  116. Hilo 3 108
  117. Hilo 1 107
  118. Hilo 3 106
  119. Hilo 3 105
  120. Hilo 1 163
  121. Hilo 3 103
  122. Hilo 1 101
  123. Hilo 1 102
  124. Hilo 3 100
  125. Hilo 2 99
  126. Hilo 2 164
  127. Hilo 3 162
  128. Hilo 3 165
  129. Hilo 2 161
  130. Hilo 2 167
  131. Hilo 3 160
  132. Hilo 1 159
  133. Hilo 3 158
  134. Hilo 3 169
  135. Hilo 2 157
  136. Hilo 3 156
  137. Hilo 2 155
  138. Hilo 3 173
  139. Hilo 3 154
  140. Hilo 1 153
  141. Hilo 1 152
  142. Hilo 2 151
  143. Hilo 3 150
  144. Hilo 3 149
  145. Hilo 1 148
  146. Hilo 2 147
  147. Hilo 1 146
  148. Hilo 3 145
  149. Hilo 2 144
  150. Hilo 3 177
  151. Hilo 2 179
  152. Hilo 3 143
  153. Hilo 3 142
  154. Hilo 2 141
  155. Hilo 2 140
  156. Hilo 3 139
  157. Hilo 1 138
  158. Hilo 2 137
  159. Hilo 1 182
  160. Hilo 3 136
  161. Hilo 3 183
  162. Hilo 2 135
  163. Hilo 2 134
  164. Hilo 2 131
  165. Hilo 1 130
  166. Hilo 3 129
  167. Hilo 3 128
  168. Hilo 3 127
  169. Hilo 1 188
  170. Hilo 2 126
  171. Hilo 2 125
  172. Hilo 1 124
  173. Hilo 3 122
  174. Hilo 3 190
  175. Hilo 3 189
  176. Hilo 2 187
  177. Hilo 2 186
  178. Hilo 3 185
  179. Hilo 1 184
  180. Hilo 2 181
  181. Hilo 3 180
  182. Hilo 2 178
  183. Hilo 1 176
  184. Hilo 1 175
  185. Hilo 2 174
  186. Hilo 2 172
  187. Hilo 1 171
  188. Hilo 1 170
  189. Hilo 1 168
  190. Hilo 3 166
  191. Hilo 2 191
  192. Hilo 1 192
  193. Hilo 2 193
  194. Hilo 3 194
  195. Hilo 1 195
  196. Hilo 3 196
  197. Hilo 1 197
  198. Hilo 2 198
  199. Hilo 1 199
  200. Hilo 2 200
  201. Hilo 1 201
  202. Hilo 2 202
  203. Hilo 2 203
  204. Hilo 2 204
  205. Hilo 1 205
  206. Hilo 3 206
  207. Hilo 2 207
  208. Hilo 2 208
  209. Hilo 3 209
  210. Hilo 3 210
  211. Hilo 1 211
  212. Hilo 1 212
  213. Hilo 1 213
  214. Hilo 2 214
  215. Hilo 1 215
  216. Hilo 3 216
  217. Hilo 2 217
  218. Hilo 1 218
  219. Hilo 3 219
  220. Hilo 2 220
  221. Hilo 2 221
  222. Hilo 3 222
  223. Hilo 3 223
  224. Hilo 2 224
  225. Hilo 1 225
  226. Hilo 3 226
  227. Hilo 1 227
  228. Hilo 3 228
  229. Hilo 2 229
  230. Hilo 2 230
  231. Hilo 1 231
  232. Hilo 2 232
  233. Hilo 3 233
  234. Hilo 1 234
  235. Hilo 3 235
  236. Hilo 1 236
  237. Hilo 3 237
  238. Hilo 1 238
  239. Hilo 2 239
  240. Hilo 3 240
  241. Hilo 1 241
  242. Hilo 3 242
  243. Hilo 1 243
  244. Hilo 1 244
  245. Hilo 3 245
  246. Hilo 2 246
  247. Hilo 1 247
  248. Hilo 2 248
  249. Hilo 3 249
  250. Hilo 1 250
  251. Hilo 2 251
  252. Hilo 3 252
  253. Hilo 1 253
  254. Hilo 2 254
  255. Hilo 3 255
  256. Hilo 1 256
  257. Hilo 2 257
  258. Hilo 3 258
  259. Hilo 2 259
  260. Hilo 3 260
  261. Hilo 1 261
  262. Hilo 1 262
  263. Hilo 2 263
  264. Hilo 3 264
  265. Hilo 2 265
  266. Hilo 3 266
  267. Hilo 1 267
  268. Hilo 2 268
  269. Hilo 3 269
  270. Hilo 1 270
  271. Hilo 3 271
  272. Hilo 1 272
  273. Hilo 2 273
  274. Hilo 3 274
  275. Hilo 2 275
  276. Hilo 3 276
  277. Hilo 1 277
  278. Hilo 3 278
  279. Hilo 1 279
  280. Hilo 2 280
  281. Hilo 1 281
  282. Hilo 3 282
  283. Hilo 2 283
  284. Hilo 3 284
  285. Hilo 1 285
  286. Hilo 2 286
  287. Hilo 3 287
  288. Hilo 2 288
  289. Hilo 2 289
  290. Hilo 1 290
  291. Hilo 1 291
  292. Hilo 1 292
  293. Hilo 3 294
  294. Hilo 3 293
  295. Hilo 3 296
  296. Hilo 2 295
  297. Hilo 2 297
  298. Hilo 3 298
  299. Hilo 2 299
  300. Hilo 1 300

caso distinto tendriamos si usamos el lock

Código
  1. private void init(final String msg) {
  2.        synchronized (this) {
  3.            System.out.println(msg + atomicInteger.incrementAndGet());
  4.        }
  5.    }
Código
  1. Hilo 1 1
  2. Hilo 3 2
  3. Hilo 2 3
  4. Hilo 2 4
  5. Hilo 1 5
  6. Hilo 1 6
  7. Hilo 2 7
  8. Hilo 1 8
  9. Hilo 3 9
  10. Hilo 2 10
  11. Hilo 1 11
  12. Hilo 2 12
  13. Hilo 1 13
  14. Hilo 3 14
  15. Hilo 2 15
  16. Hilo 3 16
  17. Hilo 2 17
  18. Hilo 1 18
  19. Hilo 2 19
  20. Hilo 3 20
  21. Hilo 1 21
  22. Hilo 3 22
  23. Hilo 2 23
  24. Hilo 3 24
  25. Hilo 1 25
  26. Hilo 3 26
  27. Hilo 2 27
  28. Hilo 1 28
  29. Hilo 3 29
  30. Hilo 3 30
  31. Hilo 2 31
  32. Hilo 1 32
  33. Hilo 3 33
  34. Hilo 1 34
  35. Hilo 2 35
  36. Hilo 3 36
  37. Hilo 2 37
  38. Hilo 1 38
  39. Hilo 3 39
  40. Hilo 2 40
  41. Hilo 2 41
  42. Hilo 1 42
  43. Hilo 1 43
  44. Hilo 2 44
  45. Hilo 3 45
  46. Hilo 3 46
  47. Hilo 2 47
  48. Hilo 1 48
  49. Hilo 2 49
  50. Hilo 1 50
  51. Hilo 2 51
  52. Hilo 3 52
  53. Hilo 1 53
  54. Hilo 3 54
  55. Hilo 1 55
  56. Hilo 3 56
  57. Hilo 2 57
  58. Hilo 1 58
  59. Hilo 2 59
  60. Hilo 3 60
  61. Hilo 3 61
  62. Hilo 2 62
  63. Hilo 1 63
  64. Hilo 2 64
  65. Hilo 1 65
  66. Hilo 3 66
  67. Hilo 3 67
  68. Hilo 1 68
  69. Hilo 2 69
  70. Hilo 3 70
  71. Hilo 1 71
  72. Hilo 1 72
  73. Hilo 2 73
  74. Hilo 3 74
  75. Hilo 3 75
  76. Hilo 2 76
  77. Hilo 1 77
  78. Hilo 3 78
  79. Hilo 2 79
  80. Hilo 1 80
  81. Hilo 3 81
  82. Hilo 2 82
  83. Hilo 1 83
  84. Hilo 3 84
  85. Hilo 2 85
  86. Hilo 1 86
  87. Hilo 3 87
  88. Hilo 2 88
  89. Hilo 1 89
  90. Hilo 3 90
  91. Hilo 3 91
  92. Hilo 2 92
  93. Hilo 1 93
  94. Hilo 1 94
  95. Hilo 3 95
  96. Hilo 2 96
  97. Hilo 3 97
  98. Hilo 1 98
  99. Hilo 2 99
  100. Hilo 3 100
  101. Hilo 1 101
  102. Hilo 2 102
  103. Hilo 2 103
  104. Hilo 3 104
  105. Hilo 1 105
  106. Hilo 3 106
  107. Hilo 1 107
  108. Hilo 2 108
  109. Hilo 1 109
  110. Hilo 1 110
  111. Hilo 3 111
  112. Hilo 1 112
  113. Hilo 1 113
  114. Hilo 2 114
  115. Hilo 3 115
  116. Hilo 3 116
  117. Hilo 3 117
  118. Hilo 1 118
  119. Hilo 1 119
  120. Hilo 2 120
  121. Hilo 3 121
  122. Hilo 1 122
  123. Hilo 2 123
  124. Hilo 3 124
  125. Hilo 1 125
  126. Hilo 2 126
  127. Hilo 3 127
  128. Hilo 2 128
  129. Hilo 2 129
  130. Hilo 2 130
  131. Hilo 3 131
  132. Hilo 1 132
  133. Hilo 2 133
  134. Hilo 3 134
  135. Hilo 1 135
  136. Hilo 2 136
  137. Hilo 3 137
  138. Hilo 1 138
  139. Hilo 2 139
  140. Hilo 3 140
  141. Hilo 1 141
  142. Hilo 2 142
  143. Hilo 1 143
  144. Hilo 2 144
  145. Hilo 3 145
  146. Hilo 2 146
  147. Hilo 3 147
  148. Hilo 2 148
  149. Hilo 1 149
  150. Hilo 2 150
  151. Hilo 3 151
  152. Hilo 1 152
  153. Hilo 3 153
  154. Hilo 3 154
  155. Hilo 1 155
  156. Hilo 2 156
  157. Hilo 1 157
  158. Hilo 2 158
  159. Hilo 3 159
  160. Hilo 2 160
  161. Hilo 1 161
  162. Hilo 3 162
  163. Hilo 2 163
  164. Hilo 3 164
  165. Hilo 2 165
  166. Hilo 3 166
  167. Hilo 1 167
  168. Hilo 1 168
  169. Hilo 1 169
  170. Hilo 3 170
  171. Hilo 2 171
  172. Hilo 3 172
  173. Hilo 3 173
  174. Hilo 1 174
  175. Hilo 1 175
  176. Hilo 3 176
  177. Hilo 1 177
  178. Hilo 2 178
  179. Hilo 1 179
  180. Hilo 3 180
  181. Hilo 1 181
  182. Hilo 3 182
  183. Hilo 1 183
  184. Hilo 2 184
  185. Hilo 3 185
  186. Hilo 3 186
  187. Hilo 1 187
  188. Hilo 2 188
  189. Hilo 3 189
  190. Hilo 1 190
  191. Hilo 2 191
  192. Hilo 2 192
  193. Hilo 1 193
  194. Hilo 2 194
  195. Hilo 2 195
  196. Hilo 3 196
  197. Hilo 1 197
  198. Hilo 3 198
  199. Hilo 2 199
  200. Hilo 2 200
  201. Hilo 2 201
  202. Hilo 2 202
  203. Hilo 1 203
  204. Hilo 2 204
  205. Hilo 3 205
  206. Hilo 1 206
  207. Hilo 2 207
  208. Hilo 2 208
  209. Hilo 1 209
  210. Hilo 3 210
  211. Hilo 3 211
  212. Hilo 3 212
  213. Hilo 1 213
  214. Hilo 2 214
  215. Hilo 2 215
  216. Hilo 2 216
  217. Hilo 1 217
  218. Hilo 3 218
  219. Hilo 1 219
  220. Hilo 3 220
  221. Hilo 3 221
  222. Hilo 1 222
  223. Hilo 1 223
  224. Hilo 1 224
  225. Hilo 1 225
  226. Hilo 1 226
  227. Hilo 2 227
  228. Hilo 2 228
  229. Hilo 3 229
  230. Hilo 2 230
  231. Hilo 3 231
  232. Hilo 2 232
  233. Hilo 2 233
  234. Hilo 3 234
  235. Hilo 1 235
  236. Hilo 2 236
  237. Hilo 1 237
  238. Hilo 2 238
  239. Hilo 2 239
  240. Hilo 3 240
  241. Hilo 3 241
  242. Hilo 3 242
  243. Hilo 3 243
  244. Hilo 3 244
  245. Hilo 1 245
  246. Hilo 3 246
  247. Hilo 2 247
  248. Hilo 2 248
  249. Hilo 2 249
  250. Hilo 3 250
  251. Hilo 1 251
  252. Hilo 2 252
  253. Hilo 3 253
  254. Hilo 3 254
  255. Hilo 1 255
  256. Hilo 3 256
  257. Hilo 2 257
  258. Hilo 1 258
  259. Hilo 1 259
  260. Hilo 1 260
  261. Hilo 2 261
  262. Hilo 2 262
  263. Hilo 2 263
  264. Hilo 3 264
  265. Hilo 1 265
  266. Hilo 1 266
  267. Hilo 2 267
  268. Hilo 1 268
  269. Hilo 3 269
  270. Hilo 2 270
  271. Hilo 1 271
  272. Hilo 1 272
  273. Hilo 2 273
  274. Hilo 3 274
  275. Hilo 3 275
  276. Hilo 1 276
  277. Hilo 2 277
  278. Hilo 3 278
  279. Hilo 1 279
  280. Hilo 2 280
  281. Hilo 3 281
  282. Hilo 1 282
  283. Hilo 2 283
  284. Hilo 1 284
  285. Hilo 3 285
  286. Hilo 1 286
  287. Hilo 3 287
  288. Hilo 1 288
  289. Hilo 3 289
  290. Hilo 2 290
  291. Hilo 3 291
  292. Hilo 1 292
  293. Hilo 3 293
  294. Hilo 1 294
  295. Hilo 2 295
  296. Hilo 2 296
  297. Hilo 1 297
  298. Hilo 3 298
  299. Hilo 3 299
  300. Hilo 2 300
  301.  

la sincronizacion tiene un efecto bastante duro por el abuso del mismo,DEADLOCK, cuando varios hilos necesitan acceso exclusivo al recurso bloqueandolo/manteniendo el bloqueo, siendo también un bug difícil de detectar  >:D
« Última modificación: 7 Febrero 2019, 07:13 am por rub'n » En línea

rubn0x52.com KNOWLEDGE  SHOULD BE FREE.
If you don't have time to read, you don't have the time (or the tools) to write, Simple as that. Stephen king
fileteruso

Desconectado Desconectado

Mensajes: 18



Ver Perfil
Re: Acceder y modificar una variable global desde distintos threads
« Respuesta #4 en: 6 Febrero 2019, 23:31 pm »

Gracias por una respuesta tan elaborada, pero justamente lo que pretendo no es que funcione de manera sincronizada, sino que falle y provocar una condición de carrera. La finalidad es meramente académica.

Muchísimas gracias otra vez.
En línea

fileteruso

Desconectado Desconectado

Mensajes: 18



Ver Perfil
Re: Acceder y modificar una variable global desde distintos threads
« Respuesta #5 en: 6 Febrero 2019, 23:58 pm »

La razón de que actualmente no te funcione es que los tipos primitivos como int se copian cuando los pasas como parámetro. Al copiarse, es otro objeto diferente, y aunque lo modifiques, el original se queda igual. Para pasar objetos, tendrías que utilizar clases en vez de tipos primitivos.

Tendrías que meter la variable dentro de una clase y compartir un objeto de esa clase con las 2 que lo van a modificar. Otra opción es tenerlo como variable static y que accedan directamente a ella.

Dado que el acceso concurrente te acabaría dando problemas, puedes usar la clase AtomicInteger que sirve precisamente para eso: Modificar un int desde varios hilos. Podrías crearla en el main, y pasárselo al constructor de los otros 2 como haces ahora.

Al final lo he resuelto pasando la propia clase como has dicho. No me ha hecho falta usar AtomicInteger.

Dejo el código para quien pudiera estar interesado:

Clase principal:
Código:
public class Principal {
public volatile static int x;

public Principal() {
x = 0;
}

public static void main(String[] args) throws InterruptedException {
Incrementador[] inc = new Incrementador[1000];
Decrementador[] dec = new Decrementador[1000];

Thread hiloinc;
Thread hilodec;

for(int i=0; i<1000; i++) {
inc[i] = new Incrementador();
dec[i] = new Decrementador();

hiloinc = new Thread(inc[i]);
hilodec = new Thread(dec[i]);

hiloinc.start();
hilodec.start();

hiloinc.join();
hilodec.join();
}

System.out.println(x);
}
}

Clase Incrementador:
Código:
public class Incrementador implements Runnable {
public void run() {
Principal.x = Principal.x+1;
}
}

Clase Decrementador:
Código:

public class Decrementador implements Runnable {
public void run() {
Principal.x = Principal.x-1;
}
}


Gracias a todos!
« Última modificación: 7 Febrero 2019, 00:12 am por fileteruso » En línea

ivancea96


Desconectado Desconectado

Mensajes: 3.412


ASMático


Ver Perfil WWW
Re: Acceder y modificar una variable global desde distintos threads
« Respuesta #6 en: 7 Febrero 2019, 20:26 pm »

@ivancea96 con clases Atómicas no basta para que un recurso compartido sea actualizado correctamente por múltiples hilos a la misma ves, tampoco basta con volatile se necesita un lock, para asegurar la correcta sincronización.

Las clases del paquete atomic son precisamente thread-safe y lock-free. Son una optimización al uso de locks, y su uso es precisamente este.

No sé que pretendías ver en tu ejemplo, pero el resultado es el esperado: La variable ha sido incrementada 10 veces, por tanto, es 10, punto.

Puedes verificarlo si quieres:

Código
  1. import java.util.Objects;
  2. import java.util.concurrent.*;
  3. import java.util.concurrent.atomic.AtomicInteger;
  4. import java.util.logging.Logger;
  5. import java.util.stream.IntStream;
  6.  
  7. public class UsoSynchronized {
  8.  
  9.    private static final Logger LOGGER = Logger.getLogger(UsoSynchronized.class.getSimpleName());
  10.    private ExecutorService executors = null;
  11.    private AtomicInteger atomicInteger = new AtomicInteger(0);
  12.  
  13.    public UsoSynchronized() {
  14.        try {
  15.            executors = Executors.newFixedThreadPool(20);
  16.            IntStream.rangeClosed(1,10)
  17.                    .forEach(e -> {
  18.                        executors.submit(()-> withAtomicInteger());
  19.                    });
  20.  
  21.        } finally {
  22.            if(Objects.nonNull(executors)) {
  23.                executors.shutdown();
  24.  
  25.                try {
  26.                    executors.awaitTermination(1, TimeUnit.DAYS);
  27.                } catch (InterruptedException exc) {
  28.                }
  29.            }
  30.        }
  31.  
  32.        LOGGER.info("Final: "+ atomicInteger.get());
  33.    }
  34.  
  35.    public void withAtomicInteger() {
  36.        LOGGER.info(""+ atomicInteger.incrementAndGet());
  37.    }
  38.  
  39.    public static void main(String... haga) {
  40.        new UsoSynchronized();
  41.    }
  42. }

Verás un 10 en Final, es lo esperado, y verás los números del 1 al 10 antes que él.

Si tú te refieres al orden, evidentemente el orden no es una necesidad en este caso, y como tal, no se mantiene.
En línea

rub'n


Conectado Conectado

Mensajes: 1.218


(e -> λ("live now")); tatuar -> λ("α");


Ver Perfil WWW
Re: Acceder y modificar una variable global desde distintos threads
« Respuesta #7 en: 8 Febrero 2019, 23:25 pm »

Claro, no me niego tampoco a las clases Atómicas, seria ilógico que no me diera cuenta del resultado si en el mensaje puse el output donde se ve clarisimamente el 10

Ese ejemplo lo puse mas bien pensando a futuro en fileteruso, porque estoy 100% que le tocara aplicar un lock  ::)

dog.


En línea

rubn0x52.com KNOWLEDGE  SHOULD BE FREE.
If you don't have time to read, you don't have the time (or the tools) to write, Simple as that. Stephen king
6666

Desconectado Desconectado

Mensajes: 146


Wikileaks.org


Ver Perfil
Re: Acceder y modificar una variable global desde distintos threads
« Respuesta #8 en: 9 Febrero 2019, 02:50 am »

static
En línea

Páginas: [1] Ir Arriba Respuesta Imprimir 

Ir a:  

Mensajes similares
Asunto Iniciado por Respuestas Vistas Último mensaje
SMF: $context variable global
PHP
[KMT] 4 3,375 Último mensaje 2 Enero 2008, 22:02 pm
por [KMT]
variable global
Programación Visual Basic
xexio 8 4,792 Último mensaje 4 Marzo 2010, 22:12 pm
por xexio
Acceder a una variable privada de una clase desde otra clase
Programación C/C++
Alvaro093 4 30,519 Último mensaje 27 Enero 2012, 09:46 am
por wachi
variable global
Programación C/C++
7emiliosk8 1 1,828 Último mensaje 12 Febrero 2017, 13:10 pm
por GGZ
No consigo modificar variable global
Programación C/C++
Gamerpc 4 2,436 Último mensaje 23 Mayo 2018, 19:07 pm
por Gamerpc
WAP2 - Aviso Legal - Powered by SMF 1.1.21 | SMF © 2006-2008, Simple Machines