julia_v_iluhina

Untitled

Sep 28th, 2016
73
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 2.01 KB | None | 0 0
  1. src / main / java / core / WaitFor.java
  2. /*
  3.     форматирование(
  4. */
  5. **********************
  6. public class ExactText extends CollectionCondition {
  7. /*
  8.     судя по реализации и конструктору - кондишен должен называться ExactTexts
  9.  
  10.     текстЫ
  11.  
  12.     и имя соответствующего метода CollectionConditions тоже подправь
  13.  
  14.     также можно немного сэкономить - отнаследовав его от TextsOf
  15.     одинаковый код можно реализовать лишь в предке)
  16.     можно и check только в предке вызвать
  17.     реально - там только одна строка отличается)
  18.     вот ее - завернуть в отдельный метод и реализовать в предке и потомке
  19.  
  20.     а check -  только в предке
  21.  
  22.     это тебе такая небольшая загадочка)
  23.  
  24.     а также можно было бы чуть имена подравнять - Texts & ExactTexts
  25. */
  26. *************************************
  27. public class Present extends ElementCondition {
  28.     boolean isPresent;
  29.     /*
  30.         переменную ты завела
  31.         это правильно
  32.         а чего не используешь? )
  33.     */
  34.  
  35.     @Override
  36.     public boolean check(WebElement element) {
  37. /*
  38.     вычисли isPresent и его верни
  39. */
  40.     @Override
  41.     public String expected() {
  42. /*
  43.     ожидаем - что true -  мы же оптимисты)
  44. */
  45.  
  46.  
  47.     @Override
  48.     public String actual() {
  49. /*
  50.     а как оно на самом деле - расскажет isPresent )
  51.  
  52.     посмотри на свой же Visible )
  53.     там все ок как раз
  54. */
  55.  
  56. ******************************
  57.  
  58. /*
  59.     По аналогии с Texts & ExactTexts -
  60.  
  61.     можно и
  62.  
  63.     ExactText от Text отнаследовать)
  64. */
Advertisement
Add Comment
Please, Sign In to add comment