[inny][laravel 5.2] Połączone zapytanie Opcje

[miccom]

Cześć
W bazie danych w tabeli user mam nazwy 'pl01', 'pl03','pl05' i każda z tych wartości może się równać '0' lub '1'

dostarczam do funkcji stringa o konstrukcji np.:

[PHP] pobierz, plaintext 
$string = 01|05
[PHP] pobierz, plaintext 

Rozbijam strninga explode
I teraz podstawiam sobie w pętli foreach

[PHP] pobierz, plaintext 
foreach ($wojewodztwa as &$value) {
			$wynik = User::where('pl'.$value,'=',1)->get();	
	}
[PHP] pobierz, plaintext 

I teraz jak wiadomo, ładuje mi tylko ostatnią pętlę
Jak uzyskać dane gdy 'pl01'=1 OR 'pl05'=1 ?

Jak do wyników USER::where dodać 'pl01' z pierwszego rozbicia?

Ten post edytował miccom 20.05.2018, 00:04:11

[SmokAnalog]

Zobacz: https://laravel.com/docs/5.2/queries#advanced-where-clauses

P.S. Coś mi się nie podoba Twoja struktura bazy :/

[miccom]

A co mogę w niej zmienić?

A odnośnie zapytań to przestudiowałem to, bardziej chodzi mi o to, jak w kodzie rozwiązać te klauzule where

mam tabelę user a w niej rekordy name, email, i 'pl01' do 'pl16' oznaczające województwa, gdzie user zapisuje, w jakich województwach jest obecny.

Jak mógłbym to inaczej rozwiązać?

Ten post edytował miccom 20.05.2018, 07:27:43

[viking]

Tabela województwa, tabela użytkownicy i tabela użytkownicy województwa. Relacja wiele do wielu.

[miccom]

viking, a za rączkę?

Tworze tabelę o nazwie wojewodztwa i w niej wbijam województwa z ich ID
Tworzę tabelę użytkownicy i w niej id usera, email i jego dane
Tworzę tabelę użWoj i w niej...
No właśnie, co w niej dać?

Ten post edytował miccom 20.05.2018, 09:52:12

[viking]

Id użytkownika oraz id województwa z kluczem głównym na obu kolumnach. Poczytaj trochę o takich powiązaniach.

[miccom]

Ściągnąłem mysql Workbech, ale nadal nie wiem jak połączyć tabele tymi kluczami

Przygotowałem sobie diagram, potem export to sql i otrzymałem:

[SQL] pobierz, plaintext 
-- MySQL Script generated by MySQL Workbench
-- Sun May 20 11:52:46 2018
-- Model: New Model    Version: 1.0
-- MySQL Workbench Forward Engineering
 
SET @OLD_UNIQUE_CHECKS=@@UNIQUE_CHECKS, UNIQUE_CHECKS=0;
SET @OLD_FOREIGN_KEY_CHECKS=@@FOREIGN_KEY_CHECKS, FOREIGN_KEY_CHECKS=0;
SET @OLD_SQL_MODE=@@SQL_MODE, SQL_MODE='TRADITIONAL,ALLOW_INVALID_DATES';
 
-- -----------------------------------------------------
-- Schema mydb
-- -----------------------------------------------------
DROP SCHEMA IF EXISTS `mydb` ;
 
-- -----------------------------------------------------
-- Schema mydb
-- -----------------------------------------------------
CREATE SCHEMA IF NOT EXISTS `mydb` DEFAULT CHARACTER SET utf8 ;
SHOW WARNINGS;
USE `mydb` ;
 
-- -----------------------------------------------------
-- Table `users`
-- -----------------------------------------------------
DROP TABLE IF EXISTS `users` ;
 
SHOW WARNINGS;
CREATE TABLE IF NOT EXISTS `users` (
  `id` INT NOT NULL,
  `email` VARCHAR(255) NULL,
  `name` VARCHAR(4255) NULL,
  PRIMARY KEY (`id`))
ENGINE = InnoDB;
 
SHOW WARNINGS;
CREATE UNIQUE INDEX `id_UNIQUE` ON `users` (`id` ASC);
 
SHOW WARNINGS;
 
