Laravel Eloquent - distinct () et count () ne fonctionnent pas correctement ensemble

J'essaye donc d'obtenir le nombre de pids distincts sur une requête, mais la valeur renvoyée est fausse.

C'est ce que j'essaye de faire:

$ad->getcodes()->groupby('pid')->distinct()->count()

ce qui renvoie la valeur "2", tandis que la valeur qu'il doit renvoyer doit être "1".

Pour contourner le problème, je fais ceci:

count($ad->getcodes()->groupby('pid')->distinct()->get())

ce qui fonctionne bien et renvoie "1"

Existe-t-il une règle où count et distinct ne peuvent pas être sur la même requête? Je trouve que la solution de contournement est "lourde", je voudrais faire fonctionner la requête d'origine :(

Ce qui suit devrait fonctionner

$ad->getcodes()->distinct('pid')->count('pid');

Une réponse plus générique qui m'aurait fait gagner du temps, et j'espère que d'autres:

Ne fonctionne pas (renvoie le nombre de toutes les lignes):

DB::table('users')
            ->select('first_name')
            ->distinct()
            ->count();

Le correctif:

DB::table('users')
            ->distinct()
            ->count('first_name');

Quelqu'un d'autre est tombé sur ce post et ne trouve pas les autres suggestions pour fonctionner?

En fonction de la requête spécifique, une approche différente peut être nécessaire. Dans mon cas, je devais soit compter les résultats d'un GROUP BY, par exemple

SELECT COUNT(*) FROM (SELECT * FROM a GROUP BY b)

ou utilisez COUNT(DISTINCT b):

SELECT COUNT(DISTINCT b) FROM a

Après quelques intrigues, j'ai réalisé qu'il n'y avait pas de fonction Laravel intégrée pour l'un ou l'autre. La solution la plus simple était donc d'utiliser l'utilisation DB::rawavec la countméthode.

$count = $builder->count(DB::raw('DISTINCT b'));

N'oubliez pas de ne pas utiliser groupByavant d'appeler count. Vous pouvez postuler groupByplus tard, si vous en avez besoin pour obtenir des lignes.

J'ai eu un problème similaire et j'ai trouvé un moyen de le contourner.

Le problème est la façon dont le générateur de requêtes de Laravel gère les agrégats. Il prend le premier résultat renvoyé, puis renvoie la valeur «agrégée». C'est généralement bien, mais lorsque vous combinez count avec groupBy, vous renvoyez un nombre par élément groupé. Ainsi, l'agrégat de la première ligne est juste un décompte du premier groupe (donc quelque chose de faible comme 1 ou 2 est probable).

Le décompte de Laravel est donc sorti, mais j'ai combiné le générateur de requêtes Laravel avec du SQL brut pour obtenir un décompte précis de mes résultats groupés.

Pour votre exemple, je m'attends à ce que ce qui suit fonctionne (et vous permette d'éviter le get):

$query = $ad->getcodes()->groupby('pid')->distinct();
$count = count(\DB::select($query->toSql(), $query->getBindings()));

Si vous voulez vous assurer de ne pas perdre de temps à sélectionner toutes les colonnes, vous pouvez éviter cela lors de la création de votre requête:

 $query = $ad->select(DB::raw(1))->getcodes()->groupby('pid')->distinct();

J'ai rencontré le même problème.

Si vous installez la barre de débogage laravel, vous pouvez voir les requêtes et voir souvent le problème

$ad->getcodes()->groupby('pid')->distinct()->count()

changer en

$ad->getcodes()->distinct()->select('pid')->count()

Vous devez définir les valeurs à renvoyer comme distinctes. Si vous ne définissez pas les champs de sélection, toutes les colonnes de la base de données seront renvoyées et toutes seront uniques. Définissez donc la requête sur distinct et ne sélectionnez que les colonnes qui constituent votre valeur `` distincte '' que vous voudrez peut-être en ajouter. ->select('pid','date')pour obtenir toutes les valeurs uniques d'un utilisateur en une journée

Vous pouvez utiliser la méthode suivante pour obtenir les données uniques selon vos besoins, comme suit,

$data = $ad->getcodes()->get()->unique('email');

$count = $data->count();

J'espère que cela fonctionnera.

Cela ne fonctionnerait-il pas?

$ad->getcodes()->distinct()->get(['pid'])->count();

Voir ici pour discussion.

$solution = $query->distinct()
            ->groupBy
            (
                [
                    'array',
                    'of',
                    'columns',
                ]
            )
            ->addSelect(
                [
                    'columns',
                    'from',
                    'the',
                    'groupby',
                ]
            )
            ->get();

N'oubliez pas que le groupe par est facultatif, cela devrait fonctionner dans la plupart des cas lorsque vous souhaitez qu'un groupe de comptage par exclure les valeurs de sélection en double, addSelect est une méthode d'instance de querybuilder.

Distinct ne prend pas d'arguments car il ajoute DISTINCT dans votre requête SQL, cependant, vous pouvez avoir besoin de définir le nom de colonne avec lequel vous souhaitez sélectionner distinct. Ainsi, si vous avez Flight->select('project_id')->distinct()->get()est équivalent à SELECT DISTINCT 'project_id' FROM flightset vous pouvez maintenant ajouter d'autres modificateurs comme count () ou même des requêtes éloquentes brutes.

Sur la base de la documentation Laravel pour les requêtes brutes, j'ai pu obtenir le décompte d'un champ de sélection pour travailler avec ce code dans le modèle de produit.

public function scopeShowProductCount($query)
{
    $query->select(DB::raw('DISTINCT pid, COUNT(*) AS count_pid'))
          ->groupBy('pid')
          ->orderBy('count_pid', 'desc');
}

Cette façade a fonctionné pour obtenir le même résultat dans le contrôleur:

$products = DB::table('products')->select(DB::raw('DISTINCT pid, COUNT(*) AS count_pid'))->groupBy('pid')->orderBy('count_pid', 'desc')->get();

Le vidage résultant pour les deux requêtes était le suivant:

#attributes: array:2 [
  "pid" => "1271"
  "count_pid" => 19
],
#attributes: array:2 [
  "pid" => "1273"
  "count_pid" => 12
],
#attributes: array:2 [
  "pid" => "1275"
  "count_pid" => 7
]

Cela fonctionnait pour moi alors essayez ceci: $ ad-> getcodes () -> distinct ('pid') -> count ()

essaye ça

$ad->getcodes()->groupby('pid')->distinct()->count('pid')