Сравнение Поплавков в векторе

Хотя вы определенно находитесь в правильном направлении (и на самом деле довольно близки к концу), в вашем коде существует несколько (imo) плохих практик (например, для инъекции целой библиотеки (jquery) для такой тривиальной задачи, объявления ненужных разрешений , делая суперплошные вызовы API-методам и т. д.). Я сам не тестировал ваш код, но из краткого обзора считаю, что исправление следующего может привести к рабочему решению (хотя и не очень близкому к оптимальному):

  1. В manifest.json: Изменить порядок скриптов содержимого, сначала добавив jquery. Согласно соответствующие документы : «js» [...] Список файлов JavaScript, которые будут вставляться в соответствующие страницы. Они вводятся в порядке их появления в этом массиве. (emphasis mine)
  2. В contentscript.js: Переместите блок chrome.runtime.sendMessage ({...}) внутри обратного вызова слушателя onMessage.

Итак, вот мой предложенный подход:

Управляющий поток:

  1. Скрипт содержимого вводится в каждый страница, соответствующая некоторым критериям.
  2. После ввода скрипты содержимого отправляют сообщение на страницу событий (непостоянная фоновая страница), а страница событий присоединяет к странице вкладку = действие.
  3. Как только всплывающее окно с действием страницы загружается, оно отправляет сообщение в скрипт контента, запрашивая информацию, в которой она нуждается.
  4. Скрипт контента обрабатывает запрос и отвечает так, чтобы действие страницы popup может отображать информацию.

Структура каталогов:

          root-directory/
           |_____img
                 |_____icon19.png
                 |_____icon38.png
           |_____manifest.json
           |_____background.js
           |_____content.js
           |_____popup.js
           |_____popup.html

manifest.json:

{
  "manifest_version": 2,
  "name":    "Test Extension",
  "version": "0.0",
  "offline_enabled": true,

  "background": {
    "persistent": false,
    "scripts": ["background.js"]
  },

  "content_scripts": [{
    "matches":    ["*://*.stackoverflow.com/*"],
    "js":         ["content.js"],
    "run_at":     "document_idle",
    "all_frames": false
  }],

  "page_action": {
    "default_title": "Test Extension",
    //"default_icon": {
    //    "19": "img/icon19.png",
    //    "38": "img/icon38.png"
    //},
    "default_popup": "popup.html"
  }

  // No special permissions required...
  //"permissions": []
}

background.js:

chrome.runtime.onMessage.addListener(function (msg, sender) {
  // First, validate the message's structure
  if ((msg.from === 'content') && (msg.subject === 'showPageAction')) {
    // Enable the page-action for the requesting tab
    chrome.pageAction.show(sender.tab.id);
  }
});

content.js:

// Inform the background page that 
// this tab should have a page-action
chrome.runtime.sendMessage({
  from:    'content',
  subject: 'showPageAction'
});

// Listen for messages from the popup
chrome.runtime.onMessage.addListener(function (msg, sender, response) {
  // First, validate the message's structure
  if ((msg.from === 'popup') && (msg.subject === 'DOMInfo')) {
    // Collect the necessary data 
    // (For your specific requirements `document.querySelectorAll(...)`
    //  should be equivalent to jquery's `$(...)`)
    var domInfo = {
      total:   document.querySelectorAll('*').length,
      inputs:  document.querySelectorAll('input').length,
      buttons: document.querySelectorAll('button').length
    };

    // Directly respond to the sender (popup), 
    // through the specified callback */
    response(domInfo);
  }
});

popup.js:

// Update the relevant fields with the new data
function setDOMInfo(info) {
  document.getElementById('total').textContent   = info.total;
  document.getElementById('inputs').textContent  = info.inputs;
  document.getElementById('buttons').textContent = info.buttons;
}

// Once the DOM is ready...
window.addEventListener('DOMContentLoaded', function () {
  // ...query for the active tab...
  chrome.tabs.query({
    active: true,
    currentWindow: true
  }, function (tabs) {
    // ...and send a request for the DOM info...
    chrome.tabs.sendMessage(
        tabs[0].id,
        {from: 'popup', subject: 'DOMInfo'},
        // ...also specifying a callback to be called 
        //    from the receiving end (content script)
        setDOMInfo);
  });
});

popup.html:



  
    
  
  
    

DOM Info

Total number of elements: N/A
Number of input elements: N/A
Number of button elements: N/A

0
задан 1201ProgramAlarm 16 January 2019 в 23:02
поделиться

1 ответ

Вы никогда не сбрасываете n, поэтому, как только вы получите простое число (когда n == (not_prime_numbers.size()-1)), остальные числа также будут добавлены в основной список.

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

0
ответ дан 1201ProgramAlarm 16 January 2019 в 23:02
поделиться
Другие вопросы по тегам:

Похожие вопросы: