'\0' конец строки

[rowlin]

Доброго времения суток
В K&R на тему хеширования приводится хеш функция :

hash(s) /* form hash value for string */
char *s;
{
int hashval;
for (hashval = 0; *s != '\0'; )
hashval += *s++;
return(hashval % HASHSIZE);
}

ввожу данные:

printf("введите имя :");
scanf("\n%s",&name);
print....

...компиляция происходит успешно .. но отладчик ругается на for (hashval=0; *s !='\0'; *s++)..
..
введите имя :lexa
введите Имя2:lena

Program received signal SIGSEGV, Segmentation fault.
0x08048541 in hash (s=0x61786500 <Address 0x61786500 out of bounds>) at first.c:18

Разве '\0' конец строки не автоматически определяется ..или где ошибка?
ps : представил тут только ключевые моменты ...по сути мы передаем name в hash() в функции записи ..в случае надобности могу выложить весь исходник..

[Syuf]

Во-первых, выборосили вы юы эту книжку; во-вторых, моя "интуиция" подсказывает, что надо написать scanf("\n%s", name); без амперсанда; что у вас есть name?

[rowlin]

char *name;

без амперсанда:
Program received signal SIGSEGV, Segmentation fault.
0xb7ebb294 in _IO_vfscanf () ;

зря вы про книжку так - это ж классика..(даж распечатал)

[Syuf]

Классика 1945 года? Почему функции в старом стиле? Зачем != 0 писать? Зачем скобки в return?
По теме: а такой ввод строк тоже из этой книженции взяли?
Либо выделяйте память (не забыв ее освободить); либо выделяйте, забыв ее освободить (шутка); либо не выделяйте ничего, и используйте статический массив символов.

даж распечатал

Это веский повод ее сжечь. (есть, что сжигать)

[Сtrl]

Syuf, глупость сказали! С каких пор scanf работает со значением, а не с адресом?
P.S. Туплю, это же C-строка. Тогда правильно говорите.

rowlin, так у вас *s++ в теле цикла или в заголовке? Вы написали:

for (hashval = 0; *s != '\0'; )

А отладчик ругается на:

for (hashval=0; *s !='\0'; *s++)

[Syuf]

С каких пор scanf работает со значением, а не с адресом?

Ctrl Так она и работает с адресом, адресом начала строки. Глупость говорите как раз вы: адресс указателя scanf'у нафиг не нужен, она читает строку. И с амперсандом работать не будет.
Другое дело, что если это статический массив, то эти адресса (str и &str) совпадают. Но тут это не статический массив.

[Syuf]

так у вас *s++ в теле цикла или в заголовке?

P.S. Тело цикла начинается после его заголовка.

[rowlin]

вот...

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

struct nlist  { 
      char *name;
      char *def;
      struct nlist *next; 
 };
#define    HASHSIZE     100
static struct nlist *hashtab[HASHSIZE];

  hash(s)   /* form hash value for string */
  char *s;
  {
   int hashval;
    for (hashval=0; *s !='\0'; *s++)
      // for (hashval = 0; *s != '\0'; )
       hashval ++;
   return(hashval % HASHSIZE);
  }

 struct nlist *lookup(s) 
 char *s;
 {
 struct nlist *np;
 for (np = hashtab[hash(s)]; np != NULL;np=np->next)
   if (strcmp(s, np->name) != 0)//если совпадений нет
   return(np);  /* found it */
 return(NULL);    /* not found */
 }

struct nlist *install(name, def) /* put (name, def) */
    char *name, *def;
    {
      struct nlist *np, *lookup();
  int hashval;
  if((np = lookup(name)) == NULL) { /* not found */
   np = (struct nlist *) malloc(sizeof(*np));
   if (np == NULL)
      return(NULL);
   if (name == NULL)
      return(NULL);
   hashval = hash(np->name);
   np->next = hashtab[hashval];
   hashtab[hashval] = np;
  } else        /* already there */
       free(np->def);/* free previous definition */
     if ((strcpy(np->def,def)) == NULL)
       return (NULL);
  return(np);
}      

char main(){
  int a;
  char *name,*def;
  for(a=0;a<3;a++){
  printf("введите имя :");
    scanf("\n%s",&name); 
    printf("\nвведите Имя2:");
    scanf("\n%s",&def);
    if (install(name,def)!=NULL) printf("end");
  }
}

Сtrl

А отладчик ругается на:
Код:

for (hashval=0; *s !='\0'; *s++)

то же самое ...
0x0804852f in hash (s=0x656565 <Address 0x656565 out of bounds>) at first.c:17

[Syuf]

Короче, hash был правильный, main надо такой:

char main(){
  int a;
  char name[80] = {'\0'}, def[80] = {'\0'};
  for(a=0;a<3;a++){
  printf("введите имя :");
    scanf("\n%s",name); 
    printf("\nвведите Имя2:");
    scanf("\n%s",def);
    if (install(name,def)!=NULL) printf("end");
  }
}

[rowlin]

Syuf

Cпасибо ..заработало ..

Насчет книжки ..возможно вы правы .. но .. мож что из нового по С посоветуете ..
ps: желательно без использования windows библиотек (<conio.h>...)