Dotaz: Prosim o pomoc s kontrolou

[Pavol]

ls -l >data.txt
sh p2.exe

-obsah souboru p2.exe

main()
{

char radek[100];
  char pismeno;
  float soucet,nasobek;
  int i;
FILE *fd;
  FILE *fw;

if((fd=fopen("data.txt","r"))==NULL) return;
  if((fw=fopen("SOUHRN.txt","w"))==NULL) return;

  soucet=0;
  printf("Zadejte pismeno, na ktere maji zacinat soubory:\n");
  scanf("%c", &pismeno);
while(!feof(fd)) /* vypis souboru */
{
  fgets (radek,100,fd);
    if(radek[56] == pismeno){
    printf("%s\n",radek);
      fprintf(fw,"%s", radek);
      i=42;
      nasobek=1;
      while ((i-- > 30) && (radek[i] != ' ')){
        soucet+=(radek[i]-48)*nasobek;
        nasobek*=10;
      }
    }
}

  fclose(fd);
  fclose(fw);
  getchar();
  system("pause");
}

Odpovědi

[©]

Pouzivej <pre> a </pre> tagy.

[Pavol]

Tak se dívám nějak se tady neobjevil celý můj dotaz. Takže znova: udělal jsem program do školy, bohužel jsem to nemohl vyzkoušet, protože se mi nepodařilo doma nainstalovat Linux(ani liveCD), a tak bych byl rad kdyby to někdo ökoukl a pomohl najít případné bagy - diky předem.
podrobnosti:

prazdne soubory: data.txt, SOUHRN.txt

SH soubor :

ls -l >data.txt
sh p2.exe

a exe soiubor programovny v jazyce C je už v zadani prispevku

[Michal Vyskočil]

1. Chybí ti deklarace standardní knihovny

#include<stdio.h>

2. Main by měl být typu int
3. Program "pause" v unixech není
4. sh p2.exe ti nebude fungovat. Jednak je v unixech koncovka exe zbytečná a druhak se ten soubor spustí ./p2 (./p2.exe, když už to musí být).

Jinak jsem tvůj program přeložil

$ gcc -c p2.c -o p2.o 
$ gcc p2.o -o p2.exe
$ ls -l >data.txt
$ cat data
total 16
-rw-r--r--  1 misak users    0 Jan  3 22:28 SOUHRN.txt
-rw-r--r--  1 misak users    0 Jan  3 22:28 data.txt
-rw-r--r--  1 misak users  744 Jan  3 22:20 p2.c
-rwxr-xr-x  1 misak users 7590 Jan  3 22:20 p2.exe
-rw-r--r--  1 misak users 1680 Jan  3 22:20 p2.o
$ ./p2.exe
Zadejte pismeno, na ktere maji zacinat soubory:
p
$ cat SOUHRN.txt
$

Tady jsou změny, které jsem musel udělat:

$ diff p2.c.original p2.c
0a1
> #include<stdio.h>
22c23
<       if (radek[56] == pismeno)
---
>       if (radek[44] == pismeno)
39c40
<   system ("pause");
---
>   //system ("pause");

Pokud ti nejede žádný unix, co takhle zkusit cygwin? Nebo vmware player s nějakým nainstalovaným unixem?

[Pavol]

Já testoval i cygwina, něco jsem podle skript vyzkoušel a něco vůbec nešlo, takže jsem byl trochu zmatený .Díky za pomoc, jen k těm změnám: include.., to jsem dodělal, system("pause") odstranil ale co ta "IF", kam s nimi ? Nebo pochopil jsem dobře, že řádek: if (radek[56] == pismeno)

mám nahradit tímto ?? (co tím vyřeším?) :

if (radek[44] == pismeno)

[Michal Vyskočil]

No mě doma výpis ls -l začínal na 45. sloupci a ne 57., ale teď se dívám, že na CentOS je to taky 56, takže ten řádek raději neměň, pokud ti to funguje

[Michal Kubeček]

Především je pošetilost snažit se parsovat výpis příkazů typu 'ls -l'. K procházení adresáře slouží opendir(), readdir() a closedir(), případně, nechce-li se tazatel zabývat přenositelností, lze uvažovat o použití scandir().

[Martin Baleja]

A když už výpis souborů pomocí ls tak bych použil ls -1 a problém s posunutím odpadá.

[Michal Vyskočil]

Aha, tak tohle mě, přiznávám nenapadlo. Já předpokládal, že zadáním bylo parsovat výstup příkazu ls -l. Sice je to přitažené za vlasy, ale některá zadání taková bývají.

[Michal Kubeček]

Nejspíš to tak bude, že je to uhozené, to je bohužel pro spoustu pedagogů vedlejší. Program vypadá typicky "příkladově", což je vidět i z toho, že zpracovává vstup ze souboru fixního jména a výsledek zapisuje do souboru fixního jména. To vídám pravidelně v zadáních příkladů, v praxi se taková bl…pardon…nesmysl moc nevidí.

[Michal Kubeček]

A aby toho nebylo málo, tak třeba u mne (SuSE 10.0) nejsou pozice sloupců pevné a poloha jména tak závisí (mimo jiné) i na tom, jak dlouhé jsou nejdelší jméno vlastníka, skupinového vlastníka a délka souboru…

[Michal Kubeček]

Další problém vidím v tom, že se pracuje s radek[56], aniž by se ověřilo, že je řádek aspoň tak dlouhý, což zrovna hned první nebude. Sice je to ve staticky alokovaném bufferu, takže nehrozí pád programu nebo zneužití, ale to, zda se bude první řádek kopírovat nebo ne, je de facto dáno náhodou. Také je podivné, že se celou dobu něco sčítá do proměnné soucet (proč je vlastně typu float?), ale hodnota se nikde nepoužije. Rozumnější verze programu by mohla vypadat např. takto:

#include <stdio.h>
#include <sys/types.h>
#include <dirent.h>
#include <sys/stat.h>
#include <unistd.h>
#include <string.h>

int main(int argc, char* argv[])
{
  DIR* dir;
  struct dirent* de;
  struct stat info;
  unsigned long long size;
  size_t prefixlen;
  const char* pgmname;
  const char* p;
  int R;

  pgmname = argv[0];
  for (p=argv[0]; *p; p++)
    if (*p == '/') pgmname = p+1;
  if (argc < 2) {
    fprintf(stderr, "usage: %s directory [prefix]\n", pgmname);
    return 1;
  }
  if (argc == 3) prefixlen = strlen(argv[2]);
  else prefixlen = 0;

  dir = opendir(argv[1]);
  if (!dir) {
    perror(pgmname);
    return 2;
  }
  R = chdir(argv[1]);
  if (R < 0) {
    perror(pgmname);
    return 3;
  }

  size = 0;
  while ((de = readdir(dir)))
    if (prefixlen==0 || !strncmp(de->d_name, argv[2], prefixlen)) {
      stat(de->d_name, &info);
      if (S_ISREG(info.st_mode)) {
        size += info.st_size;
        puts(de->d_name);
    }
  }

  closedir(dir);
  printf("Total size: %llu\n", size);

  return 0;
}

[Pavol]

Já bych taky otestoval kde co. Jenže nemám to jak otestovat, tak nemužu moc experimentovat

[Martin Baleja]

Vždycky když náhodou za někoho dělám nějaký program, dávám si pozor na to, aby na to učitel "skočil" - což [Váš|tvůj] program asi nesplňuje

[Pavol]

Pane Kubíček, kdybych tohle odevzdal panu PROFESóROVI, tak by mu asi švihlo .

No, co se týká toho školního servru , ono něco takového asi existovat bude , jenže já to mám jako volitelňák, takže jsem tam byl jen na jedné čtyřhodinovce, takže jsem to info asi ňako nezaznamenal

[Michal Vyskočil]

s/Kubíček/Kubeček/

[Michal Kubeček]

Možná by to tak bylo lepší… :-) Osobně dávám přednost programu, který

• nezávisí na verzi ls, nastavení locale a přesném vyvážení mnoha dalších okolností
• nesčítá délky souborů jako hodnoty typu float (které nakonec stejně nepoužije)
• nemá výstup závislý na stavu neinicializované paměti
• nekomunikuje s okolím prostřednictvím souborů s natvrdo zadrátovaným jménem
• detekuje chyby a sdělí uživateli, co se děje

Ale to je možná tím, že už mám po maturitě, takže mám k programování nechutně praktický přístup. :-)

[Pavol]

Máte pravdu , kdybych to odevzdal, jak radíte, tak by profes. neuveril, ze sem to delal ja.., ale muzu to mit sebou, pro jistotu

[Mortal]

nemám žádné řešení ale jen dotaz:
Co je to za školu, že tam nemáte účet na nějakém unixovém serveru?
Na něm by jsi mohl testovat

Já bych si to bez něj nedovedl představit, protože je to pro mě brána do univerzitní sítě a můžu si pak například odkudkoliv z internetu stahnout studijní materialy co jsou ve škole na samba serveru.