-- -----------------------------------------------------
-- Table `woj`
-- -----------------------------------------------------
DROP TABLE IF EXISTS `woj` ;
 
SHOW WARNINGS;
CREATE TABLE IF NOT EXISTS `woj` (
  `idWoj` INT NULL,
  `name` VARCHAR(255) NULL,
  PRIMARY KEY (`idWoj`))
ENGINE = InnoDB;
 
SHOW WARNINGS;
 
-- -----------------------------------------------------
-- Table `users_has_woj`
-- -----------------------------------------------------
DROP TABLE IF EXISTS `users_has_woj` ;
 
SHOW WARNINGS;
CREATE TABLE IF NOT EXISTS `users_has_woj` (
  `users_id` INT NOT NULL,
  `woj_idWoj` INT NOT NULL,
  PRIMARY KEY (`users_id`, `woj_idWoj`))
ENGINE = InnoDB;
 
SHOW WARNINGS;
 
SET SQL_MODE=@OLD_SQL_MODE;
SET FOREIGN_KEY_CHECKS=@OLD_FOREIGN_KEY_CHECKS;
SET UNIQUE_CHECKS=@OLD_UNIQUE_CHECKS;
 
[SQL] pobierz, plaintext 

Czy powyższe dobrze wykonałem?

Ten post edytował miccom 20.05.2018, 10:53:18

[SmokAnalog]

W tabeli `userWoj` możesz ustawić klucz podstawowy na parę kolumn: `userWoj` i `wojId`. Ale tak jak masz też jest w porządku.

Ogólnie taka struktura bazy, która idzie wszerz przez mnogość opcji, jest w 99% przypadków niewłaściwa i problematyczna. Tym bardziej u Ciebie, gdzie jak się domyślam chodzi tylko o istnienie lub nieistnienie relacji użytkownik-województwo.

[miccom]

W tabeli `userWoj` możesz ustawić klucz podstawowy na parę kolumn: `userWoj` i `wojId`. Ale tak jak masz też jest w porządku.

Ogólnie taka struktura bazy, która idzie wszerz przez mnogość opcji, jest w 99% przypadków niewłaściwa i problematyczna. Tym bardziej u Ciebie, gdzie jak się domyślam chodzi tylko o istnienie lub nieistnienie relacji użytkownik-województwo.

Edytowałem nowe połączenia tabel, czy zaktualizowany plik sql jest ok?

[SmokAnalog]

Tak, lepiej. Mógłbyś jeszcze trzymać się jednej konwencji, bo raz masz users, raz woj, razPiszeszTak, a_raz_tak. Indeksy unikalne nadałeś ręcznie? Bo nawet nie wiem czy one w dumpie się osobno pokazują dla klucza podstawowego. Jeśli ręcznie dałeś, to niepotrzebnie, bo klucz podstawowy sam w sobie jest unikalny z definicji.

[miccom]

No tak, właśnie nie wiem czy pisać wielbłądzim czy z podkreślnikami, to samo dotyczy się nazewnictwa, users i woj wiec albo tabele i ich nazwy po polsku albo po angielsku

Niby nic ważnego, ale ułatwia pracę, muszę się nauczyć raz a porządnie co i jak stosować.

Indeksy nadałem w workbench ale w większości same były już zaznaczone jako PK,
a co do users_has_woj to workbench sam tak ustawił

Czyli to będzie dobra architektura bazy?

[SQL] pobierz, plaintext 
-- MySQL Script generated by MySQL Workbench
-- Sun May 20 12:37:39 2018
-- Model: New Model    Version: 1.0
-- MySQL Workbench Forward Engineering
 
SET @OLD_UNIQUE_CHECKS=@@UNIQUE_CHECKS, UNIQUE_CHECKS=0;
SET @OLD_FOREIGN_KEY_CHECKS=@@FOREIGN_KEY_CHECKS, FOREIGN_KEY_CHECKS=0;
SET @OLD_SQL_MODE=@@SQL_MODE, SQL_MODE='TRADITIONAL,ALLOW_INVALID_DATES';
 
-- -----------------------------------------------------
-- Schema mydb
-- -----------------------------------------------------
DROP SCHEMA IF EXISTS `mydb` ;
 
