julia_v_iluhina

Untitled

Nov 29th, 2016
109
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 3.73 KB | None | 0 0
  1. /*
  2.     встречается не реформатированный код
  3.     просмотри и подправь это
  4. */
  5.  
  6. public static void searchBySubject(String subject) {
  7. /*
  8.     да, мы в метод передаем subject,  и даже  заявляем - searchBySubject
  9.  
  10.     если мы так говорим - так надо это и делать
  11.  
  12.     надо в тексте запроса для поиска писать строку subject: ...и тут текст из переданного subject....
  13.  
  14.  
  15.     или - при той реализации как сейчас - название метода и параметра надо подправить до
  16.     search(String queryText)
  17.  
  18.     когда ты даешь имя методу или параметру метода
  19.     надо перестать думать - о контексте вызова метода или о том - что за данные ты передаешь
  20.     и надо начать думать о том - что делает метод и какими данными он оперирует
  21. */
  22. **************************
  23.     public static void assertMails(String... emailSubjects) {
  24.     public static void assertMail(int n, String subject) {
  25. /*
  26.     даже если бы мы на уровне методов и правда рабтали именно с subject-ами
  27.     то хорошо бы - чтоб одинаковый термин был использован для параметров - subjects, subject
  28.  
  29.     но - на уровне методов - мы лишь проверяем вхождение нужного нам текста в текст заголовка мейла,
  30.     мы ведь в проверке - не уточняемся до элемента-subject-а и не проверяем именно его
  31.  
  32.     потому - правильнее параметр метод называть не subject, а text/texts or mailText/mailTexts or mailHeaderText/mailHeaderTexts
  33.  
  34.     да, передадим мы subject или subject-ы
  35.     но - технически - такой ассерт можно не только для проверки subject-ов писем использовать
  36.     но и для поверки любых кусков текста для мейлов
  37.  
  38.     насчет параметра int n
  39.     традиционно для номера в коллекции - используют термин index
  40.     тут - та же история - мы обращаемся к мейлу из коллекции мейлов - по его индексу - лучше используй термин index
  41.     это большему количеству людей будет со старта понятно
  42.  
  43.     это уже для общего развития в этом вопросе
  44.         https://en.wikipedia.org/wiki/Zero-based_numbering
  45.         http://c2.com/cgi/wiki?ZeroAndOneBasedIndexes
  46.         http://stackoverflow.com/questions/24841172/why-is-array-indexing-in-java-start-w
  47. */
  48. ***********************************
  49. $(byText("Inbox")).click();
  50. /*
  51.     по тексту - можно много другого наловить
  52.     http://joxi.ru/n2YkKaGUjY8Y9r
  53.  
  54.     тут надо по-другому это делать
  55.  
  56.     или оттолкнуться от элемента-контейнера для вариантов папок http://joxi.ru/GrqLOX3SN17NzA
  57.     и затем уже уточняться до Inbox
  58.  
  59.     или - постороить css Selector - title начинается с ...
  60.     http://joxi.ru/5md7jYwtvx7v9r
  61. */
  62. *********************************
  63.  System.out.println();
  64.  /*
  65.     уже не нужно
  66.  */
Advertisement
Add Comment
Please, Sign In to add comment