Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Количество ссылок в карте #65

Open
chulcha opened this issue Feb 4, 2019 · 9 comments
Open

Количество ссылок в карте #65

chulcha opened this issue Feb 4, 2019 · 9 comments

Comments

@chulcha
Copy link

chulcha commented Feb 4, 2019

при мультиязыковой карте, количество ссылок в Число языков больше

нужно

$this->urlsCount = $this->urlsCount + count($location);

@samdark
Copy link
Owner

samdark commented Feb 4, 2019

Можно пояснить "количество ссылок в Число языков больше"?

@chulcha
Copy link
Author

chulcha commented Feb 4, 2019

Если у вас 2 языка, то ссылок в карте будет 2, а не одна

у вас
$this->urlsCount число обращений в addItem, а не число переданных в нее ссылок

@chulcha
Copy link
Author

chulcha commented Feb 4, 2019

Кстати, за этим потянется еще и способ для метода flush, который подразумевает 1 инкремент

$this->urlsCount % $this->bufferSize === 0

@samdark
Copy link
Owner

samdark commented Feb 15, 2019

#66 так?

@chulcha
Copy link
Author

chulcha commented Feb 15, 2019

Почти
if ($this->urlsCount % $this->bufferSize === 0) {

у вас три языка
соответственно остаток станет равным нулю при количество языков * bufferSize

Я для этого ввел еще одно свойство для текущего значения буфера

Го это не существенно - 10 или 30

@samdark
Copy link
Owner

samdark commented Feb 15, 2019

Почему? Мы ведь уже нарастили urlsCount корректно...

@chulcha
Copy link
Author

chulcha commented Feb 15, 2019

bufferSize = 10
языков = 3

0 + 3 - остаток 3
3+3 - остаток - 3
9+3 - остаток? 2
27+3 = 30 - остаток 0

@arvai
Copy link

arvai commented Mar 24, 2021

how do you expect help if you are writing in !english ?

@chulcha
Copy link
Author

chulcha commented Mar 24, 2021

how do you expect help if you are writing in !english ?

owner reads and writes in russian

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants