c++ zadanie z klasą i szybkie pytanie

Question

Witam wszystkich bardzo serdecznie :). Mam taki problem otóż: robię zadanie w c++. Mam do zrobienia klase Tablica, która posiada argumenty : wiersze, kolumny i char**tab (dwuwymiarowa tablica dynamiczna) . Konstruktor klasy alokuje pamięc dla tab.   Oprócz konstruktora mam metodę: void readTablica(const char* nazwaPliku). Ta metoda odczytuje moją tablicę z pliku i zapisuje ją do tablicy. Wnioskuję, że mam skorzystać z mojego konstruktora gdzie mam moją dynamiczną talicę dwuwymiarową ale nie wiem jak sie do tego odwołać. Dorzucam kod mojej metody. Macie jakiś pomysł jak zapisać te znaki z pliku do tej tablicy ? Dziękuje za odp.

void Tablica::readTablica(const char* nazwaPliku){
	fstream plik;
	plik.open(nazwaPliku, ios::in);
	if(plik.good()==true){
	cout<<"it's okey";
	char a[30];
	plik.get(a,30);

	}
	else {cout<< " upsss";}
}

 

Comments

a czy w pliku sa informacje o wielkosci macierzy?

---

Tworze sobie w konstruktorze( int wiersze, int kolumny){......}

i pózniej przy tworzeniu obiektu w main podaje dowolny wymiar.

np Tablica a(10,2)

Answer 1

Wiesz, ja nie widzę całej Twojej klasy więc ... sam sobie dopisałem resztę :-) 

Można np. tak

#include <iostream>
#include <cstdlib>
#include <string>
#include <fstream>

using namespace std;

class Tablica {
public:
    Tablica(size_t wiersze, size_t kolumny): kolumny(kolumny), wiersze(wiersze) {
        dane = new int*[wiersze];
        for(size_t i = 0; i < wiersze; ++i) {
            dane[i] = new int[kolumny];
        }
    }
    void czytajPlik(const char * nazwaPliku) {
        ifstream plik(nazwaPliku);
        if(not plik) {
            cerr << "Problem z otwarciem pliku " << nazwaPliku << ".\n";
            exit(EXIT_FAILURE);
        }
        for(size_t i = 0; i < wiersze; ++i) {
            string linia;
            plik >> linia;
            for(size_t j = 0; j < kolumny; ++j) {
                dane[i][j] = linia[j];
            }
        }
    }
    void pokazTablice() const {
        for(size_t i = 0; i < wiersze; ++i) {
            cout << "Wiersz: " << i << "| ";
            for(size_t j = 0; j < kolumny; ++j) {
                cout << dane[i][j] << ' ';
            }
            putchar('\n');
        }
    }

private:
    int **dane;
    size_t kolumny;
    size_t wiersze;
};

int main() {
    Tablica t(6, 10);
    t.czytajPlik("data.txt");
    t.pokazTablice();
}

A dane mam np. takie:

1234567890 3231231412 4454545452 1231343435 zzxassdasa ffqwqdzczA

 

Comments

Dwie uwagi co do kodu:

1) Nie zwalniasz pamięci po jej zaalokowaniu (wypadałoby to zrobić, np. w destruktorze).

2) W C++ nie należy używać funkcji exit - jest ona przeznaczona dla programów w C, nie pozwala na wywołanie destruktorów przed zakończeniem pracy programu.

---

Dzięki za uwagi. Zakładałem że pytający (jak napisał) miał klasę więc to jest kod "po najmniejszej linii oporu". http://en.cppreference.com/w/cpp/utility/program/exit

Oczywiście, powinien być destruktor pracowicie usuwający tak tablice z kolumnami jak i samo miejsce na wiersze, ustosunkowanie się do kopii, przypisania, move..... ale pytającego interesował sposób implementacji czytajPlik() :-)

---

Nie pisałem o tym, co mogłoby tu jeszcze być, tylko o tym, co musi być z punktu widzenia poprawności tego prostego programu.

(jeśli linkujesz dokumentację po to, żeby pokazać że exit() wywołuje destruktory, to sugeruję przeczytać dokładnie całość linkowanej strony)

