Najlepszy sposób na sprawdzenie, czy UITableViewCell jest całkowicie widoczny

100

Mam UITableView z komórkami o różnych wysokościach i muszę wiedzieć, kiedy są one całkowicie widoczne, czy nie.

W tej chwili przeglądam każdą komórkę na liście widocznych komórek, aby sprawdzić, czy jest ona całkowicie widoczna za każdym razem, gdy widok jest przewijany. Czy to najlepsze podejście?

Oto mój kod:

- (void)scrollViewDidScroll:(UIScrollView *)aScrollView {

    CGPoint offset = aScrollView.contentOffset;
    CGRect bounds = aScrollView.bounds;    
    NSArray* cells = myTableView.visibleCells;

    for (MyCustomUITableViewCell* cell in cells) {

        if (cell.frame.origin.y > offset.y &&
            cell.frame.origin.y + cell.frame.size.height < offset.y + bounds.size.height) {

            [cell notifyCompletelyVisible];
        }
        else {

            [cell notifyNotCompletelyVisible];
        }
    }
}

Edytować:

Zwróć uwagę, że * - ( NSArray) visibleCells zwraca widoczne komórki, które są zarówno całkowicie widoczne, jak i częściowo widoczne.

Edycja 2:

Oto poprawiony kod po połączeniu rozwiązań Lnafzigera i Vadima Yelagina :

- (void)scrollViewDidScroll:(UIScrollView *)aScrollView {
    NSArray* cells = myTableView.visibleCells;
    NSArray* indexPaths = myTableView.indexPathsForVisibleRows;

    NSUInteger cellCount = [cells count];

    if (cellCount == 0) return;

    // Check the visibility of the first cell
    [self checkVisibilityOfCell:[cells objectAtIndex:0] forIndexPath:[indexPaths objectAtIndex:0]];

    if (cellCount == 1) return;

    // Check the visibility of the last cell
    [self checkVisibilityOfCell:[cells lastObject] forIndexPath:[indexPaths lastObject]];

    if (cellCount == 2) return;

    // All of the rest of the cells are visible: Loop through the 2nd through n-1 cells
    for (NSUInteger i = 1; i < cellCount - 1; i++)
        [[cells objectAtIndex:i] notifyCellVisibleWithIsCompletelyVisible:YES];
}

- (void)checkVisibilityOfCell:(MultiQuestionTableViewCell *)cell forIndexPath:(NSIndexPath *)indexPath {
    CGRect cellRect = [myTableView rectForRowAtIndexPath:indexPath];
    cellRect = [myTableView convertRect:cellRect toView:myTableView.superview];
    BOOL completelyVisible = CGRectContainsRect(myTableView.frame, cellRect);

    [cell notifyCellVisibleWithIsCompletelyVisible:completelyVisible];
}
RohinNZ
źródło
3
Na marginesie, powinieneś przejść do wszystkich swoich poprzednich pytań i zaakceptować odpowiedzi tych, którzy ci pomogli.
Baub
4
Dzięki, że mi powiedziałeś! Dałem im już +1, ale zapomniałem o ustawionej funkcji odpowiedzi.
RohinNZ
Twój kod wydaje mi się poprawny i chociaż jest skomplikowany, działa. Nie naprawiaj tego, co się nie zepsuło, co?
CodaFi

Odpowiedzi:

125

Możesz uzyskać rectForRowAtIndexPath:prostokąt komórki za pomocą metody i porównać go z prostymi granicami widoku tabeli za pomocą CGRectContainsRectfunkcji function.

Zauważ, że nie spowoduje to utworzenia instancji komórki, jeśli nie jest ona widoczna, a zatem będzie raczej szybka.

Szybki

let cellRect = tableView.rectForRowAtIndexPath(indexPath)
let completelyVisible = tableView.bounds.contains(cellRect)

Obj-C

CGRect cellRect = [tableView rectForRowAtIndexPath:indexPath];
BOOL completelyVisible = CGRectContainsRect(tableView.bounds, cellRect);

Oczywiście nie będzie to dotyczyło przycinania widoku tabeli przez nadzór lub zasłaniania przez inny widok.

Vadim Yelagin
źródło
Dziękuję Ci! Połączyłem ten kod z rozwiązaniem Lnafziger.
RohinNZ
11
Z drugiej strony, może być po prostuCGRectContainsRect(tableView.bounds, [tableView rectForRowAtIndexPath:indexPath])
Vadim Yelagin
1
dlaczego jest to początek mojego cellRecta.x = 0, origin.y = 0, szerokość = 0, wysokość = 0? chociaż w interfejsie użytkownika nie wszystkie są zerami, używam układu automatycznego, jakieś pomysły?
RainCast
7
Swift 3:let completelyVisible = tableView.bounds.contains(tableView.rectForRow(at: indexPath))
cbartel
Jak możemy obsłużyć podobną funkcjonalność dla UICollectionView?
Satyam
64

Zmieniłbym to tak:

- (void)checkVisibilityOfCell:(MyCustomUITableViewCell *)cell inScrollView:(UIScrollView *)aScrollView {
    CGRect cellRect = [aScrollView convertRect:cell.frame toView:aScrollView.superview];

    if (CGRectContainsRect(aScrollView.frame, cellRect))
        [cell notifyCompletelyVisible];
    else
        [cell notifyNotCompletelyVisible];
}