-- -----------------------------------------------------
-- Schema mydb
-- -----------------------------------------------------
CREATE SCHEMA IF NOT EXISTS `mydb` DEFAULT CHARACTER SET utf8 ;
SHOW WARNINGS;
USE `mydb` ;
 
-- -----------------------------------------------------
-- Table `users`
-- -----------------------------------------------------
DROP TABLE IF EXISTS `users` ;
 
SHOW WARNINGS;
CREATE TABLE IF NOT EXISTS `users` (
  `id` INT NOT NULL,
  `email` VARCHAR(255) NULL,
  `name` VARCHAR(255) NULL,
  `account` INT NULL COMMENT '1: użytkownik, 2:zespół / DJ',
  `password` VARCHAR(255) NOT NULL,
  `remember_token` VARCHAR(100) NULL,
  `created_at` TIMESTAMP(100) NULL DEFAULT NULL,
  `updated_at` TIMESTAMP(100) NULL DEFAULT NULL,
  `confCode` VARCHAR(255) NULL,
  PRIMARY KEY (`id`))
ENGINE = InnoDB;
 
SHOW WARNINGS;
CREATE UNIQUE INDEX `id_UNIQUE` ON `users` (`id` ASC);
 
SHOW WARNINGS;
 
-- -----------------------------------------------------
-- Table `voivodeship`
-- -----------------------------------------------------
DROP TABLE IF EXISTS `voivodeship` ;
 
SHOW WARNINGS;
CREATE TABLE IF NOT EXISTS `voivodeship` (
  `id_voivodeship` INT NOT NULL,
  `name` VARCHAR(255) NULL,
  PRIMARY KEY (`id_voivodeship`))
ENGINE = InnoDB;
 
SHOW WARNINGS;
 
-- -----------------------------------------------------
-- Table `users_has_voivodship`
-- -----------------------------------------------------
DROP TABLE IF EXISTS `users_has_voivodship` ;
 
SHOW WARNINGS;
CREATE TABLE IF NOT EXISTS `users_has_voivodship` (
  `users_id` INT NOT NULL,
  `voivodeship_id_voivodeship` INT NOT NULL,
  PRIMARY KEY (`users_id`, `voivodeship_id_voivodeship`))
ENGINE = InnoDB;
 
SHOW WARNINGS;
 
SET SQL_MODE=@OLD_SQL_MODE;
SET FOREIGN_KEY_CHECKS=@OLD_FOREIGN_KEY_CHECKS;
SET UNIQUE_CHECKS=@OLD_UNIQUE_CHECKS;
[SQL] pobierz, plaintext 

Ok, więc skleciłem takie zapytanie:

[PHP] pobierz, plaintext 
$users = DB::table('users')
            ->join('users_has_voivodeship', 'users.id', '=', 'users_id')
			->join('voivodeship', function ($join) {
            $join->on('voivodeship.id_voivodeship', '=', 'users_has_voivodeship.voivodeship_id_voivodeship')
				->where('users_has_voivodeship.voivodeship_id_voivodeship', '=', 5);
			})
			->get();
[PHP] pobierz, plaintext 

I działa dla users_has_voivodeship.voivodeship_id_voivodeship = 5, ale jak wybrać jeszcze pozostałe które zostały dostarczone stringiem?

Ten post edytował miccom 20.05.2018, 12:55:07

[viking]

Nie jest dobrze bo w ostatniej tabeli nie zdefiniowales kluczy obcych.

[miccom]

A jak to stworzyć przy użyciu workbench?

[viking]

A google działa? https://dev.mysql.com/doc/workbench/en/wb-f...ationships.html

[miccom]

[SQL] pobierz, plaintext 
-- MySQL Script generated by MySQL Workbench
-- Tue May 22 22:21:11 2018
-- Model: New Model    Version: 1.0
-- MySQL Workbench Forward Engineering
 
SET @OLD_UNIQUE_CHECKS=@@UNIQUE_CHECKS, UNIQUE_CHECKS=0;
SET @OLD_FOREIGN_KEY_CHECKS=@@FOREIGN_KEY_CHECKS, FOREIGN_KEY_CHECKS=0;
SET @OLD_SQL_MODE=@@SQL_MODE, SQL_MODE='TRADITIONAL,ALLOW_INVALID_DATES';
 
-- -----------------------------------------------------
-- Schema mydb
-- -----------------------------------------------------
DROP SCHEMA IF EXISTS `mydb` ;
 
-- -----------------------------------------------------
-- Schema mydb
-- -----------------------------------------------------
CREATE SCHEMA IF NOT EXISTS `mydb` DEFAULT CHARACTER SET utf8 ;
SHOW WARNINGS;
USE `mydb` ;
 
-- -----------------------------------------------------
-- Table `users`
-- -----------------------------------------------------
DROP TABLE IF EXISTS `users` ;
 
SHOW WARNINGS;
CREATE TABLE IF NOT EXISTS `users` (
  `id` INT NOT NULL,
  `email` VARCHAR(255) NULL,
  `name` VARCHAR(255) NULL,
  `account` INT NULL COMMENT '1: użytkownik, 2:zespół / DJ',
  `password` VARCHAR(255) NOT NULL,
  `remember_token` VARCHAR(100) NULL,
  `created_at` TIMESTAMP(100) NULL DEFAULT NULL,
  `updated_at` TIMESTAMP(100) NULL DEFAULT NULL,
  `confCode` VARCHAR(255) NULL COMMENT '\'0: nieaktywne, 1:potwierdzone\'\n',
  PRIMARY KEY (`id`))
ENGINE = InnoDB;
 
SHOW WARNINGS;
 
-- -----------------------------------------------------
-- Table `voivodeship`
-- -----------------------------------------------------
DROP TABLE IF EXISTS `voivodeship` ;
 
SHOW WARNINGS;
CREATE TABLE IF NOT EXISTS `voivodeship` (
  `id_voivodeship` INT NULL,
  `name` VARCHAR(255) NULL,
  PRIMARY KEY (`id_voivodeship`))
ENGINE = InnoDB;
 
SHOW WARNINGS;
 
-- -----------------------------------------------------
-- Table `users_has_voivodeship`
-- -----------------------------------------------------
DROP TABLE IF EXISTS `users_has_voivodeship` ;
 
SHOW WARNINGS;
CREATE TABLE IF NOT EXISTS `users_has_voivodeship` (
  `users_id` INT NOT NULL,
  `voivodeship_id_voivodeship` INT NOT NULL,
  PRIMARY KEY (`users_id`, `voivodeship_id_voivodeship`))
ENGINE = InnoDB;
 
SHOW WARNINGS;
 
SET SQL_MODE=@OLD_SQL_MODE;
SET FOREIGN_KEY_CHECKS=@OLD_FOREIGN_KEY_CHECKS;
SET UNIQUE_CHECKS=@OLD_UNIQUE_CHECKS;
[SQL] pobierz, plaintext 

Wpisuję many-to-many, dodaje mi się kolejna tabelka z relacjami, co mam jeszcze dodać?

Ponawiam pytanie.

Jak wyciągnąć dane przyporządkowane do szukanych id województw?
Dostarczam listę id województw w formacie string np. 1|5|9 rozbijam explodem i mam pojedyncze rekordy

Jak skonstruować zapytanie- aby wyciągnąć dane gdzie users_has_voivodeship.voivodeship_id_voivodeship = 1 OR users_has_voivodeship.voivodeship_id_voivodeship = 5 OR users_has_voivodeship.voivodeship_id_voivodeship = 9 ?

Nikt nie pomoże?

Kombinuję coś w ten deseń...

[PHP] pobierz, plaintext 
$count = count($wojewodztwa);
		$users=array();
		for ($while = 0; $while < $count; $while++) {
			$users = DB::table('users')
				->join('users_has_voivodeship', 'users.id', '=', 'users_id')
				->where('users_has_voivodeship.voivodeship_id_voivodeship', '=', $wojewodztwa[$while])
				->select('users.*', 'users_has_voivodeship.*')
				->get();
		}
[PHP] pobierz, plaintext 

Tylko jak teraz wtłoczyć do tablicy $users kolejne wyniki z następnego województwa?

[Pyton_000]

użycie `whereIn`?

[miccom]

Dziękuję bardzo.
Działa