Django jak działają relacje

Question

witam. czy mógłby ktoś objaśnić mi jak działają połączenia? napisałem taki kod ,ale wyskakują błędy 

from django.db import models

class users(models.Model):
    first_name = models.CharField(max_length = 50)
    surname = models.CharField(max_length = 50)

    def published(self):
        self.save()
    
class libary(models.Model):
    NameOfBook = models.CharField(max_length = 50)
    Author = models.CharField(max_length = 50)

    def published(self):
        self.save()

class loan(models.Model):
    first_name = models.ForeignKey(users, on_delete = models.CASCADE)
    surname = models.ForeignKey(users, on_delete = models.CASCADE)
    NameOfBook = models.ForeignKey(libary, on_delete = models.CASCADE)
    Author = models.ForeignKey(libary, on_delete = models.CASCADE)

    def published(self):
        self.save()

 

Comments

problem rozwiązany

Jak?

---

from django.db import models
from django.utils import timezone

class users(models.Model):
    IdOfUser = models.AutoField(primary_key = True)
    first_name = models.CharField(max_length = 50)
    surname = models.CharField(max_length = 50)

    def published(self):
        self.save()

    def __str__(self):
        return self.first_name + "  " + self.surname
    
class libary(models.Model):
    IdOfBook = models.AutoField(primary_key = True)
    NameOfBook = models.CharField(max_length = 50)
    Author = models.CharField(max_length = 50)

    def published(self):
        self.save()

    def __str__(self):
        return self.NameOfBook +"   "+ self.Author

class loan(models.Model):
    IdOfUser= models.ForeignKey(users, on_delete = models.CASCADE)
    IdOfBook = models.ForeignKey(libary, on_delete = models.CASCADE)

    def published(self):
        self.save()


    

nie chce mi się tłumaczyć ,ale obejrzałem dwa filmiki M.Z i przeczytałe dokładnie dokumentacje Django

---

@Marcins321,

Lepiej... z tym, że

IdOfBook = models.AutoField(primary_key = True)

To jest niepotrzebne; Django automatycznie dodaje primary key.

IdOfUser= models.ForeignKey

To nie jest ID; najelepiej nazwać to "user". Wtedy czytanie pól staje się czytelne, np `loan.user.surname`, `loan.book.author`.

class libary

Book.

    def published(self):
        self.save()

Nie rozumiem, po co to; wygląda na kompletnie niepotrzebne :)

---

to tak

1. Wiem że to django generuje klucze automatycznie , ale to tylko taki test czy klucz generowany automatycznie różni się od tego generowanego ręcznie ;)(wiesz o co mi chodzi? prawda?) :)

2.Class Libary ponieważ to nie jest jedna książka tylko cała biblioteka ;)

3. otóż to jest potrzebne http://prntscr.com/ldn6xd

ps.wiem że nie powinienem dodawać tam -ed ponieważ to jest czas teraźniejszy a nie przeszły ,ale tak to jest jak coś się pisze na szybko ;)

---

@adrian17, Wiesz co możesz mi pomóc ponieważ gdy usunąłem 

IdOfBook = models.AutoField(primary_key = True)

to wyskakuje mi taki błąd

http://prntscr.com/ldnj72

---

Na oko powinno działać... może to przez jakieś pośrednie migracje? Gdyby od początku był model postaci

class Book(models.Model):
    name = models.CharField(max_length=50)

To `book.id` by na pewno działało.

---

kod teraz wygląda tak

from django.db import models

class users(models.Model):
    first_name = models.CharField(max_length = 50)
    surname = models.CharField(max_length = 50)

    def publish(self):
        self.save()

    def __str__(self):
        return self.first_name + "  "+ self.surname
    
class libary(models.Model):
    NameOfBook = models.CharField(max_length = 50)
    Author = models.CharField(max_length = 50)

    def publish(self):
        self.save()

    def __str__(self):
        return self.NameOfBook +"  "+ self.Author

class loan(models.Model):
    User = models.ForeignKey(users, on_delete = models.CASCADE)
    Book = models.ForeignKey(libary, on_delete = models.CASCADE)

    def publish(self):
        self.save()


    

gdy wpisuje py manage.py migrate to wyskakuje mi takie coś 

Running migrations:
  No migrations to apply.

jedynie co może być źle to admins.py 

from django.contrib import admin
from .models import users, libary, loan

admin.site.register(users)
admin.site.register(libary)
admin.site.register(loan)

 

---

Na oko jest OK. Tyko pamiętaj też o `makemigrations`.

(i wciąż nie widzę sensu istnienia `publish()`)

---

wygląda ok ,ale nie działa :(