- (void)scrollViewDidScroll:(UIScrollView *)aScrollView { 
    NSArray* cells = myTableView.visibleCells;

    NSUInteger cellCount = [cells count];
    if (cellCount == 0)
        return;

    // Check the visibility of the first cell
    [self checkVisibilityOfCell:[cells firstObject] inScrollView:aScrollView];
    if (cellCount == 1)
        return;

    // Check the visibility of the last cell
    [self checkVisibilityOfCell:[cells lastObject] inScrollView:aScrollView];
    if (cellCount == 2)
        return;

    // All of the rest of the cells are visible: Loop through the 2nd through n-1 cells
    for (NSUInteger i = 1; i < cellCount - 1; i++)
        [[cells objectAtIndex:i] notifyCompletelyVisible];
}
lnafziger
źródło
<I> w instrukcji if powinny brzmieć <= lub> =, poprawię to w odpowiedzi ...
Aardvark
12

Możesz spróbować czegoś takiego, aby zobaczyć, ile procent jest widoczny:

-(void)scrollViewDidScroll:(UIScrollView *)sender
{
    [self checkWhichVideoToEnable];
}

-(void)checkWhichVideoToEnable
{
    for(UITableViewCell *cell in [tblMessages visibleCells])
    {
        if([cell isKindOfClass:[VideoMessageCell class]])
        {
            NSIndexPath *indexPath = [tblMessages indexPathForCell:cell];
            CGRect cellRect = [tblMessages rectForRowAtIndexPath:indexPath];
            UIView *superview = tblMessages.superview;

            CGRect convertedRect=[tblMessages convertRect:cellRect toView:superview];
            CGRect intersect = CGRectIntersection(tblMessages.frame, convertedRect);
            float visibleHeight = CGRectGetHeight(intersect);

            if(visibleHeight>VIDEO_CELL_SIZE*0.6) // only if 60% of the cell is visible
            {
                // unmute the video if we can see at least half of the cell
                [((VideoMessageCell*)cell) muteVideo:!btnMuteVideos.selected];
            }
            else
            {
                // mute the other video cells that are not visible
                [((VideoMessageCell*)cell) muteVideo:YES];
            }
        }
    }
}
Catalin
źródło
Jeśli widok tabeli może wyświetlać 2 komórki z filmami (na przykład na iPadzie), oba filmy będą odtwarzane z powyższym kodem?
Jerome
@Catalin Wypróbowałem Twoje rozwiązanie, ale mój widok tabeli zwalnia. Jakiś sposób na poprawę wydajności przewijania?
Rahul Vyas
@RahulVyas przepraszam, ale nie :( sprawdź swoje wewnętrzne elementy, być może układ można nieco zoptymalizować pod względem warstw / podwarstw
Catalin,
5

Z dokumentów:

visibleCells Zwraca komórki tabeli, które są widoczne w odbiorniku.

- (NSArray *)visibleCells

Wartość zwracana Tablica zawierająca obiekty UITableViewCell, z których każdy reprezentuje widoczną komórkę w odbierającym widoku tabeli.

Dostępność Dostępne w iOS 2.0 i nowszych.

Zobacz też - indexPathsForVisibleRows

CodaFi
źródło
4
Przepraszam, może nie byłam wystarczająco jasna. Interesują mnie tylko komórki, które są całkowicie widoczne. - (NSArray *) visibleCells i indexPathsForVisibleRows zwracają komórki, które są całkowicie widoczne i częściowo widoczne. Jak widać w moim kodzie, używam już - (NSArray *) visibleCells, aby znaleźć te, które są całkowicie widoczne. Mimo wszystko dzięki.
RohinNZ
4

Jeśli chcesz również wziąć pod uwagę contentInset i nie chcesz polegać na superviewie (ramka widoku tabeli w superview może być czymś innym niż 0,0), oto moje rozwiązanie:

extension UITableView {

    public var boundsWithoutInset: CGRect {
        var boundsWithoutInset = bounds
        boundsWithoutInset.origin.y += contentInset.top
        boundsWithoutInset.size.height -= contentInset.top + contentInset.bottom
        return boundsWithoutInset
    }

    public func isRowCompletelyVisible(at indexPath: IndexPath) -> Bool {
        let rect = rectForRow(at: indexPath)
        return boundsWithoutInset.contains(rect)
    }
}
Kukosk
źródło
1
- (BOOL)checkVisibilityOfCell{
    if (tableView.contentSize.height <= tableView.frame.size.height) {
        return YES;
    } else{
        return NO;
    }
}
Naresh Jain
źródło
0
UICollectionViewCell *cell = [self.collectionView cellForItemAtIndexPath:indexPath];
CGRect frame = cell.frame;
if (CGRectContainsRect(CGRectOffset(self.collectionView.frame, self.collectionView.contentOffset.x, self.collectionView.contentOffset.y), frame))
{
    // is on screen
}
Andy Poes
źródło
0

Nawet jeśli powiedziałeś, że chcesz go sprawdzać za każdym razem, gdy przewijasz, możesz również użyć

-(void)tableView:(UITableView *)tableView willDisplayCell:(UITableViewCell *)cell forRowAtIndexPath:(NSIndexPath *)indexPath
{
       CGRect cellRect = [tableView convertRect:cell.frame toView:tableView.superview];
       if (CGRectContainsRect(tableView.frame, cellRect)){
            //Do things in case cell is fully displayed
        }

}
iRestMyCaseYourHonor
źródło
0

Może w tym przypadku lepiej wykorzystaj następną funkcję z UITableViewDelegate

func tableView(_ tableView: UITableView, didEndDisplaying cell: UITableViewCell, forRowAt indexPath: IndexPath)
Artem
źródło
To nie zadziała. Z dokumentutells the delegate that the specified cell was removed from the table.
anatoliy_v
0

Poniższy kod umożliwia sprawdzenie, czy komórka widoku kolekcji jest całkowicie widoczna przez atrybuty układu widoku kolekcji.

guard let cellRect = collectionView.layoutAttributesForItem(at: indexPath)?.frame else { return } let isCellCompletelyVisible = collectionView.bounds.contains(cellRect)

fahlout
źródło