I nie zrozum tego jako jakiegoś ataku na siebie (akurat tak się przypadkiem złożyło, że wczoraj również zgłaszałem pewne obiekcje do Twojej wypowiedzi) - zależy mi na tym, żeby niedoświadczonym programistom pokazywać poprawny kod - tylko tyle. :)

---

Mam jeszcze takie pytanie na koniec :): Powiedzmy ze mam w pliku zapisane same znaki 'x' i 'o' kod jest np: xxxoxooxxxoxxxxx xooxooxxxoxoooxx .... i chciałbym przeciążyć operator <<(wyjscia) tak aby wyświetlał same 'x' a tak gdzie jest 'o' to luki. I tu jest moje pytanie czy mogę w ciele tego przeciążenia zrobić jakby jeszcze raz otwarcie z pliku i wypisywanie tego tylko w "szczególny" sposób ?  Dzięki za odp

---

I tu jest moje pytanie czy mogę w ciele tego przeciążenia zrobić jakby jeszcze raz otwarcie z pliku i wypisywanie tego tylko w "szczególny" sposób ?

Operacje na plikach są operacjami stosunkowo wolnymi. Skoro już masz wczytaną zawartość do pamięci, to nie ma sensu ponownie otwierać strumień do pliku. Skorzystaj z tablic, które przechowują jego zawartość w RAM.

---

Tylko że przez przeciążenie chcę wyświetlić dość zmodyfikowaną tą tablice, więc miałem pomysł, aby to zrobić za pomocą pętli i pobierania charów z pliku. Tak zrobiłem i to działa tylko, że jedyny minus jest taki, że otwieram plik jeszcze raz. Rozumiem że dzięki mojemu konstruktorowi już mam tą tablice z pliku w RAM komputera, jednak nie potrafię wprowadzić żadnych modyfikacji w ten sposób. Nawet nie wiem czy jest to możliwe. Dzięki za odpowiedz:)

---

Wyświetlenie tablicy to jedynie prosta pętla. Ominięcie wyświetlania pewnych znaków i wstawienie zamiast nich białych znaków to tylko jeden warunek.

// na przykład (przyjąłem nazewnictwo jak w klasie Użytkownika  mokrowski):

void Tablica::pokaz() const
{
    for (size_t i = 0; i < wiersze; ++i)
    {
        for (size_t j = 0; j < kolumny; ++j)
        {
            if(dane[i][j] == 'o')
            {
                std::cout<<' ';
            }
            else
            {
                std::cout<<dane[i][j];
            }
        }
        std::cout<<'\n';
    }
}

 

---

Faktycznie :) świetnie rozwiązanie. Dzięki mega mi pomogłeś, kawał dobrej roboty.Pozdrawiam. Temat do zamknięcia.

---

Na zdrowie. :)

Answer 2

Myślę że najłatwiej będzie to zrobić w dwóch pętlach korzystając z funkcji std::stoi (bądź stod w zaleznosci od typu)

Przyda się też zmienna tymczasowa string.

Comments

własnie mam taki pomysł ze w tej metodzie tworze dwie pętle i na koncu daje:

plik>>tab[i][j]; // i teoretycznie działa choc nie wiem czy to jest do końca okej :D

Dzięki za odp :)

---

możliwe że działa ale masz tutaj niejawno konwersje ze stringa do int/double

---

ale ja w moim pliku który wczytuje mam tylko chary typu 'x' albo 'o', wiec chyba będzie ok ?

---

tak to w porządku ;)

Answer 3

Jeszcze tylko odpowiedz na pytania:

1. Rozumiem że jesteś zobowiązany aby trzymać macierz w tablicy ** ? (Tak/Nie)

Jeśli nie, to z niej zrezygnuj na rzecz vector.

2. W pliku z którego czytasz zawartość macierzy, jest podany jawnie jej rozmiar (np. 2 wpisy typu ilość_kolumn ilość_wierszy)? (Tak/Nie)

Jeśli tak sprawa bardzo się uprości.

3. Plik z zawartością ma jakąś strukturę (np. pola oddzielone spacją) czy jest "strumieniem danych binarnych" ? (Tak/Nie)

Comments

1. TAK

2.NIE

3. NIE jest to bezsensowny ciąg charów ze spacjami np: xxxooxooxo xooxoxoxxx ooxoxxooxo      ( wiersze tej samej wielkosci, wszystko w jednej linii, około 6 takich ).