Discussion :

[wiztricks]

C'est bien de mettre à jour votre code.
Maintenant on a un truc qui "fonctionne" sans trop avoir à y ajouter de lignes.

Mais à part vous signaler des non sens comme:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
           for j in range(i, len(df['id'])):
                if df['date_depart'][i] < df['date_arrive'][j] and i>j:
                    surface_A += df['surface'][j]

vous ne dites rien sur ce que vous en attendiez (et que çà ne fait pas).
Et on ne va pas inventer le problème...

- W

[Naunau75]

Ce que je veux faire, c'est soustraire la surface des objets que je dépose et la deuxième boucle for me sert à comparer les dates pour retirer un objet déposé précédemment. Un objet est déposé en A à date_arrive puis est retiré à date_depart. Ce que je voudrais avoir, c'est la surface_A une fois que l'on a déposé tous les colis.

[wiztricks]

Ce que je veux faire, c'est soustraire la surface des objets que je dépose et la deuxième boucle for me sert à comparer les dates pour retirer un objet déposé précédemment.

Relisez votre code:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
    for i in range(0, len(df['id'])):
        if surface_A >= 0:
            surface_A -= df['surface'][i]
            for j in range(i, len(df['id'])):
                if df['date_depart'][i] < df['date_arrive'][j] and i>j:
                    surface_A += df['surface'][j]

L'indice i balaie les colis à déposer dans A ou B.
Et l'indice j balaie ceux qui sont encore nulle part
Donc la deuxième boucle ne fait déjà pas ce que vous voulez!
Et pourquoi comparer la date de départ du colis qu'on essaie de placer à la date d'arrivée de colis qui ne sont pas encore traités? Vos colis sont à priori indépendants non?
Et comme çà ne va traiter aucun colis (à cause du i > j): çà ne sert à rien.

Simplifiez: une fonction dépose les colis suivant la place disponible, une autre fonction les retire suivant de la date de départ (et vous allez voir ressortir le soucis "où sont les différents colis").

- W

[lg_53]

Vous avez un problème sur la gestion du temps.

La fonction que vous avez écrite, elle est censé retourné l'état de surface_A et de surface_B à quel instant ?
- A l'instant final ? Dans ce cas tout aura été expédié je présume ...
- A instant T choisi par l'utilisateur ? Dans ce cas il faut le prévoir en paramètre de votre fonction ...
- La liste des différents instants ? Donc la progression de surface_A et surface_B au cours du temps.
- autre chose ?

[Naunau75]

Simplifiez: une fonction dépose les colis suivant la place disponible, une autre fonction les retire suivant de la date de départ (et vous allez voir ressortir le soucis "où sont les différents colis").

Merci, je regarde ça dès que je trouve un peu de temps, et je reviens vers vous.

La fonction que vous avez écrite, elle est censé retourné l'état de surface_A et de surface_B à quel instant ?
- A l'instant final ? Dans ce cas tout aura été expédié je présume ...
- A instant T choisi par l'utilisateur ? Dans ce cas il faut le prévoir en paramètre de votre fonction ...
- La liste des différents instants ? Donc la progression de surface_A et surface_B au cours du temps.
- autre chose ?

Comme je passe un par un les objets par id, je n'ai la surface_A et B à la fin de la liste. Mais effectivement à l'instant final, tous les objets seront partis.
Au final j'aimerais bien avoir la surface_A et B à l'instant t et avoir l'évolution des surfaces en fonction du temps. Je vais essayer de l'implémenter.

[lg_53]

Comme je passe un par un les objets par id, je n'ai la surface_A et B à la fin de la liste. Mais effectivement à l'instant final, tous les objets seront partis.

Ca représente quoi pour vous la fin de la liste dans votre problème ? Pour moi c'est le moment où le dernier colis est arrivé, mais est ce bien cela et clair pour vous ?

[Naunau75]

Oui, celui qui a été posé le 13/01/2018

[hotcryx]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

for j in range(0, len(df['id'])):

A ta place je ne mettrais pas de len(...) dans une boucle qui risque de réévaluer la condition à chaque loop.
C'est pareil pour tes autres boucles.

Utilise des variables à cet effet.
Ex:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
size = len(df(....))
for j in range(0, size):
    ...

[wiztricks]

Au final j'aimerais bien avoir la surface_A et B à l'instant t et avoir l'évolution des surfaces en fonction du temps. Je vais essayer de l'implémenter.

Mouais, peut être faut-il commencer par un instant/état initial qui traduise le remplissage de A et de B. Puis on fait avancer le temps (la plus petite date de départ, ou l'arrivée de nouveaux colis à placer).
Vous avez aussi un soucis côté interface de votre fonction "remplissage". Il serait plus simple de lui passer une capacité disponible et une collection de colis pour qu'elle retourne la capacité restante et les colis qui n'ont pas pu être placés.
Vous allez pouvoir appliquer çà à surface_A, surface_B, surface_C,... sans retoucher au code et virer des conditions. Plus droit == plus simple... même si un peu de la complexité revient à l'appelant.

- W