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

Code pagination #11

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

evilbloodydemon
Copy link
Contributor

  • added pagination
  • extracted services

public function getEntity($category)
{
$categoryRepository = $this->entityManager->getRepository("GovnokodCodeBundle:Category");
$entities = $categoryRepository->findByName($category);
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

есть метод findOneByName, который вернет одну запись, либо null.

@wiistriker
Copy link
Owner

Думаю, существование объектов Service не совсем оправдано. Для этих целей можно и нужно использовать доктриновские репозитории.

А для пагинации есть Pagerfanta

@evilbloodydemon
Copy link
Contributor Author

не вижу, чтобы Pagerfanta поддерживала паджинацию страниц в обратном порядке

что касается сервисов, то эксперты из интернета считают, что репозитории - это просто шлюз к данным, а сервисы реализуют бизнес-логику и могут использовать несколько репозиториев одновременно (или не использовать их вовсе).
то есть, по сути своей, сервисы ближе к контроллерам, но могут использоваться не только из веб-контекста

@ghost
Copy link

ghost commented Jun 14, 2017

Чвйт не раьотает! Выложу код сюла

import java.util.*;


public class Main
{
	public static void main(String[] args) throws InterruptedException
	{
		StringBuilder s = new StringBuilder("#");
		byte a=1;
		boolean t=false;
		while(true)
		{
			if(!t)
			{
				s.append("#");
				a++;
			}
			if(a==40)
			{
				t=true;
			}
			if(a==0)
			    t=false;
			if(t)
			{
				s.deleteCharAt(s.length()-1);
				a--;
			}
			System.out.println(s);
			Thread.sleep(100);
		}
	}
}

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

Successfully merging this pull request may close these issues.

2 participants